diff --git a/docs/project/readable-codebase/PLAN.md b/docs/project/readable-codebase/PLAN.md index ae69abc..5409720 100644 --- a/docs/project/readable-codebase/PLAN.md +++ b/docs/project/readable-codebase/PLAN.md @@ -264,6 +264,21 @@ Also settle the two deviations the refactor left behind: an indirection and adds a barrel-shaped thing to a repo that deliberately has none. - Three of six sections have no story (`beheer-links`, `wat-moet-ik-regelen`, `wat-wilt-u-doen`). Add one only where the section has more than one visual state. +- **The folder name — fix it (RD-36, added after RD-04 shipped).** RD-04 recorded + `registratie/ui/dashboard/` as a stale label and left it alone. It stays stale: the folder is + named after a page that now lives in `overzicht/`, so a reader looking for the dashboard finds + four sections that are not it, and the page is somewhere else. Measured cost: 8 `git mv`s and + **four import lines in one file** (`overzicht.page.ts`). The alias does not change. + + Two names were considered. `overzicht-secties/` says what the four files are — registratie's + sections for the overzicht page — and keeps the Dutch that CLAUDE.md requires of a domain + context. Flattening the files into `registratie/ui/` was rejected: that directory holds one + folder per component, so eight loose files would break its shape. + + Fold in two stale paths the same move left behind, both naming a file that no longer exists: + `docs/reference/feature-flags.md:55` and `.claude/skills/new-ssp/SKILL.md:65` still say + `registratie/ui/dashboard.page`. The flag logic they describe lives in + `overzicht/ui/wat-wilt-u-doen.section.ts` now. --- diff --git a/docs/project/readable-codebase/README.md b/docs/project/readable-codebase/README.md index 407ff0d..20468df 100644 --- a/docs/project/readable-codebase/README.md +++ b/docs/project/readable-codebase/README.md @@ -130,12 +130,18 @@ two. Note that RD-15 exists because 22 abandoned agent worktrees are still on di | RD-33 | CLAUDE.md + `atomic-design.mdx` + the `ui-component` skill | 03, 27, 29 | yes | todo | | RD-34 | _(optional)_ `NO_SUBORGS`/`NO_TABLES` become `RemoteData.Empty` | 11 | | todo | | RD-35 | _(optional, last, alone)_ upload `type:` discriminant to `tag:` | 27 | | todo | +| RD-36 | `ui/dashboard/` → `ui/overzicht-secties/` + 2 stale `dashboard.page` paths | 04 | yes | todo | The ID order already respects every dependency, so it is the recommended running order. **Independent tickets.** RD-15 through RD-19 depend only on RD-01. Pull them forward to fill a short session. Take RD-15 early: it makes every later repository search faster. +**RD-36 is last by number, not by dependency.** It needs only RD-04, it is 8 `git mv`s plus +four import lines, and it collides with nothing else in the table — RD-27's move is confined to +`libs/shared/src/ui/`. Pull it forward into any short session. It is numbered last only because +it was added after RD-04 shipped. + **Two ordering traps the table encodes.** RD-01 must precede RD-30, because RD-01 copies its ticket template out of the directory that RD-30 archives. And four tickets edit the same two documents in different sections — RD-09 rewrites the submit-idiom teaching, while RD-31 and