basalt-ui 1.19.1 → 1.21.0

package/src/cli/init-sync.test.ts

@@ -119,6 +119,12 @@ describe('doctor — CLI vs installed basalt-ui version', () => {
   ).version
 
   it('warns when node_modules/basalt-ui version differs from the running CLI', () => {
+    // A Mantine dependency puts doctor in the framework profile — without one it reads as a
+    // tokens-only consumer, where the missing scaffold is not a failure at all.
+    writeFixture(
+      'package.json',
+      JSON.stringify({ name: 'fixture', dependencies: { '@mantine/core': '^9' } }),
+    )
     writeFixture(
       'node_modules/basalt-ui/package.json',
       JSON.stringify({ name: 'basalt-ui', version: '0.4.2' }),
@@ -132,21 +138,33 @@ describe('doctor — CLI vs installed basalt-ui version', () => {
   })
 
   it('passes when node_modules/basalt-ui version matches the running CLI', () => {
+    // A full fixture, not just the manifest: doctor also asserts the guard scans something and the
+    // oxlint preset is wired, and a green exit has to mean all of it.
+    writeFixture('package.json', JSON.stringify({ name: 'fixture', basalt: { roots: ['src'] } }))
+    writeFixture('src/app.tsx', 'export const App = () => null\n')
+    writeFixture(
+      '.oxlintrc.json',
+      '{ "extends": ["./node_modules/basalt-ui/configs/oxlint.json"] }',
+    )
     writeFixture(MANIFEST_PATH, JSON.stringify({ version: 1, files: {} }, null, 2))
     writeFixture(
       'node_modules/basalt-ui/package.json',
       JSON.stringify({ name: 'basalt-ui', version: cliVersion }),
     )
+    // The `extends` above has to RESOLVE, not merely read correctly — see the oxlint-preset check.
+    writeFixture('node_modules/basalt-ui/configs/oxlint.json', '{}')
     const { code, log } = capture(() => doctor(dir))
     expect(code).toBe(0)
     expect(log).toContain('matches the installed basalt-ui')
   })
 
-  it('is silent (no node_modules/basalt-ui present) when the package cannot be resolved', () => {
+  it('reports the version checks as SKIPPED when the package cannot be resolved', () => {
     writeFixture(MANIFEST_PATH, JSON.stringify({ version: 1, files: {} }, null, 2))
-    const { log } = capture(() => doctor(dir))
-    expect(log).not.toContain('differs from the installed basalt-ui')
+    const { code, log } = capture(() => doctor(dir))
     expect(log).not.toContain('matches the installed basalt-ui')
+    // Silence was the bug: two of five checks used to vanish while the footer still read green.
+    expect(log).toContain('SKIPPED')
+    expect(code).toBe(1)
   })
 })
 

package/src/cli/template-ledger.test.ts

@@ -116,7 +116,7 @@ describe('classify ledger: cross-version format-only drift vs a real edit', () =
     // older version token — this is what makes the scenario cross-version, not same-version.
     const olderBlock = currentBlock.replace(/basalt-ui/g, 'basalt')
     expect(olderBlock).not.toBe(currentBlock)
-    expect(olderBlock).toContain('bunx basalt sync')
+    expect(olderBlock).toContain('basalt sync')
 
     // What a PRE-1.0.2 CLI's writeUnit actually recorded at that sync: a RAW hash of the exact
     // bytes it wrote — the fix only normalizes hashes written by a CLI carrying it, so a genuine

package/src/cli/tokens-css.test.ts

@@ -1,10 +1,13 @@
 /**
  * `basalt-ui tokens:css` — the install-free path to the token system.
  *
- * The contract these tests hold is that the command is a FLAG PARSER, nothing more: whatever it
- * prints must be exactly what `buildPaletteCss` returns for the same options. If the CLI ever grows
- * emission logic of its own, a consumer who ran `bunx basalt-ui tokens:css` and a consumer who
- * imported `basalt-ui/tokens` would be looking at two different design systems.
+ * The contract these tests hold is that the command holds no emission logic of its own: the token
+ * VALUES it prints must be exactly what `buildPaletteCss` returns for the same options, or a
+ * consumer who ran `bunx basalt-ui tokens:css` and one who imported `basalt-ui/tokens` would be
+ * looking at two different design systems. What the command DOES add is file framing for an
+ * artifact a consumer commits — the `@generated` header, a trailing newline, `rgba()` spacing —
+ * which `body()` below strips before every comparison. See `./toolchain-wiring.test.ts` for the
+ * framing's own tests.
  */
 import { afterEach, beforeEach, describe, expect, it } from 'bun:test'
 import { existsSync, mkdtempSync, readFileSync, readdirSync, rmSync, writeFileSync } from 'node:fs'
@@ -12,7 +15,7 @@ import { tmpdir } from 'node:os'
 import { join, resolve } from 'node:path'
 
 import { buildPaletteCss } from '../tokens'
-import { run, tokensCss } from './index'
+import { normalizeColorFunctions, run, tokensCss } from './index'
 
 const FIXTURE = join(import.meta.dir, '..', '..', 'tests', 'fixtures', 'palette-default.css')
 
@@ -34,27 +37,37 @@ function emit(flags: string[], out = 'out.css'): string {
   return readFileSync(resolve(dir, out), 'utf8')
 }
 
+/** The emitted file minus the two-line generated header and the trailing newline. */
+function body(flags: string[], out = 'out.css'): string {
+  return emit(flags, out).split('\n').slice(2).join('\n').replace(/\n$/, '')
+}
+
+/** What `buildPaletteCss` returns, in the CLI's commit-safe formatting. */
+function expected(opts: Parameters<typeof buildPaletteCss>[0]): string {
+  return normalizeColorFunctions(buildPaletteCss(opts))
+}
+
 describe('tokens:css', () => {
-  it('with no flags emits the shipped default, byte for byte', () => {
-    expect(emit([])).toBe(readFileSync(FIXTURE, 'utf8'))
+  it('with no flags emits the shipped default, byte for byte under the header', () => {
+    expect(body([])).toBe(normalizeColorFunctions(readFileSync(FIXTURE, 'utf8')))
   })
 
   it('forwards the selector flags to buildPaletteCss and adds nothing of its own', () => {
     const flags = ['--selector-attribute', 'data-theme', '--default-scheme', 'light']
-    expect(emit(flags)).toBe(
-      buildPaletteCss({ scheme: { attribute: 'data-theme' }, defaultScheme: 'light' }),
+    expect(body(flags)).toBe(
+      expected({ scheme: { attribute: 'data-theme' }, defaultScheme: 'light' }),
     )
   })
 
   it('forwards the scheme VALUES and the media fallback', () => {
     const flags = ['--dark-value', 'night', '--light-value', 'day', '--media-fallback']
-    expect(emit(flags)).toBe(
-      buildPaletteCss({ scheme: { darkValue: 'night', lightValue: 'day' }, mediaFallback: true }),
+    expect(body(flags)).toBe(
+      expected({ scheme: { darkValue: 'night', lightValue: 'day' }, mediaFallback: true }),
     )
   })
 
   it('forwards --only core', () => {
-    expect(emit(['--only', 'core'])).toBe(buildPaletteCss({ only: 'core' }))
+    expect(body(['--only', 'core'])).toBe(expected({ only: 'core' }))
   })
 
   it('rejects an unknown --only instead of silently emitting everything', () => {
@@ -64,7 +77,7 @@ describe('tokens:css', () => {
 
   it('creates the parent directory of --out', () => {
     const css = emit([], 'nested/deep/tokens.css')
-    expect(css.startsWith(':root {')).toBe(true)
+    expect(css).toContain(':root {')
   })
 
   it('rejects an unknown --default-scheme instead of silently defaulting', () => {
@@ -87,7 +100,7 @@ describe('tokens:css', () => {
   })
 
   it('--no-legacy-aliases output is byte-identical to buildPaletteCss({ legacyAliases: false }) — the CLI holds no emission logic of its own', () => {
-    expect(emit(['--no-legacy-aliases'])).toBe(buildPaletteCss({ legacyAliases: false }))
+    expect(body(['--no-legacy-aliases'])).toBe(expected({ legacyAliases: false }))
   })
 
   it('`--help` short-circuits before the command runs — no file written', () => {