Skip to content

Commit 3c5e4ca

Browse files
StevenTCramerclaude
andcommitted
chore(kanban): 050-010 done — region backfill complete (160 files, build green)
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 98c6318 commit 3c5e4ca

1 file changed

Lines changed: 21 additions & 10 deletions

File tree

kanban/in-progress/050-010-add-region-purpose-and-region-design-to-all-migrated-source-files.md renamed to kanban/done/050-010-add-region-purpose-and-region-design-to-all-migrated-source-files.md

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -29,19 +29,30 @@ All .cs files under `source/`:
2929
## Checklist
3030

3131
### Phase 1: Inventory
32-
- [ ] List all .cs files under source/; exclude trivial ones (assembly markers, global-usings,
33-
<10 lines with no logic, generated code) and the 4 already done.
34-
- [ ] Group by project so each pass has whole-project context.
32+
- [x] Inventory: 168 non-trivial candidates (excluded markers, global-usings, generated, <12-line
33+
files, and the 4 already done).
34+
- [x] Grouped path-ordered into 14 slices of ~12 files for project-affine context.
3535

36-
### Phase 2: Add Regions (per project, with cross-file context)
37-
- [ ] `#region Purpose` — one line — and `#region Design` — the why: decisions, constraints,
38-
rationale (5–10 lines max) — at the top of each file, before the namespace.
39-
- [ ] `#region Open Questions` only where a genuine question surfaced while reading.
40-
- [ ] No temporal language; say what the code can't; skip anything that would restate the name.
36+
### Phase 2: Add Regions
37+
- [x] 14-agent workflow backfilled **160 files** (commit `98c63180`; 1,861 insertions, **0 deletions**
38+
— verified insertion-only via `git diff --numstat`).
39+
- [x] **8 skipped with recorded reasons**: dead commented-out entities (`category.cs`, `product.cs`),
40+
`#if false` body (`user-claims-base.cs`), sub-threshold stubs (`greeter-service.cs`,
41+
`SideNavigationLink.razor.cs`, `assembly-extensions.cs`), and files whose single design fact
42+
was already an inline comment (`constants.cs`, `java-script-interop-constants.cs`).
43+
- [x] Rules enforced in agent prompts: why-not-what, no temporal language, insertion-only, ≤14 lines.
4144

4245
### Phase 3: Verify
43-
- [ ] `dev build` green (regions must not break compilation, incl. source-generated partials).
44-
- [ ] Sample review for quality: regions must carry the *why*, not paraphrase the class name.
46+
- [x] `dev build` green (0/0) — regions coexist with source-generated partials.
47+
- [x] Random spot-checks carry genuine *why* (e.g. service-uri-provider: browser can't read Aspire
48+
env vars → server exposes /service-discovery; policy-registration: explicit always-true
49+
Anonymous policy avoids special-casing).
50+
51+
## Result
52+
53+
Region coverage went from 4/304 to 164/304 `.cs` files, with the remainder deliberately trivial.
54+
Freshness from here on is the `AGENTS.md` maintenance rule; presence enforcement via an analyzer
55+
remains a candidate follow-up (see Notes).
4556

4657
## Notes
4758

0 commit comments

Comments
 (0)