@klodd/ds 3.3.1 → 3.4.1

package/references/05-open-decisions/0006-bar-kind-template-migration-CLOSED.md

@@ -0,0 +1,75 @@
+# 0006 - Ekonom bar_kind()-helper - migration till kategori-id for colored-bar
+
+## Status
+**Closed 2026-05-08** via Option A (utoka bar_kind() till kategori-id).
+Ekonom-commit [4ea9415](https://github.com/drawn124578/ekonom/commit/4ea9415).
+
+bar_kind() i routers/kategorier.py returnerar nu cat.key (mat, el,
+vatten, etc.) istallet for variant-modifiers ('default'/'wealth'/'accent').
+ekonom.css .cat-bar-fill far ny background: var(--bar-accent, var(--text-tertiary))
+sa --bar-accent (satt av .cat-bar-{kid}-modifier i ekonom-domain.css)
+faktiskt blir bakgrund. 8 distinkta kategori-farger istallet for 3
+generic varianter. Pytest 636/636.
+
+## Context
+v3.1.0 introducerade paketets `.colored-bar`-komponent + Ekonoms
+bindings-klasser `.cat-bar-mat` / `.cat-bar-el` ... / `.cat-bar-privat`
+som mappar app-tokens (--cat-X-icon) till paketets `--bar-accent`.
+
+CSS-bindningarna ar redo att anvandas. Templates anvander dock idag
+ett ANNAT monster:
+
+```html
+<div class="cat-bar-track">
+  <div class="cat-bar-fill cat-bar-{{ bar_kind(g.category) }}"
+       data-bar-width="{{ pct }}%"></div>
+</div>
+```
+
+Python-helpern `bar_kind(category)` returnerar idag en av tre
+**variant-modifiers**: `default`, `wealth`, `accent`. Inte kategori-id.
+
+Aktivering av det nya kategori-monstret kraver att antingen:
+- `bar_kind()` utokas att returnera kategori-id (mat, el, noje osv.)
+- Templates byter klassnamn fran `cat-bar-{kind}` till
+  `colored-bar cat-bar-{category-id}`
+
+## Considered options
+
+### Option A: Utoka bar_kind() att returnera kategori-id
+**Pros:**
+- Aktiverar det nya kategori-monstret fullt ut
+- Templates fortsatter anvanda `cat-bar-{{ bar_kind(g) }}`-syntax -
+  bara inneliggande varde andras
+- Visuellt rikare resultat (8 distinkta kategori-farger istallet for
+  3 generic varianter)
+**Cons:**
+- Andrar Python-logik pa `bar_kind()` - kraver test-uppdatering
+- Templates behover ocksa lagga till `colored-bar`-klassen parallellt:
+  `class="colored-bar cat-bar-{{ bar_kind(g) }}"`
+- Behover bestamma vad bar_kind() returnerar for icke-kategori-rader
+  (wealth, accent-varianter for special-data)
+
+### Option B: Behall variant-modifiers som de ar
+**Pros:**
+- Inga template- eller Python-andringar
+- Default/wealth/accent-monstret ar etablerat och fungerar
+- CSS-bindningarna lever passivt for framtida anvandning utan att
+  blockera nuvarande funktionalitet
+**Cons:**
+- Det nya kategori-monstret ar redo i CSS men oanvant
+- Inga visuella forbattringar pa kategori-bars
+
+## Pending decision criteria
+- Sprint-prioritet: ar visuellt rikare kategori-fargning vard att kosta
+  Python-logik-andring + template-edit?
+- Samordning med MADR 0001 (.tx-row -> .list-row): bagge migrationer
+  beror Ekonoms transaction-listor och kan delas pa samma sprint for
+  effektivitet
+
+## Notes
+- CSS-bindningarna `.cat-bar-X` lever sedan v3.1.0 (Ekonom-commit
+  f011c2f). Inga paket-andringar behovs - bara app-side adoption
+- Fortsatt fungerande default/wealth/accent-monster betyder INGEN
+  forsamring vid val av Option B
+- Ingen tidsgrans pa beslutet

package/references/05-open-decisions/0007-ekonom-base-import-gap-CLOSED.md

@@ -0,0 +1,111 @@
+# 0007 - Ekonoms main.css-kedja importerar inte paketets ds/base/-mapp
+
+## Status
+**Closed 2026-05-07** via Option A med selektiv import. Ekonoms
+main.css importerar nu paketets `ds/base/layout.css` direkt. Skip
+av `pwa.css` (dubblett mot Ekonoms egen) och `typography.css`
+(Inter Google Fonts-konflikt med Sprint D2).
+
+## Context
+Jubb och Ekonom har olika CSS-arkitektur for hur paketets `base/`-mapp
+laddas:
+
+**Jubb:** paketets `base/`-filer kopieras till `app/static/css/base/`
+(rotnivå). `main.css` importerar `base/typography.css` direkt:
+
+```css
+/* Jubbs main.css */
+@import url("ds/base.css");
+@import url("base/typography.css");   /* paketets fil, rotniva */
+@import url("ds/components.css");
+...
+```
+
+**Ekonom:** paketets `base/`-filer kopieras till `app/static/css/ds/base/`-
+undermapp (for att inte krocka med Ekonoms egna `app/static/css/base/`
+filer). `main.css` importerar via `ds/bundles/ekonom.css` som bara
+inkluderar Ekonom-specifika `ds/base.css`, INTE paketets nyare filer:
+
+```css
+/* Ekonoms main.css */
+@import url("ds/bundles/ekonom.css");
+
+/* ds/bundles/ekonom.css */
+@import url("../ekonom-domain.css");
+@import url("../base.css");          /* Ekonoms egen, inte paketet */
+@import url("../components.css");
+@import url("../utilities.css");
+@import url("../ekonom.css");
+@import url("../components/inline-edit.css");
+```
+
+`ds/base/typography.css`, `ds/base/pwa.css`, `ds/base/layout.css`
+(paketets versioner) kopieras till disk via build:ds men laddas aldrig.
+
+### Konsekvens
+
+Framtida paket-uppdateringar i `base/`-mappen kraver manuella hot-fixes
+i Ekonom. Konkret exempel: v3.2.0 introducerade Inter @import i
+`base/typography.css`. Jubb fick det automatiskt - Ekonom kravde commit
+`5d59f1c` med Inter @import direkt i `main.css`.
+
+## Considered options
+
+### Option A: Lagg till explicita @import-rader i Ekonoms main.css
+```css
+/* Ekonoms main.css */
+@import url("ds/base/typography.css");
+@import url("ds/base/layout.css");
+@import url("ds/bundles/ekonom.css");
+```
+
+**Pros:**
+- Enklast
+- Minst risk
+- Paketets base/-filer aktiveras automatiskt vid build:ds
+- Bibehaller Ekonoms befintliga ds/-undermapp-arkitektur
+**Cons:**
+- Lite duplicering om paketet utvidgas med ytterligare base/-filer
+  (man maste komma ihag att lagga till @import)
+- `base/pwa.css` skips eftersom den redan laddas via base.html-link
+
+### Option B: Skriv om build:ds-scriptet att kopiera + auto-importera
+build:ds genererar en `ds/base/_index.css` som listar alla `base/-`
+filer i paketet. main.css importerar `ds/base/_index.css`.
+
+**Pros:**
+- Helt automatisk - inga manuella @import-rader behovs
+**Cons:**
+- Mer komplex build-script
+- Genererad fil i ds/-mappen som ingen redigerar manuellt - cognitive
+  overhead
+
+### Option C: Harmonisera Ekonoms mappstruktur med Jubbs
+Flytta paketets `base/`-filer till Ekonoms `app/static/css/base/`-
+rotniva (overskriver Ekonoms egna). Kraver migration av Ekonoms
+befintliga `base/pwa.css` och `base/typography.css`.
+
+**Pros:**
+- Identisk arkitektur Jubb/Ekonom
+- Inga manuella @import behovs
+**Cons:**
+- Mer invasivt - kraver hantering av krockande filnamn
+- Ekonoms befintliga `app/static/css/base/typography.css` har Ekonom-
+  specifika styling-regler som inte ar i paketet
+- Storre risk for visuell regression
+
+## Recommendation
+**Option A.** Enklast, minst risk, kan ske i samma sprint som annan
+Ekonom-cleanup (eventuellt MADR 0001 .tx-row -> .list-row eller MADR
+0002 parallel-classnames-cleanup).
+
+## Pending decision criteria
+- Ar paketets `base/`-utokningar tillrackligt vanliga for att
+  rattfardiga Option B (auto-importer)?
+- Ar Ekonoms befintliga `base/`-mapp viktig att behalla, eller kan
+  Option C utforas i en cleanup-sprint?
+
+## Notes
+- Hot-fix via Option A ar redan delvis utford for Inter (commit 5d59f1c
+  i Ekonom lagger till Google Fonts @import direkt i main.css). Inte
+  ren Option A men en proxy-losning som fungerar

package/references/05-open-decisions/0008-legacy-token-migration.md

@@ -0,0 +1,73 @@
+# 0008 - Legacy token-migration och `ds/components.css`-radering - v4.0.0
+
+## Status
+Pending. Trigger: innan tredje app borjar anvanda `@klodd/ds`.
+
+## Context
+`10-semantic.css` har en LEGACY ALIASES-sektion (rad 412-506) som
+holds gamla token-namn vid liv sedan migrationen pabarjades:
+`--accent`, `--accent-dim`, `--accent-link*`, `--bg-base`, `--bg-surface`,
+`--card-bg-elevated`, `--surface-medium`, `--surface-subtle`,
+`--negative-dim`, `--positive-dim`, `--shadow-float`, samt 9 radius-
+aliaser. Sektionen ar 100 rader och har funnits over ett halvar.
+
+Parallellt korr `ds/components.css` (flat-format, ~60 KB) sida vid
+sida med `ds/components/<X>.css` (BEM, paket-konvention). Bagge
+laddas i bagge appars main.css. DESIGN_TOKENS_GUIDE.md sektion 7
+dokumenterar konflikten och saver "migration sker per sida" - men
+ingen migration har skett pa manader.
+
+Audit 2026-05-08 bekraftade att Jubb-domain CSS (`jubb-theme.css`,
+`triage-v2.css`, `ds/base.css`) hanger fast i legacy-tokens, och
+samma migrations-skuld finns i Ekonom (tx-row, kategori-tokens,
+profile-edit). Det ar ingen lokal Jubb-fraga - det ar paketets
+kostnad.
+
+### Konsekvens av att leva i halv-tillstand
+
+- Varje ny komponent maste valja: gamla namnet eller nya namnet?
+  Folk valjer fel halften av gangerna.
+- Ny utvecklare (eller LLM) maste lara sig bagge systemen for att
+  lasa koden. Cognitive overhead vid varje touch.
+- Specen i DESIGN_TOKENS_GUIDE.md saver "5-raders apps/jubb.css" men
+  paketet har 36 rader. Specen ar inte sanning.
+- Migrations-skulden vaxer linjart med antal call-sites och antal
+  appar.
+
+## Decision
+Gor en fullstandig **lock-sprint** innan tredje app laggs till:
+
+1. Migrera ALLA call-sites till nya semantic token-namn
+   (`--accent` -> `--accent-9` eller `--accent-text`, `--bg-base`
+   -> `--surface-page`, `--surface-medium` -> `--surface-default`,
+   `--card-bg-elevated` -> `--surface-raised` etc).
+2. Radera LEGACY ALIASES-sektionen ur `10-semantic.css`.
+3. Migrera ALLA flat-komponenter (`.btn`, `.card`, `.panel`, `.tx-row`
+   etc) till BEM (`.btn`, `.btn--primary`, `.card`, `.card--interactive`).
+4. Radera `ds/components.css` (flat-versionen) ur bagge app-repos.
+5. Bumpa till `@klodd/ds@4.0.0` (breaking).
+
+## Consequences
+- **+** Slut pa dubbla mentala modeller. En standard.
+- **+** Spec stammer med implementationen.
+- **+** Tredje app:s onboarding gar mot ett moget system.
+- **+** `npx @klodd/ds verify` (3.4.0+) detekterar drift mekaniskt.
+- **-** Stor sprint - kraver dedikerad bandbredd (uppskattning: 2-3
+  arbetsdagar plus visuell QA per app).
+- **-** Risk for visuell regression vid migration. Kraver `/design`-
+  styleguide-jamforelse fore/efter pa bagge appar.
+
+## Pending decision criteria
+- Tidpunkt: omedelbart innan tredje app (Margevo eller annan)
+  borjar anvanda paketet. Att vanta langre forharder skulden;
+  att hasta forskuggar appar som redan ar i prod.
+- Vem kor sprinten: Calle ensam eller med assistent CC?
+- Om CC: ska den ha access till bagge app-repos under sprinten?
+
+## Notes
+- Audit 2026-05-08 (av CC) identifierade root-causes och rekommenderade
+  steg 1-3 (CLI-konsolidering) NU och steg 4 (denna migration) "nar
+  bandbredd finns".
+- Steg 1-3 ar implementerade i v3.4.0 (sync/verify/precache CLI).
+- v3.4.0 ar lokalt locked - drift detekteras av CI-job i bagge appar.
+  Det ar plattformen som later denna lock-sprint hallas i kontroll.