@genesislcap/ai-assistant 15.10.1 → 15.10.3

package/dist/dts/main/main.styles.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"main.styles.d.ts","sourceRoot":"","sources":["../../../src/main/main.styles.ts"],"names":[],"mappings":"AA21BA,eAAO,MAAM,MAAM,iDAGlB,CAAC"}
+{"version":3,"file":"main.styles.d.ts","sourceRoot":"","sources":["../../../src/main/main.styles.ts"],"names":[],"mappings":"AAi2BA,eAAO,MAAM,MAAM,iDAGlB,CAAC"}

package/dist/esm/main/main.styles.js

@@ -280,7 +280,13 @@ const baseStyles = css `
       var(--accent-fill-rest) 0%,
       color-mix(in srgb, var(--accent-fill-rest) 70%, black) 100%
     );
-    color: var(--accent-foreground-on-accent-rest);
+
+    /* The on-accent foreground token that pairs with the --accent-fill-rest background
+       above. It must be the design system's real token name: color is inherited, so an
+       undefined custom property with no fallback is invalid at computed-value time and
+       silently leaves the text at the inherited neutral foreground — which reads as
+       white-ish (fine) on a dark app and near-black (unreadable) on a light one. */
+    color: var(--foreground-on-accent-rest);
 
     /* Squared "tail" at top-right — the corner nearest the user avatar. */
     border-radius: 18px 4px 18px 18px;

package/dist/esm/main/main.styles.test.js

@@ -1,5 +1,6 @@
-var _a, _b, _c;
+var _a, _b, _c, _d, _e;
 import { assert, createLogicSuite } from '@genesislcap/foundation-testing';
+import { foregroundOnAccentRest } from '@genesislcap/foundation-ui';
 import { styles } from './main.styles';
 // CSS is not otherwise unit-tested in this package, so this covers the two
 // properties of the BLOCKED BANNER that are cheap to assert from the generated
@@ -20,6 +21,9 @@ import { styles } from './main.styles';
 // — quietly turning a banner test into a global style policy nobody agreed to, so
 // any future rule anywhere in an 800-line sheet that legitimately wanted a
 // fallback on either token would have failed a test named after the banner.
+//
+// A second suite at the bottom covers the user bubble's fill/foreground token
+// pairing, which failed the same way a reviewer's eye does (GENC-1491).
 const suite = createLogicSuite('main.styles blocked-banner rules');
 /**
  * The generated stylesheet text. `css` composes nested `ElementStyles` (this file
@@ -97,3 +101,26 @@ suite('partial exhaustion has its own modifier, layered on the visible state', (
     assert.ok(cssText.includes('.blocked-banner.is-partial'), 'the partial modifier exists');
 });
 suite.run();
+// ─── User message bubble ──────────────────────────────────────────────────────
+const bubble = createLogicSuite('main.styles user-message colour pairing');
+/** Declarations of the `.message.user` rule — the accent-filled bubble. */
+const userMessageRule = (_e = (_d = cssText.match(/\.message\.user\s*\{([^}]*)\}/)) === null || _d === void 0 ? void 0 : _d[1]) !== null && _e !== void 0 ? _e : '';
+bubble('the user bubble pairs its accent fill with the on-accent foreground', () => {
+    // The bubble fills with --accent-fill-rest, so its text MUST come from the
+    // design system's paired on-accent token. This shipped as
+    // `--accent-foreground-on-accent-rest` — a name nothing defines — and the
+    // failure was invisible in review and in dark mode: `color` is inherited, so an
+    // undefined custom property with no fallback is invalid at computed-value time
+    // and quietly leaves the text at the inherited neutral foreground. That reads
+    // white-ish on a dark app (looks intentional) and near-black on a light one,
+    // where it landed at ~1.3:1 against the fill (GENC-1491).
+    //
+    // Asserted against the token object's OWN cssCustomProperty rather than a
+    // repeated string literal: a typo here cannot agree with itself, and a rename
+    // in the design system fails this test instead of silently reintroducing the
+    // inherit.
+    assert.ok(userMessageRule.length > 0, 'the .message.user rule is in the sheet');
+    assert.ok(userMessageRule.includes('var(--accent-fill-rest)'), 'the bubble is filled from the accent token (the premise of the pairing)');
+    assert.ok(userMessageRule.includes(`color: var(${foregroundOnAccentRest.cssCustomProperty})`), `the text colour is ${foregroundOnAccentRest.cssCustomProperty}, which the design system defines`);
+});
+bubble.run();

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@genesislcap/ai-assistant",
   "description": "Genesis AI Assistant micro-frontend",
-  "version": "15.10.1",
+  "version": "15.10.3",
   "license": "SEE LICENSE IN license.txt",
   "main": "dist/esm/index.js",
   "types": "dist/ai-assistant.d.ts",
@@ -73,26 +73,26 @@
     }
   },
   "devDependencies": {
-    "@genesislcap/foundation-testing": "15.10.1",
-    "@genesislcap/genx": "15.10.1",
-    "@genesislcap/rollup-builder": "15.10.1",
-    "@genesislcap/ts-builder": "15.10.1",
-    "@genesislcap/uvu-playwright-builder": "15.10.1",
-    "@genesislcap/vite-builder": "15.10.1",
-    "@genesislcap/webpack-builder": "15.10.1",
+    "@genesislcap/foundation-testing": "15.10.3",
+    "@genesislcap/genx": "15.10.3",
+    "@genesislcap/rollup-builder": "15.10.3",
+    "@genesislcap/ts-builder": "15.10.3",
+    "@genesislcap/uvu-playwright-builder": "15.10.3",
+    "@genesislcap/vite-builder": "15.10.3",
+    "@genesislcap/webpack-builder": "15.10.3",
     "@types/dompurify": "^3.0.5",
     "@types/marked": "^5.0.2",
     "esbuild": "0.25.12"
   },
   "dependencies": {
-    "@genesislcap/foundation-ai": "15.10.1",
-    "@genesislcap/foundation-logger": "15.10.1",
-    "@genesislcap/foundation-notifications": "15.10.1",
-    "@genesislcap/foundation-redux": "15.10.1",
-    "@genesislcap/foundation-ui": "15.10.1",
-    "@genesislcap/foundation-utils": "15.10.1",
-    "@genesislcap/rapid-design-system": "15.10.1",
-    "@genesislcap/web-core": "15.10.1",
+    "@genesislcap/foundation-ai": "15.10.3",
+    "@genesislcap/foundation-logger": "15.10.3",
+    "@genesislcap/foundation-notifications": "15.10.3",
+    "@genesislcap/foundation-redux": "15.10.3",
+    "@genesislcap/foundation-ui": "15.10.3",
+    "@genesislcap/foundation-utils": "15.10.3",
+    "@genesislcap/rapid-design-system": "15.10.3",
+    "@genesislcap/web-core": "15.10.3",
     "dompurify": "^3.3.1",
     "marked": "^17.0.3"
   },
@@ -105,5 +105,5 @@
     "access": "public"
   },
   "customElements": "dist/custom-elements.json",
-  "gitHead": "37e7d73f7494436cdf07f1b34eb7563450d77428"
+  "gitHead": "6e5c9bb135b3da6792c4c3a8e632b09b0bc3fbca"
 }

package/src/main/main.styles.test.ts

@@ -1,4 +1,5 @@
 import { assert, createLogicSuite } from '@genesislcap/foundation-testing';
+import { foregroundOnAccentRest } from '@genesislcap/foundation-ui';
 import { styles } from './main.styles';
 
 // CSS is not otherwise unit-tested in this package, so this covers the two
@@ -20,6 +21,9 @@ import { styles } from './main.styles';
 // — quietly turning a banner test into a global style policy nobody agreed to, so
 // any future rule anywhere in an 800-line sheet that legitimately wanted a
 // fallback on either token would have failed a test named after the banner.
+//
+// A second suite at the bottom covers the user bubble's fill/foreground token
+// pairing, which failed the same way a reviewer's eye does (GENC-1491).
 
 const suite = createLogicSuite('main.styles blocked-banner rules');
 
@@ -128,3 +132,37 @@ suite('partial exhaustion has its own modifier, layered on the visible state', (
 });
 
 suite.run();
+
+// ─── User message bubble ──────────────────────────────────────────────────────
+
+const bubble = createLogicSuite('main.styles user-message colour pairing');
+
+/** Declarations of the `.message.user` rule — the accent-filled bubble. */
+const userMessageRule = cssText.match(/\.message\.user\s*\{([^}]*)\}/)?.[1] ?? '';
+
+bubble('the user bubble pairs its accent fill with the on-accent foreground', () => {
+  // The bubble fills with --accent-fill-rest, so its text MUST come from the
+  // design system's paired on-accent token. This shipped as
+  // `--accent-foreground-on-accent-rest` — a name nothing defines — and the
+  // failure was invisible in review and in dark mode: `color` is inherited, so an
+  // undefined custom property with no fallback is invalid at computed-value time
+  // and quietly leaves the text at the inherited neutral foreground. That reads
+  // white-ish on a dark app (looks intentional) and near-black on a light one,
+  // where it landed at ~1.3:1 against the fill (GENC-1491).
+  //
+  // Asserted against the token object's OWN cssCustomProperty rather than a
+  // repeated string literal: a typo here cannot agree with itself, and a rename
+  // in the design system fails this test instead of silently reintroducing the
+  // inherit.
+  assert.ok(userMessageRule.length > 0, 'the .message.user rule is in the sheet');
+  assert.ok(
+    userMessageRule.includes('var(--accent-fill-rest)'),
+    'the bubble is filled from the accent token (the premise of the pairing)',
+  );
+  assert.ok(
+    userMessageRule.includes(`color: var(${foregroundOnAccentRest.cssCustomProperty})`),
+    `the text colour is ${foregroundOnAccentRest.cssCustomProperty}, which the design system defines`,
+  );
+});
+
+bubble.run();

package/src/main/main.styles.ts

@@ -286,7 +286,13 @@ const baseStyles = css`
       var(--accent-fill-rest) 0%,
       color-mix(in srgb, var(--accent-fill-rest) 70%, black) 100%
     );
-    color: var(--accent-foreground-on-accent-rest);
+
+    /* The on-accent foreground token that pairs with the --accent-fill-rest background
+       above. It must be the design system's real token name: color is inherited, so an
+       undefined custom property with no fallback is invalid at computed-value time and
+       silently leaves the text at the inherited neutral foreground — which reads as
+       white-ish (fine) on a dark app and near-black (unreadable) on a light one. */
+    color: var(--foreground-on-accent-rest);
 
     /* Squared "tail" at top-right — the corner nearest the user avatar. */
     border-radius: 18px 4px 18px 18px;