Skip to content

docs: add PoC decision log (ADR-style)#64

Merged
oharsta merged 1 commit into
mainfrom
feat/decision-log
Jun 30, 2026
Merged

docs: add PoC decision log (ADR-style)#64
oharsta merged 1 commit into
mainfrom
feat/decision-log

Conversation

@sjoerdbeentjes

Copy link
Copy Markdown
Collaborator

Adds docs/decision-log.md — a single ADR-style decision log capturing the 17 repo-wide architecture and tooling decisions for the PoC.

What

  • Status legend, an index table, and one ADR-NNN entry per decision (context / decision / rationale / consequences).
  • Token-meeting decisions (naming & roles, explicit colors over opacity, modes vs themes, shadcn deviation) dated 2026-06-29 and tagged with their source meeting; the rest 2026-06-30.
  • Tailwind v4 and the demo app marked Proposed; the two still-open items (product families, final sign-off) split into an Open questions table rather than presented as decisions.

Why

The token-level decisions already live in docs/design-token-setup-exploration/ (Dutch). This captures the broader PoC layer in one place and cross-links into those records.

Notes for reviewer

  • Facts were verified against the code, surfacing a few stale spots in AGENTS.md worth a follow-up: it lists five packages (there are six) and style: base-nova (actual is base-vega).
  • English to match AGENTS.md/README; em dashes kept to match the existing beslissingen.md doc style.
  • Docs-only, so no changeset.

Record the 17 repo-wide architecture and tooling decisions for the
SURF Design System PoC as lightweight ADRs in docs/decision-log.md:
monorepo structure, Base UI over Radix, Phosphor icons, tokens as
single source of truth, Figma sync, theming, token naming/roles,
explicit colors over opacity, modes vs themes, shadcn deviation,
contracts parity, Tailwind v4, Storybook, Changesets, tree-shaking,
component scope, and the demo app.

Cross-links into the existing Dutch token-decision records and
splits out the still-open items (product families, final sign-off).
@sjoerdbeentjes sjoerdbeentjes requested review from loofpc and oharsta June 30, 2026 13:41
@oharsta oharsta merged commit c68f291 into main Jun 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants