From 4c4a10da4ea4d5536bcc27d70cdccdec58e48060 Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Sun, 14 Jun 2026 21:59:09 -0500 Subject: chore(theme-studio): remove dead code and clear a type warning - ramp (app-core.js) and its test-ramp.mjs: superseded by regenColumn, no production caller. - optList (app-core.js) and its tests: superseded by paletteOptionList. - ITALIC in generate.py: computed, never read (ITALIC_MAP is the live one). - a stray empty string in MU4E_FACES that .split() silently dropped. - the dead #familytest alias in the columntest gate, which HASHES never listed. - widen face_rows to Sequence[str], clearing the list-invariance warnings on the APPS calls. --- scripts/theme-studio/test-app-core.mjs | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) (limited to 'scripts/theme-studio/test-app-core.mjs') diff --git a/scripts/theme-studio/test-app-core.mjs b/scripts/theme-studio/test-app-core.mjs index e98e511e5..42ce4e0a2 100644 --- a/scripts/theme-studio/test-app-core.mjs +++ b/scripts/theme-studio/test-app-core.mjs @@ -7,7 +7,7 @@ import assert from 'node:assert/strict'; import { readFileSync } from 'node:fs'; import { fileURLToPath } from 'node:url'; import { - nameToHex, normalizePkgFace, buildPkgmap, packagesForExport, mergePackagesInto, effResolve, resolveSyntaxFg, resolveUiAttr, dropdownRowTextColor, optList, paletteOptionList, spanNeighborHex, slugify, + nameToHex, normalizePkgFace, buildPkgmap, packagesForExport, mergePackagesInto, effResolve, resolveSyntaxFg, resolveUiAttr, dropdownRowTextColor, paletteOptionList, spanNeighborHex, slugify, clearPalettePlan, deletePaletteColumnPlan, groundColumnMembersFromPalette, areAllLocked, lockToggleLabel, toggleLockSet, } from './app-core.js'; import { planPaletteGenerator, entriesForGeneratedColumn } from './palette-generator-core.js'; @@ -33,21 +33,6 @@ test('nameToHex: Boundary/Error — null, empty, and unknown names give null', ( assert.equal(nameToHex('chartreuse', PAL), null); }); -test('optList: Normal — default entry then the whole palette', () => { - assert.deepEqual(optList('#67809c', PAL), [['', '— default —'], ...PAL]); -}); - -test('optList: Boundary — empty cur is "have", so no (gone) entry', () => { - assert.deepEqual(optList('', PAL), [['', '— default —'], ...PAL]); -}); - -test('optList: Error — a cur not in the palette is surfaced as (gone) first', () => { - const list = optList('#123456', PAL); - assert.deepEqual(list[0], ['', '— default —']); - assert.deepEqual(list[1], ['#123456', '(gone)']); - assert.deepEqual(list.slice(2), PAL); -}); - test('paletteOptionList: Normal — color choices follow visual column ordering', () => { const pal = [ ['#67809c', 'blue'], -- cgit v1.2.3