Fix github-discussion-query payload scaling by eliminating jq argument growth

[Copilot]

github-discussion-query silently degraded coverage for higher --limit values because discussion payloads were passed through jq via shell arguments/here-strings, eventually hitting OS argument-length limits. This capped effective scan depth and caused daily report cross-check gaps.

• Pagination/data path hardening

  • Replaced in-memory JSON accumulation (--argjson all "$OUTPUT") with temp-file-backed merges (jq -s) so payload size no longer depends on shell argument limits.
  • Kept existing since pagination semantics and early-stop behavior while removing large argument transport from the hot path.

• Filter/output execution path

  • Switched jq filter execution from here-string input to file input:
    • before: jq "$JQ_FILTER" <<< "$OUTPUT"
    • after: jq "$JQ_FILTER" "$OUTPUT_FILE"
  • Updated item count and byte-size derivation to read from file-backed output while preserving prior metadata behavior.

• Temp-file lifecycle safety

  • Added explicit temp-file tracking and trap cleanup for all intermediates used during page extraction, merge, and final projection.

# Before (argument-growth prone)
OUTPUT=$(jq -cn --argjson all "$OUTPUT" --argjson page "$PAGE_NODES" '$all + $page')
jq "$JQ_FILTER" <<< "$OUTPUT"

# After (payload-size resilient)
jq -s '.[0] + .[1]' "$OUTPUT_FILE" "$PAGE_NODES_FILE" > "$MERGED_OUTPUT_FILE"
jq "$JQ_FILTER" "$OUTPUT_FILE"

• Fixes [deep-report] Fix "Argument list too long" limit (~12 items) in github-discussion-query mcp-script tool #55556

[github-actions]

Thanks for tackling the discussion-query pagination limit! This fix — switching from <<< here-strings to streaming via temp files — addresses the real bottleneck blocking the Daily Regulatory Report from scanning beyond ~12 items.

Since this is still a draft, here's what would help get it across the finish line:

• Add tests — The PR checklist notes "[ ] Add or update focused tests/fixtures for high-limit discussion queries". Specifically, add fixtures for queries with --limit values well above 12 (e.g., 50+) to confirm the argument-length issue is truly resolved and won't regress.
• Complete final checks — The checklist also flags "[ ] Run final checks (code review, CodeQL, secrets scan)". Once tests pass, push those final validation runs.

The core refactor looks solid — swapping to temp files + jq -s file-to-file operations is the right approach to sidestep shell argument limits entirely.

Once the tests are in place and the checklist is complete, this should be ready for merge!

Generated by ✅ Contribution Check · copilot · auto · 46.5 AIC · ⌖ 6.86 AIC · ⊞ 9.3K · ◷

[Copilot]

Pull request overview

Reworks github-discussion-query to use temporary files, avoiding shell argument-length limits for large result sets.

Changes:

• Uses file-backed page merging and jq filtering.
• Adds automatic temporary-file cleanup.
• Regenerates all importing workflow lock files.

Show a summary per file

File	Description
.github/workflows/shared/github-queries-mcp-script.md	Implements scalable file-backed discussion processing.
.github/workflows/daily-regulatory.lock.yml	Regenerates the imported query script.
.github/workflows/daily-performance-summary.lock.yml	Regenerates the imported query script.
.github/workflows/smoke-copilot.lock.yml	Regenerates the imported query script.
.github/workflows/smoke-copilot-arm.lock.yml	Regenerates the imported query script.
.github/workflows/smoke-copilot-aoai-entra.lock.yml	Regenerates the imported query script.
.github/workflows/smoke-copilot-aoai-apikey.lock.yml	Regenerates the imported query script.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 7/8 changed files
• Comments generated: 0
• Review effort level: Balanced

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.87.5