@invarn/cibuild 2.2.3 → 2.2.4

package/dist/src/yaml/steps/structural-facts.test.js

@@ -741,6 +741,33 @@ describe('buildStructuralBlock — appearance container (Signal C, additive)', (
             },
         ]);
     });
+    test('with reference text color → a text-channel color fact (issue 02)', () => {
+        // COLOR_DUMP renders `card` text #000000; the reference declares #ffffff for
+        // the text channel and the matching fill — so exactly one fact, channel:'text'.
+        const block = buildStructuralBlock(COLOR_DUMP, null, 2, undefined, {
+            baseline: 'figma',
+            elementIds: ['card'],
+            color: { card: { fill: '#ff0000', text: '#ffffff' } },
+        });
+        expect(block.appearance?.facts).toEqual([
+            {
+                kind: 'color',
+                elementId: 'card',
+                channel: 'text',
+                rendered: '#000000',
+                reference: '#ffffff',
+                deltaE: ciede2000(hexToLab('#000000'), hexToLab('#ffffff')),
+            },
+        ]);
+    });
+    test('an identical text color → no text fact (issue 02)', () => {
+        const block = buildStructuralBlock(COLOR_DUMP, null, 2, undefined, {
+            baseline: 'figma',
+            elementIds: ['card'],
+            color: { card: { fill: '#ff0000', text: '#000000' } },
+        });
+        expect(block.appearance?.facts).toEqual([]);
+    });
     test('an appearanceReference without color → no color facts (presence only)', () => {
         const block = buildStructuralBlock(COLOR_DUMP, null, 2, undefined, {
             baseline: 'figma',
@@ -822,6 +849,20 @@ describe('appearanceReferenceFromPreviousRender — previous render snapshot →
             color: { banner: { fill: '#fff0ee' } },
         });
     });
+    test('a previous render with per-element text colors → the text channel rides through into the reference (issue 02)', () => {
+        // The mapper carries the whole previous color map, so the foreground (text)
+        // channel reaches the reference exactly as the fill channel does — channel
+        // genericity, not a fill-only carry.
+        const previous = {
+            elementIds: ['banner', 'copy'],
+            color: { banner: { fill: '#fff0ee' }, copy: { text: '#1b1b1b' } },
+        };
+        expect(appearanceReferenceFromPreviousRender(previous)).toEqual({
+            baseline: 'previous_render',
+            elementIds: ['banner', 'copy'],
+            color: { banner: { fill: '#fff0ee' }, copy: { text: '#1b1b1b' } },
+        });
+    });
     test('a previous render with no extractable color → a presence-only reference (ids, no color)', () => {
         const previous = { elementIds: ['banner', 'voucher'] };
         expect(appearanceReferenceFromPreviousRender(previous)).toEqual({
@@ -870,6 +911,60 @@ describe('appearanceReferenceFromPreviousRender — previous render snapshot →
         ]);
         expect(colorFacts[0].kind === 'color' && colorFacts[0].deltaE).toBeGreaterThan(10);
     });
+    test('fed into buildStructuralBlock, a drifted text color produces a previous_render text-channel fact (issue 02)', () => {
+        // The render paints `copy` the wrong foreground color (#1e88e5); the previous
+        // render painted #1b1b1b. The mapper carries the text channel through, and the
+        // assembled block carries one text color fact with a large positive ΔE —
+        // distinct from fill, proven through the public interface.
+        const driftedDump = {
+            canvas: { left: 0, top: 0, right: 686, bottom: 282 },
+            nodes: [
+                { id: 'copy', bounds: { left: 0, top: 0, right: 343, bottom: 40 }, parentId: null, clipsChildren: false, textColor: '#1e88e5' },
+            ],
+        };
+        const previous = { elementIds: ['copy'], color: { copy: { text: '#1b1b1b' } } };
+        const reference = appearanceReferenceFromPreviousRender(previous);
+        const block = buildStructuralBlock(driftedDump, null, 2, undefined, reference);
+        expect(block.appearance?.baseline).toBe('previous_render');
+        const colorFacts = (block.appearance?.facts ?? []).filter((f) => f.kind === 'color');
+        expect(colorFacts).toEqual([
+            {
+                kind: 'color',
+                elementId: 'copy',
+                channel: 'text',
+                rendered: '#1e88e5',
+                reference: '#1b1b1b',
+                deltaE: ciede2000(hexToLab('#1e88e5'), hexToLab('#1b1b1b')),
+            },
+        ]);
+        expect(colorFacts[0].kind === 'color' && colorFacts[0].deltaE).toBeGreaterThan(10);
+    });
+    test('a correct fill with a wrong text color yields a text-channel fact only — the two never collapse (issue 02)', () => {
+        // One element carries both channels: fill matches the previous render, text
+        // drifts. Exactly one fact, channel:'text' — the correct fill produces none.
+        const dump = {
+            canvas: { left: 0, top: 0, right: 343, bottom: 72 },
+            nodes: [
+                { id: 'banner', bounds: { left: 0, top: 0, right: 343, bottom: 72 }, parentId: null, clipsChildren: false, fillColor: '#fff0ee', textColor: '#1e88e5' },
+            ],
+        };
+        const previous = {
+            elementIds: ['banner'],
+            color: { banner: { fill: '#fff0ee', text: '#1b1b1b' } },
+        };
+        const block = buildStructuralBlock(dump, null, 2, undefined, appearanceReferenceFromPreviousRender(previous));
+        const colorFacts = (block.appearance?.facts ?? []).filter((f) => f.kind === 'color');
+        expect(colorFacts).toEqual([
+            {
+                kind: 'color',
+                elementId: 'banner',
+                channel: 'text',
+                rendered: '#1e88e5',
+                reference: '#1b1b1b',
+                deltaE: ciede2000(hexToLab('#1e88e5'), hexToLab('#1b1b1b')),
+            },
+        ]);
+    });
 });
 // ---- Signal C: build the appearance reference from run params (issue 05) ----
 //
@@ -1059,6 +1154,22 @@ describe('embedded geometry stage parity', () => {
         };
         expect(embedded.buildStructuralBlock(dumpWithColor, null, 2, undefined, appearanceReference)).toEqual(buildStructuralBlock(dumpWithColor, null, 2, undefined, appearanceReference));
     });
+    test('embedded buildStructuralBlock matches the module for a text-only color drift (issue 02)', () => {
+        // Correct fill + wrong text: the embedded port must emit the same single
+        // text-channel fact (no collapse), matching the typed module exactly.
+        const dumpWithText = {
+            canvas: { left: 0, top: 0, right: 343, bottom: 72 },
+            nodes: [
+                { id: 'banner', bounds: { left: 0, top: 0, right: 343, bottom: 72 }, parentId: null, clipsChildren: false, fillColor: '#fff0ee', textColor: '#1e88e5' },
+            ],
+        };
+        const appearanceReference = {
+            baseline: 'previous_render',
+            elementIds: ['banner'],
+            color: { banner: { fill: '#fff0ee', text: '#1b1b1b' } },
+        };
+        expect(embedded.buildStructuralBlock(dumpWithText, null, 2, undefined, appearanceReference)).toEqual(buildStructuralBlock(dumpWithText, null, 2, undefined, appearanceReference));
+    });
     const parityCases = [
         { name: 'factsOnly', dump: HUB_PROMO_DUMP, ref: null },
         { name: 'withGeometry', dump: HUB_PROMO_DUMP, ref: HUB_PROMO_REFERENCE },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invarn/cibuild",
-  "version": "2.2.3",
+  "version": "2.2.4",
   "description": "CI Build CLI — local pipeline orchestration and validation",
   "type": "module",
   "main": "dist/cli.cjs",