You’re viewing Paradaux’s public workspaceSign in to vote Get taskl.ie for your team

PAR-408

Loading issues

PAR-408 · [Epic] Architecture rebuild: apply the commandments across… · Paradaux · taskl.ie

PAR-408

[Epic] Architecture rebuild: apply the commandments across apps/web and apps/api

The technical half of the rebuild. The UI half is PAR-357. The rules are in docs/architecture.md (the 35 commandments, amended 2026-10-18) and the agent workflow is in the root CLAUDE.md.

Decisions (Rían, 2026-10-18)

Facade: each domain's facade is its service layer and transaction boundary (rules 12 and 21 merged).
Side effects: go through a transactional outbox, not plain AFTER_COMMIT listeners (rule 19).
Layout: moves to apps/web, apps/api and contracts/ first, as its own PR.
Rule 27 (dual-write) is retired. Postgres is the only store.
Data access: each domain's data/ folder exposes server functions and client hooks on one generated client (rule 8).
Guest read path: the API has one for public workspaces (rule 7).
No feature flags. All work lands on the rebuild integration branch, which deploys to dev. One final rebuild → main PR goes to production.
Merging: the orchestrator (Claude Opus) reviews and merges agent PRs into rebuild. Rían reviews the final PR.
Agents: Sonnet 5.5 subagents do the work, one ticket per agent, each in its own worktree and PR.

Starting point (audit, 2026-10-18)

Backend: ~11.3k lines, 50 tests.
- Packages are mostly by domain, but there's a cross-domain api/ package of controllers and services.
- tasks/store/PostgresTaskSystem.java (~1,000 lines) holds most of the SQL.
- Four ThreadLocal contexts (TenantContext, ActorContext, KeyContext, EventBatch). These already caused an MCP bug.
- Three auth filters (ApiAuthFilter, TenantBearerFilter, McpAuthFilter).
- No OpenAPI and no ArchUnit.
UI: reads Postgres directly through Kysely (ui/lib/sql, 21 modules used from 47 files). Writes go through the backend API.
Lint: ESLint 9.39 (bulk suppressions are available). No boundary rules yet.

Sequence

Foundation runs in order (each depends on the one before):

Repo layout move
Lint ratchet (ArchUnit freezing plus ESLint suppressions)
OpenAPI pipeline
Platform primitives (RequestContext, Result, errors, outbox)
Reference domain end to end

After that, the domain migrations can run in parallel. Each moves one backend domain to the new structure and moves its UI reads from Kysely to the generated client.

Children are tracked by the Architecture label (subtasks are disabled). They're listed in the "Plan" comment.

Activity

taskl.ie created the issue 6d ago
taskl.ie6d ago
Plan and dependency graph (orchestrated by Claude Opus; work done by Sonnet 5.5 subagents)

Status convention:
- Todo: ready for an agent to pick up.
- Backlog: waiting on a dependency.
- In Progress: an agent is working on it.
- Pending Release: PR merged into rebuild.
- Done: shipped to main in the final PR.
Each agent comments its plan, its PR link and the lint-ratchet deltas on its ticket.

Wave 0: in order, one at a time
1. PAR-409 · A1 layout move to apps/. Blocks everything.
Wave 1: after A1, in parallel
- PAR-410 · A2 lint ratchet (ArchUnit freezing and ESLint suppressions)
- PAR-425 · U1 design foundations in the app (tokens, DM Sans, icons, primitives)
Wave 2: after A2
- PAR-411 · A3 OpenAPI pipeline
- PAR-412 · A4 platform primitives (RequestContext, Result, errors, single JWT filter, outbox)
- PAR-426 · U2 app shell (needs U1)
Wave 3: after A3 and A4
- PAR-413 · A5 reference domain: labels, including the Settings → Labels screen
Wave 4: after A5, in parallel
- PAR-414 · D1 issues reads
- PAR-417 · D4 workspaces
- PAR-423 · D10 guest read path (also needs D1)
Wave 5: in parallel as their dependencies land
- PAR-415 · D2 issues writes (D1)
- PAR-416 · D3 activity (D1)
- PAR-418 · D5 projects and milestones (D1)
- PAR-422 · D9 votes, billing, reports (D1)
- PAR-419 · D6 deployments (D5 and A4)
- PAR-420 · D7 Discord and triage (D2 and A4)
- PAR-421 · D8 GitHub (D2 and A4)
Wave 6
- PAR-424 · D11 final cleanup. Then the rebuild → main PR for Rían.
UI screen work (PAR-357 children) runs alongside, gated on the domain that provides its data. See the plan comment on PAR-357.
taskl.ie changed status from In progress to Done 5d ago
taskl.ie5d ago
Shipped: to production on 2026-10-18 in #37 (merge 3c0d36f). This epic was closed automatically by the new Deployments automation.

End state:
- Every backend domain sits behind a facade.
- The ArchUnit store is empty, so all rules are strict.
- The UI has no database access (ESLint R6 is a hard error).
- Side effects go through the outbox.
- There's a guest read path with rate limiting.
Hotfix: #84 fixed the staff break-glass audit, which was writing inside read-only transactions.

Follow-ups: PAR-445 (drop the release and discord_delivery tables, after production settles), PAR-454 (realtime across replicas), PAR-455 (ops), PAR-442 and PAR-443. All are tracked in PAR-449.

Workspace permission required to comment

Comments and reactions require workspace permission. Ask a workspace owner or admin for access. You can sign in to vote.

Decisions (Rían, 2026-10-18)

Starting point (audit, 2026-10-18)

Sequence

Activity

Plan and dependency graph (orchestrated by Claude Opus; work done by Sonnet 5.5 subagents)

Wave 0: in order, one at a time

Wave 1: after A1, in parallel

Wave 2: after A2

Wave 3: after A3 and A4

Wave 4: after A5, in parallel

Wave 5: in parallel as their dependencies land

Wave 6

Workspace permission required to comment