fix(ci): read run jobs 50 at a time so the queue evidence read succeeds by devantler · Pull Request #653 · devantler-tech/.github

devantler · 2026-10-19T10:23:11Z

🤖 Generated by the Agentic Engineer

Why

The required check on the main branch has been red since this morning even though every test job passed. Its last step reads the list of jobs in the run, and GitHub fails that read every time when it asks for 100 jobs at once on the full test catalogue, so the check gives up and reports failure.

What

The job list is now read 50 jobs at a time, which GitHub answers reliably. The same change is made in the Go cache check, which reads the same list. The check still refuses to pass on an incomplete list.

No issue: trivial fix.

Pages of 100 jobs on the full catalogue run fail with HTTP 502 on every attempt, which turns a fully green run red at the summary step. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

devantler · 2026-10-19T10:23:21Z

🤖 Generated by the Agentic Engineer

@coderabbitai review

github-code-quality · 2026-10-19T10:24:10Z

Code coverage overview

Languages: Go, C#

Go / code-coverage/test

main 8f02981	claude/ci-queue-evid... 1f58146	+/-
50%	50%	0%

C# / code-coverage/dotnet

main 8f02981	claude/ci-queue-evid... 1f58146	+/-
100%	100%	0%

Go / code-coverage/go

main 8f02981	claude/ci-queue-evid... 1f58146	+/-
50%	50%	0%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.

_{Updated October 19, 2026 10:35 UTC}

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

devantler · 2026-10-19T10:30:52Z

🤖 Generated by the Agentic Engineer

@coderabbitai review

coderabbitai · 2026-10-19T10:34:32Z

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

coderabbitai · 2026-10-19T10:38:07Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 4bffc397-3766-42f6-b1f0-9dd327879d66

📥 Commits

Reviewing files that changed from the base of the PR and between 8f02981 and 1f58146.

📒 Files selected for processing: 7

.github/scripts/observe-go-cache.sh
.github/tests/catalogue-credentials/main.go
.github/tests/test-ci-selection.sh
.github/tests/test-go-cache-native-observer.sh
.github/tests/test-native-queue-gate.sh
.github/tests/test-queue-observer.sh
.github/workflows/ci.yaml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout: 14

GitHub Check: [Test] Validate Go Project - Isolated PostgreSQL / 🧹 Lint - mega-linter
GitHub Check: [Test] Validate Go Project - Signed Fixes Suppressed / 📊 Code Coverage
GitHub Check: [Test] Validate Go Project - Signed Fixes Suppressed / 🏗️ Build
GitHub Check: [Test] Validate Go Project - Default-Branch Test Opted Out / 📊 Code Coverage
GitHub Check: [Test] Lint - Gate Pins Self-Tested Image
GitHub Check: [Test] Lint - Root Configuration
GitHub Check: [Test] Enable Auto-Merge - Queue Slot 3
GitHub Check: [Test] Validate Go Project - Gate Lockstep
GitHub Check: [Test] Govulncheck - A Timed-Out Scan Is Retried Once
GitHub Check: [Test] Create Release - Least Privilege
GitHub Check: [Test] Delete Workflow Runs - Specific Pattern / Delete workflow runs
GitHub Check: [Test] Delete Workflow Runs - All Workflows / Delete workflow runs
GitHub Check: [Test] Delete Workflow Runs - Minimal Parameters / Delete workflow runs
GitHub Check: lint-ci-coverage-parity

🧰 Additional context used

📚 Code guidelines: 3

actions/CONTRIBUTING.md — auto-discovered
.github/agent-guides/catalogue.md — auto-discovered
CONTRIBUTING.md — auto-discovered

📓 Path-based instructions: 4

Applicability reference: AGENTS.md: # AGENTS.md — devantler-tech/.github (GitHub-as-code) Full detail in [actions/CONTRIBUTING.md](actions/CONTRIBUTING.md).

📄 CodeRabbit inference engine (actions/CONTRIBUTING.md)

Files:

.github/tests/test-queue-observer.sh
.github/tests/test-native-queue-gate.sh
.github/tests/test-ci-selection.sh
.github/tests/test-go-cache-native-observer.sh

Applicability reference: AGENTS.md: # AGENTS.md — devantler-tech/.github (GitHub-as-code) Full detail in [actions/CONTRIBUTING.md](actions/CONTRIBUTING.md).

📄 CodeRabbit inference engine (actions/CONTRIBUTING.md)

Files:

.github/workflows/ci.yaml

Applicability reference: AGENTS.md: # AGENTS.md — devantler-tech/.github (GitHub-as-code) | Guide | Read before | |---|---| | [Catalogue layout and references](.github/agent-guides/catalogue.md) | Adding or moving components, self-reference...

📄 CodeRabbit inference engine (.github/agent-guides/catalogue.md)

Files:

.github/workflows/ci.yaml

Applicability reference: AGENTS.md: # AGENTS.md — devantler-tech/.github (GitHub-as-code) Three things share one repo: **The org's public profile** — `profile/README.md` (rendered on the org page), plus the org-wide commun...

📄 CodeRabbit inference engine (CONTRIBUTING.md)

Files:

.github/tests/test-queue-observer.sh
.github/tests/test-native-queue-gate.sh
.github/tests/test-ci-selection.sh
.github/scripts/observe-go-cache.sh
.github/tests/catalogue-credentials/main.go
.github/tests/test-go-cache-native-observer.sh
.github/workflows/ci.yaml

🔇 Additional comments: 7

.github/tests/catalogue-credentials/main.go: 1

1183-1183: 🎯 Functional Correctness

The digest matches the JOB_RESULTS reducer in .github/workflows/ci.yaml. No change is required.

.github/scripts/observe-go-cache.sh: 1

148-148: LGTM!

.github/workflows/ci.yaml: 1

6645-6646: LGTM!

Also applies to: 6648-6648

.github/tests/test-ci-selection.sh: 1

15-15: LGTM!

.github/tests/test-go-cache-native-observer.sh: 1

105-105: LGTM!

.github/tests/test-native-queue-gate.sh: 1

13-13: LGTM!

.github/tests/test-queue-observer.sh: 1

14-14: LGTM!

📝 Walkthrough

The observer and CI workflow now request 50 jobs per page instead of 100. The CI workflow comment records that 100-job pages returned HTTP 502 on full-catalogue runs, while 50-job pages succeeded. Related test mocks now expect 50 jobs per page. The wiring check also pins a new SHA-256 digest for the required JOB_RESULTS reducer.

Priority: ➖ Normal

Merge Risk: ⚪ Minimal · up to 1f581

The smaller job pages address the reported read failure without an identified regression; the change is ready for normal checks.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 6 files. (1 skipped: 1…	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: reducing the CI job-page size to 50 so queue evidence reads succeed.
Description check		The description explains the CI failure, the change to 50-job pages, the affected checks, and the retained incomplete-list validation. It is directly related to the changeset.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 6 files. (1 skipped: 1 unsupported.)

Resolution

Write docstrings for the functions missing them to satisfy the coverage threshold.

Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

devantler · 2026-10-19T10:39:59Z

🤖 Generated by the Agentic Engineer

Readiness at `1f58146eadc24116ccc2d74cbf009a9f296a0171`

Tested: every check is green at this head, including CI - Required Checks (its queue-evidence read now runs at 50 jobs per page), [Test] Native Go Cache Policy, and lint-ci-coverage-parity with the re-pinned reducer fingerprint. Locally the four affected stub tests and both catalogue-credential tests pass.
Reviewed: CodeRabbit reviewed this head with no actionable comments.
Tried as a user: reproduced the failure against the real red main run 39225588155 (attempt 2, 245 jobs): reading its jobs 100 per page failed with HTTP 502 on every try, while 50 per page (5 of 5 pages) and 30 per page (9 of 9) all succeeded. The same 502 is what failed [Test] Native Go Cache Policy on .github#648. After merge, main's next push run is the confirmation.

coderabbitai · 2026-10-19T10:43:40Z

⚠️ Action not completed

Pull request is closed.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

fix(ci): read run jobs 50 at a time so the queue evidence read succeeds

cf4dd61

Pages of 100 jobs on the full catalogue run fail with HTTP 502 on every attempt, which turns a fully green run red at the summary step. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

test(ci): pin the reviewed reducer after the job page size change

1f58146

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

devantler marked this pull request as ready for review October 19, 2026 10:40

devantler merged commit 001dc6d into main Oct 19, 2026
284 checks passed

devantler deleted the claude/ci-queue-evidence-page-size branch October 19, 2026 10:41

devantler mentioned this pull request Oct 19, 2026

Recover catalogue CI from primary API quota exhaustion #596

Open

5 tasks

Navigation Menu

Repository navigation

fix(ci): read run jobs 50 at a time so the queue evidence read succeeds - #653

fix(ci): read run jobs 50 at a time so the queue evidence read succeeds#653
devantler merged 2 commits into
mainfrom
claude/ci-queue-evidence-page-size

devantler commented Oct 19, 2026

Uh oh!

devantler commented Oct 19, 2026

Uh oh!

github-code-quality Bot commented Oct 19, 2026 •

edited

Loading

Uh oh!

devantler commented Oct 19, 2026

Uh oh!

coderabbitai Bot commented Oct 19, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Oct 19, 2026

Uh oh!

devantler commented Oct 19, 2026

Uh oh!

Uh oh!

coderabbitai Bot commented Oct 19, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

devantler commented Oct 19, 2026

Why

What

Uh oh!

devantler commented Oct 19, 2026

Uh oh!

github-code-quality Bot commented Oct 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code coverage overview

Go / code-coverage/test

C# / code-coverage/dotnet

Go / code-coverage/go

Uh oh!

devantler commented Oct 19, 2026

Uh oh!

coderabbitai Bot commented Oct 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Oct 19, 2026

Uh oh!

devantler commented Oct 19, 2026

Readiness at 1f58146eadc24116ccc2d74cbf009a9f296a0171

Uh oh!

Uh oh!

coderabbitai Bot commented Oct 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

github-code-quality Bot commented Oct 19, 2026 •

edited

Loading

coderabbitai Bot commented Oct 19, 2026 •

edited

Loading

Readiness at `1f58146eadc24116ccc2d74cbf009a9f296a0171`

coderabbitai Bot commented Oct 19, 2026 •

edited

Loading