@astryxdesign/cli 0.7.2-canary.fe2c655 → 0.7.2

package/clients/cli/commands/doctor-integration.test.mjs

@@ -19,8 +19,9 @@ import {Command} from 'commander';
 import {discoverCoreTemplates} from '../../../foundation/discovery/template-adapter.mjs';
 import {discoverOwnedComponents} from '../../../foundation/discovery/component-discovery.mjs';
 import {discoverBuiltinTopics} from '../../../foundation/discovery/docs-discovery.mjs';
-import {findCoreDir} from '../../../foundation/fs/paths.mjs';
+import {findCoreDir, findProjectCoreDir} from '../../../foundation/fs/paths.mjs';
 import {registerDoctor} from './doctor.mjs';
+import {expectRecordsCarry} from '../../../test-utils/text-projection.mjs';
 
 let tmpDir;
 let previousCwd;
@@ -184,7 +185,7 @@ describe('doctor integration — command', () => {
     ]);
 
     expect(logCalls.join('\n')).toContain(
-      'astryx doctor integration templates <package>',
+      ' doctor integration templates <package>',
     );
     expect(process.exitCode).toBeUndefined();
   });
@@ -209,6 +210,470 @@ describe('doctor integration — command', () => {
     expect(process.exitCode).toBeUndefined();
   });
 
+  it('validate prints the fix and the file an issue carries, and JSON carries the same issue', async () => {
+    writeDocIntegration({name: 'deploying'});
+    fs.writeFileSync(
+      path.join(tmpDir, 'docs', 'ghost.doc.zh.mjs'),
+      "export default {title: 'ghost'};\n",
+    );
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      'doctor',
+      'integration',
+      'validate',
+    ]);
+    const output = logCalls.join('\n');
+    expect(output).toContain('orphan_doc_overlay');
+    expect(output).toMatch(
+      /fix:\s+Delete docs\/ghost\.doc\.zh\.mjs, or add the topic it translates/,
+    );
+    expect(output).toMatch(/file:\s+docs\/ghost\.doc\.zh\.mjs/);
+    expect(output).toMatch(/line:\s+1\b/);
+
+    logCalls = [];
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      '--json',
+      'doctor',
+      'integration',
+      'validate',
+    ]);
+    const parsed = JSON.parse(logCalls.join('\n'));
+    expect(parsed.data.issues).toEqual([
+      {
+        code: 'orphan_doc_overlay',
+        severity: 'warning',
+        message: expect.stringContaining('docs/ghost.doc.zh.mjs'),
+        fix: expect.stringContaining('Delete docs/ghost.doc.zh.mjs'),
+        file: 'docs/ghost.doc.zh.mjs',
+        line: 1,
+      },
+    ]);
+  });
+
+  it('validate warns about each doc field that does not match its authoring type, and still exits 0', async () => {
+    writeIntegration({id: 'hero', name: 'Hero', type: 'block'});
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      'doctor',
+      'integration',
+      'validate',
+    ]);
+
+    const output = logCalls.join('\n');
+    expect(output).toContain('[ok] No integration issues found.');
+    expect(output).toMatch(
+      /Checking 1 doc\(s\) against the authoring types \(TypeScript \d+\.\d+/,
+    );
+    expect(output).toContain('templates/hero.doc.mjs');
+    expect(output).toMatch(/export:\s+default/);
+    expect(output).toMatch(/type:\s+IntegrationTemplateDoc/);
+    expect(output).toContain(
+      'Add `aspectRatio`: IntegrationBlockTemplateDoc requires it.',
+    );
+    expect(output).toContain(
+      'Add `displayName`: IntegrationBlockTemplateDoc requires it.',
+    );
+    expect(output).toContain(
+      '2 doc field(s) in 1 file(s) do not match the authoring types.',
+    );
+    expect(process.exitCode).toBeUndefined();
+  });
+
+  it('validate says there is no doc to type-check, rather than asking for TypeScript', async () => {
+    const outside = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-no-docs-'));
+    try {
+      fs.writeFileSync(
+        path.join(outside, 'package.json'),
+        JSON.stringify({name: '@acme/widgets', version: '1.2.3'}),
+      );
+      fs.writeFileSync(
+        path.join(outside, 'astryx.integration.mjs'),
+        "export default {components: './components'};\n",
+      );
+      fs.mkdirSync(path.join(outside, 'components'));
+      fs.writeFileSync(
+        path.join(outside, 'components', 'Widget.tsx'),
+        'export function Widget() { return null; }\n',
+      );
+      process.chdir(outside);
+
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        'doctor',
+        'integration',
+        'validate',
+      ]);
+
+      const output = logCalls.join('\n');
+      expect(output).toContain('[info] No doc files to type-check.');
+      expect(output).not.toContain('Docs not type-checked');
+      expect(output).not.toContain('against the authoring types');
+    } finally {
+      process.chdir(previousCwd);
+      fs.rmSync(outside, {recursive: true, force: true});
+    }
+  });
+
+  it('validate says the docs matched even when another check failed', async () => {
+    writeIntegration({id: 'hero', missingRoot: true});
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      "export default {templates: './missing', components: './components'};\n",
+    );
+    fs.mkdirSync(path.join(tmpDir, 'components'), {recursive: true});
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.doc.mjs'),
+      "export default {type: 'component', name: 'AcmeBadge', displayName: 'Badge', usage: {description: 'A badge.'}, props: []};\n",
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.tsx'),
+      'export function AcmeBadge() { return null; }\n',
+    );
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      'doctor',
+      'integration',
+      'validate',
+    ]);
+
+    const output = logCalls.join('\n');
+    expect(process.exitCode).toBe(1);
+    expect(output).toContain(
+      '[info] No doc field differs from its authoring type (1 checked).',
+    );
+    expect(output).not.toContain('[ok] Every doc');
+  });
+
+  it('validate prints every field of the doc type check that JSON carries', async () => {
+    writeIntegration({id: 'hero', name: 'Hero', type: 'block'});
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      "export default {templates: './templates', components: './components'};\n",
+    );
+    fs.mkdirSync(path.join(tmpDir, 'components'), {recursive: true});
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.doc.mjs'),
+      "export default {type: 'component', name: 'AcmeBadge', displayName: 5, usage: {description: 'A badge.'}, props: [], keyword: ['x']};\n",
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.tsx'),
+      'export function AcmeBadge() { return null; }\n',
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'shared.mjs'),
+      `export default {type: 'component', name: 'AcmeAgain', displayName: 'Again', usage: {description: 'x'}, props: []};\n`,
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeAgain.doc.mjs'),
+      "export {default} from './shared.mjs';\n",
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeAgain.tsx'),
+      'export function AcmeAgain() { return null; }\n',
+    );
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'validate',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const report = JSON.parse(await run(['--json'])).data.docTypes;
+    const output = await run([]);
+
+    expect([...new Set(report.findings.map(f => f.problem))].sort()).toEqual([
+      'missing_field',
+      'unknown_field',
+      'wrong_type',
+    ]);
+    expect(report.unchecked).toHaveLength(1);
+    expectRecordsCarry(output, report.findings, ['doc', 'field', 'problem']);
+    expectRecordsCarry(output, report.unchecked, ['doc', 'reason']);
+    expect(output).toContain(
+      `Checking ${report.docs} doc(s) against the authoring types (TypeScript ${report.typescript})`,
+    );
+  });
+
+  it('validate type-checks each doc component discovery serves, in __fixtures__, in a dot-folder or behind a symlink', async () => {
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      JSON.stringify({name: '@acme/widgets', version: '1.2.3'}),
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      `export default {components: './components'};\n`,
+    );
+    for (const [rel, name] of [
+      ['components/__fixtures__/AcmeFixture.doc.mjs', 'AcmeFixture'],
+      ['components/.drafts/AcmeDraft.doc.mjs', 'AcmeDraft'],
+      ['shared/AcmeLinked.doc.mjs', 'AcmeLinked'],
+    ]) {
+      fs.mkdirSync(path.dirname(path.join(tmpDir, rel)), {recursive: true});
+      fs.writeFileSync(
+        path.join(tmpDir, rel),
+        `export default {type: 'component', name: '${name}', displayName: '${name}', usage: {description: 'Fixture.'}, props: [], keyword: []};\n`,
+      );
+    }
+    fs.symlinkSync(
+      path.join(tmpDir, 'shared', 'AcmeLinked.doc.mjs'),
+      path.join(tmpDir, 'components', 'AcmeLinked.doc.mjs'),
+    );
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      '--json',
+      'doctor',
+      'integration',
+      'validate',
+    ]);
+
+    const {docTypes} = JSON.parse(logCalls.join('\n')).data;
+    expect(docTypes).toMatchObject({status: 'checked', docs: 3, unchecked: []});
+    expect(
+      docTypes.findings.map(
+        (/** @type {any} */ f) => `${f.doc} ${f.field} ${f.problem}`,
+      ),
+    ).toEqual([
+      'components/__fixtures__/AcmeFixture.doc.mjs keyword unknown_field',
+      'components/.drafts/AcmeDraft.doc.mjs keyword unknown_field',
+      'components/AcmeLinked.doc.mjs keyword unknown_field',
+    ]);
+  });
+
+  it('validate type-checks a hidden doc and a second doc for one name, and counts a test fixture', async () => {
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      JSON.stringify({name: '@acme/widgets', version: '1.2.3'}),
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      `export default {components: './components'};\n`,
+    );
+    for (const [rel, name, extra] of [
+      ['components/AcmeBadge.doc.mjs', 'AcmeBadge', ''],
+      ['components/again/AcmeBadge.doc.mjs', 'AcmeBadge', ', keyword: []'],
+      [
+        'components/AcmeHidden.doc.mjs',
+        'AcmeHidden',
+        ', hidden: true, keyword: []',
+      ],
+      [
+        'components/__tests__/AcmeTested.doc.mjs',
+        'AcmeTested',
+        ', keyword: []',
+      ],
+    ]) {
+      fs.mkdirSync(path.dirname(path.join(tmpDir, rel)), {recursive: true});
+      fs.writeFileSync(
+        path.join(tmpDir, rel),
+        `export default {type: 'component', name: '${name}', displayName: '${name}', usage: {description: 'Fixture.'}, props: []${extra}};\n`,
+      );
+      fs.writeFileSync(
+        path.join(tmpDir, rel.replace('.doc.mjs', '.tsx')),
+        'export default function Fixture() { return null; }\n',
+      );
+    }
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'validate',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const {docTypes} = JSON.parse(await run(['--json'])).data;
+    const output = await run([]);
+
+    expect(docTypes).toMatchObject({docs: 3, fixtures: 1, unchecked: []});
+    expect(
+      docTypes.findings.map(
+        (/** @type {any} */ f) => `${f.doc} ${f.field} ${f.problem}`,
+      ),
+    ).toEqual([
+      'components/AcmeHidden.doc.mjs keyword unknown_field',
+      'components/again/AcmeBadge.doc.mjs keyword unknown_field',
+    ]);
+    expect(output).toContain('[info] 1 test fixture doc(s) not checked.');
+    expect(output).not.toContain('[ok] Every doc matches');
+  });
+
+  it('validate keeps the fixture count when test fixtures are the only docs', async () => {
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      JSON.stringify({name: '@acme/widgets', version: '1.2.3'}),
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      `export default {components: './components'};\n`,
+    );
+    const rel = 'components/__tests__/AcmeTested.doc.mjs';
+    fs.mkdirSync(path.dirname(path.join(tmpDir, rel)), {recursive: true});
+    fs.writeFileSync(
+      path.join(tmpDir, rel),
+      `export default {type: 'component', name: 'AcmeTested', displayName: 'AcmeTested', usage: {description: 'Fixture.'}, props: []};\n`,
+    );
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'validate',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const {docTypes} = JSON.parse(await run(['--json'])).data;
+    const output = await run([]);
+
+    expect(docTypes).toMatchObject({docs: 0, fixtures: 1, findings: []});
+    expect(output).toContain('[info] No doc files to type-check.');
+    expect(output).toContain('[info] 1 test fixture doc(s) not checked.');
+  });
+
+  it("validate keeps main's issue for a root it cannot read, and lists a root with a folder it cannot read", async () => {
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      JSON.stringify({name: '@acme/widgets', version: '1.2.3'}),
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      `export default {components: './components', docs: './docs'};\n`,
+    );
+    fs.mkdirSync(path.join(tmpDir, 'components', 'locked'), {recursive: true});
+    fs.mkdirSync(path.join(tmpDir, 'docs'));
+    const locked = [
+      path.join(tmpDir, 'components', 'locked'),
+      path.join(tmpDir, 'docs'),
+    ];
+    for (const dir of locked) fs.chmodSync(dir, 0o000);
+    process.chdir(tmpDir);
+    try {
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        '--json',
+        'doctor',
+        'integration',
+        'validate',
+      ]);
+    } finally {
+      for (const dir of locked) fs.chmodSync(dir, 0o755);
+    }
+
+    const {data} = JSON.parse(logCalls.join('\n'));
+    expect(data.issues.map((/** @type {any} */ i) => i.code).sort()).toEqual([
+      'invalid_component',
+      'unreadable_root',
+    ]);
+    expect(data.docTypes.unchecked).toEqual([
+      {
+        doc: 'components',
+        reason: expect.stringMatching(
+          /^the components root does not load, so no doc in it was checked: .*EACCES/,
+        ),
+      },
+    ]);
+  });
+
+  it('validate reports the doc type check as additive JSON', async () => {
+    writeIntegration({id: 'hero', name: 'Hero', type: 'block'});
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      '--json',
+      'doctor',
+      'integration',
+      'validate',
+    ]);
+
+    const parsed = JSON.parse(logCalls.join('\n'));
+    expect(parsed.data.issues).toEqual([]);
+    expect(parsed.data.docTypes).toMatchObject({
+      status: 'checked',
+      docs: 1,
+      unchecked: [],
+    });
+    expect(parsed.data.docTypes.typescript).toMatch(/^\d+\.\d+/);
+    expect(parsed.data.docTypes.findings).toEqual([
+      expect.objectContaining({
+        doc: 'templates/hero.doc.mjs',
+        export: 'default',
+        type: 'IntegrationTemplateDoc',
+        field: 'aspectRatio',
+        problem: 'missing_field',
+        severity: 'warning',
+      }),
+      expect.objectContaining({field: 'displayName', problem: 'missing_field'}),
+    ]);
+    expect(process.exitCode).toBeUndefined();
+  });
+
+  it('validate says, in one visible line, that docs were not type-checked when the project has no TypeScript', async () => {
+    const outside = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-no-ts-'));
+    const inside = tmpDir;
+    try {
+      // The fixture helpers write to tmpDir; point it outside the repository,
+      // where no TypeScript resolves, and restore it for afterEach.
+      tmpDir = outside;
+      writeIntegration({id: 'hero', name: 'Hero', type: 'block'});
+      tmpDir = inside;
+      process.chdir(outside);
+
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        'doctor',
+        'integration',
+        'validate',
+      ]);
+
+      const output = logCalls.join('\n');
+      expect(output).toContain(
+        '[skip] Docs not type-checked: TypeScript is not installed in this project',
+      );
+      expect(output).not.toMatch(
+        /Checking \d+ doc\(s\) against the authoring types/,
+      );
+      expect(process.exitCode).toBeUndefined();
+    } finally {
+      process.chdir(previousCwd);
+      fs.rmSync(outside, {recursive: true, force: true});
+    }
+  });
+
   it('templates warns on a Core id and writes the exact --package command', async () => {
     const [core] = await discoverCoreTemplates();
     expect(core).toBeDefined();
@@ -352,7 +817,8 @@ describe('doctor integration — command', () => {
     tmpDir = outside;
     try {
       writeComponentIntegration('AcmeCarousel');
-      expect(findCoreDir(outside)).toBeNull();
+      // Doctor judges the project's own Core, never the one a copy carries.
+      expect(findProjectCoreDir(outside)).toBeNull();
       process.chdir(outside);
 
       await createProgram().parseAsync([
@@ -366,6 +832,10 @@ describe('doctor integration — command', () => {
       const printed = logCalls.join('\n');
       expect(printed).toContain('core_not_found');
       expect(printed).not.toContain('[ok]');
+      // No TypeScript out here either: the props comparison says it skipped.
+      expect(printed).toContain(
+        '[skip] Props not compared: TypeScript is not installed in this project',
+      );
       expect(process.exitCode).toBe(1);
     } finally {
       tmpDir = previousTmp;
@@ -396,6 +866,356 @@ describe('doctor integration — command', () => {
     expect(process.exitCode).toBeUndefined();
   });
 
+  it('components prints each documented prop that differs from the code, and still exits 0', async () => {
+    writeComponentIntegration('AcmeBadge');
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeBadge', displayName: 'AcmeBadge', description: 'Fixture component.', props: [{name: 'label', type: 'string', description: 'Text.'}]};\n`,
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.tsx'),
+      'export function AcmeBadge(props: {label: string}) { return null; }\n',
+    );
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      'doctor',
+      'integration',
+      'components',
+    ]);
+
+    const output = logCalls.join('\n');
+    expect(output).toMatch(/^kind:\s+required_mismatch$/m);
+    expect(output).toContain(
+      '"label" is required in the code, but the doc does not mark it required.',
+    );
+    expect(output).toContain(
+      '1 prop(s) differ between the docs and the code in 1 of 1 component(s) compared (TypeScript',
+    );
+    expect(output).not.toContain('[ok] Documented prop');
+    expect(process.exitCode).toBeUndefined();
+  });
+
+  it('components compares a hidden doc and a family entry with no record, so their drift is never an [ok]', async () => {
+    writeComponentIntegration('AcmeMenu');
+    const dir = path.join(tmpDir, 'components');
+    fs.writeFileSync(
+      path.join(dir, 'AcmeMenu.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeMenuRoot', displayName: 'AcmeMenuRoot', description: 'Fixture component.', hiddenComponents: ['AcmeMenuItem'], components: [{name: 'AcmeMenuItem', displayName: 'AcmeMenuItem', description: 'Fixture item.', props: [{name: 'label', type: 'string', description: 'Text.'}]}]};\n`,
+    );
+    fs.writeFileSync(
+      path.join(dir, 'AcmeMenu.tsx'),
+      'export function AcmeMenuRoot() { return null; }\nexport function AcmeMenuItem(props: {value?: string}) { return null; }\n',
+    );
+    fs.writeFileSync(
+      path.join(dir, 'AcmeSecret.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeSecret', displayName: 'AcmeSecret', description: 'Fixture component.', hidden: true, props: [{name: 'tone', type: 'string', description: 'Tone.'}]};\n`,
+    );
+    fs.writeFileSync(
+      path.join(dir, 'AcmeSecret.tsx'),
+      'export function AcmeSecret(props: {size?: number}) { return null; }\n',
+    );
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'components',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const report = JSON.parse(await run(['--json'])).data.propDrift;
+    const output = await run([]);
+
+    expect(
+      report.findings.map(f => `${f.component}.${f.prop} ${f.kind}`),
+    ).toEqual([
+      'AcmeMenuItem.label not_in_code',
+      'AcmeMenuItem.value not_in_doc',
+      'AcmeSecret.size not_in_doc',
+      'AcmeSecret.tone not_in_code',
+    ]);
+    expect(output).not.toContain('[ok] Documented prop');
+    expect(process.exitCode).toBeUndefined();
+  });
+
+  it('components lists props a mapped type builds from keys it cannot trace, and says the [ok] leaves them out', async () => {
+    writeComponentIntegration('AcmeChip');
+    const dir = path.join(tmpDir, 'components');
+    fs.writeFileSync(
+      path.join(dir, 'AcmeChip.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeChip', displayName: 'AcmeChip', description: 'Fixture.', props: [{name: 'label', type: 'string', description: 'Text.'}]};\n`,
+    );
+    fs.writeFileSync(
+      path.join(dir, 'AcmeChip.tsx'),
+      "export function AcmeChip(props: {label?: string} & {[K in `on${Capitalize<'open'>}`]?: () => void}) { return null; }\n",
+    );
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'components',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const {propsNotChecked} = JSON.parse(await run(['--json'])).data.propDrift;
+    const output = await run([]);
+
+    expect(propsNotChecked).toEqual([
+      expect.objectContaining({component: 'AcmeChip', props: ['onOpen']}),
+    ]);
+    expect(output).toContain(
+      '[info] Props not checked for a doc entry in 1 component(s):',
+    );
+    expect(output).toContain(
+      'in 1 component(s), apart from the props not checked (TypeScript',
+    );
+  });
+
+  it('components compares the entries of a family doc whose own source is missing', async () => {
+    writeComponentIntegration('AcmeTabs');
+    const dir = path.join(tmpDir, 'components');
+    const entry = (/** @type {string} */ name, /** @type {string} */ prop) =>
+      `{name: '${name}', displayName: '${name}', description: 'Fixture.', props: [{name: '${prop}', type: 'string', description: 'Text.'}]}`;
+    fs.writeFileSync(
+      path.join(dir, 'AcmeTabs.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeTabs', displayName: 'AcmeTabs', description: 'Fixture.', props: [], hiddenComponents: ['AcmeTabsPanel'], components: [${entry('AcmeTabsItem', 'label')}, ${entry('AcmeTabsPanel', 'ghost')}]};\n`,
+    );
+    fs.rmSync(path.join(dir, 'AcmeTabs.tsx'), {force: true});
+    // The package's entry exports the members, so discovery records the
+    // listed one, though the family's own file is missing.
+    for (const name of ['AcmeTabsItem', 'AcmeTabsPanel']) {
+      fs.writeFileSync(
+        path.join(dir, `${name}.tsx`),
+        `export function ${name}(props: {label?: string; open?: boolean}) { return null; }\n`,
+      );
+      fs.appendFileSync(
+        path.join(tmpDir, 'index.ts'),
+        `export {${name}} from './components/${name}';\n`,
+      );
+    }
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      '--json',
+      'doctor',
+      'integration',
+      'components',
+    ]);
+    const report = JSON.parse(logCalls.join('\n')).data.propDrift;
+
+    expect(
+      report.findings.map(
+        (/** @type {any} */ f) => `${f.component}.${f.prop} ${f.kind}`,
+      ),
+    ).toEqual([
+      'AcmeTabsItem.open not_in_doc',
+      'AcmeTabsPanel.ghost not_in_code',
+      'AcmeTabsPanel.label not_in_doc',
+      'AcmeTabsPanel.open not_in_doc',
+    ]);
+    expect(report.unchecked).toEqual([
+      {
+        component: 'AcmeTabs',
+        doc: 'components/AcmeTabs.doc.mjs',
+        reason: 'no source file for AcmeTabs',
+      },
+    ]);
+  });
+
+  it('components counts two docs of one name as two components, and compares a .doc.ts beside a .doc.mjs', async () => {
+    writeComponentIntegration('AcmeTwin');
+    const dir = path.join(tmpDir, 'components');
+    const doc = (/** @type {string} */ name, /** @type {string} */ prop) =>
+      `export default {type: 'component', name: '${name}', displayName: '${name}', description: 'Fixture.', props: [{name: '${prop}', type: 'string', description: 'Text.'}]};\n`;
+    fs.writeFileSync(
+      path.join(dir, 'AcmeTwin.doc.mjs'),
+      doc('AcmeTwin', 'tone'),
+    );
+    fs.writeFileSync(
+      path.join(dir, 'AcmeTwin.doc.ts'),
+      doc('AcmeTwin', 'size'),
+    );
+    fs.writeFileSync(
+      path.join(dir, 'AcmeTwin.tsx'),
+      'export function AcmeTwin(props: {label?: string}) { return null; }\n',
+    );
+    for (const folder of ['a', 'b']) {
+      fs.mkdirSync(path.join(dir, folder));
+      fs.writeFileSync(
+        path.join(dir, folder, 'AcmeX.doc.mjs'),
+        doc('AcmeX', folder),
+      );
+      fs.writeFileSync(
+        path.join(dir, folder, 'AcmeX.tsx'),
+        'export function AcmeX() { return null; }\n',
+      );
+    }
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'components',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const report = JSON.parse(await run(['--json'])).data.propDrift;
+    const output = await run([]);
+
+    expect(
+      report.findings
+        .filter(f => f.kind === 'not_in_code')
+        .map(f => `${f.doc} ${f.prop}`)
+        .sort(),
+    ).toEqual([
+      'components/AcmeTwin.doc.mjs tone',
+      'components/AcmeTwin.doc.ts size',
+      'components/a/AcmeX.doc.mjs a',
+      'components/b/AcmeX.doc.mjs b',
+    ]);
+    expect(output).toMatch(/ in 4 of 4 component\(s\) compared /);
+  });
+
+  it('components prints every field of the props comparison that JSON carries', async () => {
+    writeComponentIntegration('AcmeBadge');
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeBadge', displayName: 'AcmeBadge', description: 'Fixture component.', props: [${[
+        "{name: 'label', type: 'string', description: 'Text.'}",
+        "{name: 'elevation', type: 'number', description: 'Shadow.'}",
+        "{name: 'size', type: 'string', description: 'Size.'}",
+        "{name: 'meta', type: '{id: string; extra?: number}', description: 'Data.'}",
+        "{name: 'gap', type: 'number', description: 'Gap.', default: '8'}",
+      ].join(', ')}]};\n`,
+    );
+    // `key` is a prop a doc never needs, though the component declares it.
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeBadge.tsx'),
+      "export function AcmeBadge({gap = 4}: {label: string; tone?: 'neutral' | 'info'; size?: number; meta?: {id: string}; gap?: number; key?: string}) { return null; }\n",
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeLonely.doc.mjs'),
+      `export default {type: 'component', name: 'AcmeLonely', displayName: 'AcmeLonely', description: 'Fixture component.', props: []};\n`,
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeLonely.tsx'),
+      'export function Other() { return null; }\n',
+    );
+    process.chdir(tmpDir);
+    const run = async (/** @type {string[]} */ flags) => {
+      logCalls.length = 0;
+      await createProgram().parseAsync([
+        'node',
+        'astryx',
+        ...flags,
+        'doctor',
+        'integration',
+        'components',
+      ]);
+      return logCalls.join('\n');
+    };
+
+    const report = JSON.parse(await run(['--json'])).data.propDrift;
+    const output = await run([]);
+
+    expect(report.findings.map(f => `${f.prop} ${f.kind}`)).toEqual([
+      'elevation not_in_code',
+      'gap default_mismatch',
+      'label required_mismatch',
+      'size type_mismatch',
+      'tone not_in_doc',
+    ]);
+    expectRecordsCarry(output, report.findings, ['component', 'prop', 'kind']);
+    expectRecordsCarry(output, report.unchecked, ['component', 'reason']);
+    expectRecordsCarry(output, report.typesNotCompared, [
+      'component',
+      'reason',
+      'props',
+    ]);
+    expect(report.typesNotCompared).toHaveLength(1);
+    expect(output).toContain(
+      `Documented prop types judged: ${report.typesCompared}; not compared: 1.`,
+    );
+    // The response-type reference, which the README's JSON table comes from,
+    // names every field too.
+    const {doc: responseTypes} =
+      await import('../../../foundation/response/response-types.doc.mjs');
+    const member = responseTypes.members.find(
+      m => m.value === 'integration.component-conflicts',
+    );
+    for (const field of Object.keys(report)) {
+      expect(member?.description).toMatch(new RegExp(`\\b${field}\\b`));
+    }
+    expect(output).toContain(
+      `in 1 of ${report.components} component(s) compared (TypeScript ${report.typescript}).`,
+    );
+  });
+
+  it('components says documented props match when they do', async () => {
+    writeComponentIntegration('AcmeOnlyWidget');
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      'doctor',
+      'integration',
+      'components',
+    ]);
+
+    const output = logCalls.join('\n');
+    expect(output).toMatch(
+      /\[ok\] Documented prop names and required flags match the code in 1 component\(s\) \(TypeScript \d/,
+    );
+    expect(output).toMatch(
+      /\[info\] Documented prop types judged: \d+; not compared: 0\./,
+    );
+  });
+
+  it('components prints no [ok] for props it could not compare', async () => {
+    writeComponentIntegration('AcmeLonely');
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeLonely.tsx'),
+      'export function Other() { return null; }\n',
+    );
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      'doctor',
+      'integration',
+      'components',
+    ]);
+
+    const output = logCalls.join('\n');
+    expect(output).toContain(
+      '[info] Props not compared for 1 documented component(s):',
+    );
+    expect(output).not.toContain('[ok] Documented prop');
+  });
+
   it('components exposes conflicts as structured JSON warnings', async () => {
     const coreDir = findCoreDir(tmpDir);
     expect(coreDir).not.toBeNull();
@@ -425,6 +1245,71 @@ describe('doctor integration — command', () => {
     expect(process.exitCode).toBeUndefined();
   });
 
+  it('docs resolves a link to a component or a template, its own or Core’s, and still flags one that names no doc', async () => {
+    const page = (await discoverCoreTemplates()).find(
+      template => template.type === 'page',
+    );
+    expect(page).toBeDefined();
+    fs.writeFileSync(
+      path.join(tmpDir, 'package.json'),
+      JSON.stringify({name: '@acme/widgets', version: '1.2.3'}),
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.integration.mjs'),
+      "export default {components: './components', templates: './templates', docs: './docs'};\n",
+    );
+    for (const dir of ['components', 'templates', 'docs']) {
+      fs.mkdirSync(path.join(tmpDir, dir), {recursive: true});
+    }
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeButton.doc.mjs'),
+      "export default {type: 'component', name: 'AcmeButton', displayName: 'Acme Button', description: 'A button.', props: []};\n",
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'components', 'AcmeButton.tsx'),
+      'export function AcmeButton() { return null; }\n',
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'templates', 'acme-card.doc.mjs'),
+      "export default {type: 'block', name: 'Acme card', description: 'A card.'};\n",
+    );
+    fs.writeFileSync(
+      path.join(tmpDir, 'templates', 'acme-card.tsx'),
+      'export default function Fixture() { return null; }\n',
+    );
+    const text =
+      'Start with {@link component:AcmeButton} or {@link block:acme-card}, ' +
+      'built on {@link @astryxdesign/core:component:Button} and ' +
+      `{@link @astryxdesign/core:page:${page?.dirName}}. Not {@link component:AcmeMissing}.`;
+    fs.writeFileSync(
+      path.join(tmpDir, 'docs', 'using-widgets.doc.mjs'),
+      `export default ${JSON.stringify({
+        type: 'generic',
+        name: 'using-widgets',
+        title: 'Using widgets',
+        description: 'Fixture topic.',
+        sections: [{title: 'Overview', content: [{type: 'prose', text}]}],
+      })};\n`,
+    );
+    process.chdir(tmpDir);
+
+    await createProgram().parseAsync([
+      'node',
+      'astryx',
+      '--json',
+      'doctor',
+      'integration',
+      'docs',
+    ]);
+
+    const parsed = JSON.parse(logCalls.join('\n'));
+    const unresolved = parsed.data.issues
+      .filter(issue => issue.code === 'invalid_doc_graph')
+      .map(issue => issue.message);
+    expect(unresolved).toHaveLength(1);
+    expect(unresolved[0]).toContain('"component:AcmeMissing" names no doc');
+  });
+
   it('docs reports an intentional replacement as info', async () => {
     const [coreTopic] = Object.keys(discoverBuiltinTopics());
     writeDocIntegration({name: 'acme-setup', replaces: coreTopic});