@adia-ai/a2ui-compose 0.7.25 → 0.7.27

package/CHANGELOG.md

@@ -1,4 +1,19 @@
 # Changelog — @adia-ai/a2ui-compose
+## [0.7.27] — 2026-07-12
+
+### Fixed
+- **Transpiler — `[span]` is now universally preserved (TKT-0002).** `extractProps()` had a universal preserve for `[slot]` but not `[span]` — the same class of cross-cutting, layout-bearing attribute, consumed purely via `grid.css`'s `& > [span="N"]` child selectors and never declared in any component's own yaml `props:`. Every transpiled asymmetric grid collapsed to equal columns (the admin-dashboard's 3:2 Overview/Recent-Sales split was the operator-reported case). Regression tests cover `span="3"`, `span="full"`, and the absent case.
+- **Transpiler — `Tag` is a leaf type; labels ride the `text` prop (TKT-0004 A).** `Tag` was missing from `LEAF_TYPES`, so `<tag-ui size="sm">label</tag-ui>` transpiled to `Tag > Text variant=body` — a full body-size paragraph inside the pill, inflating every converted chip (581 instances across 233 docs pages measured). Labels now ride tag.yaml's documented `attr(text)` contract, in both the pure-text and element-children extraction paths.
+- **`strategies/zettel/issue-reporter.js` — issue storage repointed to `qa/findings/issues/`.** The previous path would have silently recreated the retired `.brain/` tree at runtime on the first `report_issue` call.
+
+### Maintenance
+- Transpiler test suite: `extract-props.test.js` 19 → 24 cases (span preservation ×3, Tag text contract ×2).
+
+## [0.7.26] — 2026-07-04
+
+### Fixed
+- **Monolithic engine no longer teaches `variant="outline"` as the encoding for "secondary button."** `strategies/monolithic/_shared.js`'s hardcoded system-prompt text — sent to the LLM on every generation call — read `variant="primary" for main action, "outline" for secondary, "ghost" for tertiary`, actively instructing every generation to reach for `outline` on any non-primary button. `button-ui`'s unstyled default already renders the neutral/secondary look (see `@adia-ai/web-components` changelog); corrected the prompt text to say so explicitly and to leave `variant` unset for secondary/default actions, reserving `variant="primary"` for the one CTA that should stand out and `variant="ghost"` for genuinely lower-emphasis tertiary actions. Also fixed the same anti-pattern hardcoded into the file's fallback-error-UI generator — a "Copy Debug Info" button built with `variant: 'outline'` right next to a `variant: 'primary'` "Try Again" — dropped the attribute there too.
+- Corpus evidence that motivated the investigation: of 446 `<button-ui>` tags across harvested demo-page chunks, only 30 were bare vs. 91 explicit `variant="outline"` + 215 `variant="ghost"` — confirming the prompt text (not incidental author habit) was the dominant driver.
 
 ## [0.7.25] — 2026-07-03
 
@@ -910,7 +925,7 @@ See root [CHANGELOG.md `[Unreleased]`](../../../CHANGELOG.md) for the v0.4.5 ove
 - **`compose/strategies/monolithic/generate-pro.js` + companion `_shared/fragment-expander.js` — fragment machinery removed (§37).** Fragments retired; 85 compositions inlined with previously-referenced fragment content. Expander deleted; its caller's try/catch in `generator-adapter.js` handles the absence gracefully.
 - **MCP server `mcp/server.js` (§37).** Removed `get_fragment` tool, simplified `zettel_stats`, updated imports + boot logs + engine description to reflect the post-fragment world.
 
-See root [CHANGELOG.md `[Unreleased]`](../../../CHANGELOG.md) for the cross-cutting arc narrative + [.claude/docs/journal/2026/05/2026-05-12.md](../../../.claude/docs/journal/2026/05/2026-05-12.md) §§ 37 / 38 / 41 / 47 for per-§ details.
+See root [CHANGELOG.md `[Unreleased]`](../../../CHANGELOG.md) for the cross-cutting arc narrative + `.claude/docs/journal/2026/05/2026-05-12.md` §§ 37 / 38 / 41 / 47 for per-§ details.
 ## [0.4.3] - 2026-05-11
 
 ### Fixed
@@ -1194,7 +1209,7 @@ packages now share one version, governed by
 [`.claude/docs/specs/package-architecture.md` § 15 (Versioning Policy)](../../../.claude/docs/specs/package-architecture.md#15-versioning-policy).
 This release also lands the `engine/` ↔ `engines/` collision fix from
 T3 of the
-[`.claude/docs/plans/packages-architecture-fixes-2026-05-02.md`](../../../.brain/archive/2026-Q2/PLAN-packages-architecture-fixes-2026-05-02.md)
+`.claude/docs/plans/packages-architecture-fixes-2026-05-02.md`
 plan.
 
 ### Changed
@@ -1249,8 +1264,8 @@ multi-turn surface, plus a first-class telemetry surface that auto-fires
 issue tickets on engine failure paths.
 
 Spec: [`.claude/docs/specs/genui-multiturn-architecture.md`](../../../.claude/docs/specs/genui-multiturn-architecture.md) (Active v0.1.0).
-Plan: [`.claude/docs/plans/genui-multiturn-rollout-2026-04-28.md`](../../../.claude/docs/plans/genui-multiturn-rollout-2026-04-28.md) (Phase A scoped).
-ADR: [`0008-multiturn-genui-architecture.md`](../../../.brain/adrs/0008-multiturn-genui-architecture.md).
+Plan: `.claude/docs/plans/genui-multiturn-rollout-2026-04-28.md` (Phase A scoped).
+ADR: `0008-multiturn-genui-architecture.md`.
 
 ### Added (engine modules)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adia-ai/a2ui-compose",
-  "version": "0.7.25",
+  "version": "0.7.27",
   "description": "AdiaUI A2UI compose engine \u2014 framework-agnostic. Takes natural-language intents + a catalog and produces A2UI protocol messages. Pairs with `@adia-ai/a2ui-retrieval` (intent classification, catalog lookup) and `@adia-ai/a2ui-validator` (schema + semantic checks).",
   "type": "module",
   "exports": {

package/strategies/monolithic/_shared.js

@@ -245,7 +245,7 @@ KEY RULES:
 - Use Table for tabular data (set data/columns via JS props, not Text children).
 - Use EmptyState for no-content/error states (icon, heading, description, action slot).
 - Use Skeleton for loading placeholders (variant: text|circle|rect).
-- Button: use variant="primary" for main action, "outline" for secondary, "ghost" for tertiary.
+- Button: use variant="primary" for the one main CTA that should stand out. Leave variant unset for secondary/default actions — the default (no attribute) already IS the secondary look; do NOT add variant="outline" just to mark something as non-primary. Reserve variant="ghost" for a genuinely lower-emphasis tertiary action.
 - Button: use text prop for button label.
 - NEVER use Text for clickable/interactive actions. "View logs", "Contact support", "Learn more" = Button (variant="ghost" or "link"), NOT Text. Text is for static content only.
 - For error/status dialogs: use Alert for the error message, Button for recovery actions (retry, view logs, contact support). Don't render action labels as Text captions.
@@ -1391,7 +1391,7 @@ function fallbackMessage(reason, opts = {}) {
 
   if (executionId || rawResponse) {
     footerChildren.push('copy-btn');
-    components.push({ id: 'copy-btn', component: 'Button', text: 'Copy Debug Info', icon: 'clipboard', variant: 'outline', 'data-clipboard': debugPayload });
+    components.push({ id: 'copy-btn', component: 'Button', text: 'Copy Debug Info', icon: 'clipboard', 'data-clipboard': debugPayload });
   }
 
   components.push(

package/strategies/zettel/issue-reporter.js

@@ -7,7 +7,7 @@
  *   3. Engine auto-fire — internal failure paths trigger `autoReport(reason)`.
  *      ctx.reporter = 'auto'. Suppressed when ctx.evalMode is true.
  *
- * Every issue lands as immutable JSON under `.brain/audit-history/issues/`.
+ * Every issue lands as immutable JSON under `qa/findings/issues/`.
  * Traces > 200KB spill to a sidecar `.trace.json` next to the issue.
  *
  * Spec: .claude/docs/specs/genui-multiturn-architecture.md §3.5 + §4.6 + §11.
@@ -23,7 +23,7 @@ import { fileURLToPath } from 'node:url';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const REPO_ROOT = resolve(__dirname, '../../../../..');
-const DEFAULT_STORAGE_ROOT = resolve(REPO_ROOT, '.brain/audit-history/issues');
+const DEFAULT_STORAGE_ROOT = resolve(REPO_ROOT, 'qa/findings/issues');
 const TRACE_INLINE_THRESHOLD_BYTES = 200 * 1024;
 
 const VALID_TYPES = new Set(['bug', 'training-gap', 'protocol-gap', 'ux-feedback']);

package/transpiler/extract-props.test.js

@@ -225,4 +225,39 @@ describe('§163 extractProps — catalog-driven prop fidelity', () => {
       expect(props.variant).toBe('outline');
     });
   });
+
+  describe('TKT-0004 — Tag label via text prop (attr(text) contract)', () => {
+    it('falls back to light-DOM textContent when no text= attribute', () => {
+      const el = elem({ size: 'sm', variant: 'success' }, { text: '~1 min · no tools' });
+      const props = extractProps(el, 'Tag');
+      expect(props.text).toBe('~1 min · no tools');
+      expect(props.size).toBe('sm');
+    });
+
+    it('an explicit text= attribute wins over textContent', () => {
+      const el = elem({ text: 'explicit' }, { text: 'ignored light-dom' });
+      const props = extractProps(el, 'Tag');
+      expect(props.text).toBe('explicit');
+    });
+  });
+
+  describe('TKT-0002 — [span] universal preservation', () => {
+    it('preserves span on a Card (grid-ui child), same as slot', () => {
+      const el = elem({ span: '3' });
+      const props = extractProps(el, 'Card');
+      expect(props.span).toBe('3');
+    });
+
+    it('preserves span="full" verbatim (not coerced to a number)', () => {
+      const el = elem({ span: 'full' });
+      const props = extractProps(el, 'Card');
+      expect(props.span).toBe('full');
+    });
+
+    it('omits span when absent — no stray empty prop', () => {
+      const el = elem({});
+      const props = extractProps(el, 'Card');
+      expect(props.span).toBeUndefined();
+    });
+  });
 });

package/transpiler/transpiler-maps.js

@@ -317,6 +317,17 @@ export function extractProps(el, a2uiType) {
   const slotAttr = attr('slot');
   if (slotAttr) props.slot = slotAttr;
 
+  // Universal: preserve [span] for every element. Span is a layout-bearing
+  // attribute grid-ui's child reads via a pure CSS attribute selector
+  // (grid.css: `& > [span="3"] { grid-column: span 3; }`) — it belongs to
+  // "whatever this grid child happens to be," never to a specific
+  // component's own prop schema, so no .yaml declares it and the
+  // catalog-driven pass below silently drops it (TKT-0002: collapsed the
+  // admin-dashboard Overview/Recent Sales 3:2 split to two equal, unspanned
+  // columns).
+  const spanAttr = attr('span');
+  if (spanAttr) props.span = spanAttr;
+
   // §163 (v0.5.4) — Catalog-driven generic extraction FIRST. Fills in
   // every prop the v0.9 catalog declares for this type (label, icon,
   // variant=, etc.). Legacy per-type blocks below run AFTER and can
@@ -351,6 +362,15 @@ export function extractProps(el, a2uiType) {
     }
   }
 
+  // ── Tag ── (TKT-0004: tag-ui renders its label via CSS attr(text) —
+  // tag.yaml documents textContent as routing to the text attribute. The
+  // authored form is light-DOM content (<tag-ui size="sm">label</tag-ui>),
+  // so fall back to textContent when no text= attribute is set.)
+  if (a2uiType === 'Tag' && props.text === undefined) {
+    const text = (el.textContent || '').trim().replace(/\s+/g, ' ');
+    if (text) props.text = text;
+  }
+
   // ── Link ── (§47 — <a> tags map to Link, not Button; carries href, target, rel)
   if (a2uiType === 'Link') {
     const text = (el.textContent || '').trim().replace(/\s+/g, ' ');

package/transpiler/transpiler.js

@@ -201,9 +201,13 @@ function walkNode(el, context) {
     return id;
   }
 
-  // Leaf types: these render their own text, don't recurse into children
+  // Leaf types: these render their own text, don't recurse into children.
+  // Tag joined 2026-07-12 (TKT-0004): tag-ui renders its label via CSS
+  // attr(text) — as a container, its light-DOM label became a child Text
+  // at body size, inflating every converted chip site-wide (581 instances
+  // across 233 pages measured).
   const LEAF_TYPES = new Set([
-    'Text', 'Button', 'Kbd', 'Code', 'Badge', 'Image', 'Divider',
+    'Text', 'Button', 'Kbd', 'Code', 'Badge', 'Tag', 'Image', 'Divider',
     'TextField', 'CheckBox', 'Toggle', 'Radio', 'Slider', 'Search',
     'Upload', 'ColorPicker', 'DateTimeInput', 'TextArea', 'OtpInput',
   ]);
@@ -244,7 +248,7 @@ function walkNode(el, context) {
     if (directText) {
       // Preserve the direct text on whichever prop the type uses
       if (a2uiType === 'Text' || a2uiType === 'Kbd' || a2uiType === 'Code') containerProps.textContent = directText;
-      else if (a2uiType === 'Button' || a2uiType === 'Badge') {
+      else if (a2uiType === 'Button' || a2uiType === 'Badge' || a2uiType === 'Tag') {
         if (containerProps.text === undefined) containerProps.text = directText;
       }
     } else {