basalt-ui 1.19.0 → 1.20.0

package/configs/oxlint-plugin.test.ts

@@ -11,6 +11,9 @@ import { afterEach, beforeEach, describe, expect, it } from 'bun:test'
 import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
 import { tmpdir } from 'node:os'
 import { dirname, resolve } from 'node:path'
+// eslint-disable-next-line -- the plugin is plain JS; the ledger + id set are named exports beside it
+import basaltPlugin, { KNOWN_RULE_IDS, PLUGIN_RULE_GRACE } from './oxlint-plugin.js'
+import { GUARD_RULES } from '../src/guard/index.ts'
 
 const PLUGIN_PATH = resolve(import.meta.dirname, 'oxlint-plugin.js')
 const OXLINT_BIN = resolve(import.meta.dirname, '..', '..', '..', 'node_modules', '.bin', 'oxlint')
@@ -30,6 +33,8 @@ beforeEach(() => {
         'basalt/chart-in-raw-surface': 'error',
         'basalt/hand-rolled-plot': 'error',
         'basalt/chart-legend-literal': 'error',
+        'basalt/shadow-basalt-export': 'error',
+        'basalt/hand-rolled-shell': 'error',
         'basalt/visx-boundary': 'error',
         'basalt/visx-tooltip': 'error',
         'basalt/token-layer-boundary': 'error',
@@ -355,28 +360,40 @@ describe('basalt/agent-no-raw-usechat', () => {
 
 describe('basalt/ai-sdk-major', () => {
   it("flags when the nearest package.json declares a different ai major (5 vs basalt's 7)", () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '5.0.196' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '5.0.196' } }),
+    )
     const { code, rules } = run(`import { streamText } from 'ai'\n`, 'lib.ts')
     expect(code).toBe(1)
     expect(rules).toContain('ai-sdk-major')
   })
 
   it('does NOT flag when the nearest package.json declares the same major (^7.0.18)', () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '^7.0.18' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '^7.0.18' } }),
+    )
     const { code, rules } = run(`import { streamText } from 'ai'\n`, 'lib.ts')
     expect(code).toBe(0)
     expect(rules).not.toContain('ai-sdk-major')
   })
 
   it('does NOT flag when there is no ai in the nearest package.json', () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: {} }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1' } }),
+    )
     const { code, rules } = run(`import { streamText } from 'ai'\n`, 'lib.ts')
     expect(code).toBe(0)
     expect(rules).not.toContain('ai-sdk-major')
   })
 
   it('does NOT flag an unrelated import even with a skewed nearest package.json', () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '5.0.196' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '5.0.196' } }),
+    )
     const { code, rules } = run(`import { useState } from 'react'\n`, 'lib.ts')
     expect(code).toBe(0)
     expect(rules).not.toContain('ai-sdk-major')
@@ -385,14 +402,20 @@ describe('basalt/ai-sdk-major', () => {
   // The matcher covers TWO forms (source === 'ai' OR source.startsWith('@ai-sdk/')) — every
   // fixture above only exercises the bare 'ai' specifier; this pins the scoped-package half.
   it("flags a scoped '@ai-sdk/*' import when the nearest package.json declares a different ai major", () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '5.0.196' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '5.0.196' } }),
+    )
     const { code, rules } = run(`import { openai } from '@ai-sdk/openai'\n`, 'lib.ts')
     expect(code).toBe(1)
     expect(rules).toContain('ai-sdk-major')
   })
 
   it("does NOT flag a scoped '@ai-sdk/*' import when the nearest package.json declares the same major", () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '^7.0.18' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '^7.0.18' } }),
+    )
     const { code, rules } = run(`import { openai } from '@ai-sdk/openai'\n`, 'lib.ts')
     expect(code).toBe(0)
     expect(rules).not.toContain('ai-sdk-major')
@@ -402,7 +425,10 @@ describe('basalt/ai-sdk-major', () => {
   // token as its two agent-chat siblings above — not `theme-allow`. A skewed producer/consumer
   // import that is genuinely intentional (see doctor's `aiMajorSkewReason`) can mark the line.
   it('does NOT flag an import marked basalt-agent-allow, even with a skewed nearest package.json', () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '5.0.196' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '5.0.196' } }),
+    )
     const { code, rules } = run(
       `// basalt-agent-allow\nimport { streamText } from 'ai'\n`,
       'lib.ts',
@@ -412,11 +438,44 @@ describe('basalt/ai-sdk-major', () => {
   })
 
   it('STILL flags an import marked only theme-allow (wrong escape token)', () => {
-    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '5.0.196' } }))
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ dependencies: { 'basalt-ui': '^1.19.1', ai: '5.0.196' } }),
+    )
     const { code, rules } = run(`// theme-allow\nimport { streamText } from 'ai'\n`, 'lib.ts')
     expect(code).toBe(1)
     expect(rules).toContain('ai-sdk-major')
   })
+
+  // The scoping fix: a workspace package with no basalt-ui dependency is not a basalt consumer, so
+  // its `ai` pin is none of this rule's business. rb's `apps/api` took 3 errors for exactly this,
+  // while `doctor` — which scopes the same concern by workspace — reported green.
+  it('does NOT flag a package that does not depend on basalt-ui at all', () => {
+    writeFileSync(resolve(dir, 'package.json'), JSON.stringify({ dependencies: { ai: '5.0.196' } }))
+    const { code, rules } = run(`import { streamText } from 'ai'\n`, 'lib.ts')
+    expect(code).toBe(0)
+    expect(rules).not.toContain('ai-sdk-major')
+  })
+
+  it('does NOT flag a file outside a declared basalt.roots', () => {
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ basalt: { roots: ['web/src'] }, dependencies: { ai: '5.0.196' } }),
+    )
+    const { code, rules } = run(`import { streamText } from 'ai'\n`, 'api/src/llm.ts')
+    expect(code).toBe(0)
+    expect(rules).not.toContain('ai-sdk-major')
+  })
+
+  it('DOES flag a file inside a declared basalt.roots', () => {
+    writeFileSync(
+      resolve(dir, 'package.json'),
+      JSON.stringify({ basalt: { roots: ['web/src'] }, dependencies: { ai: '5.0.196' } }),
+    )
+    const { code, rules } = run(`import { streamText } from 'ai'\n`, 'web/src/llm.ts')
+    expect(code).toBe(1)
+    expect(rules).toContain('ai-sdk-major')
+  })
 })
 
 // ── raw-scroll-container — promotion (off → warn → error shipped) ───────────
@@ -500,11 +559,13 @@ describe('basalt/hand-rolled-plot', () => {
     expect(rules).not.toContain('hand-rolled-plot')
   })
 
-  it('reports ONCE per file — declaring the exception is a file-level decision', () => {
+  // A `theme-allow` that NAMES the rule and gives a reason is a written declaration about the file,
+  // and still covers every assembly node in it — the DualPanel shape.
+  it('a named-with-a-reason declaration covers the whole file', () => {
     const { code, rules } = run(
       `${CHARTS_IMPORT}export const C = () => (
   <svg>
-    {/* theme-allow: multi-pane shape, not a single cartesian plot */}
+    {/* theme-allow basalt/hand-rolled-plot: multi-pane shape, not a single cartesian plot */}
     <AxisLeftNumeric scale={a} />
     <HoverOverlay width={1} height={1} />
     <Crosshair x={0} top={0} bottom={1} />
@@ -515,6 +576,37 @@ describe('basalt/hand-rolled-plot', () => {
     expect(rules).not.toContain('hand-rolled-plot')
   })
 
+  // The linewatch defect: a BARE comment used to buy the whole file permanent immunity, because
+  // only the first node was reported and the waiver was tested there. It now waives its own node
+  // and nothing else, so everything added to the file afterwards is still reported.
+  it('a bare theme-allow waives only the node it sits above, not the file', () => {
+    const { code, rules } = run(
+      `${CHARTS_IMPORT}export const C = () => (
+  <svg>
+    {/* theme-allow */}
+    <AxisLeftNumeric scale={a} />
+    <HoverOverlay width={1} height={1} />
+  </svg>
+)\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('hand-rolled-plot')
+  })
+
+  // A comment written about something else must not double as a licence to hand-assemble a plot.
+  it('a theme-allow scoped to a DIFFERENT rule does not waive this one', () => {
+    const { code, rules } = run(
+      `${CHARTS_IMPORT}export const C = () => (
+  <svg>
+    {/* theme-allow raw-hex — the swatch below is a measured pixel */}
+    <AxisLeftNumeric scale={a} />
+  </svg>
+)\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('hand-rolled-plot')
+  })
+
   it('does NOT flag a same-named component that is NOT a basalt chart primitive', () => {
     const { code, rules } = run(
       `import { Crosshair } from './my-own-widgets'
@@ -583,3 +675,284 @@ describe('basalt/chart-legend-literal', () => {
     expect(rules).not.toContain('chart-legend-literal')
   })
 })
+
+// ── shadow-basalt-export ─────────────────────────────────────────────────────
+
+describe('basalt/shadow-basalt-export', () => {
+  it('flags a local component that collides with a basalt-ui root export', () => {
+    const { code, rules } = run(`export function EmptyState() { return null }\n`)
+    expect(code).toBe(1)
+    expect(rules).toContain('shadow-basalt-export')
+  })
+
+  it('flags the arrow/const form too (rb shipped a local StatCard)', () => {
+    const { code, rules } = run(`const StatCard = () => null\nexport default StatCard\n`)
+    expect(code).toBe(1)
+    expect(rules).toContain('shadow-basalt-export')
+  })
+
+  it('does NOT flag a name basalt does not export', () => {
+    const { code, rules } = run(`export function ExerciseSummaryCard() { return null }\n`)
+    expect(code).toBe(0)
+    expect(rules).not.toContain('shadow-basalt-export')
+  })
+
+  // The export list is derived from the real barrel, so a TYPE export must not create a collision:
+  // a local `StatCardProps` is an ordinary thing for a consumer to declare.
+  it('does NOT flag a name that basalt exports only as a type', () => {
+    const { code, rules } = run(`export type StatCardProps = { label: string }\n`, 'types.ts')
+    expect(code).toBe(0)
+    expect(rules).not.toContain('shadow-basalt-export')
+  })
+
+  it('is waivable with a scoped theme-allow', () => {
+    const { code, rules } = run(
+      `// theme-allow basalt/shadow-basalt-export — domain-specific, unrelated to the shipped one\nexport function EmptyState() { return null }\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('shadow-basalt-export')
+  })
+})
+
+// ── hand-rolled-shell ────────────────────────────────────────────────────────
+
+describe('basalt/hand-rolled-shell', () => {
+  it('flags a hand-assembled Mantine AppShell', () => {
+    const { code, rules } = run(
+      `import { AppShell, Burger } from '@mantine/core'
+export const Shell = () => (
+  <AppShell>
+    <AppShell.Header><Burger opened={false} /></AppShell.Header>
+    <AppShell.Navbar>nav</AppShell.Navbar>
+  </AppShell>
+)\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('hand-rolled-shell')
+  })
+
+  it('does NOT flag a file that renders BasaltShell', () => {
+    const { code, rules } = run(
+      `import { AppShell } from '@mantine/core'
+import { BasaltShell } from 'basalt-ui'
+export const Shell = () => <BasaltShell sections={[]}><AppShell.Main>x</AppShell.Main></BasaltShell>\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('hand-rolled-shell')
+  })
+
+  // Exempt by DECLARATION, not by path — a rule saying "compose X" cannot fire inside X. (The
+  // fixture also trips `shadow-basalt-export`, which is correct for a consumer file and irrelevant
+  // here, so this asserts the rule set rather than the exit code.)
+  it('does NOT flag the module that DEFINES BasaltShell', () => {
+    const { rules } = run(
+      `import { AppShell } from '@mantine/core'
+export function BasaltShell() { return <AppShell.Navbar>nav</AppShell.Navbar> }\n`,
+    )
+    expect(rules).not.toContain('hand-rolled-shell')
+  })
+
+  it("does NOT flag a consumer's own AppShell-named component with no @mantine import", () => {
+    const { code, rules } = run(
+      `import { AppShell } from './my-layout'
+export const C = () => <AppShell.Navbar>nav</AppShell.Navbar>\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('hand-rolled-shell')
+  })
+
+  it('does NOT flag a standalone Mantine NavLink (deliberately not a trigger)', () => {
+    const { code, rules } = run(
+      `import { NavLink } from '@mantine/core'
+export const C = () => <NavLink label="Settings" />\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('hand-rolled-shell')
+  })
+})
+
+// ── grace ledger ↔ shipped preset ────────────────────────────────────────────
+
+describe('PLUGIN_RULE_GRACE', () => {
+  const shipped = JSON.parse(readFileSync(resolve(import.meta.dirname, 'oxlint.json'), 'utf8')) as {
+    rules: Record<string, unknown>
+  }
+  const graceIds = Object.keys(PLUGIN_RULE_GRACE)
+  const shippedBasaltIds = Object.keys(shipped.rules).filter((id) => id.startsWith('basalt/'))
+
+  // The mechanism whose ABSENCE let three rules sit at `warn` for up to twelve minors with nothing
+  // tracking them. Deleting a ledger entry IS the promotion, and this test makes flipping the
+  // shipped level part of the same commit.
+  it('every ledger entry is warn in the shipped preset', () => {
+    for (const id of graceIds) expect(shipped.rules[`basalt/${id}`]).toBe('warn')
+  })
+
+  it('every shipped rule NOT in the ledger is error', () => {
+    for (const id of shippedBasaltIds) {
+      if (graceIds.includes(id.slice('basalt/'.length))) continue
+      expect([id, shipped.rules[id]]).toEqual([id, 'error'])
+    }
+  })
+
+  it('every ledger entry carries a written promotion note', () => {
+    for (const id of graceIds) {
+      expect((PLUGIN_RULE_GRACE as Record<string, string>)[id]?.length ?? 0).toBeGreaterThan(40)
+    }
+  })
+})
+
+// ── chart-legend-literal: the .map() half ────────────────────────────────────
+
+describe('basalt/chart-legend-literal — derived-from-series', () => {
+  const IMPORT = `import { ChartLegend } from 'basalt-ui/charts'\n`
+
+  // Deriving from *an* array is not deriving from *the* series: every field of a
+  // `refLines.map(...)` / `PACE_ZONES.map(...)` legend is authored at the call site, and the list
+  // can keep naming a band the plot no longer draws exactly like a `[...]` literal can.
+  it('flags a .map() over a non-series array', () => {
+    const { code, rules } = run(
+      `${IMPORT}export const C = () => <ChartLegend items={refLines.map((r) => ({ key: r.k, label: r.l }))} />\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('chart-legend-literal')
+  })
+
+  it('does NOT flag a .map() over the series', () => {
+    const { code, rules } = run(
+      `${IMPORT}export const C = () => <ChartLegend items={series.map((s) => ({ key: s.key }))} />\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('chart-legend-literal')
+  })
+
+  it('does NOT flag a .map() over a derived series binding', () => {
+    const { code, rules } = run(
+      `${IMPORT}export const C = () => <ChartLegend items={visibleSeries.map((s) => ({ key: s.key }))} />\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('chart-legend-literal')
+  })
+
+  it('does NOT flag deriveLegend(series)', () => {
+    const { code, rules } = run(
+      `${IMPORT}export const C = () => <ChartLegend items={deriveLegend(series)} />\n`,
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('chart-legend-literal')
+  })
+})
+
+// ── no-raw-font-size: style-context scoping ──────────────────────────────────
+
+describe('basalt/no-raw-font-size — style context', () => {
+  it('flags a fontSize inside a style attribute', () => {
+    const { code, rules } = run(`export const C = () => <div style={{ fontSize: 11 }} />\n`)
+    expect(code).toBe(1)
+    expect(rules).toContain('no-raw-font-size')
+  })
+
+  it('flags a fontSize inside a Mantine styles per-part object', () => {
+    const { code, rules } = run(
+      `export const C = () => <TextInput styles={{ input: { fontSize: 12 } }} />\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('no-raw-font-size')
+  })
+
+  it('flags a fontSize in a CSSProperties-annotated const', () => {
+    const { code, rules } = run(
+      `import type { CSSProperties } from 'react'\nconst rowLook: CSSProperties = { fontSize: 11 }\nexport { rowLook }\n`,
+      'look.ts',
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('no-raw-font-size')
+  })
+
+  // The obsidian false positive: a `fontSize` key in a plain data object, in a package with no
+  // React and no Mantine, where "route it through VX.text.*" is advice about another domain.
+  it('does NOT flag a fontSize key in a plain data object', () => {
+    const { code, rules } = run(
+      `export const iconizeData = { settings: { fontSize: 16 }, Inbox: 'LiInbox' }\n`,
+      'data.ts',
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('no-raw-font-size')
+  })
+
+  it('does NOT run in a test file at all', () => {
+    const { code, rules } = run(
+      `export const C = () => <div style={{ fontSize: 11 }} />\n`,
+      'nav-config.test.tsx',
+    )
+    expect(code).toBe(0)
+    expect(rules).not.toContain('no-raw-font-size')
+  })
+})
+
+// ── KNOWN_RULE_IDS — the hand-maintained list a typo'd theme-allow is measured against ──────────
+
+describe('KNOWN_RULE_IDS', () => {
+  // Forgetting an entry is not cosmetic: an id this set does not know is treated as a typo, so the
+  // annotation naming it waives nothing. Before the fail-closed fix it was worse — an unknown id
+  // fell through to the empty-`rules` branch and became a BLANKET waiver, which is how a
+  // hand-maintained duplicate turned a missing entry into a hole in the guard.
+  it('contains every rule the plugin actually exports', () => {
+    for (const id of Object.keys(basaltPlugin.rules)) {
+      expect([id, KNOWN_RULE_IDS.has(id)]).toEqual([id, true])
+    }
+  })
+
+  it('contains every guard kind', () => {
+    for (const kind of Object.keys(GUARD_RULES)) {
+      expect([kind, KNOWN_RULE_IDS.has(kind)]).toEqual([kind, true])
+    }
+  })
+
+  it('contains nothing else — it is exactly the union of the two registries', () => {
+    const expected = [...Object.keys(basaltPlugin.rules), ...Object.keys(GUARD_RULES)].toSorted()
+    expect([...KNOWN_RULE_IDS].toSorted()).toEqual(expected)
+  })
+})
+
+// ── theme-allow fails closed on a typo ─────────────────────────────────────────────────────────
+
+describe('theme-allow with an unrecognized rule id', () => {
+  it('waives nothing rather than escalating to a blanket suppression', () => {
+    // `no-raw-font-size` would be waived by a bare `theme-allow`. A typo'd id must not do that.
+    const { code, rules } = run(
+      `export const C = () => <Text fz={10} /> // theme-allow no-raw-font-sizee — legacy\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('no-raw-font-size')
+  })
+
+  it('a correctly spelled id still waives', () => {
+    const { code, rules } = run(
+      `export const C = () => <Text fz={10} /> // theme-allow no-raw-font-size — legacy\n`,
+    )
+    expect(rules).not.toContain('no-raw-font-size')
+    expect(code).toBe(0)
+  })
+
+  it('a bare theme-allow still waives everything', () => {
+    const { rules } = run(`export const C = () => <Text fz={10} /> // theme-allow\n`)
+    expect(rules).not.toContain('no-raw-font-size')
+  })
+
+  it('does not resolve an Object.prototype key as a rule id', () => {
+    const { code, rules } = run(
+      `export const C = () => <Text fz={10} /> // theme-allow constructor — inherited key\n`,
+    )
+    expect(code).toBe(1)
+    expect(rules).toContain('no-raw-font-size')
+  })
+
+  // The id slot closes at the first space no comma opened — mirrors `parseAllowAnnotation`.
+  it('reads an unknown word after a resolved id as the reason, not as a second id', () => {
+    const { code, rules } = run(
+      `export const C = () => <Text fz={10} /> // theme-allow no-raw-font-size sub-scale micro label\n`,
+    )
+    expect(rules).not.toContain('no-raw-font-size')
+    expect(code).toBe(0)
+  })
+})

package/configs/oxlint.json

@@ -16,12 +16,21 @@
     "typescript/no-explicit-any": "warn",
     "import/no-duplicates": "error",
     "import/no-unassigned-import": "off",
+    "jsx-a11y/prefer-tag-over-role": "off",
+    "no-underscore-dangle": [
+      "warn",
+      {
+        "allow": ["__APP_VERSION__"]
+      }
+    ],
     "basalt/no-raw-font-size": "error",
-    "basalt/raw-size-literal": "warn",
+    "basalt/raw-size-literal": "error",
     "basalt/card-inset": "error",
     "basalt/chart-in-raw-surface": "error",
     "basalt/hand-rolled-plot": "warn",
     "basalt/chart-legend-literal": "warn",
+    "basalt/shadow-basalt-export": "warn",
+    "basalt/hand-rolled-shell": "warn",
     "basalt/raw-scroll-container": "error",
     "basalt/visx-boundary": "error",
     "basalt/visx-tooltip": "error",

package/dist/agent/stick-to-bottom.js

@@ -3,7 +3,7 @@ import { lazy, Suspense } from "react";
 import { VX } from "../tokens/index.js";
 function PlainContainerFallback({ children, className, style }) {
   const props = {
-    // theme-allow — scroll anchoring needs a real scrollable node it owns, not a ScrollArea.
+    // theme-allow raw-scroll-container — scroll anchoring needs a real scrollable node it owns, not a ScrollArea.
     style: { overflowY: "auto", ...style }
   };
   if (className !== void 0) props.className = className;

package/dist/agent-chat/thread-detail-panel.js

@@ -67,7 +67,7 @@ function ThreadDetailPanel({
       {
         style: {
           height: "100%",
-          // theme-allow — BasaltStickToBottom owns this scroll node (see stick-to-bottom.tsx).
+          // theme-allow raw-scroll-container — BasaltStickToBottom owns this scroll node (see stick-to-bottom.tsx).
           overflowY: "auto",
           padding: "var(--vx-space-agent-transcript-inset)"
         },

package/dist/agent-chat/thread-message.js

@@ -242,7 +242,7 @@ const DEFAULT_VIRTUALIZE_INITIAL_SCROLL = "end";
 const MAX_INITIAL_SCROLL_ATTEMPTS = 5;
 function NonVirtualizedRows({ rows, height }) {
   return (
-    // theme-allow — degrade target owns its own scroll node, same as the real virtualizer below.
+    // theme-allow raw-scroll-container — degrade target owns its own scroll node, same as the real virtualizer below.
     /* @__PURE__ */ jsx(Box, { style: { height, overflow: "auto" }, children: /* @__PURE__ */ jsx(Stack, { gap: "sm", children: rows.map((row) => /* @__PURE__ */ jsx(Fragment, { children: row.node }, row.key)) }) })
   );
 }
@@ -331,7 +331,7 @@ const LazyVirtualizedRows = lazy(
         );
       });
       return (
-        // theme-allow — TanStack Virtual measures/scrolls this element (never nest in BasaltStickToBottom, see ./virtualize).
+        // theme-allow raw-scroll-container — TanStack Virtual measures/scrolls this element (never nest in BasaltStickToBottom, see ./virtualize).
         /* @__PURE__ */ jsx(Box, { ref: parentRef, style: { height, overflow: "auto" }, children: /* @__PURE__ */ jsx(Box, { style: { height: virtualizer.getTotalSize(), position: "relative" }, children: virtualizer.getVirtualItems().map((virtualItem) => {
           const row = rows[virtualItem.index];
           if (row === void 0) return null;

package/dist/agent-chat/tool-chip.js

@@ -73,7 +73,7 @@ function ToolChip({
             height: 6,
             flexShrink: 0,
             borderRadius: "50%",
-            // theme-allow: a 6px state dot is a circle, not a surface corner — no radius token expresses it
+            // theme-allow raw-surface — a 6px state dot is a circle, not a surface corner
             backgroundColor: dotToken
           }
         }

package/dist/charts/kinds/Donut.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Donut.d.ts","sourceRoot":"","sources":["../../../src/charts/kinds/Donut.tsx"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAgB,SAAS,EAAE,MAAM,OAAO,CAAA;AAKpD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,gBAAgB,CAAA;AAE/C;;;;GAIG;AACH,MAAM,MAAM,UAAU,CAAC,CAAC,SAAS,MAAM,GAAG,SAAS,IAAI;IAAE,GAAG,EAAE,CAAC,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAA;AAEhF,MAAM,MAAM,UAAU,CAAC,CAAC,SAAS,MAAM,GAAG,SAAS,IAAI;IACrD,IAAI,EAAE,UAAU,CAAC,CAAC,CAAC,EAAE,CAAA;IACrB,mFAAmF;IACnF,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,WAAW,EAAE,CAAC,GAAG,EAAE,CAAC,KAAK,MAAM,CAAA;IAC/B,WAAW,EAAE,CAAC,CAAC,EAAE,MAAM,KAAK,MAAM,CAAA;IAClC,WAAW,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,MAAM,CAAA;IACrC,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,cAAc,CAAC,EAAE,MAAM,CAAA;IACvB;;;;;;;;;;;;;;;;;;;;;;OAsBG;IACH,aAAa,CAAC,EAAE,SAAS,CAAA;IACzB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,+FAA+F;IAC/F,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,4FAA4F;IAC5F,SAAS,CAAC,EAAE,OAAO,CAAA;CACpB,CAAA;AAED;;;;;;;;GAQG;AACH,iBAAS,UAAU,CAAC,CAAC,SAAS,MAAM,GAAG,SAAS,EAAE,KAAK,EAAE,UAAU,CAAC,CAAC,CAAC,+BAoBrE;AA2JD;;;GAGG;AACH,eAAO,MAAM,KAAK,EAAuB,OAAO,UAAU,CAAA"}
+{"version":3,"file":"Donut.d.ts","sourceRoot":"","sources":["../../../src/charts/kinds/Donut.tsx"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAgB,SAAS,EAAE,MAAM,OAAO,CAAA;AAKpD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,gBAAgB,CAAA;AAE/C;;;;GAIG;AACH,MAAM,MAAM,UAAU,CAAC,CAAC,SAAS,MAAM,GAAG,SAAS,IAAI;IAAE,GAAG,EAAE,CAAC,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAA;AAEhF,MAAM,MAAM,UAAU,CAAC,CAAC,SAAS,MAAM,GAAG,SAAS,IAAI;IACrD,IAAI,EAAE,UAAU,CAAC,CAAC,CAAC,EAAE,CAAA;IACrB,mFAAmF;IACnF,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,WAAW,EAAE,CAAC,GAAG,EAAE,CAAC,KAAK,MAAM,CAAA;IAC/B,WAAW,EAAE,CAAC,CAAC,EAAE,MAAM,KAAK,MAAM,CAAA;IAClC,WAAW,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,MAAM,CAAA;IACrC,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,cAAc,CAAC,EAAE,MAAM,CAAA;IACvB;;;;;;;;;;;;;;;;;;;;;;OAsBG;IACH,aAAa,CAAC,EAAE,SAAS,CAAA;IACzB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,+FAA+F;IAC/F,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,4FAA4F;IAC5F,SAAS,CAAC,EAAE,OAAO,CAAA;CACpB,CAAA;AAED;;;;;;;;GAQG;AACH,iBAAS,UAAU,CAAC,CAAC,SAAS,MAAM,GAAG,SAAS,EAAE,KAAK,EAAE,UAAU,CAAC,CAAC,CAAC,+BAoBrE;AA4JD;;;GAGG;AACH,eAAO,MAAM,KAAK,EAAuB,OAAO,UAAU,CAAA"}

package/dist/charts/kinds/Donut.js

@@ -129,8 +129,8 @@ function DonutPlot(props) {
         style: {
           position: "absolute",
           inset: 0,
+          // Centering over the donut hole; charts/ is Mantine-free, so Center is unavailable here.
           display: "flex",
-          // theme-allow: centering over the donut hole — charts/ is Mantine-free, so Center is unavailable here
           alignItems: "center",
           justifyContent: "center",
           pointerEvents: "none"

package/dist/charts/primitives/ChartLegend.js

@@ -109,7 +109,7 @@ function LegendSwatch({ item, idPrefix }) {
           width: 11,
           height: 11,
           borderRadius: 3,
-          // theme-allow: sub-scale legend-swatch corner, below the 4px radius floor
+          // theme-allow raw-surface — sub-scale legend-swatch corner, below the 4px radius floor
           backgroundColor: item.color,
           opacity: item.fillOpacity ?? 0.7,
           flexShrink: 0
@@ -152,7 +152,7 @@ function LegendChildSwatch({ item }) {
           width: 10,
           height: 10,
           borderRadius: 2,
-          // theme-allow: sub-scale legend-swatch corner, below the 4px radius floor
+          // theme-allow raw-surface — sub-scale legend-swatch corner, below the 4px radius floor
           backgroundColor: item.color,
           opacity: item.fillOpacity ?? 0.7,
           flexShrink: 0

package/dist/charts/primitives/ChartTooltip.js

@@ -101,7 +101,7 @@ function TooltipRow({
                 width: 8,
                 height: 8,
                 borderRadius: 2,
-                // theme-allow: sub-scale tooltip micro-corner, below the 4px radius floor
+                // theme-allow raw-surface — sub-scale tooltip micro-corner, below the 4px radius floor
                 backgroundColor: color,
                 flexShrink: 0
               }