@sentry/junior-github 0.133.0 → 0.134.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/SETUP.md CHANGED
@@ -1,6 +1,6 @@
1
1
  # GitHub plugin setup
2
2
 
3
- This plugin exposes two skills — `github-code` (clone, source-code investigation, pull requests) and `github-issues` (issue workflows). Junior uses GitHub App installation tokens for reads, workflow dispatches, allowlisted issue and pull request writes, and Git branch pushes. Human OAuth is reserved for explicitly personal operations such as pull request reviews and user-attachment uploads.
3
+ This plugin exposes two skills — `github-code` (clone, source-code investigation, pull requests) and `github-issues` (issue workflows). Junior uses GitHub App installation tokens for reads, workflow dispatches, allowlisted issue and pull request writes (including pull request reviews and inline review comments), and Git branch pushes. Human OAuth is reserved for explicitly personal operations such as user-attachment uploads.
4
4
 
5
5
  ## 1) Create/install GitHub App
6
6
 
@@ -153,7 +153,7 @@ Use `additionalUserScopes` only when a human-identity integration flow requires
153
153
 
154
154
  - When either GitHub skill is active, authenticated `gh` and `git` commands cause the runtime to inject GitHub credentials automatically for the current turn.
155
155
  - The plugin classifies GitHub traffic from the forwarded HTTP request. Reads use `installation-read`, while `GET /user` uses `user-read`. Allowlisted App-owned mutations and Git smart-HTTP pushes use repository-scoped `installation-write`. User-attachment uploads to `uploads.github.com/user-attachments/assets` use `user-write`. Unknown REST writes and GraphQL mutations are denied.
156
- - `user-read` and explicitly human `user-write` operations require the actor, or an explicitly delegated user subject, to authorize the GitHub App through the private OAuth flow. Junior-owned issue, pull request, and branch operations do not fall back to user OAuth.
156
+ - `user-read` and explicitly human `user-write` operations require the actor, or an explicitly delegated user subject, to authorize the GitHub App through the private OAuth flow. Junior-owned issue, pull request, review, inline review comment, and branch operations do not fall back to user OAuth.
157
157
  - Headless resource-event turns use the `resource-event` system actor and may receive the same repository-scoped installation grants. This lets Junior respond to subscribed pull request events by committing and pushing fixes without inheriting a subscriber's OAuth credential.
158
158
  - Git commits use Junior as author and committer. Resolvable human run actors are credited once with `Co-Authored-By` trailers.
159
159
  - Issued credentials are reused only within the current turn, credential leases are cached by plugin grant and repository lease scope, and upstream 403 permission denials clear the cached lease before the next retry.
@@ -182,7 +182,7 @@ new resources through the typed tools, for example
182
182
  Use `gh` for the allowlisted lifecycle operations described by the skill.
183
183
 
184
184
  `gh` supports either direct `GITHUB_TOKEN` (for local debugging) or sandbox-level header injection.
185
- The plugin uses installation credentials for read-only GitHub traffic, workflow dispatches, allowlisted issue and pull request mutations, and Git smart-HTTP pushes. The installed GitHub App permissions still need to cover the operation: actions for workflow dispatches, issues for issue edits/comments/labels, contents for pushes, pull requests for PR mutations, and workflows for workflow-file pushes.
185
+ The plugin uses installation credentials for read-only GitHub traffic, workflow dispatches, allowlisted issue and pull request mutations (including reviews and inline review comments), and Git smart-HTTP pushes. The installed GitHub App permissions still need to cover the operation: actions for workflow dispatches, issues for issue edits/comments/labels, contents for pushes, pull requests for PR mutations/reviews/inline comments, and workflows for workflow-file pushes.
186
186
 
187
187
  Committing and pushing code uses more than one GitHub surface:
188
188
 
@@ -251,7 +251,7 @@ jr-rpc config set github.repo getsentry/junior
251
251
  5. Verify the issue is authored by the GitHub App bot and includes `Requested by` attribution.
252
252
  6. Exercise `github-issues` to update title/body, add/remove labels, and add a comment.
253
253
  7. Push a test branch and exercise `github-code` to create a draft PR using explicit repo targeting and `--head`.
254
- 8. Verify allowlisted mutations are attributed to the GitHub App bot, while a pull request review still requires delegated human authorization.
254
+ 8. Verify allowlisted mutations, including pull request reviews and inline review comments, are attributed to the GitHub App bot. Confirm merge remains unsupported.
255
255
  9. Verify GitHub API calls succeed while this skill is active without writing tokens into sandbox env/files.
256
256
  10. Verify raw token values are never printed in output or logs.
257
257
  11. Check logs for: