@3fn/core 14.1.0 → 15.0.0

package/src/__tests__/browser-distribution/token-loading.test.ts

@@ -1,344 +0,0 @@
-/**
- * @jest-environment jsdom
- * @category evergreen
- * @purpose Verify token loading behavior in browser environment
- */
-
-/**
- * Token Loading Behavior Tests
- *
- * Tests that verify:
- * 1. Tokens load correctly via link tag (Requirements 3.2, 3.3)
- * 2. Console warning appears when tokens not loaded (Requirements 3.4)
- *
- * @see .kiro/specs/028-web-component-browser-distribution/design.md
- * @see Requirements: 3.2, 3.3, 3.4
- */
-
-import * as fs from 'fs';
-import * as path from 'path';
-
-describe('Token Loading Behavior', () => {
-  const BROWSER_DIST_DIR = path.join(process.cwd(), 'dist', 'browser');
-  const TOKENS_CSS_PATH = path.join(BROWSER_DIST_DIR, 'tokens.css');
-
-  beforeAll(() => {
-    // Ensure tokens.css exists before running tests
-    if (!fs.existsSync(TOKENS_CSS_PATH)) {
-      throw new Error(
-        'tokens.css not found. Run "npm run build:browser" first.'
-      );
-    }
-  });
-
-  describe('Token CSS File Availability', () => {
-    /**
-     * Requirement 3.1: Build system SHALL copy DesignTokens.web.css to dist/browser/tokens.css
-     */
-    it('should have tokens.css in browser distribution directory', () => {
-      expect(fs.existsSync(TOKENS_CSS_PATH)).toBe(true);
-    });
-
-    it('should have non-empty tokens.css file', () => {
-      const stats = fs.statSync(TOKENS_CSS_PATH);
-      expect(stats.size).toBeGreaterThan(0);
-    });
-
-    it('should contain CSS custom property definitions', () => {
-      const content = fs.readFileSync(TOKENS_CSS_PATH, 'utf-8');
-      
-      // Should contain :root selector with CSS custom properties
-      expect(content).toContain(':root');
-      expect(content).toMatch(/--[a-zA-Z0-9-]+\s*:/);
-    });
-
-    /**
-     * Requirement 3.2: All CSS custom properties required by components SHALL be available
-     * Note: Token name updated from --color-primary to --color-action-primary per Spec 052
-     */
-    it('should contain color-action-primary token used for token detection', () => {
-      const content = fs.readFileSync(TOKENS_CSS_PATH, 'utf-8');
-      
-      // The browser-entry.ts checks for --color-action-primary to detect if tokens are loaded
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      expect(content).toContain('--color-action-primary');
-    });
-  });
-
-  describe('Token Loading via Link Tag', () => {
-    let originalGetComputedStyle: typeof window.getComputedStyle;
-
-    beforeEach(() => {
-      // Store original getComputedStyle
-      originalGetComputedStyle = window.getComputedStyle;
-    });
-
-    afterEach(() => {
-      // Restore original getComputedStyle
-      window.getComputedStyle = originalGetComputedStyle;
-      
-      // Clean up any added style elements
-      const styleElements = document.querySelectorAll('style[data-test]');
-      styleElements.forEach(el => el.remove());
-    });
-
-    /**
-     * Requirement 3.2: WHEN a developer includes <link rel="stylesheet" href="dist/browser/tokens.css">
-     * THEN all CSS custom properties required by components SHALL be available
-     */
-    it('should make CSS custom properties available when tokens are loaded', () => {
-      // Read actual tokens.css content
-      const tokensContent = fs.readFileSync(TOKENS_CSS_PATH, 'utf-8');
-      
-      // Inject tokens into JSDOM
-      const styleElement = document.createElement('style');
-      styleElement.setAttribute('data-test', 'tokens');
-      styleElement.textContent = tokensContent;
-      document.head.appendChild(styleElement);
-
-      // Verify tokens are available via getComputedStyle
-      const computedStyle = window.getComputedStyle(document.documentElement);
-      
-      // Check for essential tokens that components use
-      // Note: JSDOM has limited CSS support, so we verify the style element was added
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      expect(styleElement.textContent).toContain('--color-action-primary');
-      expect(styleElement.textContent).toContain('--color-text-default');
-      expect(styleElement.textContent).toContain('--space-inset-100');
-    });
-
-    it('should contain all essential semantic tokens for components', () => {
-      const tokensContent = fs.readFileSync(TOKENS_CSS_PATH, 'utf-8');
-      
-      // Essential tokens that components reference
-      // Note: Token names updated per Spec 052 semantic token naming restructure
-      const essentialTokens = [
-        '--color-action-primary',  // Renamed from --color-primary
-        '--color-feedback-error-text',  // Renamed from --color-error-strong
-        '--color-feedback-success-text',  // Renamed from --color-success-strong
-        '--color-text-default',
-        '--color-text-muted',
-        '--color-structure-canvas',  // Renamed from --color-background
-        '--color-structure-surface',  // Renamed from --color-surface
-        '--color-structure-border',  // Renamed from --color-border
-        '--space-inset-100',
-        '--radius-100',
-        '--duration-150',
-        '--accessibility-focus-width',
-        '--accessibility-focus-color',
-      ];
-
-      for (const token of essentialTokens) {
-        expect(tokensContent).toContain(token);
-      }
-    });
-  });
-
-  describe('Console Warning When Tokens Not Loaded', () => {
-    let consoleWarnSpy: jest.SpyInstance;
-    let originalGetComputedStyle: typeof window.getComputedStyle;
-
-    beforeEach(() => {
-      // Spy on console.warn
-      consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation();
-      
-      // Store original getComputedStyle
-      originalGetComputedStyle = window.getComputedStyle;
-    });
-
-    afterEach(() => {
-      // Restore console.warn
-      consoleWarnSpy.mockRestore();
-      
-      // Restore original getComputedStyle
-      window.getComputedStyle = originalGetComputedStyle;
-    });
-
-    /**
-     * Requirement 3.4: IF Token_CSS is not loaded THEN the Browser_Bundle SHALL log a warning
-     * to the browser console indicating that tokens are required for proper styling
-     */
-    it('should warn when tokens are not detected', () => {
-      // Mock getComputedStyle to return empty value (simulating no tokens loaded)
-      window.getComputedStyle = jest.fn().mockReturnValue({
-        getPropertyValue: jest.fn().mockReturnValue(''),
-      });
-
-      // Import and call the checkTokensLoaded function behavior
-      // We simulate what browser-entry.ts does
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      const testProperty = window.getComputedStyle(document.documentElement)
-        .getPropertyValue('--color-action-primary');
-      
-      if (!testProperty || testProperty.trim() === '') {
-        console.warn(
-          '[DesignerPunk] Design tokens not detected. ' +
-          'Include tokens.css before using components: ' +
-          '<link rel="stylesheet" href="dist/browser/tokens.css">'
-        );
-      }
-
-      // Verify warning was logged
-      expect(consoleWarnSpy).toHaveBeenCalledTimes(1);
-      expect(consoleWarnSpy).toHaveBeenCalledWith(
-        expect.stringContaining('[DesignerPunk]')
-      );
-      expect(consoleWarnSpy).toHaveBeenCalledWith(
-        expect.stringContaining('Design tokens not detected')
-      );
-      expect(consoleWarnSpy).toHaveBeenCalledWith(
-        expect.stringContaining('tokens.css')
-      );
-    });
-
-    it('should not warn when tokens are detected', () => {
-      // Mock getComputedStyle to return a value (simulating tokens loaded)
-      window.getComputedStyle = jest.fn().mockReturnValue({
-        getPropertyValue: jest.fn().mockReturnValue('#FF00FF'),
-      });
-
-      // Simulate the checkTokensLoaded function behavior
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      const testProperty = window.getComputedStyle(document.documentElement)
-        .getPropertyValue('--color-action-primary');
-      
-      if (!testProperty || testProperty.trim() === '') {
-        console.warn(
-          '[DesignerPunk] Design tokens not detected. ' +
-          'Include tokens.css before using components: ' +
-          '<link rel="stylesheet" href="dist/browser/tokens.css">'
-        );
-      }
-
-      // Verify no warning was logged
-      expect(consoleWarnSpy).not.toHaveBeenCalled();
-    });
-
-    it('should include actionable guidance in warning message', () => {
-      // Mock getComputedStyle to return empty value
-      window.getComputedStyle = jest.fn().mockReturnValue({
-        getPropertyValue: jest.fn().mockReturnValue(''),
-      });
-
-      // Simulate the checkTokensLoaded function behavior
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      const testProperty = window.getComputedStyle(document.documentElement)
-        .getPropertyValue('--color-action-primary');
-      
-      if (!testProperty || testProperty.trim() === '') {
-        console.warn(
-          '[DesignerPunk] Design tokens not detected. ' +
-          'Include tokens.css before using components: ' +
-          '<link rel="stylesheet" href="dist/browser/tokens.css">'
-        );
-      }
-
-      // Verify warning includes actionable guidance
-      const warningMessage = consoleWarnSpy.mock.calls[0][0];
-      expect(warningMessage).toContain('<link rel="stylesheet"');
-      expect(warningMessage).toContain('dist/browser/tokens.css');
-    });
-  });
-
-  describe('Token Detection Logic in Browser Entry', () => {
-    /**
-     * Verify the browser-entry.ts contains the correct token detection logic
-     */
-    it('should have checkTokensLoaded function in browser entry', () => {
-      const browserEntryPath = path.join(process.cwd(), 'src', 'browser-entry.ts');
-      const content = fs.readFileSync(browserEntryPath, 'utf-8');
-
-      // Verify checkTokensLoaded function exists
-      expect(content).toContain('function checkTokensLoaded');
-      
-      // Verify it checks for --color-action-primary (renamed from --color-primary in Spec 052)
-      expect(content).toContain('--color-action-primary');
-      
-      // Verify it uses getComputedStyle
-      expect(content).toContain('getComputedStyle');
-      
-      // Verify it logs a warning
-      expect(content).toContain('console.warn');
-      expect(content).toContain('[DesignerPunk]');
-    });
-
-    it('should check tokens after DOM is ready', () => {
-      const browserEntryPath = path.join(process.cwd(), 'src', 'browser-entry.ts');
-      const content = fs.readFileSync(browserEntryPath, 'utf-8');
-
-      // Verify DOM ready check
-      expect(content).toContain('document.readyState');
-      expect(content).toContain('DOMContentLoaded');
-    });
-
-    it('should handle both loading and ready states', () => {
-      const browserEntryPath = path.join(process.cwd(), 'src', 'browser-entry.ts');
-      const content = fs.readFileSync(browserEntryPath, 'utf-8');
-
-      // Verify both states are handled
-      expect(content).toMatch(/if\s*\(\s*document\.readyState\s*===\s*['"]loading['"]\s*\)/);
-      expect(content).toContain('addEventListener');
-      expect(content).toContain('checkTokensLoaded');
-    });
-  });
-
-  describe('ESM Bundle Token Warning', () => {
-    /**
-     * Verify the ESM bundle contains the token warning logic
-     */
-    it('should contain token detection warning in ESM bundle', () => {
-      const esmBundlePath = path.join(BROWSER_DIST_DIR, 'designerpunk.esm.js');
-      
-      if (!fs.existsSync(esmBundlePath)) {
-        throw new Error('ESM bundle not found. Run "npm run build:browser" first.');
-      }
-
-      const bundleContent = fs.readFileSync(esmBundlePath, 'utf-8');
-
-      // Verify warning message is in bundle
-      expect(bundleContent).toContain('[DesignerPunk]');
-      expect(bundleContent).toContain('Design tokens not detected');
-      expect(bundleContent).toContain('tokens.css');
-    });
-
-    it('should check for --color-action-primary token in ESM bundle', () => {
-      const esmBundlePath = path.join(BROWSER_DIST_DIR, 'designerpunk.esm.js');
-      const bundleContent = fs.readFileSync(esmBundlePath, 'utf-8');
-
-      // Verify the token check is in the bundle
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      expect(bundleContent).toContain('--color-action-primary');
-      expect(bundleContent).toContain('getPropertyValue');
-    });
-  });
-
-  describe('UMD Bundle Token Warning', () => {
-    /**
-     * Verify the UMD bundle contains the token warning logic
-     */
-    it('should contain token detection warning in UMD bundle', () => {
-      const umdBundlePath = path.join(BROWSER_DIST_DIR, 'designerpunk.umd.js');
-      
-      if (!fs.existsSync(umdBundlePath)) {
-        throw new Error('UMD bundle not found. Run "npm run build:browser" first.');
-      }
-
-      const bundleContent = fs.readFileSync(umdBundlePath, 'utf-8');
-
-      // Verify warning message is in bundle
-      expect(bundleContent).toContain('[DesignerPunk]');
-      expect(bundleContent).toContain('Design tokens not detected');
-      expect(bundleContent).toContain('tokens.css');
-    });
-
-    it('should check for --color-action-primary token in UMD bundle', () => {
-      const umdBundlePath = path.join(BROWSER_DIST_DIR, 'designerpunk.umd.js');
-      const bundleContent = fs.readFileSync(umdBundlePath, 'utf-8');
-
-      // Verify the token check is in the bundle
-      // Note: color.primary was renamed to color.action.primary in Spec 052
-      expect(bundleContent).toContain('--color-action-primary');
-      expect(bundleContent).toContain('getPropertyValue');
-    });
-  });
-});

package/src/__tests__/browser-distribution/umd-bundle-loading.test.ts

@@ -1,169 +0,0 @@
-/**
- * @category evergreen
- * @purpose Verify UMD bundle loading functionality for browser distribution
- */
-
-/**
- * UMD Bundle Loading Tests
- *
- * Tests that the UMD bundle loads correctly via script tag
- * and exposes the global DesignerPunk namespace.
- *
- * @see .kiro/specs/028-web-component-browser-distribution/design.md
- * @see Requirements: 4.1, 4.2, 4.3
- */
-
-import * as fs from 'fs';
-import * as path from 'path';
-
-describe('UMD Bundle Loading', () => {
-  const BROWSER_DIST_DIR = path.join(process.cwd(), 'dist', 'browser');
-  const UMD_BUNDLE = path.join(BROWSER_DIST_DIR, 'designerpunk.umd.js');
-  const UMD_MIN_BUNDLE = path.join(BROWSER_DIST_DIR, 'designerpunk.umd.min.js');
-
-  beforeAll(() => {
-    // Ensure bundles exist before running tests
-    if (!fs.existsSync(BROWSER_DIST_DIR)) {
-      throw new Error(
-        'Browser bundles not found. Run "npm run build:browser" first.'
-      );
-    }
-  });
-
-  describe('UMD Bundle Structure', () => {
-    it('should have UMD bundle file', () => {
-      expect(fs.existsSync(UMD_BUNDLE)).toBe(true);
-    });
-
-    it('should have minified UMD bundle file', () => {
-      expect(fs.existsSync(UMD_MIN_BUNDLE)).toBe(true);
-    });
-
-    it('should have source map for UMD bundle', () => {
-      const sourceMapPath = `${UMD_BUNDLE}.map`;
-      expect(fs.existsSync(sourceMapPath)).toBe(true);
-    });
-
-    it('should have source map for minified UMD bundle', () => {
-      const sourceMapPath = `${UMD_MIN_BUNDLE}.map`;
-      expect(fs.existsSync(sourceMapPath)).toBe(true);
-    });
-  });
-
-  describe('Global Namespace', () => {
-    it('should define DesignerPunk global namespace', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // UMD/IIFE bundles should define the global namespace
-      expect(bundleContent).toContain('DesignerPunk');
-    });
-
-    it('should use IIFE pattern for global assignment', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // esbuild generates IIFE format with var assignment
-      expect(bundleContent).toMatch(/var\s+DesignerPunk\s*=/);
-    });
-
-    it('should expose components via global namespace', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // Components should be accessible via the namespace
-      // Actual component names from browser-entry.ts exports (Stemma System naming)
-      expect(bundleContent).toContain('InputTextBase');
-      expect(bundleContent).toContain('ButtonCTA');
-      expect(bundleContent).toContain('IconBaseElement');
-      expect(bundleContent).toContain('ContainerBaseWeb');
-    });
-  });
-
-  describe('Script Tag Compatibility', () => {
-    it('should be valid JavaScript that can be parsed', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // Try to parse the bundle as JavaScript
-      expect(() => {
-        new Function(bundleContent);
-      }).not.toThrow();
-    });
-
-    it('should be self-executing (IIFE)', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // IIFE pattern: starts with ( or var and contains function execution
-      expect(bundleContent).toMatch(/\(\s*\(\s*\)\s*=>|\(\s*function\s*\(/);
-    });
-
-    it('should not require module system', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // UMD bundles should work without require/import
-      // They should not have top-level import statements
-      expect(bundleContent).not.toMatch(/^import\s+/m);
-    });
-  });
-
-  describe('Minified Bundle', () => {
-    it('should be smaller than non-minified bundle', () => {
-      const umdSize = fs.statSync(UMD_BUNDLE).size;
-      const umdMinSize = fs.statSync(UMD_MIN_BUNDLE).size;
-
-      expect(umdMinSize).toBeLessThan(umdSize);
-    });
-
-    it('should still define DesignerPunk namespace', () => {
-      const bundleContent = fs.readFileSync(UMD_MIN_BUNDLE, 'utf-8');
-
-      // Minified bundle should still have the namespace
-      expect(bundleContent).toContain('DesignerPunk');
-    });
-
-    it('should still contain component exports', () => {
-      const bundleContent = fs.readFileSync(UMD_MIN_BUNDLE, 'utf-8');
-
-      // Minified bundle should still export components
-      // Note: Minification may mangle internal names, but exports should remain
-      // Check for custom element tag names which are preserved as strings
-      // Note: Stemma System migration - text-input-field removed, using input-text-base instead
-      // Note: dp-icon and dp-container deprecated names removed
-      expect(bundleContent).toContain('input-text-base');
-      expect(bundleContent).toContain('button-cta');
-      expect(bundleContent).toContain('icon-base');
-      expect(bundleContent).toContain('container-base');
-    });
-
-    it('should be valid JavaScript', () => {
-      const bundleContent = fs.readFileSync(UMD_MIN_BUNDLE, 'utf-8');
-
-      expect(() => {
-        new Function(bundleContent);
-      }).not.toThrow();
-    });
-  });
-
-  describe('Browser Target Compatibility', () => {
-    it('should be self-contained (no external dependencies)', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // Bundle should not have import statements for external packages
-      // All dependencies should be inlined
-      const importMatches = bundleContent.match(/import\s+.*from\s+['"][^'"]+['"]/g) || [];
-      
-      // Filter out any internal imports (if any)
-      const externalImports = importMatches.filter(
-        (imp) => !imp.includes('./') && !imp.includes('../')
-      );
-
-      expect(externalImports.length).toBe(0);
-    });
-
-    it('should work in browser environment without Node.js APIs', () => {
-      const bundleContent = fs.readFileSync(UMD_BUNDLE, 'utf-8');
-
-      // Should not reference Node.js-specific globals
-      expect(bundleContent).not.toMatch(/\bprocess\.env\b/);
-      expect(bundleContent).not.toMatch(/\b__dirname\b/);
-      expect(bundleContent).not.toMatch(/\b__filename\b/);
-    });
-  });
-});

package/src/__tests__/console-allowlist.json

@@ -1,14 +0,0 @@
-[
-  { "suite": "src/components/core/Avatar-Base/__tests__/Avatar.accessibility.test.ts", "pattern": "^AvatarBaseElement: \"alt\" is required when \"src\" is provided\\.", "reason": "Spec 126 (O2, ratified 2026-07-09) resolved the original Req 5.4 / Req 9.2 tension this entry was seeded against: the warn condition is now `type === 'human' && src && alt == null && !decorative`, so a genuinely decorative or empty-alt (via setAttribute) fixture no longer warns at all. This entry is retained as a safety net, not a live suppression path — every fixture in this suite that legitimately triggers the warn (human + src + absent alt, incl. the property-form alt='' collapse) wraps it in a local jest.spyOn(console,'warn').mockImplementation(() => {}), which per console-fail-setup.ts's documented shadow behavior fully swallows the output before this hook ever sees it. Pattern updated to match the new Spec 126 message string (was: '\"alt\" prop is required when \"src\" is provided')." },
-  { "suite": "src/components/core/Chip-Base/__tests__/ChipBase.test.ts", "pattern": "^ChipBase: Could not calculate blend colors", "reason": "jsdom stylesheet limitation: jsdom never applies real CSS, so the blend-color token lookup cannot succeed and the component's designed CSS-fallback warn fires every run (fired async via rAF). PR #39 adjudicated as environmental noise, not a defect. Component-prefixed pattern (Lina review, PR #39) — a bare 'Could not calculate blend colors' substring is shared by Button components too; prefix-matching prevents accidentally swallowing a cross-component warn. Discharges the pending jsdom-stylesheet-limitation doc-addition (125-B-backlog.md item 5; Thurgood chip, 2026-07-09)." },
-  { "suite": "src/components/core/Chip-Filter/__tests__/ChipFilter.test.ts", "pattern": "^ChipFilter: Could not calculate blend colors", "reason": "jsdom stylesheet limitation: jsdom never applies real CSS, so the blend-color token lookup cannot succeed and the component's designed CSS-fallback warn fires every run (fired async via rAF). PR #39 adjudicated as environmental noise, not a defect. Component-prefixed pattern (Lina review, PR #39) — a bare 'Could not calculate blend colors' substring is shared by Button components too; prefix-matching prevents accidentally swallowing a cross-component warn. Discharges the pending jsdom-stylesheet-limitation doc-addition (125-B-backlog.md item 5; Thurgood chip, 2026-07-09)." },
-  { "suite": "src/components/core/Chip-Input/__tests__/ChipInput.test.ts", "pattern": "^ChipInput: Could not calculate blend colors", "reason": "jsdom stylesheet limitation: jsdom never applies real CSS, so the blend-color token lookup cannot succeed and the component's designed CSS-fallback warn fires every run (fired async via rAF). PR #39 adjudicated as environmental noise, not a defect. Component-prefixed pattern (Lina review, PR #39) — a bare 'Could not calculate blend colors' substring is shared by Button components too; prefix-matching prevents accidentally swallowing a cross-component warn. Discharges the pending jsdom-stylesheet-limitation doc-addition (125-B-backlog.md item 5; Thurgood chip, 2026-07-09)." },
-  { "suite": "src/build/errors/__tests__/ErrorHandler.test.ts", "pattern": "^\\[(ERROR|WARNING|INFO)\\]|^Suggestions:|^ {2}\\u2022 |^ {2}\\.\\.\\. and \\d+ more", "reason": "Deliberate error-path logging: this suite constructs BuildErrors with fake TEST_ERROR/CONFIG_ERROR/BUILD_ERROR codes specifically to assert ErrorHandler's own console-emitting default logger (ErrorHandler.ts defaultLogger/formatErrorMessage) — the console output IS the behavior under test. PR #39 adjudicated as expected noise (96 error blocks silenced fleet-wide, this suite's share among them)." },
-  { "suite": "src/build/errors/__tests__/ErrorHandler.integration.test.ts", "pattern": "^\\[(ERROR|WARNING|INFO)\\]|^Suggestions:|^ {2}\\u2022 |^ {2}\\.\\.\\. and \\d+ more", "reason": "Deliberate error-path logging: this suite exercises ErrorHandler/ErrorReporter integration with fake BUILD_ERROR/CONFIG_ERROR codes — the console output IS the behavior under test. PR #39 adjudicated as expected noise." },
-  { "suite": "product-mcp-server/src/__tests__/GapDetector.test.ts", "pattern": "^\\[mcp-product-server\\] Component directory not found: .* \\u2014 gap detection disabled$", "reason": "Deliberate error-path logging: the 'with missing component directory' tests intentionally point GapDetector at a nonexistent path to assert its disabled-gracefully behavior — the console.error IS the assertion target for the first test and expected repeat noise for the others in the same describe block. PR #39 adjudicated as expected." },
-  { "suite": "product-mcp-server/src/__tests__/ProductIndexerWalk.test.ts", "pattern": "^\\[mcp-product-server\\] Indexed: \\d+ screens, \\d+ domain objects, \\d+ templates, \\d+ one-off components$", "reason": "ProductIndexer.index() logs an informational indexing summary via console.error (not a failure signal) on every index() call. PR #39 adjudicated as expected per-run noise, silenced via a beforeAll spy in this suite." },
-  { "suite": "product-mcp-server/src/__tests__/Spec108-ProductTokens.test.ts", "pattern": "^\\[mcp-product-server\\] Indexed: \\d+ screens, \\d+ domain objects, \\d+ templates, \\d+ one-off components$", "reason": "ProductIndexer.index() logs an informational indexing summary via console.error (not a failure signal) on every index() call. PR #39 adjudicated as expected per-run noise, silenced via a beforeAll spy in this suite." },
-  { "suite": "src/__tests__/integration/Spec107-DesignLanguageContext.test.ts", "pattern": "^\\[mcp-product-server\\] Indexed: \\d+ screens, \\d+ domain objects, \\d+ templates, \\d+ one-off components$", "reason": "ProductIndexer.index() logs an informational indexing summary via console.error (not a failure signal); this suite's ProductIndexer instances are constructed inside nested beforeAll hooks, so PR #39 installed the spy in an outer beforeAll (not beforeEach) to be active before them. Adjudicated as expected noise." },
-  { "suite": "src/cli/__tests__/figma-extract.test.ts", "pattern": "^\\u274c |^Usage: npm run figma:extract|^\\s+npm run figma:extract|^\\s+Run the DTCG generator first|^$", "reason": "Genuinely novel class, adjudicated in this task (125-B Task 4.4, not part of PR #39): figma-extract is a CLI tool whose console.error output IS its product surface — every message this suite asserts on is the deliberate, deterministic error/usage text figma-extract.ts emits (arg validation, DTCG-load failures, extraction failures, usage banners, blank-line separators). This suite installs its own module-top-level console.error spy (mockConsoleError, restored per-file, not per-test) and reads its .mock.calls directly; this hook's beforeEach captures the CURRENT console.error before spying and calls through to it, so the suite's own assertions keep working unaffected." },
-  { "suite": "src/cli/__tests__/figma-push.test.ts", "pattern": "^\\u274c |^Resolution:|^\\s+\\d+\\.\\s|^\\s+-\\s|^\\s+Run the DTCG generator first|^$", "reason": "Genuinely novel class, adjudicated in this task (125-B Task 4.4, not part of PR #39): figma-push is a CLI tool whose console.error output IS its product surface — every message this suite asserts on is the deliberate, deterministic error/resolution text figma-push.ts emits (arg validation, DTCG-load failures, drift/sync-failure reporting with its numbered resolution steps and '  - <detail>' lists, blank-line separators). Same module-top-level console.error spy pattern as figma-extract.test.ts; this hook chains through to it rather than shadowing it." }
-]