@astryxdesign/cli 0.6.4-canary.f0355e3 → 0.6.4

package/foundation/discovery/authoring-self-docs.mjs

@@ -33,7 +33,6 @@ export const AUTHORING_SELF_DOCS = [
   'config/config.doc.mjs',
   'debug/debug.doc.mjs',
   'gap-report/gap-report.doc.mjs',
-  'discover/discover.doc.mjs',
   'codemod/codemod.doc.mjs',
   'identity/identity.doc.mjs',
   'doctypes/base/graph-fields.doc.mjs',

package/foundation/discovery/authoring-self-docs.test.mjs

@@ -63,9 +63,7 @@ describe('authoring self-docs', () => {
 describe('what the authoring docs say about the docs tree', () => {
   it('marks exactly the fields the docs tree does not read yet', () => {
     const notReadYet = graphFieldsDoc.fields
-      .filter(field =>
-        /^Reserved: .*Nothing reads it today/.test(field.description),
-      )
+      .filter(field => /Not read yet/.test(field.description))
       .map(field => field.name);
     // The tree reads `placement` for every guide (spec:AST-046); the other
     // graph fields are still refused by every topic reader.
@@ -81,9 +79,7 @@ describe('what the authoring docs say about the docs tree', () => {
     expect(placement.description).toMatch(
       /Read for every guide, the CLI's and each integration's/,
     );
-    expect(graphFieldsDoc.description).toMatch(
-      /Nothing reads `aliases` or `audience` today/,
-    );
+    expect(graphFieldsDoc.description).toMatch(/not built yet/);
   });
 
   it('keeps graph blocks behind the separate GraphContentBlock type, and a section takes a reference block', () => {

package/foundation/discovery/cli-self-docs.mjs

@@ -315,20 +315,12 @@ function functionSection(fn, index) {
   }
   const params = fn.params ?? [];
   if (params.length > 0) {
-    // A Default column only when some parameter declares a default, so a
-    // function with none keeps a three-column table.
-    const defaults = params.some(
-      (/** @type {any} */ p) => typeof p.default === 'string' && p.default !== '',
-    );
     content.push({
       type: 'table',
-      headers: defaults
-        ? ['Parameter', 'Type', 'Default', 'Description']
-        : ['Parameter', 'Type', 'Description'],
+      headers: ['Parameter', 'Type', 'Description'],
       rows: params.map((/** @type {any} */ p) => [
-        `\`${p.name}\`${p.required ? ' (required)' : ''}`,
+        `\`${p.name}\``,
         `\`${p.type ?? ''}\``,
-        ...(defaults ? [p.default ?? ''] : []),
         p.description ?? '',
       ]),
     });
@@ -413,12 +405,6 @@ export function cliDocSection(doc, index) {
   if (doc.type === 'command') return commandSection(doc, index);
   if (doc.type === 'function') return functionSection(doc, index);
   if (doc.type === 'enum') return enumSection(doc);
-  if (doc.type === 'namespace')
-    return {
-      id: routeSegment(doc.name),
-      title: doc.title,
-      content: doc.blocks ?? [],
-    };
   return {...selfDocSection(doc), id: routeSegment(doc.name)};
 }
 

package/foundation/discovery/cli-self-docs.test.mjs

@@ -315,26 +315,6 @@ describe('cliDocSection', () => {
     ]);
   });
 
-  it('marks required parameters and adds a Default column when a parameter has a default', () => {
-    const beta = fn('beta', {
-      params: [
-        {name: 'slug', type: 'string', description: 'The slug.', required: true},
-        {name: 'options.limit', type: 'number', description: 'How many.', default: '20'},
-      ],
-    });
-    const table = cliDocSection(beta, indexOf([beta])).content.find(
-      (/** @type {any} */ block) => block.type === 'table',
-    );
-    expect(table).toEqual({
-      type: 'table',
-      headers: ['Parameter', 'Type', 'Default', 'Description'],
-      rows: [
-        ['`slug` (required)', '`string`', '', 'The slug.'],
-        ['`options.limit`', '`number`', '20', 'How many.'],
-      ],
-    });
-  });
-
   it('renders an API function, an enum, and a schema', () => {
     const alpha = fn('alpha', {
       description: 'Longer.',

package/foundation/discovery/docs-discovery.mjs

@@ -626,11 +626,7 @@ function findMergeTarget(sections, section) {
   const sameTitle = sections.findIndex(
     candidate => sourceTitle(candidate) === title,
   );
-  if (sameTitle !== -1) return sameTitle;
-  // A base section retitled later keeps its old key as its `id`, so an
-  // extension that still names it by the old title finds it by that id. A
-  // title variant of a section with no id stays a separate section.
-  return sections.findIndex(candidate => candidate.id === key);
+  return sameTitle;
 }
 
 /**

package/foundation/discovery/docs-discovery.test.mjs

@@ -509,27 +509,6 @@ describe('mergeTopic', () => {
     expect(base.sections[0].content[0].text).toBe('npm i');
   });
 
-  it('finds a retitled base section by the key its old title derives', () => {
-    // A base section retitled later keeps its old key as its `id`; an
-    // extension that still names it by the old title replaces it.
-    const merged = mergeTopic(
-      {
-        ...base,
-        sections: [
-          {id: 'quick-start', title: 'Wrap your app in a theme', content: []},
-          {id: 'tokens', title: 'Tokens', content: []},
-        ],
-      },
-      {sections: [{title: 'Quick Start', content: []}]},
-    );
-    expect(merged.sections.map(section => [section.id, section.title])).toEqual(
-      [
-        ['quick-start', 'Quick Start'],
-        ['tokens', 'Tokens'],
-      ],
-    );
-  });
-
   it('replaces a section by stable ID even when its title changes', () => {
     const merged = mergeTopic(
       {

package/foundation/discovery/docs-section-key.d.mts

@@ -32,7 +32,7 @@ export function sectionTitleKey(title: unknown): string;
 export function sectionKey(section: any): string;
 /**
  * A name as a docs-tree route segment: lowercase words joined by hyphens, so
- * `integrationPackCheck` and `integration verify` both read naturally.
+ * `integrationPackCheck` and `integration pack` both read naturally.
  * @param {string} name
  * @returns {string}
  */

package/foundation/discovery/docs-section-key.mjs

@@ -95,7 +95,7 @@ function legacyTitleMatchIndex(sections, query) {
 
 /**
  * A name as a docs-tree route segment: lowercase words joined by hyphens, so
- * `integrationPackCheck` and `integration verify` both read naturally.
+ * `integrationPackCheck` and `integration pack` both read naturally.
  * @param {string} name
  * @returns {string}
  */

package/foundation/discovery/template-adapter.mjs

@@ -4,7 +4,7 @@
  * @file Shared template discovery + IO.
  *
  * Owns everything the template leaves (list/show/skeleton/copy) AND other
- * commands (component, search, init, discover, Doctor integration)
+ * commands (component, layout, search, init, discover, Doctor integration)
  * share: template discovery across core/external/integration sources, the
  * template-spec loaders, and the cross-command helpers (stripTemplateAssetRefs,
  * findShowcase, findRelatedBlocks, extractComponents, listTemplates). The

package/foundation/doc-compiler/doc-loads.test.mjs

@@ -53,8 +53,8 @@ const RUNNERS = {
     sites: ['import(node:child_process)', 'worker_threads.Worker'],
   },
   'api/integration/pack-check.mjs': {
-    runs: '`npm pack`, `tar`, and a Node script that resolves the packed specifiers, to check a package as published, and the package\'s doc modules, to see which need a newer CLI',
-    sites: ['child_process.spawnSync ×3', 'loadTopicModule'],
+    runs: '`npm pack`, `tar`, and a Node script that resolves the packed specifiers, to check a package as published',
+    sites: ['child_process.spawnSync ×3'],
   },
   'api/swizzle/_github.mjs': {
     runs: '`gh auth status`, to check the GitHub CLI is signed in',
@@ -188,10 +188,21 @@ const RUNNERS = {
       'import ../../../api/integration/integrationPackCheck.doc.mjs',
       'import ./integration-add.doc.mjs',
       'import ./integration-pack.doc.mjs',
-      'import ./integration-verify.doc.mjs',
       'import ./integration.doc.mjs',
     ],
   },
+  'clients/cli/commands/layout.mjs': {
+    runs: SELF_DOCS,
+    sites: [
+      'import ../../../api/layout/layoutCheck.doc.mjs',
+      'import ../../../api/layout/layoutExpand.doc.mjs',
+      'import ../../../api/layout/layoutGrammar.doc.mjs',
+      'import ./layout-check.doc.mjs',
+      'import ./layout-expand.doc.mjs',
+      'import ./layout-grammar.doc.mjs',
+      'import ./layout.doc.mjs',
+    ],
+  },
   'clients/cli/commands/search.mjs': {
     runs: SELF_DOCS,
     sites: [

package/foundation/doc-compiler/tree.d.mts

@@ -240,10 +240,6 @@ export type TreeNode = {
     name: string;
     title: string;
     summary: string;
-    /**
-     * an authored namespace's search keywords
-     */
-    keywords?: string[] | undefined;
     /**
      * the parent's route; null at the top
      */

package/foundation/doc-compiler/tree.mjs

@@ -58,7 +58,7 @@ export const KIND_GROUPS = Object.freeze({
   schema: {
     segment: 'schemas',
     title: 'Schemas',
-    summary: 'Data shapes, field by field.',
+    summary: 'Every shape the CLI reads or returns, field by field.',
   },
   enum: {
     segment: 'enums',
@@ -140,7 +140,6 @@ export const UNORGANIZED = 'unorganized';
  * @property {string} name
  * @property {string} title
  * @property {string} summary
- * @property {string[]} [keywords] an authored namespace's search keywords
  * @property {string | null} parent the parent's route; null at the top
  * @property {string | null} slot the parent slot this node sits in
  * @property {number | null} order
@@ -421,14 +420,10 @@ export function buildDocsTree({namespaces, docs, topics = []}) {
         name: input.doc.name,
         title: input.doc.title,
         summary: input.doc.summary,
-        ...(Array.isArray(input.doc.keywords)
-          ? {keywords: input.doc.keywords}
-          : {}),
         parent: parentRoute,
         slot: home?.slot ?? null,
         order: home?.order ?? null,
         generated: false,
-        ref: {selfDoc: input.doc},
         slots: Object.entries(input.doc.slots).map(([name, slot]) => ({
           name,
           title: slot.title,