Skip to content

Commit 1450e2c

Browse files
fix(autofix): carry the raised repair bounds through sibling prose
1 parent 8202c94 commit 1450e2c

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

.github/scripts/run-autofix-review-verification.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ reject_fix() {
119119
if [[ "${preexisting}" == 'true' ]]; then
120120
# NOT retryable: the repair agent is only allowed to amend this round's
121121
# fix, and a failure that exists without the fix is outside that boundary
122-
# by definition — the 18-minute repair budget cannot reach it. The remedy
122+
# by definition — the 45-minute repair budget cannot reach it. The remedy
123123
# is a base update (merge main into the branch), not a repair.
124124
echo "preexisting=true" >> "${GITHUB_OUTPUT}"
125125
elif [[ "${retryable}" == 'true' ]]; then
@@ -1047,7 +1047,7 @@ fi
10471047
# a DEFECT-CLAIM round only when resolved-comments.txt marks a finding
10481048
# resolved-in-code whose thread is Critical-tagged or belongs to a
10491049
# CHANGES_REQUESTED review (matched in rc.json/rv.json). Those rounds get a
1050-
# non-retryable rejection on all-green — the 18-minute repair pass cannot
1050+
# non-retryable rejection on all-green — the 45-minute repair pass cannot
10511051
# make a nonexistent defect reproduce; the next full round re-reads the
10521052
# feedback with the evidence in LAST_REJECTION and can decline or escalate
10531053
# instead. Every OTHER src+test round (a refactor pinning existing

.github/workflows/qwen-autofix.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2173,7 +2173,7 @@ jobs:
21732173
# Keep the predicate as narrow as the job's own `if:` — concurrency is
21742174
# evaluated BEFORE it, so without the do_review conjunct a dispatch with
21752175
# `phase: issue` + `pr_number: N` (route emits pr_number unconditionally)
2176-
# would park this skipped job in that PR's shared slot behind a 300-minute
2176+
# would park this skipped job in that PR's shared slot behind a 330-minute
21772177
# address round, stalling the issue phase that `needs` it.
21782178
concurrency:
21792179
group: "qwen-pr-head-write-${{ needs.route.outputs.do_review == 'true' && needs.route.outputs.pr_number || github.run_id }}"
@@ -2776,7 +2776,7 @@ jobs:
27762776
# The dispatch-pending marker is exempted by context: it is this
27772777
# loop's own StatusContext busy signal (no .workflowName/.name, so
27782778
# it passes the filters above) and its dedicated TTL check above is
2779-
# the authority on it — the 330-minute horizon here would keep a
2779+
# the authority on it — the 360-minute horizon here would keep a
27802780
# stranded marker blocking long past its TTL.
27812781
HAS_PENDING_CHECKS="$(jq -r --arg cut "${PENDING_CUTOFF}" \
27822782
--argjson nonblocking "${NON_BLOCKING_CHECKS}" \
@@ -3888,7 +3888,7 @@ jobs:
38883888
;;
38893889
esac
38903890
# The label routing pins ecs-qwen, but a mis-labelled registration
3891-
# must not silently claim a PAT-bearing 300-minute job — assert the
3891+
# must not silently claim a PAT-bearing 330-minute job — assert the
38923892
# pool by name on the self-hosted branch too.
38933893
if [[ "${RUNNER_ENVIRONMENT}" == 'self-hosted' ]]; then
38943894
case "${RUNNER_NAME}" in

scripts/tests/qwen-autofix-workflow.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4144,7 +4144,7 @@ describe('qwen-autofix workflow', () => {
41444144
// No rollup entries → dispatchable.
41454145
expect(runMarkerCheck([])).toBe('pass');
41464146

4147-
// A stranded marker must NOT keep blocking through the 330-minute
4147+
// A stranded marker must NOT keep blocking through the 360-minute
41484148
// HAS_PENDING_CHECKS gate after its TTL expired: replay the gate's jq
41494149
// over fixture rollups.
41504150
const pendingGate = reviewScanJob.match(
@@ -4194,7 +4194,7 @@ describe('qwen-autofix workflow', () => {
41944194
checkRun('build', 'IN_PROGRESS', '2026-08-17T07:50:00Z'),
41954195
]),
41964196
).toBe('true');
4197-
// ...a check stuck past the 330-minute horizon is aged out...
4197+
// ...a check stuck past the 360-minute horizon is aged out...
41984198
expect(
41994199
runPendingGate([
42004200
checkRun('build', 'IN_PROGRESS', '2026-08-17T01:00:00Z'),

0 commit comments

Comments
 (0)