@astryxdesign/cli 0.6.4-canary.f0355e3 → 0.6.4

package/api/doctor/doctor.test.mjs

@@ -354,31 +354,16 @@ describe('checkImplicitIntegrations', () => {
   });
 
   it('names the package, the field, and what it contributes', () => {
-    // Roots count only when they exist, so this one gives them real folders.
-    const root = fs.mkdtempSync(path.join(os.tmpdir(), 'astryx-implicit-'));
-    try {
-      const dir = name => {
-        fs.mkdirSync(path.join(root, name));
-        return path.join(root, name);
-      };
-      const c = checkImplicitIntegrations({
-        integrations: [
-          autolinked({
-            components: dir('components'),
-            templates: dir('templates'),
-            themes: dir('themes'),
-          }),
-        ],
-      });
-      expect(c.message).toContain('@acme/widgets@1.0.0');
-      expect(c.message).toContain('from dependencies');
-      expect(c.message).toContain(
-        'contributing components, templates, themes',
-      );
-      expect(c.message).not.toContain('missing on disk');
-    } finally {
-      fs.rmSync(root, {recursive: true, force: true});
-    }
+    const c = checkImplicitIntegrations({
+      integrations: [
+        autolinked({templates: '/abs/templates', themes: '/abs/themes'}),
+      ],
+    });
+    expect(c.message).toContain('@acme/widgets@1.0.0');
+    expect(c.message).toContain('from dependencies');
+    expect(c.message).toContain(
+      'contributing components, templates, themes',
+    );
   });
 
   it('names the declared key too when an npm alias makes them differ', () => {
@@ -921,100 +906,3 @@ describe('checkDocsProgressiveDisclosure languages', () => {
     expect(c.message).not.toMatch(/deploying overview:/);
   });
 });
-
-describe('doctor says what it could not check', () => {
-  const CORE = {
-    'node_modules/@astryxdesign/core/package.json': JSON.stringify({
-      name: '@astryxdesign/core',
-      version: '0.6.3',
-    }),
-  };
-
-  /** Installed dependency whose manifest declares roots that do not exist. */
-  const dangling = {
-    'node_modules/@acme/dangling/package.json': JSON.stringify({
-      name: '@acme/dangling',
-      version: '2.0.0',
-    }),
-    'node_modules/@acme/dangling/astryx.integration.mjs':
-      "export default {providerId: 'acme-dangling', components: './components', templates: './templates', docs: './docs'};\n",
-  };
-
-  /** Installed dependency whose manifest cannot be parsed at all. */
-  const broken = {
-    'node_modules/@acme/broken/package.json': JSON.stringify({
-      name: '@acme/broken',
-      version: '1.0.0',
-    }),
-    'node_modules/@acme/broken/astryx.integration.mjs':
-      'export default {  this is not valid javascript ((\n',
-  };
-
-  /** @param {Record<string, string>} extra @param {string[]} deps */
-  const project = (extra, deps) =>
-    mkProject({
-      'package.json': JSON.stringify({
-        name: 'consumer',
-        version: '1.0.0',
-        dependencies: Object.fromEntries(
-          ['@astryxdesign/core', ...deps].map(d => [d, '1.0.0']),
-        ),
-      }),
-      ...CORE,
-      ...extra,
-    });
-
-  // An unparseable manifest is kept out of the loaded set on purpose, and
-  // doctor used to report that no installed dependency ships a manifest.
-  it('names an installed dependency whose manifest cannot be loaded', async () => {
-    const report = (await doctor({cwd: project(broken, ['@acme/broken'])})).data;
-    const check = report.checks.find(c => c.id === 'implicit-integrations');
-
-    expect(check.status).toBe('info');
-    expect(check.message).toContain('@acme/broken');
-    expect(check.message).toContain('could not be loaded');
-    expect(check.message).not.toContain('no installed dependency ships');
-    expect(report.summary.fail).toBe(0);
-  }, SLOW);
-
-  it('does not claim contributions from roots that are missing on disk', async () => {
-    const report = (await doctor({cwd: project(dangling, ['@acme/dangling'])}))
-      .data;
-    const implicit = report.checks.find(c => c.id === 'implicit-integrations');
-    const issues = report.checks.find(c => c.id === 'integration-issues');
-
-    expect(implicit.message).toContain('contributing nothing');
-    expect(implicit.message).toContain('missing on disk');
-    expect(implicit.message).not.toContain('contributing components');
-    expect(issues.status).toBe('warn');
-    expect(issues.message).toContain('@acme/dangling');
-  }, SLOW);
-
-  it('still says no dependency ships a manifest when none does', async () => {
-    const report = (await doctor({cwd: project({}, [])})).data;
-    const check = report.checks.find(c => c.id === 'implicit-integrations');
-
-    expect(check.message).toBe(
-      'None — no installed dependency ships an astryx.integration.* manifest.',
-    );
-  }, SLOW);
-
-  it('says how many integrations could not be read, rather than counting silently', () => {
-    /** @type {any} */
-    const ctx = {
-      cwd: '/x',
-      nodeVersion: process.versions.node,
-      coreDir: null,
-      configPath: null,
-      configTheme: null,
-      integrations: [
-        {name: '@acme/ok', __spec: '@acme/ok', providerId: 'ok'},
-        {name: '@acme/bad', __spec: '@acme/bad', __loadError: 'boom'},
-      ],
-    };
-    const check = checkProviderIdentity(ctx);
-
-    expect(check.message).toContain('1 loaded integration has its own provider ID.');
-    expect(check.message).toContain('could not be read');
-  });
-});

package/api/doctor/doctor.type.d.mts

@@ -23,7 +23,7 @@ export type DoctorCheck = {
      */
     message: string;
     /**
-     * - Actionable remediation: always present on 'warn' and 'fail'; some 'info' checks carry one too.
+     * - Actionable remediation, present when status is not 'pass'.
      */
     fix?: string | undefined;
 };

package/api/doctor/doctor.type.mjs

@@ -19,7 +19,7 @@
  * @property {string} label - Human-readable check name.
  * @property {DoctorStatus} status
  * @property {string} message - One-line result summary.
- * @property {string} [fix] - Actionable remediation: always present on 'warn' and 'fail'; some 'info' checks carry one too.
+ * @property {string} [fix] - Actionable remediation, present when status is not 'pass'.
  */
 
 /**

package/api/gap-report/gap-report.doc.mjs

@@ -12,16 +12,12 @@ export const doc = {
   name: 'gapReport',
   namespace: 'cli/api',
   displayName: 'gapReport()',
-  summary:
-    'Report a missing or hard-to-use design-system capability to the package that owns it.',
+  summary: 'Route a design-system gap through the fan-out handler composition.',
   description:
-    'Sends a gap report to every configured handler: the project config handler first, then each integration handler in config order. ' +
-    'Each handler has 30 s to finish, and its output goes to stderr. Public handlers run only with confirmPublic; internal handlers always run. ' +
-    "With no handler, it files a GitHub issue for the owning package only with confirmPublic (without it nothing is sent), or returns the package's issues URL when that is not on GitHub. " +
-    'The report records whether an agent or a person ran it, and the response lists each handler outcome in order.',
+    'Creates a normalized gap report and fans it out to every effective handler: the project config handler first, then each loaded integration handler in config order, deduplicated by handle function identity. Each handler receives a structuredClone of the report and an AbortSignal, then runs in its own worker with a 30 s timeout and stdout redirected to stderr. A timed-out worker is terminated before the next handler starts, so process.exit, process.exitCode, and late continuations cannot affect the CLI process. A public handler requires confirmPublic per handler; internal handlers always run. When no handlers exist, a built-in GitHub/routed-only fallback runs. The aggregate response carries ordered deliveries with per-handler outcomes.',
   importPath: '@astryxdesign/cli/api',
   signature:
-    'gapReport(component: string | undefined, options?: GapReportOptions): Promise<GapReportCategoriesResponse | GapReportReceiptResponse>',
+    'gapReport(component?: string, options?: GapReportOptions): Promise<GapReportCategoriesResponse | GapReportReceiptResponse>',
   keywords: [
     'gap',
     'report',
@@ -60,13 +56,13 @@ export const doc = {
       name: 'options.package',
       type: 'string',
       description:
-        'Package that owns the gap: @astryxdesign/core, or a loaded integration by package name or config entry. Overrides automatic owner routing; required when more than one package provides the component.',
+        'Explicit owning package when automatic routing is ambiguous.',
     },
     {
       name: 'options.confirmPublic',
       type: 'boolean',
       description:
-        'Allow public delivery: public handlers run, and with no handler a GitHub issue is filed through the gh CLI.',
+        'Explicitly consent to invoking public handlers or creating a GitHub issue.',
       default: 'false',
     },
     {
@@ -81,7 +77,6 @@ export const doc = {
       type: 'string',
       description:
         'Directory used to load project config and component ownership.',
-      default: 'process.cwd()',
     },
   ],
   returns: [
@@ -92,21 +87,17 @@ export const doc = {
     {
       type: 'gap-report.file',
       description:
-        'Receipt: status (filed, partial, failed, routed_only, consent_required, skipped), package, issuesUrl, ordered deliveries (handlerType, handler, audience, status, url, message), filedCount, and routedOnlyCount.',
+        'An aggregate receipt with per-handler deliveries, filedCount/routedOnlyCount totals, and overall status.',
     },
   ],
   throws: [
-    {
-      code: 'ERR_MISSING_ARGUMENT',
-      when: 'component, category, or reason is missing, blank, or not a string (unless listCategories is true)',
-    },
     {
       code: 'ERR_UNKNOWN_CATEGORY',
       when: 'category is not one of the fixed gap-report values',
     },
     {
       code: 'ERR_INVALID_ARGUMENT',
-      when: 'component is over 120 characters, category is over 80, reason is over 2000, or detail is not a string or is over 8000 characters',
+      when: 'a field value is invalid',
     },
     {
       code: 'ERR_AMBIGUOUS_COMPONENT',
@@ -118,7 +109,7 @@ export const doc = {
     },
     {
       code: 'ERR_NOT_FOUND',
-      when: 'no report handler is configured and the owning package has no issues URL',
+      when: 'no handler and no issues URL available',
     },
   ],
   examples: [
@@ -131,8 +122,8 @@ export const doc = {
       code: "const receipt = await gapReport('Button', {category: 'missing_variant', reason: 'Need a compact size'});",
     },
     {
-      label: 'Name the owning package',
-      code: "const receipt = await gapReport('Button', {category: 'docs_gap', reason: 'Missing keyboard example', package: '@astryxdesign/core'});",
+      label: 'Confirm public filing',
+      code: "await gapReport('Button', {category: 'docs_gap', reason: 'Missing keyboard example', confirmPublic: true});",
     },
   ],
   command: 'gap-report',

package/api/hook/hook.doc.mjs

@@ -34,7 +34,6 @@ export const doc = {
       name: 'options.cwd',
       type: 'string',
       description: 'Directory to resolve @astryxdesign/core from.',
-      default: 'process.cwd()',
     },
     {
       name: 'options.list',
@@ -44,8 +43,7 @@ export const doc = {
     {
       name: 'options.category',
       type: 'string',
-      description:
-        "List only the hooks in this category, a key of the unfiltered list such as 'Media' or 'Focus'.",
+      description: 'List only hooks in this category.',
     },
     {
       name: 'options.params',
@@ -56,8 +54,7 @@ export const doc = {
       name: 'options.detail',
       type: "'full' | 'compact' | 'brief'",
       description: 'Detail level for list views.',
-      default:
-        "'full' for a named hook; 'brief' for lists (returned as data.detail: 'names')",
+      default: "'full' for a named hook, 'brief' for list views",
     },
     {
       name: 'options.lang',
@@ -107,7 +104,7 @@ export const doc = {
     },
     {
       label: 'Browse a category',
-      code: "await hook(undefined, {category: 'Media', detail: 'compact'});",
+      code: "await hook(undefined, {category: 'State', detail: 'compact'});",
     },
   ],
   command: 'hook',

package/api/index.d.mts

@@ -18,7 +18,6 @@ export { integrationAddTheme } from "./integration/add-theme.mjs";
 export { integrationPackCheck } from "./integration/pack-check.mjs";
 export { AstryxError } from "./error.mjs";
 export { logger } from "./logger.mjs";
-export * from "../foundation/response/batch.type.mjs";
 export * from "./component/component.type.mjs";
 export * from "./docs/docs.type.mjs";
 export * from "./blog/blog.type.mjs";
@@ -33,12 +32,14 @@ export * from "./gap-report/gap-report.type.mjs";
 export * from "./upgrade/upgrade.type.mjs";
 export * from "./init/init.type.mjs";
 export * from "./doctor/doctor.type.mjs";
+export * from "./layout/layout.type.mjs";
 export * from "./integration/integration-authoring.type.mjs";
 export * from "./integration/pack-check.type.mjs";
 export * from "./integration/validate-integration.type.mjs";
 export * from "./integration/authoring-checks.type.mjs";
 export type Logger = import("./logger.mjs").Logger;
 export { themeBuild, themeAdd, themeTemplate, themeList, themeListAvailable, themeTargets, themePaletteGenerate, generateTonalPalette, listThemes } from "./theme/theme.mjs";
+export { layoutExpand, layoutCheck, layoutGrammar } from "./layout/layout.mjs";
 export { integrationAdd, integrationAddAgentDoc, integrationAddCodemod, integrationAddComponent, integrationAddDoc, integrationAddTemplate } from "./integration/add-contribution.mjs";
 export { validateIntegration, summarizeIssues } from "./integration/validate-integration.mjs";
 export { integrationTemplateConflicts, integrationComponentConflicts, integrationDocConflicts } from "./integration/authoring-checks.mjs";

package/api/index.mjs

@@ -3,7 +3,7 @@
 /**
  * @file Programmatic API for the Astryx CLI.
  *
- * Every function returns the same { type, data } envelope that `astryx --json` outputs.
+ * Every function returns the same { type, data } envelope that `xds --json` outputs.
  * Errors throw AstryxError (with optional .suggestions).
  *
  * @example
@@ -44,6 +44,7 @@ export {gapReport} from './gap-report/gap-report.mjs';
 export {upgrade} from './upgrade/upgrade.mjs';
 export {init} from './init/init.mjs';
 export {doctor} from './doctor/doctor.mjs';
+export {layoutExpand, layoutCheck, layoutGrammar} from './layout/layout.mjs';
 export {
   integrationAdd,
   integrationAddAgentDoc,
@@ -72,12 +73,10 @@ export {logger} from './logger.mjs';
  * @typedef {import('./logger.mjs').Logger} Logger
  */
 
-// ── Types (re-exported from the shared response foundation and each command's
-// colocated `.type.mjs`) ──────────────────────────────────────────────────
+// ── Types (re-exported from each command's colocated `.type.mjs`) ─────
 // Runtime no-ops (the .type.mjs files are `export {}`); tsc carries these
 // through to the generated api/index.d.mts so the public type surface exposes
-// the shared receipt vocabulary and every command's Options + response types.
-export * from '../foundation/response/batch.type.mjs';
+// every command's Options + response types by name.
 export * from './component/component.type.mjs';
 export * from './docs/docs.type.mjs';
 export * from './blog/blog.type.mjs';
@@ -92,6 +91,7 @@ export * from './gap-report/gap-report.type.mjs';
 export * from './upgrade/upgrade.type.mjs';
 export * from './init/init.type.mjs';
 export * from './doctor/doctor.type.mjs';
+export * from './layout/layout.type.mjs';
 export * from './integration/integration-authoring.type.mjs';
 export * from './integration/pack-check.type.mjs';
 export * from './integration/validate-integration.type.mjs';

package/api/init/init.doc.mjs

@@ -19,9 +19,10 @@ export const doc = {
     'Sets a project up with NO prompts, so it behaves identically for humans, ' +
     'agents, CI, and piped I/O. By default it installs the AGENTS.md/CLAUDE.md ' +
     'agent-docs cheat sheet, including guidance from configured integrations, and ' +
-    "prints getting-started guidance; `features` / `all` print only the chosen features' " +
-    'guidance instead (the theme feature also writes the annotated theme template). ' +
-    'With `removeAgents` it removes the managed agent-docs block instead of installing.',
+    'prints getting-started guidance; `features` / ' +
+    '`all` add theme and page-building guidance and can scaffold a starter ' +
+    'template. With `removeAgents` it removes the managed agent-docs block ' +
+    'instead of installing.',
   importPath: '@astryxdesign/cli/api',
   signature:
     'init(options?: InitOptions, ctx?: {cwd?: string}): Promise<InitRunResponse | InitRemoveResponse>',
@@ -31,7 +32,7 @@ export const doc = {
       name: 'options.features',
       type: 'string',
       description:
-        'Comma-separated features to install: agents (agent docs), theme (writes the annotated theme.template.ts), template (the page-building commands; writes a page only with templateName).',
+        'Comma-separated features to install: agents (agent docs), theme (writes the annotated theme.template.ts), template (page-template guidance).',
     },
     {
       name: 'options.all',
@@ -46,8 +47,7 @@ export const doc = {
     {
       name: 'options.agent',
       type: 'string',
-      description:
-        'Agent preset: claude, cursor, codex, hermes, muse, all. Used only when the agents feature runs.',
+      description: 'Agent preset: claude, cursor, codex, hermes, muse, all.',
     },
     {
       name: 'options.agentDocsPath',
@@ -58,14 +58,9 @@ export const doc = {
       name: 'options.templateName',
       type: 'string',
       description:
-        'Scaffold this page template to src/pages/<name>/page.tsx. Used only when the template feature runs (features includes template, or all); otherwise ignored. Programmatic only; the CLI never sets it.',
-    },
-    {
-      name: 'ctx.cwd',
-      type: 'string',
-      description: 'Directory to set up.',
-      default: 'process.cwd()',
+        'Scaffold a named page template (programmatic only; the CLI never sets it).',
     },
+    {name: 'ctx.cwd', type: 'string', description: 'Directory to set up.'},
   ],
   returns: [
     {
@@ -76,13 +71,13 @@ export const doc = {
     {
       type: 'init.remove',
       description:
-        'Returned when `removeAgents` is set. `data.removed` is always true, even when no managed block was found.',
+        'Confirmation that the managed agent-docs block was removed (`data.removed: true`), returned when `removeAgents` is set.',
     },
   ],
   throws: [
     {
       code: 'ERR_UNKNOWN_AGENT',
-      when: 'the agents feature runs (default mode, features includes agents, or all) and `agent` is not one of claude, cursor, codex, hermes, muse, all',
+      when: '`agent` is not one of claude, cursor, codex, hermes, muse, all',
     },
     {
       code: 'ERR_UNKNOWN_FEATURE',
@@ -90,7 +85,7 @@ export const doc = {
     },
     {
       code: 'ERR_UNKNOWN_TEMPLATE',
-      when: 'the template feature runs and `templateName` matches no bundled template',
+      when: '`templateName` matches no bundled template',
     },
     {
       code: 'ERR_FILE_EXISTS',
@@ -98,7 +93,7 @@ export const doc = {
     },
     {
       code: 'ERR_PATH_TRAVERSAL',
-      when: 'the template output path resolves outside cwd (for example through a symlinked src directory), or, with removeAgents, a managed agent-docs file resolves outside cwd through a symlink',
+      when: 'the template output path resolves outside cwd, for example through a symlinked src directory',
     },
   ],
   examples: [

package/api/integration/add-helpers.d.mts

@@ -8,7 +8,7 @@ export function findPackageDir(startDir: string): string;
 /**
  * @typedef {object} WritePlan
  * @property {string} path
- * @property {string | Buffer} contents a Buffer is written byte for byte
+ * @property {string} contents
  * @property {boolean} createOnly
  * @property {Buffer} [expectedOriginal] bytes captured before validation;
  *   a different current file is a concurrent edit and must never be overwritten
@@ -56,10 +56,7 @@ export function packageJsonUpdate(packageFile: string, rootPath: string, manifes
 export function applyWrites(plans: WritePlan[]): () => void;
 export type WritePlan = {
     path: string;
-    /**
-     * a Buffer is written byte for byte
-     */
-    contents: string | Buffer;
+    contents: string;
     createOnly: boolean;
     /**
      * bytes captured before validation;

package/api/integration/add-helpers.mjs

@@ -5,8 +5,7 @@
  *
  * Hosts the atomic staged-write transaction, package.json files-array
  * maintenance, package-dir resolution, and project-path normalization that
- * every `integration add <kind>` command needs. `theme add` and `theme build`
- * write through the same transaction. Stateless and side-effect-free
+ * every `integration add <kind>` command needs. Stateless and side-effect-free
  * outside of {@link applyWrites}.
  */
 
@@ -41,7 +40,7 @@ export function findPackageDir(startDir) {
 /**
  * @typedef {object} WritePlan
  * @property {string} path
- * @property {string | Buffer} contents a Buffer is written byte for byte
+ * @property {string} contents
  * @property {boolean} createOnly
  * @property {Buffer} [expectedOriginal] bytes captured before validation;
  *   a different current file is a concurrent edit and must never be overwritten
@@ -193,29 +192,21 @@ export function packageJsonUpdate(
 
 // ── Atomic staged-write transaction ─────────────────────────────────
 
-/**
- * @param {string} file
- * @returns {boolean} false when the file is still there
- */
+/** @param {string} file */
 function removeTemporary(file) {
   try {
     fs.rmSync(file, {force: true});
-    return true;
   } catch {
     // Best effort. The transaction error remains the actionable failure.
-    return false;
   }
 }
 
 /**
  * Restore writes that already published.  Best-effort so callers preserve
- * the original actionable error; returns every path it could not put back.
+ * the original actionable error.
  * @param {Array<WritePlan & {temporary: string, original: Buffer|null, mode: number}>} published
- * @returns {string[]}
  */
 function rollbackWrites(published) {
-  /** @type {string[]} */
-  const unrestored = [];
   for (const plan of [...published].reverse()) {
     let restore = null;
     try {
@@ -234,18 +225,12 @@ function rollbackWrites(published) {
         fs.renameSync(restore, plan.path);
         restore = null;
       }
-    } catch (error) {
-      // A created file that is already gone needs nothing. Any other failure
-      // leaves this call's bytes, or no bytes, where the original was.
-      const gone =
-        error instanceof Error &&
-        /** @type {NodeJS.ErrnoException} */ (error).code === 'ENOENT';
-      if (!(gone && plan.original == null)) unrestored.push(plan.path);
+    } catch {
+      // Best effort. A concurrent edit belongs to its writer, not this rollback.
     } finally {
       if (restore != null) removeTemporary(restore);
     }
   }
-  return unrestored;
 }
 
 /** @param {string} file */
@@ -340,22 +325,10 @@ export function applyWrites(plans) {
       }
       published.push(plan);
     }
-    return () => {
-      rollbackWrites(published);
-    };
+    return () => rollbackWrites(published);
   } catch (error) {
-    const leftovers = staged
-      .filter(plan => !removeTemporary(plan.temporary))
-      .map(plan => plan.temporary);
-    const unrestored = rollbackWrites(published);
-    if (error instanceof Error) {
-      if (unrestored.length > 0) {
-        error.message += ` Could not restore: ${unrestored.join(', ')}.`;
-      }
-      if (leftovers.length > 0) {
-        error.message += ` Could not remove temporary files: ${leftovers.join(', ')}.`;
-      }
-    }
+    for (const plan of staged) removeTemporary(plan.temporary);
+    rollbackWrites(published);
     throw error;
   }
 }

package/api/integration/add-theme.mjs

@@ -24,10 +24,6 @@ import {
 import {loadManifestObject} from '../../foundation/integrations/integrations.mjs';
 import {themeDescriptorSource} from '../../foundation/integrations/theme-descriptor.mjs';
 import {assertContributionVisible} from '../../foundation/integrations/contribution-inventory.mjs';
-import {
-  themesCliProblem,
-  withDocsTreeCli,
-} from '../../foundation/integrations/cli-requirement.mjs';
 import {
   applyWrites,
   findPackageDir,
@@ -237,28 +233,11 @@ export async function integrationAddTheme(name, options = {}) {
       createOnly: true,
     },
   ];
-  let packageUpdate = packageJsonUpdate(
+  const packageUpdate = packageJsonUpdate(
     packageFile,
     rootPath,
     path.basename(manifestFile),
   );
-  // A CLI older than the one that reads typed theme descriptors rejects the
-  // themes root and withholds the package's themes and docs. Declare the CLI
-  // that reads them as a peer, so an older one is flagged at install instead.
-  {
-    const expectedOriginal =
-      packageUpdate?.expectedOriginal ?? fs.readFileSync(packageFile);
-    const text = packageUpdate?.contents ?? expectedOriginal.toString('utf-8');
-    const current = JSON.parse(text);
-    if (themesCliProblem(current) != null) {
-      packageUpdate = {
-        contents:
-          JSON.stringify(withDocsTreeCli(current), null, 2) +
-          (text.endsWith('\n') ? '\n' : ''),
-        expectedOriginal,
-      };
-    }
-  }
   if (packageUpdate != null) {
     plans.push({
       path: packageFile,

package/api/integration/add-theme.test.mjs

@@ -77,40 +77,6 @@ describe('integrationAddTheme', () => {
     expect((await validateLocalIntegration(tmpDir)).issues).toEqual([]);
   });
 
-  it('declares the optional CLI peer that reads typed theme descriptors', async () => {
-    setup({includeFiles: false});
-    const result = await integrationAddTheme('ocean', {cwd: tmpDir});
-    expect(result.data.files).toContain('package.json');
-    const pkg = JSON.parse(
-      fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8'),
-    );
-    expect(pkg.peerDependencies).toEqual({'@astryxdesign/cli': '>=0.7.0'});
-    expect(pkg.peerDependenciesMeta).toEqual({
-      '@astryxdesign/cli': {optional: true},
-    });
-  });
-
-  it('keeps a CLI peer that already reads typed theme descriptors', async () => {
-    const pkg = {
-      name: '@acme/themes',
-      version: '1.0.0',
-      peerDependencies: {'@astryxdesign/cli': '^0.7.2'},
-    };
-    fs.writeFileSync(
-      path.join(tmpDir, 'package.json'),
-      `${JSON.stringify(pkg, null, 2)}\n`,
-    );
-    fs.writeFileSync(
-      path.join(tmpDir, 'astryx.integration.mjs'),
-      'export default {};\n',
-    );
-    const result = await integrationAddTheme('ocean', {cwd: tmpDir});
-    expect(result.data.files).not.toContain('package.json');
-    expect(
-      JSON.parse(fs.readFileSync(path.join(tmpDir, 'package.json'), 'utf-8')),
-    ).toEqual(pkg);
-  });
-
   it('dry-runs the identical receipt without writing anything', async () => {
     setup();
     const beforeManifest = fs.readFileSync(

package/api/integration/authoring-checks.mjs

@@ -368,11 +368,11 @@ export async function integrationDocConflicts(pkg, options = {}) {
   // and placed guides this package adds to the docs tree, and every link in
   // its docs (spec:AST-046, spec:AST-047).
   if (errors.length === 0) {
-    for (const {severity, message} of await packageDocsProblems(
+    for (const message of await packageDocsProblems(
       /** @type {{name: string}} */ (resolved.integration),
       discovered,
     )) {
-      issues.push({code: 'invalid_doc_graph', severity, message});
+      issues.push({code: 'invalid_doc_graph', severity: 'warning', message});
     }
 
     // A reference block includes content rather than linking to it, so one