@starklab/stark-mcp 0.1.0 → 0.2.1

package/src/adopt/rnTokenAliasResolver.test.js

@@ -1,260 +0,0 @@
-import { mkdtempSync, rmSync, mkdirSync, writeFileSync } from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-import { describe, it, expect, afterEach } from 'vitest';
-
-import { resolveRnTokenAliases } from './rnTokenAliasResolver.js';
-
-let tmpDirs = [];
-
-afterEach(() => {
-  for (const dir of tmpDirs) rmSync(dir, { recursive: true, force: true });
-  tmpDirs = [];
-});
-
-function fixture(files) {
-  const root = mkdtempSync(path.join(os.tmpdir(), 'stark-adopt-rn-alias-'));
-  tmpDirs.push(root);
-  for (const [rel, content] of Object.entries(files)) {
-    const full = path.join(root, rel);
-    mkdirSync(path.dirname(full), { recursive: true });
-    writeFileSync(full, content);
-  }
-  return root;
-}
-
-function usage(result, stkToken) {
-  return result.usages.find((u) => u.stkToken === stkToken);
-}
-
-function property(result, name) {
-  const found = result.properties.find((p) => p.name === name);
-  if (!found) throw new Error(`No property "${name}" in result. Found: ${result.properties.map((p) => p.name).join(', ')}`);
-  return found;
-}
-
-function findingsFor(result, rule, propName) {
-  return result.findings.filter((f) => f.rule === rule && f.property === propName);
-}
-
-describe('resolveRnTokenAliases — detection', () => {
-  it('reports detected:false with no report when no file imports a Stark RN token package', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        const styles = StyleSheet.create({ container: { backgroundColor: '#1956dd' } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    expect(result.detected).toBe(false);
-    expect(result.report).toBeNull();
-  });
-
-  it('detects an import from @starklab/stk/rn even if unused elsewhere', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    expect(result.detected).toBe(true);
-    expect(result.confidence).toBe('uncertain');
-  });
-});
-
-describe('resolveRnTokenAliases — usage classification (Pass 2)', () => {
-  it('classifies a directly-imported token referenced with no indirection as direct', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-        const styles = StyleSheet.create({ container: { backgroundColor: stkSurfaceBrand1Strong } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    expect(usage(result, 'stkSurfaceBrand1Strong').classification).toBe('direct');
-    expect(result.report.direct).toBe(1);
-  });
-
-  it('classifies a reference through a scalar local const alias as aliased', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-        const primary = stkSurfaceBrand1Strong;
-        const styles = StyleSheet.create({ container: { backgroundColor: primary } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const u = result.usages.find((x) => x.ref === 'primary');
-    expect(u.classification).toBe('aliased');
-    expect(result.report.aliased).toBe(1);
-  });
-
-  it('classifies a reference through an object-literal property alias as aliased', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-        const theme = { primary: stkSurfaceBrand1Strong };
-        const styles = StyleSheet.create({ container: { backgroundColor: theme.primary } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const u = result.usages.find((x) => x.ref === 'theme.primary');
-    expect(u.classification).toBe('aliased');
-  });
-
-  it('classifies a reference through a raw-literal object alias as drift, not aliased', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSpacingMd } from '@starklab/stk/rn-spacing'; // unused, only to satisfy detection
-        const theme = { primary: '#1956dd' };
-        const styles = StyleSheet.create({ container: { backgroundColor: theme.primary } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const p = property(result, 'theme.primary');
-    expect(p.state).toBe('drift');
-    const u = result.usages.find((x) => x.ref === 'theme.primary');
-    expect(u.classification).toBe('drift');
-    expect(findingsFor(result, 'drift-behind-alias', 'theme.primary')).toHaveLength(1);
-  });
-
-  it('classifies a reference through an alias to a foreign (non-Stark) import as broken', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSpacingMd } from '@starklab/stk/rn-spacing'; // unused, only to satisfy detection
-        import { someColor } from 'some-other-lib';
-        const theme = { primary: someColor };
-        const styles = StyleSheet.create({ container: { backgroundColor: theme.primary } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const p = property(result, 'theme.primary');
-    expect(p.state).toBe('broken');
-    expect(findingsFor(result, 'broken-alias', 'theme.primary')).toHaveLength(1);
-  });
-
-  it('applies the name-only heuristic when the containing object has no static origin (e.g. a useTokens() hook)', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSpacingMd } from '@starklab/stk/rn-spacing'; // unused, only to satisfy detection
-        import { useTokens } from '../theme/ThemeContext';
-        function useStyles() {
-          const t = useTokens();
-          return StyleSheet.create({ container: { backgroundColor: t.stkSurfaceBrand1Strong } });
-        }
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const u = usage(result, 'stkSurfaceBrand1Strong');
-    expect(u.classification).toBe('direct');
-    expect(u.heuristic).toBe(true);
-  });
-
-  it('does not count an unrelated member/identifier access as a usage', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSpacingMd } from '@starklab/stk/rn-spacing'; // unused, only to satisfy detection
-        const styles = StyleSheet.create({ container: { flexDirection: 'row' } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    expect(result.report.total).toBe(0);
-  });
-});
-
-describe('resolveRnTokenAliases — layer violations', () => {
-  it('flags a scalar alias resolving straight to a primitive RN token', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSpacingMd } from '@starklab/stk/rn-spacing';
-        const gap = stkSpacingMd;
-        const styles = StyleSheet.create({ container: { padding: gap } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const p = property(result, 'gap');
-    expect(p.state).toBe('conformant');
-    const findings = findingsFor(result, 'layer-violation', 'gap');
-    expect(findings).toHaveLength(1);
-    expect(findings[0].severity).toBe('critical');
-  });
-});
-
-describe('resolveRnTokenAliases — depth', () => {
-  it('warns past ~3 hops on a multi-hop conformant scalar chain', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-        const hop4 = stkSurfaceBrand1Strong;
-        const hop3 = hop4;
-        const hop2 = hop3;
-        const hop1 = hop2;
-        const styles = StyleSheet.create({ container: { backgroundColor: hop1 } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    const p = property(result, 'hop1');
-    expect(p.state).toBe('conformant');
-    expect(findingsFor(result, 'deep-alias-chain', 'hop1')).toHaveLength(1);
-  });
-
-  it('does not warn on a short chain within the depth budget', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-        const hop2 = stkSurfaceBrand1Strong;
-        const hop1 = hop2;
-        const styles = StyleSheet.create({ container: { backgroundColor: hop1 } });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    expect(findingsFor(result, 'deep-alias-chain', 'hop1')).toHaveLength(0);
-  });
-});
-
-describe('resolveRnTokenAliases — alias count and report', () => {
-  it('counts every distinct alias-graph entry and reports the three-number summary', () => {
-    const root = fixture({
-      'src/App.js': `
-        import { StyleSheet } from 'react-native';
-        import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';
-        const primary = stkSurfaceBrand1Strong;
-        const secondary = '#1956dd';
-        const styles = StyleSheet.create({
-          a: { backgroundColor: stkSurfaceBrand1Strong },
-          b: { backgroundColor: primary },
-          c: { backgroundColor: secondary },
-        });
-      `,
-    });
-    const result = resolveRnTokenAliases(root, { platform: 'native' });
-    expect(result.aliasCount).toBe(2); // primary, secondary
-    expect(result.report.direct).toBe(1); // a
-    expect(result.report.aliased).toBe(1); // b
-    // c aliases a plain string local const — the alias graph tracked it (it's
-    // a real consumer-authored indirection), it just doesn't terminate at a
-    // Stark token, so it folds into "unresolved" same as CSS drift does on web.
-    expect(result.report.unresolved).toBe(1); // c
-    expect(result.report.total).toBe(3);
-  });
-});
-
-describe('resolveRnTokenAliases — platform scope', () => {
-  it('throws for a non-native platform since JS-object token indirection has no CSS/Tailwind equivalent', () => {
-    const root = fixture({
-      'src/App.js': `import { stkSurfaceBrand1Strong } from '@starklab/stk/rn';`,
-    });
-    expect(() => resolveRnTokenAliases(root, { platform: 'web' })).toThrow(/native/i);
-  });
-});

package/src/adopt/tailwindResolver.test.js

@@ -1,178 +0,0 @@
-import { mkdtempSync, rmSync, mkdirSync, writeFileSync } from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-import { describe, it, expect, afterEach } from 'vitest';
-
-import { resolveTailwindTokens } from './tailwindResolver.js';
-
-let tmpDirs = [];
-
-afterEach(() => {
-  for (const dir of tmpDirs) rmSync(dir, { recursive: true, force: true });
-  tmpDirs = [];
-});
-
-function fixture(files) {
-  const root = mkdtempSync(path.join(os.tmpdir(), 'stark-adopt-tailwind-'));
-  tmpDirs.push(root);
-  for (const [rel, content] of Object.entries(files)) {
-    const full = path.join(root, rel);
-    mkdirSync(path.dirname(full), { recursive: true });
-    writeFileSync(full, content);
-  }
-  return root;
-}
-
-function usage(result, className) {
-  return result.usages.find((u) => u.className === className);
-}
-
-describe('resolveTailwindTokens — detection', () => {
-  it('reports detected:false with no report when neither a config nor an @theme block exists', () => {
-    const root = fixture({
-      'src/app.css': `.btn { color: red; }`,
-      'src/App.jsx': `export const App = () => <div className="flex p-4" />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(result.detected).toBe(false);
-    expect(result.report).toBeNull();
-  });
-
-  it('detects a v4 @theme block even with no config file', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(result.detected).toBe(true);
-    expect(result.source).toBe('v4');
-  });
-
-  it('detects a v3 tailwind.config.js even with no @theme block', () => {
-    const root = fixture({
-      'tailwind.config.js': `module.exports = { theme: { extend: { colors: { primary: 'var(--stk-surface-brand-1-strong)' } } } };`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(result.detected).toBe(true);
-    expect(result.source).toBe('v3');
-  });
-
-  it('reports themeParseIncomplete instead of a zero score for a dynamic/function-based v3 config', () => {
-    const root = fixture({
-      'tailwind.config.js': `module.exports = { theme: (helpers) => ({ colors: { primary: helpers.colors.blue } }) };`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(result.detected).toBe(true);
-    expect(result.themeParseIncomplete).toBe(true);
-    expect(result.report).toBeNull();
-  });
-});
-
-describe('resolveTailwindTokens — v4 @theme classification', () => {
-  it('classifies a @theme entry aliasing an inventory stk token as conformant', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    const prop = result.properties.find((p) => p.name === '--color-primary');
-    expect(prop.state).toBe('conformant');
-  });
-
-  it('classifies a @theme entry with a raw value as drift', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: #1956dd; }`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    const prop = result.properties.find((p) => p.name === '--color-primary');
-    expect(prop.state).toBe('drift');
-  });
-});
-
-describe('resolveTailwindTokens — v3 tailwind.config theme.extend.colors classification', () => {
-  it('classifies a flat color entry aliasing an inventory stk token as conformant', () => {
-    const root = fixture({
-      'tailwind.config.js': `module.exports = { theme: { extend: { colors: { primary: 'var(--stk-surface-brand-1-strong)' } } } };`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    const prop = result.properties.find((p) => p.name === '--color-primary');
-    expect(prop.state).toBe('conformant');
-  });
-
-  it('classifies a nested DEFAULT/shade color entry', () => {
-    const root = fixture({
-      'tailwind.config.js': `module.exports = { theme: { extend: { colors: { brand: { DEFAULT: 'var(--stk-surface-brand-1-strong)', 500: '#1956dd' } } } } };`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(result.properties.find((p) => p.name === '--color-brand').state).toBe('conformant');
-    expect(result.properties.find((p) => p.name === '--color-brand-500').state).toBe('drift');
-  });
-});
-
-describe('resolveTailwindTokens — JSX className usage scanning', () => {
-  it('classifies a utility class resolving to a conformant theme entry as aliased', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-      'src/App.jsx': `export const App = () => <div className="bg-primary" />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(usage(result, 'bg-primary').classification).toBe('aliased');
-    expect(result.report.aliased).toBe(1);
-  });
-
-  it('classifies a utility class resolving to a drifted theme entry as drift, not aliased', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: #1956dd; }`,
-      'src/App.jsx': `export const App = () => <div className="bg-primary" />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(usage(result, 'bg-primary').classification).toBe('drift');
-  });
-
-  it('strips a variant prefix before matching the base utility', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-      'src/App.jsx': `export const App = () => <div className="hover:bg-primary dark:[&:hover]:bg-primary" />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(usage(result, 'hover:bg-primary').classification).toBe('aliased');
-    expect(usage(result, 'dark:[&:hover]:bg-primary').classification).toBe('aliased');
-  });
-
-  it('classifies an arbitrary-value var() utility as direct', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-      'src/App.jsx': `export const App = () => <div className="bg-[var(--stk-surface-brand-1-strong)]" />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(usage(result, 'bg-[var(--stk-surface-brand-1-strong)]').classification).toBe('direct');
-    expect(result.report.direct).toBe(1);
-  });
-
-  it('does not count an unrelated utility class or Tailwind default-palette color as a usage', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-      'src/App.jsx': `export const App = () => <div className="flex bg-red-500 p-4" />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(usage(result, 'flex')).toBeUndefined();
-    expect(usage(result, 'bg-red-500')).toBeUndefined();
-    expect(usage(result, 'p-4')).toBeUndefined();
-    expect(result.report.total).toBe(0);
-  });
-
-  it('does not count a dynamic className expression (clsx/ternary) since it is not statically resolvable', () => {
-    const root = fixture({
-      'src/app.css': `@theme { --color-primary: var(--stk-surface-brand-1-strong); }`,
-      'src/App.jsx': `export const App = ({ active }) => <div className={active ? 'bg-primary' : 'bg-primary'} />;`,
-    });
-    const result = resolveTailwindTokens(root, { platform: 'web' });
-    expect(result.report.total).toBe(0);
-  });
-});
-
-describe('resolveTailwindTokens — platform scope', () => {
-  it('throws for a non-web platform since Tailwind utility classes have no RN equivalent', () => {
-    const root = fixture({ 'tailwind.config.js': `module.exports = {};` });
-    expect(() => resolveTailwindTokens(root, { platform: 'native' })).toThrow(/web/i);
-  });
-});

package/src/adopt/targetDiscovery.test.js

@@ -1,227 +0,0 @@
-import { mkdtempSync, rmSync, mkdirSync, writeFileSync } from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-import { describe, it, expect, afterEach } from 'vitest';
-
-import { discoverTargets, workspacePackageMap } from './targetDiscovery.js';
-
-let tmpDirs = [];
-
-afterEach(() => {
-  for (const dir of tmpDirs) rmSync(dir, { recursive: true, force: true });
-  tmpDirs = [];
-});
-
-function fixture(files) {
-  const root = mkdtempSync(path.join(os.tmpdir(), 'stark-adopt-targets-'));
-  tmpDirs.push(root);
-  for (const [rel, content] of Object.entries(files)) {
-    const full = path.join(root, rel);
-    mkdirSync(path.dirname(full), { recursive: true });
-    writeFileSync(full, content);
-  }
-  return root;
-}
-
-function target(discovery, dir) {
-  const found = discovery.targets.find((t) => t.dir === dir);
-  if (!found) throw new Error(`No target "${dir}". Found: ${discovery.targets.map((t) => t.dir).join(', ')}`);
-  return found;
-}
-
-describe('discoverTargets — manifest source resolution', () => {
-  it('reads npm/yarn array-form package.json#workspaces', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/a/package.json': JSON.stringify({ name: 'a' }),
-      'packages/b/package.json': JSON.stringify({ name: 'b' }),
-    });
-    const d = discoverTargets(root);
-    expect(d.manifestSource).toBe('package.json#workspaces');
-    expect(d.targets.map((t) => t.dir).sort()).toEqual(['packages/a', 'packages/b']);
-  });
-
-  it('reads yarn object-form package.json#workspaces.packages', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: { packages: ['apps/*'] } }),
-      'apps/one/package.json': JSON.stringify({ name: 'one' }),
-    });
-    const d = discoverTargets(root);
-    expect(d.manifestSource).toBe('package.json#workspaces');
-    expect(d.targets.map((t) => t.dir)).toEqual(['apps/one']);
-  });
-
-  it('falls back to pnpm-workspace.yaml when package.json has no workspaces field', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root' }),
-      'pnpm-workspace.yaml': "packages:\n  - 'packages/*'\n  - 'apps/*'\n",
-      'packages/x/package.json': JSON.stringify({ name: 'x' }),
-      'apps/y/package.json': JSON.stringify({ name: 'y' }),
-    });
-    const d = discoverTargets(root);
-    expect(d.manifestSource).toBe('pnpm-workspace.yaml');
-    expect(d.targets.map((t) => t.dir).sort()).toEqual(['apps/y', 'packages/x']);
-  });
-
-  it('falls back to a whole-repo scan when no manifest declares globs', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root' }),
-      'nested/thing/package.json': JSON.stringify({ name: 'thing' }),
-    });
-    const d = discoverTargets(root);
-    expect(d.manifestSource).toBeNull();
-    expect(d.targets.map((t) => t.dir)).toEqual(['nested/thing']);
-  });
-
-  it('treats the root itself as the sole target when nothing nested exists either', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root' }),
-    });
-    const d = discoverTargets(root);
-    expect(d.targets.map((t) => t.dir)).toEqual(['.']);
-  });
-
-  it('records turbo.json/nx.json presence as informational monorepoTooling flags', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'turbo.json': '{}',
-      'packages/a/package.json': JSON.stringify({ name: 'a' }),
-    });
-    const d = discoverTargets(root);
-    expect(d.monorepoTooling).toEqual({ turbo: true, nx: false });
-  });
-});
-
-describe('discoverTargets — scope classification', () => {
-  it('marks a direct @starklab/stk* dependent in scope', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/consumer/package.json': JSON.stringify({
-        name: 'consumer',
-        dependencies: { '@starklab/stk-components': '^1.0.0' },
-      }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/consumer');
-    expect(t.inScope).toBe(true);
-    expect(t.scopeReason).toMatch(/depends on @starklab\/stk\*/);
-  });
-
-  it('marks a target with no stk* dependency, direct or transitive, out of scope — not 0%', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/unrelated/package.json': JSON.stringify({ name: 'unrelated', dependencies: { lodash: '^4.0.0' } }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/unrelated');
-    expect(t.inScope).toBe(false);
-    expect(t.scopeReason).toBe('no @starklab/stk* dependency, direct or transitive');
-    expect(d.excluded.map((x) => x.dir)).toContain('packages/unrelated');
-  });
-
-  it('marks a transitive dependent (via a workspace sibling) in scope', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/core/package.json': JSON.stringify({
-        name: 'core',
-        dependencies: { '@starklab/stk': '^1.0.0' },
-      }),
-      'packages/app/package.json': JSON.stringify({ name: 'app', dependencies: { core: '1.0.0' } }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/app');
-    expect(t.inScope).toBe(true);
-    expect(t.scopeReason).toMatch(/via a workspace sibling/);
-  });
-
-  it('flags an excluded target with a UI framework dependency as an opportunity', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/ui-no-stk/package.json': JSON.stringify({ name: 'ui-no-stk', dependencies: { react: '^18.0.0' } }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/ui-no-stk');
-    expect(t.inScope).toBe(false);
-    expect(t.rendersUi).toBe(true);
-    expect(d.opportunities.map((x) => x.dir)).toContain('packages/ui-no-stk');
-  });
-
-  it('does not list an excluded, non-UI target as an opportunity', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/plain/package.json': JSON.stringify({ name: 'plain', dependencies: { lodash: '^4.0.0' } }),
-    });
-    const d = discoverTargets(root);
-    expect(d.opportunities.map((x) => x.dir)).not.toContain('packages/plain');
-  });
-});
-
-describe('discoverTargets — dominion.config.json overrides', () => {
-  it('force-includes a target with no stk* dependency, unrestricted', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/early/package.json': JSON.stringify({ name: 'early', dependencies: { lodash: '^4.0.0' } }),
-      'dominion.config.json': JSON.stringify({ targets: { forceInclude: ['packages/early'] } }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/early');
-    expect(t.inScope).toBe(true);
-    expect(t.forceIncluded).toBe(true);
-    expect(t.scopeReason).toBe('force-included');
-  });
-
-  it('force-excludes a target that would otherwise be in scope, and requires a reason', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/gaming/package.json': JSON.stringify({
-        name: 'gaming',
-        dependencies: { '@starklab/stk-components': '^1.0.0' },
-      }),
-      'dominion.config.json': JSON.stringify({
-        targets: { forceExclude: { 'packages/gaming': 'prototype, not tracked yet' } },
-      }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/gaming');
-    expect(t.inScope).toBe(false);
-    expect(t.forceExcluded).toBe(true);
-    expect(t.scopeReason).toBe('force-excluded: prototype, not tracked yet');
-  });
-
-  it('gives force-exclude priority over force-include when both name the same target', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/contested/package.json': JSON.stringify({ name: 'contested' }),
-      'dominion.config.json': JSON.stringify({
-        targets: {
-          forceInclude: ['packages/contested'],
-          forceExclude: { 'packages/contested': 'explicitly out' },
-        },
-      }),
-    });
-    const d = discoverTargets(root);
-    const t = target(d, 'packages/contested');
-    expect(t.inScope).toBe(false);
-    expect(t.scopeReason).toBe('force-excluded: explicitly out');
-  });
-});
-
-describe('workspacePackageMap', () => {
-  it('maps every target with a package.json#name to its absolute dir, in and out of scope alike', () => {
-    const root = fixture({
-      'package.json': JSON.stringify({ name: 'root', workspaces: ['packages/*'] }),
-      'packages/consumer/package.json': JSON.stringify({
-        name: 'consumer',
-        dependencies: { '@starklab/stk-components': '^1.0.0' },
-      }),
-      'packages/unrelated/package.json': JSON.stringify({ name: 'unrelated' }),
-    });
-    const d = discoverTargets(root);
-    const map = workspacePackageMap(d);
-    expect(map.get('consumer')).toBe(path.join(root, 'packages/consumer'));
-    // Out-of-scope targets are still mapped — a cross-workspace wrapper
-    // chain can legitimately pass through one on its way to a DS component.
-    expect(map.get('unrelated')).toBe(path.join(root, 'packages/unrelated'));
-  });
-});