Skip to content

Commit 494bb71

Browse files
committed
docs: sync Bundle B evidence and Week 7 plan status with landed work
1 parent 061df44 commit 494bb71

8 files changed

Lines changed: 1209 additions & 288 deletions

File tree

design/20260319_agent-terminal-v1/15-week-7-plan.md

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,12 +15,16 @@ Week 7 is about ratifying the shipped v1 surface so the docs, examples, JSON env
1515

1616
## Status update (2026-03-25)
1717

18-
This Week 7 plan now reflects two landed phases of the contract-closeout work:
18+
This Week 7 plan now reflects the full landed Phase 1–5 contract-closeout sequence plus the follow-up review-hardening pass:
1919

2020
- **Phase 1 — contract audit and doc ratification:** the high-value examples in `02-cli-contract.md` now match the shipped CLI syntax/result shapes for `create`, `list`, `type`, `paste`, `send-keys`, `snapshot`, `screenshot`, `resize`, and `destroy`.
2121
- **Phase 2 — targeted code/schema alignment:** the shipped `send-keys` result now exposes `accepted`, `bytesWritten`, and `seq`, and the shipped `destroy` result now exposes `{ sessionId, destroyed }`.
22+
- **Phase 3 — golden-envelope expansion:** `test/unit/commands/golden-envelopes.test.ts` now covers 10 command surfaces with valid/invalid/extra-field locks for the representative public JSON contracts.
23+
- **Phase 4 — proof bundles and review normalization:** four Week 7 proof bundles now exist with generated review pages, and the browser-verification pass added reviewer screenshots for those pages.
24+
- **Phase 5 — design/doc synchronization:** the CLI contract doc is ratified, the Week 7 proof/docs set is in sync, and the gap-tracker/status docs now describe the same shipped surface.
25+
- **Review hardening:** defensive copies, type import alignment, error-path tests, and tightened assertions landed in follow-up commits, bringing the suite to 510 tests total.
2226

23-
The main open Week 7 carry-over is now proof-bundle completeness/review normalization. Broader platform/runtime work remains intentionally future scope and is tracked separately in [`../WEEK2-GAPS.md`](../WEEK2-GAPS.md).
27+
The main open Week 7 carry-over is now only the lower-priority "should lock if time permits" surfaces (`gc`, `record export`, and `doctor`) plus broader platform/runtime work tracked separately in [`../WEEK2-GAPS.md`](../WEEK2-GAPS.md).
2428

2529
See [14-week-6-status.md](./14-week-6-status.md), [02-cli-contract.md](./02-cli-contract.md), [`../WEEK2-GAPS.md`](../WEEK2-GAPS.md), and this file for the current execution focus.
2630

@@ -49,9 +53,9 @@ Week 7 is done only when every required checkbox below is complete.
4953

5054
- [x] Every high-value example in `02-cli-contract.md` is either implemented as written or explicitly annotated as historical / aspirational / future scope.
5155
- [x] The remaining syntax/result-shape mismatches for `create`, `list`, `type`, `paste`, `send-keys`, `snapshot`, `screenshot`, and `destroy` are closed or explicitly ratified.
52-
- [ ] Golden-envelope or equivalent contract tests cover the representative public JSON surfaces most likely to drift.
53-
- [ ] The Week 6 proof bundles, and any critical carried-forward Week 4/5 proof gaps, meet the documented review-bundle minimum or the docs are explicitly updated to describe the lighter accepted standard.
54-
- [ ] The top-level design entrypoint, validation docs, Week 6 status record, and gap tracker all describe the same current reality.
56+
- [x] Golden-envelope or equivalent contract tests cover the representative public JSON surfaces most likely to drift.
57+
- [x] The Week 6 proof bundles, and any critical carried-forward Week 4/5 proof gaps, meet the documented review-bundle minimum or the docs are explicitly updated to describe the lighter accepted standard.
58+
- [x] The top-level design entrypoint, validation docs, Week 6 status record, and gap tracker all describe the same current reality.
5559
- [ ] The remaining post-Week-7 gap is clearly future-scope platform/runtime work rather than unfinished contract or proof closure.
5660

5761
## Scope boundaries
@@ -112,7 +116,7 @@ Make the public CLI examples and the shipped command surface tell the same truth
112116

113117
## Workstream B — contract locks and representative JSON coverage
114118

115-
**Status (2026-03-25): Partially complete.** Phase 2 targeted code/schema changes landed for `send-keys` result enrichment and `destroy` result-shape alignment, but the broader representative-envelope lock work remains open.
119+
**Status (2026-03-25): Complete.** `19a7223` expanded the golden-envelope suite across 10 representative command surfaces, and follow-up hardening in `652f657` and `061df44` aligned result-type imports, hardened send-keys handling with defensive copies, and tightened the contract/error-path assertions.
116120

117121
### Goal
118122

@@ -133,7 +137,7 @@ Prevent the ratified contract from drifting again.
133137

134138
## Workstream C — proof-bundle completeness and review normalization
135139

136-
**Status (2026-03-25): Open.** The Week 6 bundles still need the remaining review-surface normalization and screenshot/video completeness work described below.
140+
**Status (2026-03-25): Complete.** The four Week 7 proof bundles are checked in, their review pages have been regenerated, and the browser-verification pass in `4fe94b3` added reviewer screenshots for the generated review surfaces.
137141

138142
### Goal
139143

@@ -158,7 +162,7 @@ Bring the checked-in proof bundles back in line with the validation doc's stated
158162

159163
## Workstream D — design and gap-tracker synchronization
160164

161-
**Status (2026-03-25): Mostly complete.** The top-level design entrypoint and `WEEK2-GAPS.md` now reflect Week 7 reality; validation-doc/proof-bundle follow-through still depends on Workstream C.
165+
**Status (2026-03-25): Complete.** The top-level design entrypoint, CLI contract doc, validation/proof docs, and Week 7 status sync now describe the same shipped reality, and Workstream C's bundle/browser pass closed the last proof-tracking dependency.
162166

163167
### Goal
164168

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
step command exit_code status
2-
01 npx vitest run test/unit/commands/golden-envelopes.test.ts --reporter=verbose 0 ok
3-
02 npx vitest run test/unit/commands/golden-envelopes.test.ts --reporter=json 0 ok
4-
03 cat test/unit/commands/golden-envelopes.test.ts 0 ok
2+
01 ./node_modules/.bin/vitest run test/unit/commands/golden-envelopes.test.ts --reporter=verbose 0 ok
3+
02 ./node_modules/.bin/vitest run test/unit/commands/golden-envelopes.test.ts --reporter=json 0 ok
4+
03 ./node_modules/.bin/prettier --write dogfood/20260325-week7-b-envelope-locks/logs/02-vitest-json.json 0 ok
5+
04 cp test/unit/commands/golden-envelopes.test.ts dogfood/20260325-week7-b-envelope-locks/logs/03-test-source.txt 0 ok
6+
05 ./node_modules/.bin/tsx src/tools/review-bundle.ts dogfood/20260325-week7-b-envelope-locks 0 ok
7+
06 ./node_modules/.bin/prettier --write dogfood/20260325-week7-b-envelope-locks/notes.md dogfood/20260325-week7-b-envelope-locks/index.html 0 ok
8+
07 PATH=/usr/local/nvm/versions/node/v22.19.0/bin:$PATH npm run format:check 0 ok

0 commit comments

Comments
 (0)