crowmaki/ai-studio
Watch
1
0
Fork
You've already forked ai-studio
 0
Code Issues 8 Pull requests Projects Releases Packages Wiki Activity Actions

Upload PR images with an issue-scoped token #103

Merged
crowmaki merged 1 commit from tools/pr-images-issue-scope into main 2026-10-08 21:55:21 +00:00
Conversation 0 Commits 1 Files changed 1 +2 -2
crowmaki commented 2026-10-08 21:55:04 +00:00
Owner
Copy link

Follow-up to #102.

scripts/pr-images.mjs read and edited the PR body through /repos/{repo}/pulls/{n}. That endpoint needs the read:repository scope. A token with only the issue scope, which is what docs/agents/issue-tracker.md asks for, failed with a 403. Forgejo's /repos/{repo}/issues/{n} endpoints also work on PRs, so the script now uses them to read and update the body. The attachment upload was already on the issue endpoint.

Tests

  • bun test scripts passes.
  • Ran the fixed script against the real Forgejo on #101 with an issue-only token. It uploaded the four screenshots and linked them in the PR body, and each attachment URL returns 200 image/png.
Follow-up to #102. `scripts/pr-images.mjs` read and edited the PR body through `/repos/{repo}/pulls/{n}`. That endpoint needs the `read:repository` scope. A token with only the issue scope, which is what `docs/agents/issue-tracker.md` asks for, failed with a 403. Forgejo's `/repos/{repo}/issues/{n}` endpoints also work on PRs, so the script now uses them to read and update the body. The attachment upload was already on the issue endpoint. ## Tests - `bun test scripts` passes. - Ran the fixed script against the real Forgejo on #101 with an issue-only token. It uploaded the four screenshots and linked them in the PR body, and each attachment URL returns 200 `image/png`. 
The pulls endpoints need the repository scope; the issue endpoints serve
PRs too and need only the issue scope the docs ask for.
crowmaki deleted branch tools/pr-images-issue-scope 2026-10-08 21:55:21 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
needs-info
Waiting on reporter for more information

  
needs-triage
Maintainer needs to evaluate this issue

  
question
More information is needed

  
ready-for-agent
Fully specified, ready for an AFK agent

  
ready-for-human
Requires human implementation

  
wayfinder:grilling
Needs a decision talked through with a human

  
wayfinder:map
Index of decisions and open questions for a large effort

  
wayfinder:prototype
Needs a rough artifact built to react to

  
wayfinder:research
Needs facts gathered from docs or external sources

  
wayfinder:task
Manual work that must happen before a decision

  
wontfix
Will not be actioned

No labels
bug
duplicate
enhancement
help wanted
invalid
needs-info
needs-triage
question
ready-for-agent
ready-for-human
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
crowmaki/ai-studio!103
No description provided.