Skip to content

test(fixture): adr-migrate stops at 0 lint errors, 50 turns - #611

Merged
aaronsb merged 1 commit into
mainfrom
fix/tier2-adr-migrate-turns
Sep 28, 2026
Merged

aaronsb merged 1 commit into
mainfrom
fix/tier2-adr-migrate-turns

Conversation

@aaronsb

@aaronsb aaronsb commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Summary

Tier 2 run 36436570079 on main (post-#609) failed adr-migrate with error_max_turns at 40. Every corpus check passed: both records declare adr/v1, carry the imported: block from adr import apply, lint with 0 errors, invent no operator statement and keep their sections. The rubric scored 0 of 3 because the session ended before its summary.

The prompt asked to lint "until they are clean", and imported records always warn that they have no ## Summary yet (ADR-306 §4). The prompt now stops at 0 errors and names that warning as expected. max_turns goes from 40 to 50.

adr-consider (2/2) and adr-way (5/5) passed in the same run.

Test plan

  • CI green
  • After merge: gh workflow run live-fixture.yml -f tier2_scenarios=adr-migrate passes

Run 36436570079 on main hit error_max_turns at 40 after the conversion
had already passed every corpus check. The prompt asked for clean lint,
and imported records always warn that they lack a Summary (ADR-306 §4).
@aaronsb
aaronsb merged commit 79dc18a into main Sep 28, 2026
4 checks passed
@aaronsb
aaronsb deleted the fix/tier2-adr-migrate-turns branch September 28, 2026 14:42
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.

1 participant