fix(#190): remove Jira remnants and add issues:write to custom agent guide#209
Open
fullsend-ai-coder[bot] wants to merge 1 commit into
Open
fix(#190): remove Jira remnants and add issues:write to custom agent guide#209fullsend-ai-coder[bot] wants to merge 1 commit into
fullsend-ai-coder[bot] wants to merge 1 commit into
Conversation
…guide Remove the Jira code path from the pre-script example (which left an orphaned elif after PR fullsend-ai#2568 removed Jira support), consolidate permissions into a single block with contents:read, id-token:write, and issues:write, and drop the now-unused issue_source workflow input.
ReviewFindingsLow
Labels: PR fixes documentation bugs in user guide Previous runReviewFindingsMedium
Low
|
Owner
|
/fs-review |
| permissions: | ||
| contents: read | ||
| id-token: write | ||
| issues: write |
There was a problem hiding this comment.
[low] permission-expansion
The diff adds issues: write to the main agent workflow example (Step 7), but the pre-script only uses gh issue view (a read operation) and the example post-script does not write to issues. At most issues: read is needed for the example as written. Granting issues: write in example code that users will copy violates least-privilege.
Suggested fix: Replace issues: write with issues: read in the main workflow example, since only gh issue view is used. Add an inline comment noting that users should upgrade to issues: write if their post-script creates or updates issues.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove the Jira code path from the pre-script example (which left an orphaned elif after PR fullsend-ai#2568 removed Jira support), consolidate permissions into a single block with contents:read, id-token:write, and issues:write, and drop the now-unused issue_source workflow input.
Closes #190
Post-script verification
agent/190-fix-docs-building-custom-agents)f96750babbed5ada406a9ae04e8068449701d9c7..HEAD)