@astryxdesign/cli 0.7.2-canary.e2909cf → 0.7.2-canary.e4bdd70

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (194) hide show
  1. package/README.md +15 -14
  2. package/api/build/help/help.mjs +1 -1
  3. package/api/component/_adapter.d.mts +41 -1
  4. package/api/component/_adapter.mjs +93 -7
  5. package/api/component/component.mjs +22 -5
  6. package/api/component/integration-export-name.test.mjs +644 -0
  7. package/api/docs/_adapter.d.mts +11 -0
  8. package/api/docs/_adapter.mjs +244 -1
  9. package/api/docs/integrationDocs.test.mjs +124 -0
  10. package/api/doctor/theme-checks.mjs +1 -1
  11. package/api/gap-report/gap-report.mjs +67 -4
  12. package/api/index.d.mts +2 -0
  13. package/api/index.mjs +2 -0
  14. package/api/init/run/run.mjs +2 -2
  15. package/api/integration/_adapter.d.mts +12 -0
  16. package/api/integration/_adapter.mjs +54 -0
  17. package/api/integration/add-contribution.component-names.test.mjs +66 -0
  18. package/api/integration/add-contribution.mjs +8 -4
  19. package/api/integration/add-contribution.test.mjs +4 -2
  20. package/api/integration/authoring-checks.mjs +11 -2
  21. package/api/integration/authoring-checks.test.mjs +39 -0
  22. package/api/integration/authoring-checks.type.d.mts +9 -0
  23. package/api/integration/authoring-checks.type.mjs +14 -1
  24. package/api/integration/check/check.d.mts +96 -0
  25. package/api/integration/check/check.mjs +357 -0
  26. package/api/integration/check/check.test.mjs +489 -0
  27. package/api/integration/integration-check.type.d.mts +118 -0
  28. package/api/integration/integration-check.type.mjs +84 -0
  29. package/api/integration/integrationCheck.doc.d.mts +10 -0
  30. package/api/integration/integrationCheck.doc.mjs +73 -0
  31. package/api/integration/integrationComponentConflicts.doc.mjs +36 -5
  32. package/api/integration/pack-check.mjs +51 -97
  33. package/api/integration/pack-check.test.mjs +359 -0
  34. package/api/integration/validate-integration.mjs +8 -0
  35. package/api/integration/validate-integration.test.mjs +11 -0
  36. package/api/integration/validate-integration.type.d.mts +8 -0
  37. package/api/integration/validate-integration.type.mjs +12 -1
  38. package/api/integration/validateIntegration.doc.mjs +17 -3
  39. package/api/search/search.d.mts +22 -6
  40. package/api/search/search.domain-priority.test.mjs +93 -18
  41. package/api/search/search.integration-check.test.mjs +24 -0
  42. package/api/search/search.mjs +133 -22
  43. package/api/swizzle/copy/copy.mjs +28 -14
  44. package/api/theme/build/build.family.test.mjs +54 -6
  45. package/api/theme/build/build.mjs +114 -17
  46. package/api/theme/themeBuild.doc.mjs +4 -4
  47. package/assets/docs/README.md +10 -0
  48. package/assets/docs/author-a-theme.doc.mjs +1 -1
  49. package/assets/docs/getting-started.doc.mjs +3 -3
  50. package/assets/docs/styling-overview.doc.mjs +1 -1
  51. package/assets/docs/theme.doc.dense.mjs +1 -1
  52. package/assets/docs/theme.doc.mjs +1 -1
  53. package/assets/docs/theme.doc.zh.mjs +1 -1
  54. package/assets/docs/tree/add-a-component.doc.mjs +11 -1
  55. package/assets/docs/tree/add-a-theme.doc.mjs +1 -1
  56. package/assets/docs/tree/add-a-topic.doc.mjs +10 -7
  57. package/assets/docs/tree/agent-guidance.doc.mjs +2 -2
  58. package/assets/docs/tree/block-template.doc.mjs +8 -20
  59. package/assets/docs/tree/build-the-template.doc.mjs +1 -1
  60. package/assets/docs/tree/check-the-props.doc.mjs +196 -0
  61. package/assets/docs/tree/check-your-docs.doc.mjs +51 -5
  62. package/assets/docs/tree/checks.doc.mjs +24 -16
  63. package/assets/docs/tree/codemods.doc.mjs +4 -4
  64. package/assets/docs/tree/component-family.doc.mjs +8 -6
  65. package/assets/docs/tree/component-imports.doc.mjs +10 -2
  66. package/assets/docs/tree/components.doc.mjs +1 -1
  67. package/assets/docs/tree/debug-and-gap-reports.doc.mjs +7 -18
  68. package/assets/docs/tree/define-the-theme.doc.mjs +6 -5
  69. package/assets/docs/tree/describe-the-component.doc.mjs +2 -2
  70. package/assets/docs/tree/docs.doc.mjs +3 -3
  71. package/assets/docs/tree/document-the-template.doc.mjs +1 -1
  72. package/assets/docs/tree/document-the-theme.doc.mjs +1 -1
  73. package/assets/docs/tree/export-template-assets.doc.mjs +1 -1
  74. package/assets/docs/tree/extend-or-replace.doc.mjs +6 -6
  75. package/assets/docs/tree/fonts-and-assets.doc.mjs +2 -2
  76. package/assets/docs/tree/generate-a-palette.doc.mjs +5 -3
  77. package/assets/docs/tree/grade-template-with-agent.doc.mjs +1 -1
  78. package/assets/docs/tree/integrations.doc.mjs +8 -0
  79. package/assets/docs/tree/links.doc.mjs +6 -5
  80. package/assets/docs/tree/package-and-test.doc.mjs +1 -1
  81. package/assets/docs/tree/page-template.doc.mjs +4 -8
  82. package/assets/docs/tree/publishing.doc.mjs +1 -1
  83. package/assets/docs/tree/quick-start.doc.mjs +37 -21
  84. package/assets/docs/tree/replace-a-core-component.doc.mjs +4 -6
  85. package/assets/docs/tree/replace-a-core-template.doc.mjs +6 -8
  86. package/assets/docs/tree/sections-and-placement.doc.mjs +4 -6
  87. package/assets/docs/tree/see-it-in-an-app.doc.mjs +4 -2
  88. package/assets/docs/tree/short-and-findable.doc.mjs +6 -5
  89. package/assets/docs/tree/single-component.doc.mjs +15 -6
  90. package/assets/docs/tree/start-a-template.doc.mjs +3 -3
  91. package/assets/docs/tree/subcomponent.doc.mjs +6 -6
  92. package/assets/docs/tree/template-assets.doc.mjs +2 -2
  93. package/assets/docs/tree/template-doc-overview.doc.mjs +4 -15
  94. package/assets/docs/tree/template-fonts.doc.mjs +1 -1
  95. package/assets/docs/tree/template-icons.doc.mjs +1 -1
  96. package/assets/docs/tree/template-images-media.doc.mjs +1 -1
  97. package/assets/docs/tree/template-styles.doc.mjs +2 -2
  98. package/assets/docs/tree/templates.doc.mjs +1 -1
  99. package/assets/docs/tree/test-in-an-app.doc.mjs +3 -3
  100. package/assets/docs/tree/test-template-in-app.doc.mjs +1 -1
  101. package/assets/docs/tree/tokens-and-setup.doc.mjs +19 -15
  102. package/assets/docs/tree/troubleshooting.doc.mjs +13 -5
  103. package/assets/docs/tree/type-scale.doc.mjs +3 -3
  104. package/assets/docs/tree/upgrading.doc.mjs +3 -3
  105. package/assets/docs/tree/use-a-theme-in-an-app.doc.mjs +3 -1
  106. package/assets/docs/tree/verify-packed-template.doc.mjs +1 -1
  107. package/assets/docs/tree/versioning.doc.mjs +7 -6
  108. package/assets/docs/tree/write-the-template-file.doc.mjs +4 -3
  109. package/assets/docs/use-a-theme.doc.mjs +2 -2
  110. package/assets/theme.template.ts +1 -1
  111. package/authoring/codemod/codemod.doc.mjs +7 -3
  112. package/authoring/config/config.doc.mjs +9 -6
  113. package/authoring/config/type.ts +4 -2
  114. package/authoring/discover/discover.doc.mjs +1 -1
  115. package/authoring/doctypes/base/graph-fields.doc.mjs +6 -4
  116. package/authoring/doctypes/command/command.doc.mjs +14 -12
  117. package/authoring/doctypes/component/component.doc.mjs +29 -26
  118. package/authoring/doctypes/component/type.ts +13 -8
  119. package/authoring/doctypes/enum/enum.doc.mjs +7 -4
  120. package/authoring/doctypes/function/function.doc.mjs +9 -8
  121. package/authoring/doctypes/hook/hook.doc.mjs +5 -4
  122. package/authoring/doctypes/namespace/namespace.doc.mjs +7 -6
  123. package/authoring/doctypes/reference/reference.doc.mjs +11 -10
  124. package/authoring/doctypes/schema/schema.doc.mjs +9 -9
  125. package/authoring/doctypes/template/integration-template.doc.d.mts +11 -0
  126. package/authoring/doctypes/template/integration-template.doc.mjs +49 -0
  127. package/authoring/doctypes/template/parse.d.mts +6 -2
  128. package/authoring/doctypes/template/parse.mjs +3 -1
  129. package/authoring/doctypes/template/template.doc.mjs +3 -3
  130. package/authoring/doctypes/template/type.ts +22 -0
  131. package/authoring/doctypes/theme/theme.doc.mjs +2 -2
  132. package/authoring/gap-report/gap-report.doc.mjs +2 -2
  133. package/authoring/identity/identity.doc.mjs +3 -3
  134. package/authoring/index.d.ts +1 -0
  135. package/authoring/integration/integration.doc.mjs +12 -4
  136. package/authoring/integration/parse.test.mjs +10 -0
  137. package/authoring/integration/schema.d.mts +2 -0
  138. package/authoring/integration/schema.mjs +1 -0
  139. package/authoring/integration/type.ts +4 -0
  140. package/clients/cli/commands/build-theme.batch-report.test.mjs +3 -1
  141. package/clients/cli/commands/build-theme.mjs +1 -1
  142. package/clients/cli/commands/build-theme.path-safety.test.mjs +155 -21
  143. package/clients/cli/commands/doctor-integration-components.doc.mjs +15 -4
  144. package/clients/cli/commands/doctor-integration-validate.doc.mjs +11 -2
  145. package/clients/cli/commands/doctor-integration.test.mjs +837 -0
  146. package/clients/cli/commands/doctor.mjs +190 -0
  147. package/clients/cli/commands/integration-check.doc.mjs +85 -0
  148. package/clients/cli/commands/integration-check.test.mjs +267 -0
  149. package/clients/cli/commands/integration.doc.mjs +6 -2
  150. package/clients/cli/commands/integration.mjs +107 -1
  151. package/clients/cli/commands/text-json-parity.test.mjs +5 -0
  152. package/clients/cli/commands/theme-build.doc.mjs +2 -2
  153. package/clients/cli/index.mjs +1 -0
  154. package/foundation/config/project.mjs +82 -0
  155. package/foundation/discovery/authoring-self-docs.mjs +2 -1
  156. package/foundation/discovery/component-discovery.d.mts +178 -10
  157. package/foundation/discovery/component-discovery.export-names.test.mjs +760 -0
  158. package/foundation/discovery/component-discovery.mjs +659 -62
  159. package/foundation/discovery/component-replacement.mjs +18 -0
  160. package/foundation/discovery/component-replacement.test.mjs +40 -0
  161. package/foundation/discovery/doc-types.d.mts +157 -0
  162. package/foundation/discovery/doc-types.mjs +1224 -0
  163. package/foundation/discovery/doc-types.test.mjs +652 -0
  164. package/foundation/discovery/docs-discovery.d.mts +81 -0
  165. package/foundation/discovery/docs-discovery.mjs +57 -0
  166. package/foundation/discovery/integration-doc-files.d.mts +22 -0
  167. package/foundation/discovery/integration-doc-files.mjs +231 -0
  168. package/foundation/discovery/integration-doc-files.test.mjs +246 -0
  169. package/foundation/discovery/module-exports.d.mts +27 -0
  170. package/foundation/discovery/module-exports.mjs +356 -31
  171. package/foundation/discovery/module-exports.test.mjs +172 -0
  172. package/foundation/discovery/package-tsconfig.d.mts +14 -0
  173. package/foundation/discovery/package-tsconfig.mjs +228 -0
  174. package/foundation/discovery/package-tsconfig.test.mjs +210 -0
  175. package/foundation/discovery/project-typescript.d.mts +23 -0
  176. package/foundation/discovery/project-typescript.mjs +61 -0
  177. package/foundation/discovery/prop-drift.d.mts +278 -0
  178. package/foundation/discovery/prop-drift.mjs +2263 -0
  179. package/foundation/discovery/prop-drift.test.mjs +2219 -0
  180. package/foundation/discovery/type-match.d.mts +53 -0
  181. package/foundation/discovery/type-match.mjs +660 -0
  182. package/foundation/discovery/type-match.test.mjs +169 -0
  183. package/foundation/discovery/type-text.d.mts +68 -0
  184. package/foundation/discovery/type-text.mjs +658 -0
  185. package/foundation/discovery/type-text.test.mjs +196 -0
  186. package/foundation/discovery/unresolved-references.d.mts +62 -0
  187. package/foundation/discovery/unresolved-references.mjs +790 -0
  188. package/foundation/discovery/unresolved-references.test.mjs +567 -0
  189. package/foundation/doc-compiler/doc-loads.test.mjs +6 -0
  190. package/foundation/doc-compiler/links.d.mts +9 -5
  191. package/foundation/doc-compiler/links.mjs +4 -2
  192. package/foundation/integrations/contribution-inventory.mjs +6 -1
  193. package/foundation/response/response-types.doc.mjs +7 -2
  194. package/package.json +9 -9
@@ -17,8 +17,11 @@
17
17
  * foundation/discovery/docs-discovery, which the catalog comes from.
18
18
  */
19
19
 
20
+ import * as path from 'node:path';
20
21
  import {Project} from '../../foundation/config/project.mjs';
21
22
  import {DocsCatalog} from '../../foundation/discovery/docs-discovery.mjs';
23
+ import {checkDocTypes} from '../../foundation/discovery/doc-types.mjs';
24
+ import {integrationDocFiles} from '../../foundation/discovery/integration-doc-files.mjs';
22
25
  import {
23
26
  linkReferenceTopic,
24
27
  lowerReferenceTopic,
@@ -53,6 +56,17 @@ import {
53
56
  selfDocSection,
54
57
  } from '../../foundation/discovery/authoring-self-docs.mjs';
55
58
  import {CLI_PROVIDER_ID} from '../../foundation/identity/providers.mjs';
59
+ import {
60
+ CORE_PACKAGE,
61
+ discoverOwnedComponents,
62
+ discoverValidIntegrationComponents,
63
+ } from '../../foundation/discovery/component-discovery.mjs';
64
+ import {loadComponentDoc} from '../../foundation/discovery/component-loader.mjs';
65
+ import {
66
+ discoverCoreTemplates,
67
+ discoverIntegrationTemplatesForOne,
68
+ } from '../../foundation/discovery/template-adapter.mjs';
69
+ import {findCoreDir} from '../../foundation/fs/paths.mjs';
56
70
  import {AstryxError} from '../error.mjs';
57
71
  import {ERROR_CODES} from '../../foundation/response/error-codes.mjs';
58
72
 
@@ -495,12 +509,205 @@ async function docFinder(catalog, fromProvider) {
495
509
  };
496
510
  }
497
511
  }
512
+ if (CONTRIBUTED_KINDS.has(parsed.kind)) {
513
+ const link = await contributedDocLink(
514
+ catalog,
515
+ provider,
516
+ parsed.kind,
517
+ parsed.name,
518
+ );
519
+ if (link)
520
+ return {link: {target, ...link}, kind: parsed.kind, typed: null};
521
+ }
522
+ const searchType =
523
+ parsed.kind === 'component'
524
+ ? 'component'
525
+ : parsed.kind === 'page' || parsed.kind === 'block'
526
+ ? 'template'
527
+ : 'doc';
498
528
  return {
499
- problem: `"${target}" names no doc. Find it with \`astryx search ${parsed.name} --type doc\`, then name it as \`[<provider>:]<kind>:<name>\`.`,
529
+ problem: `"${target}" names no doc. Find it with \`astryx search ${parsed.name} --type ${searchType}\`, then name it as \`[<provider>:]<kind>:<name>\`.`,
500
530
  };
501
531
  };
502
532
  }
503
533
 
534
+ /** The doc kinds no docs tree holds: components and templates. */
535
+ const CONTRIBUTED_KINDS = new Set(['component', 'page', 'block']);
536
+
537
+ /**
538
+ * The link to a component, page, or block doc (spec:AST-047 FR9). No docs tree
539
+ * holds these, so a link finds them among the component and template records
540
+ * the catalog carries, the ones `astryx component` and `astryx template` open,
541
+ * and reads as that command. A component is named by its export name, a
542
+ * template by its id; anything but Core's names its package.
543
+ * @param {DocsCatalog} catalog
544
+ * @param {string} provider normalized provider id
545
+ * @param {string} kind `component`, `page`, or `block`
546
+ * @param {string} name
547
+ * @returns {Promise<Omit<import('../../foundation/doc-compiler/links.mjs').DocLink, 'target'> | null>}
548
+ */
549
+ async function contributedDocLink(catalog, provider, kind, name) {
550
+ const contributions = await catalog.contributions();
551
+ if (!contributions) return null;
552
+ const sameProvider = (/** @type {string} */ providerId) => {
553
+ try {
554
+ return normalizeProviderId(providerId) === provider;
555
+ } catch {
556
+ return false;
557
+ }
558
+ };
559
+ if (kind === 'component') {
560
+ const record = contributions.components.find(
561
+ component =>
562
+ component.name === name && sameProvider(component.providerId),
563
+ );
564
+ if (!record) return null;
565
+ /** @type {any} */
566
+ let doc = null;
567
+ try {
568
+ doc = record.docPath ? await loadComponentDoc(record.docPath) : null;
569
+ } catch {
570
+ // The component checks report a doc that does not load; the link
571
+ // still opens the component.
572
+ }
573
+ const entry =
574
+ doc?.name === name
575
+ ? doc
576
+ : doc?.components?.find(
577
+ (/** @type {any} */ each) => each.name === name,
578
+ );
579
+ // Bare, Core's name opens Core's component unless another package ships
580
+ // the same name or replaces it (spec:AST-035 FR11); then the link selects
581
+ // Core, as it always selects any other package.
582
+ const selectPackage =
583
+ record.package !== CORE_PACKAGE ||
584
+ contributions.components.some(
585
+ other =>
586
+ other.package !== CORE_PACKAGE &&
587
+ (other.name === name || other.replaces === name),
588
+ );
589
+ return {
590
+ id: createDocId(provider, 'component', name),
591
+ route: null,
592
+ title: entry?.displayName ?? name,
593
+ summary: entry?.description ?? doc?.description ?? '',
594
+ command: `astryx component ${name}${selectPackage ? ` --package ${record.package}` : ''}`,
595
+ };
596
+ }
597
+ const template = contributions.templates.find(
598
+ each =>
599
+ each.type === kind && each.id === name && sameProvider(each.providerId),
600
+ );
601
+ if (!template) return null;
602
+ // The same for a Core template: another package's template of its kind
603
+ // with its id, or one that replaces it, takes the bare id.
604
+ const selectPackage =
605
+ template.package !== CORE_PACKAGE ||
606
+ contributions.templates.some(
607
+ other =>
608
+ other.package !== CORE_PACKAGE &&
609
+ other.type === template.type &&
610
+ (other.id === name || other.replaces === name),
611
+ );
612
+ return {
613
+ id: createDocId(provider, template.type, name),
614
+ route: null,
615
+ title: template.displayName ?? template.name,
616
+ summary: template.description ?? '',
617
+ command: `astryx template ${name} --type ${template.type}${selectPackage ? ` --package ${template.package}` : ''}`,
618
+ };
619
+ }
620
+
621
+ /**
622
+ * The component and template docs one integration package's links can name:
623
+ * Core's and its own, as `doctor integration docs` checks it before any app
624
+ * configures it.
625
+ * @param {any} integration a loaded integration: its name, provider id, roots,
626
+ * and package directory
627
+ * @returns {Promise<import('../../foundation/discovery/docs-discovery.mjs').LinkableContributions>}
628
+ */
629
+ async function packageContributions(integration) {
630
+ const own = integration.providerId ?? integration.name;
631
+ /** @type {import('../../foundation/discovery/docs-discovery.mjs').LinkableComponent[]} */
632
+ const components = [];
633
+ const coreDir = findCoreDir(integration.__packageDir ?? process.cwd());
634
+ if (coreDir) {
635
+ try {
636
+ for (const record of discoverOwnedComponents(coreDir, [])) {
637
+ components.push({
638
+ providerId: CORE_PACKAGE,
639
+ package: CORE_PACKAGE,
640
+ name: record.name,
641
+ docPath: record.docPath,
642
+ });
643
+ }
644
+ } catch {
645
+ // The components check reports a Core it cannot read.
646
+ }
647
+ }
648
+ if (integration.components) {
649
+ try {
650
+ const {components: records} =
651
+ await discoverValidIntegrationComponents(integration);
652
+ for (const record of records) {
653
+ components.push({
654
+ providerId: own,
655
+ package: integration.name,
656
+ name: record.name,
657
+ docPath: record.docPath,
658
+ // Declared: this package alone cannot tell whether an app applies
659
+ // it, so a link to the Core name it names selects Core.
660
+ ...(typeof record.replaces === 'string'
661
+ ? {replaces: record.replaces}
662
+ : {}),
663
+ });
664
+ }
665
+ } catch {
666
+ // `validate` reports components that do not load.
667
+ }
668
+ }
669
+ /** @type {import('../../foundation/discovery/docs-discovery.mjs').LinkableTemplate[]} */
670
+ const templates = [];
671
+ /** @param {any} template @param {string} providerId @param {string} pkg */
672
+ const addTemplate = (template, providerId, pkg) => {
673
+ if (template.type !== 'page' && template.type !== 'block') return;
674
+ templates.push({
675
+ providerId,
676
+ package: pkg,
677
+ type: template.type,
678
+ id: template.dirName,
679
+ name: template.name,
680
+ ...(template.displayName == null
681
+ ? {}
682
+ : {displayName: template.displayName}),
683
+ ...(template.description == null
684
+ ? {}
685
+ : {description: template.description}),
686
+ ...(typeof template.replaces === 'string'
687
+ ? {replaces: template.replaces}
688
+ : {}),
689
+ });
690
+ };
691
+ try {
692
+ for (const template of await discoverCoreTemplates()) {
693
+ addTemplate(template, CORE_PACKAGE, CORE_PACKAGE);
694
+ }
695
+ } catch {
696
+ // Core templates that do not load leave nothing to link to.
697
+ }
698
+ if (integration.templates) {
699
+ try {
700
+ const {templates: records} =
701
+ await discoverIntegrationTemplatesForOne(integration);
702
+ for (const template of records)
703
+ addTemplate(template, own, integration.name);
704
+ } catch {
705
+ // `validate` reports templates that do not load.
706
+ }
707
+ }
708
+ return {components, templates};
709
+ }
710
+
504
711
  /**
505
712
  * How a doc's links find their targets (spec:AST-047 FR9), as
506
713
  * {@link docFinder} finds them: each resolves to the link a read shows.
@@ -701,6 +908,7 @@ export async function docsLinkProblems(
701
908
  */
702
909
  export async function packageDocsProblems(integration, discovered) {
703
910
  const catalog = DocsCatalog.fromBuiltins();
911
+ catalog.setContributions(() => packageContributions(integration));
704
912
  for (const record of discovered.records) catalog.add(record);
705
913
  catalog.addTreeInputs({
706
914
  namespaces: discovered.namespaces.map(input => ({...input, rank: 1})),
@@ -794,6 +1002,7 @@ async function namespaceReferenceTarget(catalog, route, lang) {
794
1002
  */
795
1003
  export async function packageReferenceProblems(integration, discovered) {
796
1004
  const catalog = DocsCatalog.fromBuiltins();
1005
+ catalog.setContributions(() => packageContributions(integration));
797
1006
  for (const record of discovered.records) catalog.add(record);
798
1007
  catalog.addTreeInputs({
799
1008
  namespaces: discovered.namespaces.map(input => ({...input, rank: 1})),
@@ -1125,3 +1334,37 @@ export async function resolveTopicDocs(topic, options = {}) {
1125
1334
  const node = await lowerTopic(catalog, entry, effectiveLang);
1126
1335
  return {catalog, node, lang: effectiveLang, entry};
1127
1336
  }
1337
+
1338
+ /**
1339
+ * Each doc one integration package ships, checked against the published
1340
+ * authoring type for its kind with the project's own TypeScript (spec:AST-039
1341
+ * FR13). Without TypeScript the report says nothing was checked.
1342
+ * @param {any} integration a loaded integration
1343
+ * @param {string} cwd
1344
+ * @returns {Promise<import('../../foundation/discovery/doc-types.mjs').DocTypesReport & {fixtures?: number}>}
1345
+ */
1346
+ export async function integrationDocTypes(integration, cwd) {
1347
+ const packageDir = /** @type {{__packageDir?: string}} */ (integration)
1348
+ .__packageDir;
1349
+ const {files, unread, fixtures} = await integrationDocFiles(integration);
1350
+ const checked = await checkDocTypes(files, {
1351
+ packageDir: packageDir ?? cwd,
1352
+ resolveFrom: [cwd, packageDir],
1353
+ });
1354
+ // Test fixtures no reader serves don't ship as docs: only counted.
1355
+ const report =
1356
+ checked.status === 'checked' && fixtures > 0
1357
+ ? {...checked, fixtures}
1358
+ : checked;
1359
+ if (report.status !== 'checked' || unread.length === 0) return report;
1360
+ // A root its reader cannot read serves no doc; say so rather than skip it.
1361
+ const rel = (/** @type {string} */ file) =>
1362
+ path.relative(packageDir ?? cwd, file).split(path.sep).join('/') || '.';
1363
+ return {
1364
+ ...report,
1365
+ unchecked: [
1366
+ ...report.unchecked,
1367
+ ...unread.map(entry => ({doc: rel(entry.path), reason: entry.reason})),
1368
+ ].sort((a, b) => a.doc.localeCompare(b.doc)),
1369
+ };
1370
+ }
@@ -22,6 +22,7 @@ import {docs} from './docs.mjs';
22
22
  import {search} from '../search/search.mjs';
23
23
  import {loadDocsCatalog} from './_adapter.mjs';
24
24
  import {AstryxError} from '../error.mjs';
25
+ import {discoverCoreTemplates} from '../../foundation/discovery/template-adapter.mjs';
25
26
 
26
27
  const SLOW = 30_000;
27
28
 
@@ -110,6 +111,129 @@ describe('integration-contributed topics', () => {
110
111
  expect(section.data.title).toBe('Overview');
111
112
  }, SLOW);
112
113
 
114
+ it('reads a link to a component or a template as the command that opens it', async () => {
115
+ scaffold({
116
+ 'using-widgets.doc.mjs': topic({
117
+ name: 'using-widgets',
118
+ title: 'Using widgets',
119
+ sections: [
120
+ {
121
+ title: 'Overview',
122
+ content: [
123
+ {
124
+ type: 'prose',
125
+ text: 'Start with {@link component:AcmeButton} or {@link block:acme-card}, built on {@link @astryxdesign/core:component:Button}.',
126
+ },
127
+ ],
128
+ },
129
+ ],
130
+ }),
131
+ });
132
+ const pkgDir = path.join(tmpDir, 'node_modules', '@acme', 'widgets');
133
+ fs.writeFileSync(
134
+ path.join(pkgDir, 'astryx.integration.mjs'),
135
+ "export default {components: './components', templates: './templates', docs: './docs'};\n",
136
+ );
137
+ fs.mkdirSync(path.join(pkgDir, 'components'));
138
+ fs.writeFileSync(
139
+ path.join(pkgDir, 'components', 'AcmeButton.doc.mjs'),
140
+ "export default {type: 'component', name: 'AcmeButton', displayName: 'Acme Button', description: 'A button.', props: []};\n",
141
+ );
142
+ fs.writeFileSync(
143
+ path.join(pkgDir, 'components', 'AcmeButton.tsx'),
144
+ 'export function AcmeButton() { return null; }\n',
145
+ );
146
+ fs.mkdirSync(path.join(pkgDir, 'templates'));
147
+ fs.writeFileSync(
148
+ path.join(pkgDir, 'templates', 'acme-card.doc.mjs'),
149
+ "export default {type: 'block', name: 'Acme card', description: 'A card.'};\n",
150
+ );
151
+ fs.writeFileSync(
152
+ path.join(pkgDir, 'templates', 'acme-card.tsx'),
153
+ 'export default function Fixture() { return null; }\n',
154
+ );
155
+
156
+ const detail = await docs('using-widgets', undefined, {cwd: tmpDir, full: true});
157
+
158
+ const prose = JSON.stringify(detail.data.sections);
159
+ expect(prose).toContain('`astryx component AcmeButton --package @acme/widgets`');
160
+ expect(prose).toContain(
161
+ '`astryx template acme-card --type block --package @acme/widgets`',
162
+ );
163
+ expect(prose).toContain('`astryx component Button`');
164
+ expect(prose).not.toContain('{@link');
165
+ }, SLOW);
166
+
167
+ it('selects Core in a link only where the bare name would open something else', async () => {
168
+ const core = await discoverCoreTemplates();
169
+ const page = core.find(template => template.type === 'page')?.dirName;
170
+ const block = core.find(template => template.type === 'block')?.dirName;
171
+ scaffold({
172
+ 'using-widgets.doc.mjs': topic({
173
+ name: 'using-widgets',
174
+ title: 'Using widgets',
175
+ sections: [
176
+ {
177
+ title: 'Overview',
178
+ content: [
179
+ {
180
+ type: 'prose',
181
+ text:
182
+ 'Core: {@link @astryxdesign/core:component:Button}, {@link @astryxdesign/core:component:Badge}, ' +
183
+ `{@link @astryxdesign/core:page:${page}}, {@link @astryxdesign/core:block:${block}}.`,
184
+ },
185
+ ],
186
+ },
187
+ ],
188
+ }),
189
+ });
190
+ const pkgDir = path.join(tmpDir, 'node_modules', '@acme', 'widgets');
191
+ // The CLI range that turns component replacement on.
192
+ fs.writeFileSync(
193
+ path.join(pkgDir, 'package.json'),
194
+ JSON.stringify({
195
+ name: '@acme/widgets',
196
+ version: '1.0.0',
197
+ peerDependencies: {'@astryxdesign/cli': '>=0.6.7'},
198
+ peerDependenciesMeta: {'@astryxdesign/cli': {optional: true}},
199
+ }),
200
+ );
201
+ fs.writeFileSync(
202
+ path.join(pkgDir, 'astryx.integration.mjs'),
203
+ "export default {components: './components', templates: './templates', docs: './docs'};\n",
204
+ );
205
+ fs.mkdirSync(path.join(pkgDir, 'components'));
206
+ // AcmeButton replaces Core's Button, so bare `Button` opens AcmeButton.
207
+ fs.writeFileSync(
208
+ path.join(pkgDir, 'components', 'AcmeButton.doc.mjs'),
209
+ "export default {type: 'component', name: 'AcmeButton', displayName: 'Acme Button', description: 'A button.', replaces: 'Button', props: []};\n",
210
+ );
211
+ fs.writeFileSync(
212
+ path.join(pkgDir, 'components', 'AcmeButton.tsx'),
213
+ 'export function AcmeButton() { return null; }\n',
214
+ );
215
+ // A page template with a Core page's id makes the bare id ambiguous.
216
+ fs.mkdirSync(path.join(pkgDir, 'templates'));
217
+ fs.writeFileSync(
218
+ path.join(pkgDir, 'templates', `${page}.doc.mjs`),
219
+ "export default {type: 'page', name: 'Acme page', displayName: 'Acme page', description: 'A page.'};\n",
220
+ );
221
+ fs.writeFileSync(
222
+ path.join(pkgDir, 'templates', `${page}.tsx`),
223
+ 'export default function Fixture() { return null; }\n',
224
+ );
225
+
226
+ const detail = await docs('using-widgets', undefined, {cwd: tmpDir, full: true});
227
+
228
+ const prose = JSON.stringify(detail.data.sections);
229
+ expect(prose).toContain('`astryx component Button --package @astryxdesign/core`');
230
+ expect(prose).toContain('`astryx component Badge`');
231
+ expect(prose).toContain(
232
+ `\`astryx template ${page} --type page --package @astryxdesign/core\``,
233
+ );
234
+ expect(prose).toContain(`\`astryx template ${block} --type block\``);
235
+ }, SLOW);
236
+
113
237
  it('is invisible to a project that does not configure the integration', async () => {
114
238
  scaffold({'deploying.doc.mjs': topic()}, {config: 'export default {};\n'});
115
239
  const listed = await docs(undefined, undefined, {cwd: tmpDir});
@@ -736,7 +736,7 @@ export async function checkAppThemes(cwd) {
736
736
  label: 'App imports the theme module',
737
737
  status: 'warn',
738
738
  message: `Could not prove that project source imports ${state.module.path}${importEvidence.parseFailures ? `; ${importEvidence.parseFailures} source file(s) could not be parsed` : ''}.`,
739
- fix: `Import {themes, defaultThemeSlug} from the generated module and {Theme} from '@astryxdesign/core', then render <Theme theme={themes[defaultThemeSlug]}>.`,
739
+ fix: `Import {themes, defaultThemeSlug} from the generated module and {Theme} from '@astryxdesign/core/theme', then render <Theme theme={themes[defaultThemeSlug]}>.`,
740
740
  });
741
741
  }
742
742
 
@@ -17,7 +17,18 @@
17
17
  import {Worker} from 'node:worker_threads';
18
18
  import {AstryxError} from '../error.mjs';
19
19
  import {Project} from '../../foundation/config/project.mjs';
20
- import {CORE_PACKAGE} from '../../foundation/discovery/component-discovery.mjs';
20
+ import {
21
+ CORE_PACKAGE,
22
+ discoverIntegrationComponents,
23
+ docStem,
24
+ findComponentReadme,
25
+ findIntegrationComponent,
26
+ integrationRecordHold,
27
+ isIntegrationMember,
28
+ unscopedComponentOwners,
29
+ } from '../../foundation/discovery/component-discovery.mjs';
30
+ import {legacyExternalKeeps} from '../component/_adapter.mjs';
31
+ import {findCoreDir} from '../../foundation/fs/paths.mjs';
21
32
  import {captureEnv} from '../../foundation/debug/event.mjs';
22
33
  import {ERROR_CODES} from '../../foundation/response/error-codes.mjs';
23
34
  import {parseGapReportReceipt} from '../../authoring/gap-report/parse.mjs';
@@ -178,13 +189,65 @@ async function selectTarget(project, component, packageName) {
178
189
  coreName,
179
190
  );
180
191
  if (replacement) return targetForPackage(project, replacement.package);
192
+ // The owner rule runs over every owner discovery finds, loadable or not
193
+ // (a doc recorded under another export still answers to its file's name);
194
+ // then only owners whose docs load route a report.
195
+ const wanted = bareComponent.toLowerCase();
196
+ const loads = new Set(records.map(r => `${r.package}\0${r.docPath}`));
197
+ const recorded = project.loadedIntegrations.flatMap(integration =>
198
+ discoverIntegrationComponents(integration).flatMap(r =>
199
+ r.name.toLowerCase() === wanted
200
+ ? [{...r, hold: integrationRecordHold(r)}]
201
+ : docStem(r.docPath).toLowerCase() === wanted && !isIntegrationMember(r)
202
+ ? [{...r, hold: 'doc'}]
203
+ : [],
204
+ ),
205
+ );
206
+ // The checks below find files by name, so they run on every spelling the
207
+ // case-blind match above found, as well as the one typed.
208
+ const spellings = new Set([
209
+ bareComponent,
210
+ ...recorded.flatMap(r =>
211
+ [r.name, docStem(r.docPath)].filter(n => n.toLowerCase() === wanted),
212
+ ),
213
+ ]);
214
+ // Core also holds a name only its doc lookup finds, as in detail: a name a
215
+ // doc records yields to it, though it routes no report itself.
216
+ const coreDir = findCoreDir(project.cwd);
217
+ const coreDocs =
218
+ coreDir != null &&
219
+ [...spellings].some(name => findComponentReadme(coreDir, name) != null);
220
+ const ruled = unscopedComponentOwners([
221
+ ...records
222
+ .filter(
223
+ r => r.package === CORE_PACKAGE && r.name.toLowerCase() === wanted,
224
+ )
225
+ .map(r => ({...r, hold: 'core'})),
226
+ ...recorded,
227
+ // A hidden doc named after its file still holds that name, as the
228
+ // component lookup finds it; it routes no report.
229
+ ...project.loadedIntegrations.flatMap(integration =>
230
+ [...spellings].flatMap(spelling => {
231
+ const found = findIntegrationComponent(integration, spelling);
232
+ return found?.hold === 'file'
233
+ ? [{...found, package: integration.name}]
234
+ : [];
235
+ }),
236
+ ),
237
+ ]).filter(r => !coreDocs || r.hold !== 'doc');
238
+ // A legacy docs package keeps an export name a doc records, as in detail.
181
239
  const owners = [
182
240
  ...new Set(
183
- records
241
+ ([...spellings].some(name =>
242
+ legacyExternalKeeps(project.cwd, name, ruled),
243
+ )
244
+ ? []
245
+ : ruled
246
+ )
184
247
  .filter(
185
- record => record.name.toLowerCase() === bareComponent.toLowerCase(),
248
+ r => r.hold === 'core' || loads.has(`${r.package}\0${r.docPath}`),
186
249
  )
187
- .map(record => record.package),
250
+ .map(r => r.package),
188
251
  ),
189
252
  ];
190
253
 
package/api/index.d.mts CHANGED
@@ -16,6 +16,7 @@ export { init } from "./init/init.mjs";
16
16
  export { doctor } from "./doctor/doctor.mjs";
17
17
  export { integrationAddTheme } from "./integration/add-theme.mjs";
18
18
  export { integrationPackCheck } from "./integration/pack-check.mjs";
19
+ export { integrationCheck } from "./integration/check/check.mjs";
19
20
  export { AstryxError } from "./error.mjs";
20
21
  export { logger } from "./logger.mjs";
21
22
  export * from "../foundation/response/batch.type.mjs";
@@ -37,6 +38,7 @@ export * from "./integration/integration-authoring.type.mjs";
37
38
  export * from "./integration/pack-check.type.mjs";
38
39
  export * from "./integration/validate-integration.type.mjs";
39
40
  export * from "./integration/authoring-checks.type.mjs";
41
+ export * from "./integration/integration-check.type.mjs";
40
42
  export type Logger = import("./logger.mjs").Logger;
41
43
  export { themeBuild, themeAdd, themeRemove, themeUse, themeEject, themeTemplate, themeList, themeListAvailable, themeListCopySources, themeTargets, themePaletteGenerate, generateTonalPalette, listThemes } from "./theme/theme.mjs";
42
44
  export { integrationAdd, integrationAddAgentDoc, integrationAddCodemod, integrationAddComponent, integrationAddDoc, integrationAddTemplate } from "./integration/add-contribution.mjs";
package/api/index.mjs CHANGED
@@ -67,6 +67,7 @@ export {
67
67
  integrationComponentConflicts,
68
68
  integrationDocConflicts,
69
69
  } from './integration/authoring-checks.mjs';
70
+ export {integrationCheck} from './integration/check/check.mjs';
70
71
  export {AstryxError} from './error.mjs';
71
72
  // The one shared logger: the `logger` instance side-effecting commands write
72
73
  // through, plus its generated `Logger` type. Part of the public surface so an
@@ -100,3 +101,4 @@ export * from './integration/integration-authoring.type.mjs';
100
101
  export * from './integration/pack-check.type.mjs';
101
102
  export * from './integration/validate-integration.type.mjs';
102
103
  export * from './integration/authoring-checks.type.mjs';
104
+ export * from './integration/integration-check.type.mjs';
@@ -50,11 +50,11 @@ export function getNextSteps(invocation) {
50
50
  ` (run \`${invocation} doctor\` to verify)`,
51
51
  " 2. Import base styles: import '@astryxdesign/core/reset.css'",
52
52
  " and import '@astryxdesign/core/astryx.css'",
53
- " 3. Import components: import { Button } from '@astryxdesign/core'",
53
+ " 3. Import components: import { Button } from '@astryxdesign/core/Button'",
54
54
  ' 4. Add and wire a theme:',
55
55
  ' npm install @astryxdesign/theme-neutral',
56
56
  ` ${invocation} theme add neutral`,
57
- " import { Theme } from '@astryxdesign/core'",
57
+ " import { Theme } from '@astryxdesign/core/theme'",
58
58
  " import { themes, defaultThemeSlug } from './astryx-themes'",
59
59
  ' <Theme theme={themes[defaultThemeSlug]}>...</Theme>',
60
60
  ` Extend an imported theme to customize it. Run \`${invocation} theme eject <slug>\` only to fork source.`,
@@ -0,0 +1,12 @@
1
+ // @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
2
+ // DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
3
+
4
+ /**
5
+ * Each component doc's props compared with the component's TypeScript props,
6
+ * read with the project's own compiler (spec:AST-039 FR12). Without one, the
7
+ * report says the comparison was skipped.
8
+ * @param {any} integration a loaded integration
9
+ * @param {string} cwd
10
+ * @returns {Promise<import('../../foundation/discovery/prop-drift.mjs').PropDriftReport>}
11
+ */
12
+ export function integrationPropDrift(integration: any, cwd: string): Promise<import("../../foundation/discovery/prop-drift.mjs").PropDriftReport>;
@@ -0,0 +1,54 @@
1
+ // Copyright (c) Meta Platforms, Inc. and affiliates.
2
+
3
+ /**
4
+ * @file The integration subject's access to the environment (INV21): reading
5
+ * one package's component docs and sources with the project's own TypeScript.
6
+ *
7
+ * @input a loaded integration and the directory the command runs in
8
+ * @output the reports the integration checks attach to their results
9
+ * @position api/integration (internal; not exported from ./api)
10
+ */
11
+
12
+ import {
13
+ discoverValidIntegrationComponents,
14
+ forEachIntegrationDoc,
15
+ integrationSourceForDoc,
16
+ } from '../../foundation/discovery/component-discovery.mjs';
17
+ import {comparePropDocs} from '../../foundation/discovery/prop-drift.mjs';
18
+
19
+ /**
20
+ * Each component doc's props compared with the component's TypeScript props,
21
+ * read with the project's own compiler (spec:AST-039 FR12). Without one, the
22
+ * report says the comparison was skipped.
23
+ * @param {any} integration a loaded integration
24
+ * @param {string} cwd
25
+ * @returns {Promise<import('../../foundation/discovery/prop-drift.mjs').PropDriftReport>}
26
+ */
27
+ export async function integrationPropDrift(integration, cwd) {
28
+ const packageDir = /** @type {{__packageDir?: string}} */ (integration)
29
+ .__packageDir;
30
+ const {components} = await discoverValidIntegrationComponents(integration);
31
+ // Every doc the package ships has its prop tables compared or listed as not
32
+ // compared. A doc discovery leaves out (a hidden one, a second doc for one
33
+ // name, or one whose source or own file doesn't load) gets a record under
34
+ // its file name.
35
+ const recorded = new Set(components.map(c => c.docPath));
36
+ /** @type {typeof components} */
37
+ const left = [];
38
+ forEachIntegrationDoc(integration?.components, (docPath, stem) => {
39
+ if (recorded.has(docPath)) return;
40
+ recorded.add(docPath);
41
+ left.push({
42
+ name: stem,
43
+ package: integration.name,
44
+ docPath,
45
+ sourcePath: integrationSourceForDoc(docPath),
46
+ issuesUrl: integration.issuesUrl,
47
+ group: null,
48
+ });
49
+ });
50
+ return comparePropDocs([...components, ...left], {
51
+ packageDir: packageDir ?? cwd,
52
+ resolveFrom: [cwd, packageDir],
53
+ });
54
+ }