diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-28 00:00:42 -0400 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-28 00:00:42 -0400 |
| commit | edb07cd1059cc74fa4d159c81caa9fbc36cdc973 (patch) | |
| tree | 7742a9a9b72f520d405c333cadd28972cc59c2fc | |
| parent | a5fd0b4d1e4dfea0f51c811f94c7cb1b96afcf30 (diff) | |
| download | dotemacs-edb07cd1059cc74fa4d159c81caa9fbc36cdc973.tar.gz dotemacs-edb07cd1059cc74fa4d159c81caa9fbc36cdc973.zip | |
chore(todo): re-scope wttrin task to the verified root cause
Replaces the wrong "blocked on shipping" read. The real cause was the inventory capture matching only versioned package dirs, fixed in a5fd0b4d and verified on a clean load. Remaining work is a clean-Emacs inventory regen and getting the new face into the checkout source.
| -rw-r--r-- | todo.org | 5 |
1 files changed, 4 insertions, 1 deletions
@@ -430,7 +430,10 @@ Expected: d duplicates; D names the exact targets and only deletes on yes; the f ** PROJECT [#A] Theme-Studio Open Work Parent grouping the open theme-studio / theming issues; close each child independently. *** TODO [#C] wttrin preview in theme-studio :feature: -emacs-wttrin handed off its faces (incl. the new =wttrin-instructions-header= for the two-column footer headers). But wttrin is not a studio app: =package-inventory.json= (generated from installed packages by =build-inventory.el=) has no wttrin entry, so there is no wttrin face data, no preview, and no app-dispatch entry. Not the one-line add the handoff implied. And wttrin is on the unmerged =feature/named-locations= branch, so the inventory capture cannot see it yet. Cleanest path once it ships: regenerate the inventory (wttrin then becomes a generic-preview app for free), then optionally add a bespoke two-column-footer =renderWttrin=. Probably batch with the other pending cross-project packages (chime, pearl). Inventory + rendered sample in the inbox handoff (2026-06-26). +Root cause found, capture fixed (commit a5fd0b4d). The studio inventory (=build-inventory.el=) only captured faces defined in versioned dirs (=/elpa/PKG-VERSION/=); wttrin is an unversioned local checkout (=/elpa/wttrin/=) and was silently excluded. Not a branch dependency, despite my first read. The regex now matches unversioned checkouts too, verified on a clean wttrin load. Remaining: +1. Regenerate the inventory in a *clean* Emacs (restart first; the live daemon's wttrin face attribution got disturbed during the investigation). wttrin then becomes a generic-preview app with its four file-loaded faces, and the same regen picks up the other local-checkout packages the fix unblocked. +2. =wttrin-instructions-header= is eval-defined from outside this checkout (absent from =elpa/wttrin/wttrin.el=), so it is not captured until it lands in the checkout's compiled source. +3. Optional: a bespoke two-column-footer =renderWttrin= instead of the generic preview. *** 2026-06-27 Sat @ 22:05:31 -0400 Rebuilt the dirvish preview as a realistic two-pane Shipped in 61b68fcf (option 2). Two-pane dirvish: an active listing (per-type nerd-icons, sizes, hl-line, dimmed backup) beside an =ls -l= preview pane, the remaining faces in a labeled extras strip, all 38 dirvish faces still covered. Visual verified via off-screen screenshot. *** TODO [#A] theme-studio: consistent assignment-view table columns :feature:studio:next: |