chore(brand): finish PR #44 relocation — empty retired Brand/Icons, move Rituals UI sources #46

Merged

peterswimm merged 1 commit from chore/pr44-audit-followups into main

2026-07-24 04:08:14 +00:00

peterswimm commented

2026-07-24 02:00:39 +00:00

Owner

Follow-ups from the PR #44 audit (ville's app-icon relocation + KeFF's icon/annotation commits):

Empty the retired Brand/Icons/ tree — 73e2dfb moved 15 of 16 files; the leftover rituals-app-icon.af~lock~ (Affinity lock file, matches the *~lock~ gitignore rule) was the sole remaining tracked file. Removed; Brand/Icons/ is now fully retired again, matching CLAUDE.md.
Finish the source/ relocation — move Brand/Rituals/UI/ (KeFF's Modecard-consentgate annotation .af/.pdf, UI mockup WIPs, rituals-logo.af, and the three annotation-export SVGs) to Brand/source/components/Rituals/UI/, completing what ville started for the app-icon subset. Pure 100% renames; nothing references these paths (verified against assets.yml, sync-manifest, design-sync config, forgeblock configs).
Document the convention — CLAUDE.md now names Brand/source/components/ as the sanctioned editor-source tree (app icons at components/Icons/app-icon/<product>/), distinct from the lowercase pipeline roots.

Deliberately NOT done: rituals-icon-{consentgate,info,warning}.svg are annotation-artboard exports (giant off-canvas <rect id="Annotations"> + serif:id groups), not pipeline-ready icons — registering them in udg/config/assets.yml as-is would produce a broken viewBox. Follow-up for @ville/KeFF: clean single-artboard re-exports into Brand/source/icons/, then register.

Verified locally: just check all 9 gates green; pytest generator/tests/ 95/95.

Follow-ups from the PR #44 audit (ville's app-icon relocation + KeFF's icon/annotation commits): - **Empty the retired `Brand/Icons/` tree** — 73e2dfb moved 15 of 16 files; the leftover `rituals-app-icon.af~lock~` (Affinity lock file, matches the `*~lock~` gitignore rule) was the sole remaining tracked file. Removed; `Brand/Icons/` is now fully retired again, matching CLAUDE.md. - **Finish the source/ relocation** — move `Brand/Rituals/UI/` (KeFF's Modecard-consentgate annotation .af/.pdf, UI mockup WIPs, rituals-logo.af, and the three annotation-export SVGs) to `Brand/source/components/Rituals/UI/`, completing what ville started for the app-icon subset. Pure 100% renames; nothing references these paths (verified against assets.yml, sync-manifest, design-sync config, forgeblock configs). - **Document the convention** — CLAUDE.md now names `Brand/source/components/` as the sanctioned editor-source tree (app icons at `components/Icons/app-icon/<product>/`), distinct from the lowercase pipeline roots. **Deliberately NOT done:** `rituals-icon-{consentgate,info,warning}.svg` are annotation-artboard exports (giant off-canvas `<rect id="Annotations">` + serif:id groups), not pipeline-ready icons — registering them in `udg/config/assets.yml` as-is would produce a broken viewBox. Follow-up for @ville/KeFF: clean single-artboard re-exports into `Brand/source/icons/`, then register. Verified locally: `just check` all 9 gates green; `pytest generator/tests/` 95/95.

peterswimm added 1 commit

2026-07-24 02:00:39 +00:00

chore(brand): finish PR #44 relocation — empty retired Brand/Icons, move Rituals UI sources

UDG dist drift gate / dist/ matches dtcg + udg sources (pull_request) Failing after 21s

Details

Test Generator / lint-workflows (pull_request) Successful in 23s

Details

Test Generator / lint-prose (pull_request) Successful in 24s

Details

Security Scans / Gitleaks secret scan (pull_request) Successful in 27s

Details

Security Scans / Trivy filesystem scan (pull_request) Successful in 34s

Details

Test Generator / test (pull_request) Failing after 44s

Details

c2b5491b8c

- Remove the orphaned rituals-app-icon.af~lock~ that 73e2dfb left behind
  (matches the *~lock~ gitignore rule; was the sole remaining file in the
  retired Brand/Icons/ tree, which is now fully gone)
- Move Brand/Rituals/UI/ (Modecard-consentgate annotations .af/.pdf, UI
  mockup WIPs, rituals-logo.af, annotation-export SVGs) to
  Brand/source/components/Rituals/UI/, completing the source/ layout
  reorganization ville started for the app-icon subset
- Document the editor-source tree convention in CLAUDE.md: sanctioned home
  is Brand/source/components/ (app icons at components/Icons/app-icon/
  <product>/), distinct from the lowercase pipeline roots

Note: the three rituals-icon-{consentgate,info,warning}.svg files are
annotation-artboard exports (off-canvas Annotations rect), NOT pipeline-
ready icons — intentionally not registered in udg/config/assets.yml until
clean single-artboard re-exports exist.

peterswimm referenced this pull request

2026-07-24 02:17:45 +00:00

fix(ci): dist gate + test job failures — miniyaml silent truncation & block-coverage gaps #47

peterswimm merged commit a157a14595 into main

2026-07-24 04:08:14 +00:00

peterswimm deleted branch chore/pr44-audit-followups

2026-07-24 04:08:14 +00:00

peterswimm referenced this pull request from a commit

2026-07-24 04:08:16 +00:00

Merge pull request 'chore(brand): finish PR #44 relocation — empty retired Brand/Icons, move Rituals UI sources' (#46) from chore/pr44-audit-followups into main