mirror of
https://github.com/garrytan/gstack.git
synced 2026-11-03 07:18:03 +01:00
eval(plan-eng-review-artifact): periodic checkpoint resumes at Test review; the fresh full run moves to marathon
A fresh interactive /plan-eng-review cannot reach Test review inside the 600 s periodic budget: startup plus one-at-a-time Scope Challenge questions took 595 s with six questions and no Test review in census 37174266054 and in a local run (one earlier fresh run passed at 498 s). Per the eval-speed policy, the long run moves to the non-blocking marathon lane and the blocking case becomes a focused checkpoint. - plan-eng-review-artifact (periodic, 600 s unchanged): resumes the real interactive skill from a saved review record: the plan, the Scope Challenge result, Sections 1-2 complete, and a Decision ledger rebuilt from that census run's six actual native questions and answers (test/fixtures/plan-eng-artifact-resume-37174266054.json). The live skill runs Section 3, Test review. Assertion unchanged: exactly one new eng-review test plan in the run's own state root, about the dashboard change. Paid trial (local, Claude Code 2.1.284): PASS, 364 s. - plan-eng-review-artifact-full (marathon, PTY_LONG_MS): the fresh start-to-finish run with the same assertion; budget measured beside it. - test/plan-eng-resume.test.ts proves the rebuilt record is a valid resume point and rejects missing, unanswered, edited or duplicated records. - Registry: touchfiles, tiers, kinds, test-name labels, durations, paid file count 115, PTY consumer list; CHANGELOG and TODOS.
This commit is contained in:
1 parent
307ba98623
commit
7249f01464
14 files changed
+405
-43
No files matched your search
@@ -87,6 +87,7 @@ export const CASE_TEST_NAMES: Record<string, string> = {
|
||||
'cso-infra-scope': '/cso --infra finds an attacker-to-credential execution path',
|
||||
'plan-ceo-review-plan-mode': 'first terminal outcome is asked (Step 0 fires before any plan write)',
|
||||
'plan-eng-review-artifact': 'an interactive review writes one QA test plan about the reviewed change',
|
||||
'plan-eng-review-artifact-full': 'a fresh interactive review reaches Test review and writes one QA test plan',
|
||||
};
|
||||
|
||||
export const CASE_KEY_SEPARATOR = '#';
|
||||
|
||||
@@ -172,6 +172,7 @@
|
||||
"test/skill-e2e-plan-design-finding-floor.test.ts": 155538,
|
||||
"test/skill-e2e-plan-design-plan-mode.test.ts": 106608,
|
||||
"test/skill-e2e-plan-devex-peer-comparison-classification.test.ts": 79786,
|
||||
"test/skill-e2e-plan-eng-artifact-full.test.ts": 595009,
|
||||
"test/skill-e2e-plan-eng-artifact.test.ts": 498474,
|
||||
"test/skill-e2e-plan-eng-finding-floor.test.ts": 388666,
|
||||
"test/skill-e2e-plan-eng-multi-finding-batching.test.ts": 597500,
|
||||
|
||||
Reference in new issue
Block a user