docs(claude.md): refresh Repository status with ADR-0022/0023/0025 shipments #212

Merged
julien merged 1 commits from docs/claude-md-refresh-may-2026 into main 2026-05-24 01:32:20 +02:00
Owner

Summary

CLAUDE.md's Repository status section drifted from main while three ADRs landed back-to-back. The roadmap block still listed ADR-0022 (docs static site) and ADR-0023 (charts lib + audit dashboards) as upcoming chantiers even though both shipped, and the new libs/shared/auth/ lib introduced by ADR-0025 was not in the lib-roots list.

This PR refreshes the section to reflect what main actually carries today. No code touched.

What lands

File Change
CLAUDE.md Repository status section rewritten: lib roots updated, three new "Shipped on main" bullets (ADR-0022 / 0023 / 0025), roadmap pruned and rebalanced around what is genuinely outstanding.

Notes for the reviewer

  • Lib roots. Was: "four lib roots (libs/feature/, libs/shared/state, libs/shared/tokens, libs/shared/ui, libs/shared/util)" — five entries called "four". Now: "seven lib roots" listing libs/feature/auth, libs/shared/auth, libs/shared/charts, libs/shared/state, libs/shared/tokens, libs/shared/ui, libs/shared/util. The count is correct and the new libs from ADR-0023 and ADR-0025 are explicit.

  • CI line now mentions the catalogue-drift gate alongside the standard format:check / lint / test / build quartet — readers grepping for "what CI runs" land on the actual gate set.

  • AI relay entry merged with its live consumer. The previous wording said "Live consumer (chatbot widget on portal-shell) and the proto-drift CI gate ship next" — both half-true: chatbot widget shipped (under apps/portal-shell/src/app/features/chatbot/), proto-drift gate did not. Entry now says chatbot is live and moves the proto-drift gate to the roadmap.

  • Phase-3a admin app phrasing updated. Previous wording: "business modules (CMS, menu management, user list, audit log viewer) not yet implemented". Two of those four exist on main today: /admin/users (user-list reader via admin-users-reader.service.ts) and /admin/audit (audit-log viewer + statistics + integrated charts). Entry now names them and notes that only CMS + menu management remain.

  • Roadmap entry split for the guards. The old line bundled @RequireMfa() and @RequireAdmin() as both "designed-in, awaiting first consumer route". @RequireAdmin is wired into every /api/admin/* controller today; only @RequireMfa is still consumer-less. Roadmap line narrowed to mention only @RequireMfa.

  • ADR-0026 referenced as (proposed). The link target is a placeholder (#) because the ADR file does not exist yet — kept lowercase-light so the link does not render as a dead anchor in editors. Will become a real link in the same PR that drafts ADR-0026.

Test plan

  • pnpm exec prettier --check CLAUDE.md — clean.
  • Manual cross-check against the workspace state:
    • find libs/shared -maxdepth 1 -type d matches the listed lib roots.
    • ls docs/.vitepress/ confirms VitePress is wired.
    • ls libs/shared/charts/src/lib/ confirms the three chart components exist.
    • grep -l BarChart\\|DonutChart\\|StackedBarChart apps/portal-admin/src/app/pages/audit/ confirms the audit-page integration.
    • ls scripts/check-catalogue-drift* confirms the drift gate exists.
    • grep -rn @RequireAdmin apps/portal-bff/src/admin/ confirms the admin gate is wired today.

No CI gates affected — this is a doc-only change.

## Summary `CLAUDE.md`'s **Repository status** section drifted from `main` while three ADRs landed back-to-back. The roadmap block still listed ADR-0022 (docs static site) and ADR-0023 (charts lib + audit dashboards) as upcoming chantiers even though both shipped, and the new `libs/shared/auth/` lib introduced by ADR-0025 was not in the lib-roots list. This PR refreshes the section to reflect what `main` actually carries today. No code touched. ## What lands | File | Change | | --- | --- | | `CLAUDE.md` | Repository status section rewritten: lib roots updated, three new "Shipped on `main`" bullets (ADR-0022 / 0023 / 0025), roadmap pruned and rebalanced around what is genuinely outstanding. | ## Notes for the reviewer - **Lib roots.** Was: "four lib roots (`libs/feature/`, `libs/shared/state`, `libs/shared/tokens`, `libs/shared/ui`, `libs/shared/util`)" — five entries called "four". Now: "seven lib roots" listing `libs/feature/auth`, `libs/shared/auth`, `libs/shared/charts`, `libs/shared/state`, `libs/shared/tokens`, `libs/shared/ui`, `libs/shared/util`. The count is correct and the new libs from ADR-0023 and ADR-0025 are explicit. - **CI line** now mentions the catalogue-drift gate alongside the standard `format:check / lint / test / build` quartet — readers grepping for "what CI runs" land on the actual gate set. - **AI relay entry merged with its live consumer.** The previous wording said "Live consumer (chatbot widget on `portal-shell`) and the proto-drift CI gate ship next" — both half-true: chatbot widget shipped (under `apps/portal-shell/src/app/features/chatbot/`), proto-drift gate did not. Entry now says chatbot is live and moves the proto-drift gate to the roadmap. - **Phase-3a admin app phrasing updated.** Previous wording: "business modules (CMS, menu management, user list, audit log viewer) not yet implemented". Two of those four exist on `main` today: `/admin/users` (user-list reader via `admin-users-reader.service.ts`) and `/admin/audit` (audit-log viewer + statistics + integrated charts). Entry now names them and notes that only CMS + menu management remain. - **Roadmap entry split for the guards.** The old line bundled `@RequireMfa()` and `@RequireAdmin()` as both "designed-in, awaiting first consumer route". `@RequireAdmin` is wired into every `/api/admin/*` controller today; only `@RequireMfa` is still consumer-less. Roadmap line narrowed to mention only `@RequireMfa`. - **ADR-0026 referenced as `(proposed)`.** The link target is a placeholder `(#)` because the ADR file does not exist yet — kept lowercase-light so the link does not render as a dead anchor in editors. Will become a real link in the same PR that drafts ADR-0026. ## Test plan - [x] `pnpm exec prettier --check CLAUDE.md` — clean. - [x] Manual cross-check against the workspace state: - `find libs/shared -maxdepth 1 -type d` matches the listed lib roots. - `ls docs/.vitepress/` confirms VitePress is wired. - `ls libs/shared/charts/src/lib/` confirms the three chart components exist. - `grep -l BarChart\\|DonutChart\\|StackedBarChart apps/portal-admin/src/app/pages/audit/` confirms the audit-page integration. - `ls scripts/check-catalogue-drift*` confirms the drift gate exists. - `grep -rn @RequireAdmin apps/portal-bff/src/admin/` confirms the admin gate is wired today. No CI gates affected — this is a doc-only change.
julien added 1 commit 2026-05-24 01:30:32 +02:00
docs(claude.md): refresh Repository status with ADR-0022/0023/0025 shipments
CI / scan (pull_request) Successful in 3m26s
CI / commits (pull_request) Successful in 3m26s
CI / check (pull_request) Successful in 3m43s
CI / a11y (pull_request) Successful in 3m42s
CI / perf (pull_request) Successful in 7m13s
baa73fd33f
CLAUDE.md drifted from main while three ADRs landed back-to-back.
The roadmap section still listed docs-static-site, charts lib +
audit-page dashboards as "next" even though both shipped, and the
new shared-auth lib was not in the lib roots list.

Changes scoped to the Repository status section:

- Lib roots: was four (listed five), now seven — adds
  libs/feature/auth (already existed), libs/shared/auth (new with
  ADR-0025), libs/shared/charts (new with ADR-0023).
- CI line mentions the catalogue-drift gate alongside
  format:check / lint / test / build.
- New "Shipped on main" entries:
  - Docs static site (ADR-0022): VitePress + Mermaid + workflow.
  - Charts lib + audit-page dashboards (ADR-0023): 3 components
    integrated on /admin/audit.
  - Authorization model + guards (ADR-0025): catalogues +
    PrincipalBuilder + 3 new decorators + drift gate.
- Phase-3a updated: user-list and audit-log viewer modules now
  exist; only CMS + menu management remain in v1 scope.
- AI relay entry merged with its live consumer (chatbot widget in
  portal-shell); proto-drift CI gate moved to the roadmap.
- Roadmap pruned of the two shipped chantiers; added entries for
  the @RequireScope Prisma-backed resolver (depends on the
  proposed ADR-0026), the proto-drift gate, and the admin-app
  CMS/menu pair.
- @RequireMfa / @RequireAdmin entry split: @RequireAdmin is on
  main (used by the admin app already); only @RequireMfa
  consumer routes remain on the roadmap.

No code touched.
julien merged commit ca3963b2e3 into main 2026-05-24 01:32:20 +02:00
julien deleted branch docs/claude-md-refresh-may-2026 2026-05-24 01:32:22 +02:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: julien/apf_portal#212