Skip to content

ci: Redline caller pinned to askalf/askalf@35a0ee3 (#62 + #63) - #87

Merged
askalf merged 1 commit into
mainfrom
ci/redline-pin-63
Sep 25, 2026
Merged

askalf merged 1 commit into
mainfrom
ci/redline-pin-63

Conversation

@askalf

@askalf askalf commented Sep 25, 2026

Copy link
Copy Markdown
Owner

What

.github/workflows/redline.yml calls askalf/askalf/.github/workflows/redline-review.yml at a fixed commit. This moves the pin from 63e007e to 35a0ee3, which carries askalf/askalf#62 and #63.

Why

At 18:24Z on 2026-09-25 eight repos ran their first Redline-in-CI job at once and five died with "no review submitted within 30 turns", while dario sat at its concurrency ceiling. The script at 63e007e nudged the model 24 times and paid 25 model calls per run for nothing, and logged nothing about what the replies held. At 35a0ee3 three replies without a tool call end the run, each such reply is logged with its text, an empty reply stays out of the history, and a reply with no content fails at once.

Nothing else changes

Same runner label, same guard, same permissions. The workflow is a pin bump only.

@github-actions github-actions Bot added github_actions Pull requests that update GitHub Actions code size/XS Under 10 hand-written lines labels Sep 25, 2026

@sprayberry-redline sprayberry-redline left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review from the Sprayberry Labs fleet code reviewer.

Reviewed by the gating lane (gating review).

Verdict: approve. Pin bump only: .github/workflows/redline.yml:23 moves the reusable workflow reference from 63e007e... to the full SHA 35a0ee3805b3d0ae6df3a7ca63bfebf35a76c6af and updates the trailing comment to askalf/askalf#63. Verified the rest of the file is unchanged: read-only permissions (contents/pull-requests read), pull_request trigger, same-repo and non-draft guard on line 21, per-PR concurrency group, and the runner-label: redline input. The reference is a full commit SHA, so the callee remains immutable. PR text is plain and factual with no AI attribution or secrets. The callee workflow at 35a0ee3 lives in askalf/askalf and is not in this checkout, so its described behaviour changes were not independently read; the caller's security posture does not depend on them.

@askalf
askalf merged commit bef7399 into main Sep 25, 2026
14 checks passed
@askalf
askalf deleted the ci/redline-pin-63 branch September 25, 2026 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code size/XS Under 10 hand-written lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants