From 2b5ce8a2336b0934071abfa0d1a2604a2f26c9da Mon Sep 17 00:00:00 2001 From: Ninosaurier Date: Fri, 17 Jul 2026 18:36:11 +0200 Subject: [PATCH] Update(docs): align ADR-0001 and ADR-0002 with implemented architecture Corrects two Architecture Decision Records that had drifted from the actual implementation during this sprint. - ADR-0002 (Technology Stack): unit testing now uses Vitest instead of Jest; Playwright's role is clarified as the browser provider used by Vitest's browser mode rather than a standalone test runner. - ADR-0001 (Repository Structure): repository diagram updated to remove the unused apps/ directory, correct the docs/ path to match the actual Starlight content location, and add the previously missing adm/ folder. Both ADRs now include a Change Log section documenting what changed and why, per the Architecture Change Management process (ADM Phase H). --- .../adr/000100-repository-structure.mdx | 40 +++++++++++++++---- 1 file changed, 33 insertions(+), 7 deletions(-) diff --git a/docs/src/content/docs/architecture/adr/000100-repository-structure.mdx b/docs/src/content/docs/architecture/adr/000100-repository-structure.mdx index 16515e6..da4d526 100644 --- a/docs/src/content/docs/architecture/adr/000100-repository-structure.mdx +++ b/docs/src/content/docs/architecture/adr/000100-repository-structure.mdx @@ -5,6 +5,7 @@ description: "Decision to use a pnpm-based monorepo structure for LUNA Charts." * **Status:** Accepted * **Date:** 2026-07-12 +* **Last Updated:** 2026-07-17 * **Decision Makers:** LUNA Charts Maintainers --- @@ -66,20 +67,23 @@ luna-charts/ │ ├── packages/ │ └── luna-charts/ -│ -├── apps/ -│ ├── storybook/ -│ └── docs/ +│ └── .storybook/ │ ├── docs/ -│ ├── architecture/ -│ └── adr/ +│ └── src/content/docs/architecture/ +│ ├── adr/ +│ └── adm/ │ ├── examples/ │ +├── .changeset/ +│ └── .github/ ``` +Storybook is not maintained as a separate app but lives directly inside the `packages/luna-charts` package, alongside the component source code it documents. +The `docs/` workspace is not nested under an `apps/` directory; it is a top-level workspace built with Starlight, with architecture documentation (ADR and ADM) organized under `docs/src/content/docs/architecture/`. + Internally, the library is organized into architectural modules such as: * Core @@ -135,4 +139,26 @@ This structure supports the project's architecture-first approach and simplifies * LUNA Charts Architecture Wiki * ADR-0002: Technology Stack -* pnpm Workspaces Documentation \ No newline at end of file +* pnpm Workspaces Documentation + +--- + +## Change Log + +### 2026-07-17 – Repository structure diagram updated (removal of `apps/`) + +**Reason** + +The originally planned `apps/` workspace (containing `apps/storybook` and `apps/docs`) was never adopted in this form during implementation. Storybook was instead kept inside `packages/luna-charts` alongside the component source it documents, and the documentation workspace (`docs/`) was placed directly at the repository root rather than nested under `apps/`. The repository structure diagram in this ADR no longer matched the actual repository layout, while the Getting Started documentation (`project-structure.mdx`) already reflected the current structure. + +**Changes** + +* Removed the `apps/` directory (and its `storybook/` and `docs/` subentries) from the structure diagram +* Corrected the `docs/` path to reflect the actual Starlight content location (`docs/src/content/docs/architecture/`) +* Added the `adm/` folder to the diagram, alongside `adr/`, since Architecture Development Method documentation is stored there as well +* Added `.changeset/` to the diagram, as it is a workspace-level configuration directory used for release management +* Added a note clarifying that Storybook is maintained as part of the `packages/luna-charts` package rather than as a separate app + +**Impact** + +The overall decision (pnpm-based monorepo with logical workspace separation) is unaffected. Only the illustrative directory layout was corrected to match the implemented structure. No other ADRs are impacted by this change. \ No newline at end of file