Skip to content

fix: enable bash tool and add GraphQL pagination in firewall-issue-dispatcher#1905

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-review-comments-implementation
Closed

fix: enable bash tool and add GraphQL pagination in firewall-issue-dispatcher#1905
Copilot wants to merge 2 commits intomainfrom
copilot/fix-review-comments-implementation

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 11, 2026

The optimized firewall-issue-dispatcher workflow instructs the agent to run gh api graphql but never grants execution capability, and the query silently truncates results at 50 issues / 100 comments with no pagination.

Changes

  • Enable constrained bash tool — adds bash: ["gh api graphql:*"] to the frontmatter so the agent can actually execute the batch fetch step
  • Add cursor pagination — query now accepts $endCursor: String, passes after: $endCursor to the issues connection, and returns pageInfo { hasNextPage endCursor }; command switches to gh api graphql --paginate to fetch all pages automatically
- gh api graphql -f query='
-   query {
-     repository(owner: "github", name: "gh-aw") {
-       issues(labels: ["awf"], states: [OPEN], first: 50) {
+ gh api graphql --paginate -f query='
+   query($endCursor: String) {
+     repository(owner: "github", name: "gh-aw") {
+       issues(labels: ["awf"], states: [OPEN], first: 50, after: $endCursor) {
+         pageInfo { hasNextPage endCursor }

Lock file recompiled and post-processed after frontmatter changes.

Copilot AI changed the title [WIP] Fix code based on review comments fix: enable bash tool and add GraphQL pagination in firewall-issue-dispatcher Apr 11, 2026
Copilot AI requested a review from lpcox April 11, 2026 14:55
@lpcox lpcox closed this Apr 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants