docs(front): audit the front against TanStack Router, Query and Form docs #533

Merged

root merged 3 commits from claude/nestjs-best-practices-audit-716d79 into main

2026-10-15 13:06:05 +00:00

root commented

2026-10-15 12:57:55 +00:00

(Migrated from gitlab.core-project.space)

Summary

A read-only audit of apps/front against the TanStack Router, Query and Form docs for the installed versions, community practice, and the front's own rules. It records the owner's decisions and is the source the specs #407, #408, #409 and #410 cite.

 docs/research/
 ├── gitlab-poc.md
+└── front-audit.md   # findings per objective, rules challenged, migration plan, decisions, sources

front-audit.md
  1. Summary                    ten points, ranked by gain over effort
  2. Findings                   Router + Query, Form (v1, then v2 alpha), simplification with file counts
  3. Rules challenged           each front lint rule, review rule and ADR: keep, amend or delete
  4. Migration plan             independent merge requests, now specs #407-#410
  5. Questions                  decided: auth guards, Form v2 now, view/UI merge, cross-vertical reads in lib/
  6. Sources                    primary docs pinned to our versions, community posts tagged by date

Evidence

Before: no record of why the front's specs #407-#410 exist, or which rules they change.
After: each spec's "Source" line resolves to a section of this file.
Docs only: no code, rule, lint config or ADR changes. Every proposed rule change is a proposal here and lands through #408.
Gate green on each commit (pre-commit typecheck and tests, Node 24).

Merge Danger

Door: two-way

A Markdown file; reverting it costs nothing.

Blast Radius: none

Nothing imports or builds it. The specs link to it, so deleting it later would leave their "Source" lines dangling.

## Summary A read-only audit of `apps/front` against the TanStack Router, Query and Form docs for the installed versions, community practice, and the front's own rules. It records the owner's decisions and is the source the specs #407, #408, #409 and #410 cite. ```diff docs/research/ ├── gitlab-poc.md +└── front-audit.md # findings per objective, rules challenged, migration plan, decisions, sources ``` ```text front-audit.md 1. Summary ten points, ranked by gain over effort 2. Findings Router + Query, Form (v1, then v2 alpha), simplification with file counts 3. Rules challenged each front lint rule, review rule and ADR: keep, amend or delete 4. Migration plan independent merge requests, now specs #407-#410 5. Questions decided: auth guards, Form v2 now, view/UI merge, cross-vertical reads in lib/ 6. Sources primary docs pinned to our versions, community posts tagged by date ``` ## Evidence - **Before:** no record of why the front's specs #407-#410 exist, or which rules they change. **After:** each spec's "Source" line resolves to a section of this file. - Docs only: no code, rule, lint config or ADR changes. Every proposed rule change is a proposal here and lands through #408. - Gate green on each commit (pre-commit typecheck and tests, Node 24). ## Merge Danger **Door:** two-way A Markdown file; reverting it costs nothing. **Blast Radius:** none Nothing imports or builds it. The specs link to it, so deleting it later would leave their "Source" lines dangling.