@astryxdesign/cli 0.6.4-canary.ed2e54e → 0.6.4

package/api/layout/layout.type.d.mts

@@ -0,0 +1,89 @@
+// @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
+// DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
+
+/**
+ * The input surface an expression was parsed as.
+ */
+export type LayoutForm = "compact" | "outline" | "auto";
+/**
+ * A validation issue with its formatted, human-readable rendering.
+ */
+export type LayoutIssue = {
+    line?: number | undefined;
+    col?: number | undefined;
+    message: string;
+    formatted: string;
+    suggestions?: string[] | undefined;
+};
+/**
+ * A block referenced by a layout expression and how it was spliced in.
+ */
+export type LayoutBlockReference = {
+    name: string;
+    mode: string;
+};
+/**
+ * astryx --json layout expand "<expr>" [path]
+ */
+export type LayoutExpandResponse = {
+    type: "layout.expand";
+    data: {
+        form: LayoutForm;
+        code: string;
+        componentsUsed: string[];
+        states: number;
+        todos: string[];
+        blocksReferenced: LayoutBlockReference[];
+        warnings: string[];
+        written: string | null;
+    };
+};
+/**
+ * astryx --json layout check "<expr>"
+ */
+export type LayoutCheckResponse = {
+    type: "layout.check";
+    data: {
+        valid: boolean;
+        form: LayoutForm;
+        errors: LayoutIssue[];
+        warnings: string[];
+        compact: string;
+        outline: string;
+    };
+};
+/**
+ * astryx --json layout grammar
+ */
+export type LayoutGrammarResponse = {
+    type: "layout.grammar";
+    data: {
+        text: string;
+        aliases: Record<string, string>;
+    };
+};
+export type LayoutResponse = LayoutExpandResponse | LayoutCheckResponse | LayoutGrammarResponse;
+/**
+ * Options for `layoutExpand()`.
+ */
+export type LayoutExpandOptions = {
+    targetPath?: string | undefined;
+    form?: LayoutForm | undefined;
+    loose?: boolean | undefined;
+    name?: string | undefined;
+    cwd?: string | undefined;
+};
+/**
+ * Options for `layoutCheck()`.
+ */
+export type LayoutCheckOptions = {
+    form?: LayoutForm | undefined;
+    loose?: boolean | undefined;
+    cwd?: string | undefined;
+};
+/**
+ * Options for `layoutGrammar()`.
+ */
+export type LayoutGrammarOptions = {
+    cwd?: string | undefined;
+};

package/api/layout/layout.type.mjs

@@ -0,0 +1,103 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file Colocated types for the `layout` command — source of truth for its JSON
+ * responses. `astryx layout expand|check|grammar` turn compressed XLE/XLO
+ * expressions into validated XDS TSX (or echo canonical surfaces / a grammar
+ * cheatsheet). The `types/layout.d.ts` barrel re-exports from here.
+ *
+ * Invocation                                 -> type discriminator
+ * -----------------------------------------------------------------
+ * astryx --json layout expand "<expr>"       -> layout.expand
+ * astryx --json layout check "<expr>"        -> layout.check
+ * astryx --json layout grammar               -> layout.grammar
+ */
+
+/**
+ * The input surface an expression was parsed as.
+ * @typedef {'compact' | 'outline' | 'auto'} LayoutForm
+ */
+
+/**
+ * A validation issue with its formatted, human-readable rendering.
+ * @typedef {object} LayoutIssue
+ * @property {number} [line]
+ * @property {number} [col]
+ * @property {string} message
+ * @property {string} formatted
+ * @property {string[]} [suggestions]
+ */
+
+/**
+ * A block referenced by a layout expression and how it was spliced in.
+ * @typedef {object} LayoutBlockReference
+ * @property {string} name
+ * @property {string} mode
+ */
+
+/**
+ * astryx --json layout expand "<expr>" [path]
+ * @typedef {object} LayoutExpandResponse
+ * @property {'layout.expand'} type
+ * @property {object} data
+ * @property {LayoutForm} data.form
+ * @property {string} data.code
+ * @property {string[]} data.componentsUsed
+ * @property {number} data.states Number of useState hooks the expansion scaffolded.
+ * @property {string[]} data.todos
+ * @property {LayoutBlockReference[]} data.blocksReferenced
+ * @property {string[]} data.warnings
+ * @property {string | null} data.written
+ */
+
+/**
+ * astryx --json layout check "<expr>"
+ * @typedef {object} LayoutCheckResponse
+ * @property {'layout.check'} type
+ * @property {object} data
+ * @property {boolean} data.valid
+ * @property {LayoutForm} data.form
+ * @property {LayoutIssue[]} data.errors
+ * @property {string[]} data.warnings
+ * @property {string} data.compact
+ * @property {string} data.outline
+ */
+
+/**
+ * astryx --json layout grammar
+ * @typedef {object} LayoutGrammarResponse
+ * @property {'layout.grammar'} type
+ * @property {object} data
+ * @property {string} data.text
+ * @property {Record<string, string>} data.aliases Alias table (short name -> canonical component), from the registry.
+ */
+
+/**
+ * @typedef {LayoutExpandResponse | LayoutCheckResponse | LayoutGrammarResponse} LayoutResponse
+ */
+
+/**
+ * Options for `layoutExpand()`.
+ * @typedef {object} LayoutExpandOptions
+ * @property {string} [targetPath]
+ * @property {LayoutForm} [form]
+ * @property {boolean} [loose]
+ * @property {string} [name]
+ * @property {string} [cwd]
+ */
+
+/**
+ * Options for `layoutCheck()`.
+ * @typedef {object} LayoutCheckOptions
+ * @property {LayoutForm} [form]
+ * @property {boolean} [loose]
+ * @property {string} [cwd]
+ */
+
+/**
+ * Options for `layoutGrammar()`.
+ * @typedef {object} LayoutGrammarOptions
+ * @property {string} [cwd]
+ */
+
+export {};

package/api/layout/layoutCheck.doc.d.mts

@@ -0,0 +1,11 @@
+// @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
+// DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
+
+/**
+ * @file FunctionDoc for `layoutCheck()` / `astryx layout check`. Colocated with
+ * the API function it documents; the response-shape source of truth stays in
+ * `layout.type.mjs`.
+ * @position packages/cli/api/layout — function documentation
+ */
+/** @type {import('@astryxdesign/cli/authoring').FunctionDoc} */
+export const doc: import("@astryxdesign/cli/authoring").FunctionDoc;

package/api/layout/layoutCheck.doc.mjs

@@ -0,0 +1,85 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file FunctionDoc for `layoutCheck()` / `astryx layout check`. Colocated with
+ * the API function it documents; the response-shape source of truth stays in
+ * `layout.type.mjs`.
+ * @position packages/cli/api/layout — function documentation
+ */
+
+/** @type {import('@astryxdesign/cli/authoring').FunctionDoc} */
+export const doc = {
+  type: 'function',
+  kind: 'api',
+  name: 'layoutCheck',
+  namespace: 'cli/api',
+  displayName: 'layoutCheck()',
+  summary: 'Validate a layout expression without expanding it.',
+  description:
+    'The validator behind `astryx layout check`. Parses and validates a compressed XLE/XLO ' +
+    'expression without generating any TSX, and echoes it back in both canonical surfaces ' +
+    '(compact and outline). Validation failures are reported in the layout.check envelope ' +
+    '(valid: false) with line/col and suggestions (not thrown) so callers can lint an ' +
+    'expression and surface fixes.',
+  importPath: '@astryxdesign/cli/api',
+  signature:
+    'layoutCheck(expression: string, options?: LayoutCheckOptions): Promise<LayoutCheckResponse>',
+  keywords: ['layout', 'check', 'validate', 'xle', 'xlo', 'lint'],
+  params: [
+    {
+      name: 'expression',
+      type: 'string',
+      description: 'The layout expression to validate.',
+      required: true,
+    },
+    {
+      name: 'options.form',
+      type: "'compact' | 'outline' | 'auto'",
+      description:
+        'Force which input surface the expression is parsed as, or auto-detect it.',
+      default: "'auto'",
+    },
+    {
+      name: 'options.loose',
+      type: 'boolean',
+      description:
+        'Treat unknown {hint} references as warnings instead of errors.',
+      default: 'false',
+    },
+    {
+      name: 'options.cwd',
+      type: 'string',
+      description: 'Directory the block catalog and registry resolve against.',
+    },
+  ],
+  returns: [
+    {
+      type: 'layout.check',
+      description:
+        'The validation result: a valid flag, the detected form, errors (each with line/col, message, formatted text, and suggestions), warnings, and the expression re-printed in both canonical surfaces (compact and outline).',
+    },
+  ],
+  throws: [
+    {code: 'ERR_INVALID_ARGUMENT', when: 'the expression is empty'},
+    {
+      code: 'ERR_INVALID_OPTION',
+      when: 'form is not one of "compact", "outline", or "auto"',
+    },
+    {
+      code: 'ERR_LAYOUT_PARSE',
+      when: 'the expression has a syntax error (reported with line/col)',
+    },
+  ],
+  examples: [
+    {
+      label: 'Validate an expression',
+      code: 'const r = await layoutCheck(\'VStack[g4] > Text"Hi"\');',
+    },
+    {
+      label: 'Read the canonical forms',
+      code: 'const {data} = await layoutCheck(\'Card > Text"Hi"\');',
+    },
+  ],
+  command: 'layout check',
+  related: ['layoutExpand', 'layoutGrammar'],
+};

package/api/layout/layoutExpand.doc.d.mts

@@ -0,0 +1,11 @@
+// @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
+// DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
+
+/**
+ * @file FunctionDoc for `layoutExpand()` / `astryx layout expand`. Colocated
+ * with the API function it documents; the response-shape source of truth stays
+ * in `layout.type.mjs`.
+ * @position packages/cli/api/layout — function documentation
+ */
+/** @type {import('@astryxdesign/cli/authoring').FunctionDoc} */
+export const doc: import("@astryxdesign/cli/authoring").FunctionDoc;

package/api/layout/layoutExpand.doc.mjs

@@ -0,0 +1,107 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file FunctionDoc for `layoutExpand()` / `astryx layout expand`. Colocated
+ * with the API function it documents; the response-shape source of truth stays
+ * in `layout.type.mjs`.
+ * @position packages/cli/api/layout — function documentation
+ */
+
+/** @type {import('@astryxdesign/cli/authoring').FunctionDoc} */
+export const doc = {
+  type: 'function',
+  kind: 'api',
+  name: 'layoutExpand',
+  namespace: 'cli/api',
+  displayName: 'layoutExpand()',
+  summary: 'Expand a validated layout expression into XDS TSX.',
+  description:
+    'The generator behind `astryx layout expand`. Parses and validates a compressed XLE/XLO ' +
+    'expression, then expands it into ready-to-use XDS TSX, auto-routing structural children ' +
+    'into the right slots, scaffolding typed useState for interactive controls, and splicing or ' +
+    'importing any referenced template blocks. Returns the code (and metadata) in a layout.expand ' +
+    'envelope, optionally writing it to a path within cwd.',
+  importPath: '@astryxdesign/cli/api',
+  signature:
+    'layoutExpand(expression: string, options?: LayoutExpandOptions): Promise<LayoutExpandResponse>',
+  keywords: ['layout', 'expand', 'xle', 'xlo', 'tsx', 'scaffold', 'generate'],
+  params: [
+    {
+      name: 'expression',
+      type: 'string',
+      description:
+        'The layout expression to expand (XLE compact or XLO outline form).',
+      required: true,
+    },
+    {
+      name: 'options.targetPath',
+      type: 'string',
+      description:
+        'Write the generated TSX here (validated to stay within cwd). A path that ends in .tsx, .ts, .jsx, .js, .mjs, .cjs, .css, .scss, .json, .md or .html is used as-is; any other path is a directory that gets <name>.tsx. An existing file is replaced. Omit to return the code without writing.',
+    },
+    {
+      name: 'options.form',
+      type: "'compact' | 'outline' | 'auto'",
+      description:
+        'Force which input surface the expression is parsed as, or auto-detect it.',
+      default: "'auto'",
+    },
+    {
+      name: 'options.loose',
+      type: 'boolean',
+      description:
+        'Downgrade unknown {hint} references to TODO warnings instead of hard errors.',
+      default: 'false',
+    },
+    {
+      name: 'options.name',
+      type: 'string',
+      description: 'PascalCase name for the generated component.',
+      default: "'GeneratedLayout'",
+    },
+    {
+      name: 'options.cwd',
+      type: 'string',
+      description:
+        'Directory the block catalog, registry, and target path resolve against.',
+    },
+  ],
+  returns: [
+    {
+      type: 'layout.expand',
+      description:
+        'The expansion: the parsed form, the generated TSX code, componentsUsed, states (count of useState hooks scaffolded), todos, blocksReferenced (each {name, mode}), warnings, and written (the relative output path, or null when nothing was written).',
+    },
+  ],
+  throws: [
+    {
+      code: 'ERR_INVALID_ARGUMENT',
+      when: 'the expression is empty, or name is not a PascalCase identifier',
+    },
+    {
+      code: 'ERR_INVALID_OPTION',
+      when: 'form is not one of "compact", "outline", or "auto"',
+    },
+    {
+      code: 'ERR_LAYOUT_PARSE',
+      when: 'the expression has a syntax error (reported with line/col)',
+    },
+    {
+      code: 'ERR_LAYOUT_INVALID',
+      when: 'the expression parses but fails validation (unknown component/prop/enum/block)',
+    },
+    {code: 'ERR_PATH_TRAVERSAL', when: 'the target path escapes cwd'},
+  ],
+  examples: [
+    {
+      label: 'Expand to TSX',
+      code: 'const r = await layoutExpand(\'VStack[g4] > Heading"Title" + Text"Body"\');',
+    },
+    {
+      label: 'Write to a file',
+      code: "await layoutExpand('Card > Text\"Hi\"', {targetPath: 'src/Generated.tsx', name: 'Generated'});",
+    },
+  ],
+  command: 'layout expand',
+  related: ['layoutCheck', 'layoutGrammar'],
+};

package/api/layout/layoutGrammar.doc.d.mts

@@ -0,0 +1,11 @@
+// @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
+// DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
+
+/**
+ * @file FunctionDoc for `layoutGrammar()` / `astryx layout grammar`. Colocated
+ * with the API function it documents; the response-shape source of truth stays
+ * in `layout.type.mjs`.
+ * @position packages/cli/api/layout — function documentation
+ */
+/** @type {import('@astryxdesign/cli/authoring').FunctionDoc} */
+export const doc: import("@astryxdesign/cli/authoring").FunctionDoc;

package/api/layout/layoutGrammar.doc.mjs

@@ -0,0 +1,57 @@
+// Copyright (c) Meta Platforms, Inc. and affiliates.
+
+/**
+ * @file FunctionDoc for `layoutGrammar()` / `astryx layout grammar`. Colocated
+ * with the API function it documents; the response-shape source of truth stays
+ * in `layout.type.mjs`.
+ * @position packages/cli/api/layout — function documentation
+ */
+
+/** @type {import('@astryxdesign/cli/authoring').FunctionDoc} */
+export const doc = {
+  type: 'function',
+  kind: 'api',
+  name: 'layoutGrammar',
+  namespace: 'cli/api',
+  displayName: 'layoutGrammar()',
+  summary: 'Return the XLE/XLO grammar cheatsheet for this install.',
+  description:
+    'The reference behind `astryx layout grammar`: the agent cheatsheet for writing XLE/XLO ' +
+    "layout expressions, with the alias table generated from this branch's registry rather than " +
+    'hand-maintained, so short names always reflect the components actually installed.',
+  importPath: '@astryxdesign/cli/api',
+  signature:
+    'layoutGrammar(options?: LayoutGrammarOptions): Promise<LayoutGrammarResponse>',
+  keywords: [
+    'layout',
+    'grammar',
+    'cheatsheet',
+    'xle',
+    'xlo',
+    'aliases',
+    'reference',
+  ],
+  params: [
+    {
+      name: 'options.cwd',
+      type: 'string',
+      description:
+        'Directory the component registry (and its alias table) resolves against.',
+    },
+  ],
+  returns: [
+    {
+      type: 'layout.grammar',
+      description:
+        "The cheatsheet: a text field with the full grammar reference, plus an aliases map (short name → canonical component) generated from this install's registry.",
+    },
+  ],
+  examples: [
+    {
+      label: 'Get the cheatsheet',
+      code: 'const {data} = await layoutGrammar();',
+    },
+  ],
+  command: 'layout grammar',
+  related: ['layoutExpand', 'layoutCheck'],
+};

package/api/search/search.test.mjs

@@ -127,24 +127,6 @@ describe('search leaf — docs at the grain a reader reads them', () => {
     SLOW,
   );
 
-  it(
-    'finds the component batch guide from task language',
-    async () => {
-      const result = await search('look up several components', {
-        cwd,
-        type: 'doc',
-      });
-      expect(result.data.results).toContainEqual(
-        expect.objectContaining({
-          name: 'cli/component-lookups',
-          section: 'several',
-          command: 'astryx docs cli/component-lookups several',
-        }),
-      );
-    },
-    SLOW,
-  );
-
   it(
     'gives a top-level namespace hit the topic list as its parent',
     async () => {

package/api/template/template-integration.test.mjs

@@ -19,6 +19,7 @@ import {
 } from './template.mjs';
 import {search} from '../search/search.mjs';
 import {build} from '../build/build.mjs';
+import {layoutExpand} from '../layout/layout.mjs';
 import {runCli} from '../../test-utils/run-cli.mjs';
 
 let tmpDir;
@@ -517,7 +518,53 @@ describe('integration template discovery', () => {
     });
   });
 
-  it('resolves chained block aliases consistently in template', async () => {
+  it('resolves a replacement block through the layout alias', async () => {
+    const pkgDir = installWidgets(tmpDir);
+    writeTemplate(pkgDir, 'acme-card-callout', {
+      kind: 'block',
+      source:
+        'export default function AcmeCardCallout() { return <span>Acme replacement block</span>; }\n',
+    });
+    declareReplaces(pkgDir, {
+      'acme-card-callout': 'CardCallout',
+    });
+
+    const accidental = path.join(tmpDir, 'node_modules', '@acme', 'extra');
+    fs.mkdirSync(path.join(accidental, 'templates'), {recursive: true});
+    fs.writeFileSync(
+      path.join(accidental, 'package.json'),
+      JSON.stringify({name: '@acme/extra', version: '1.0.0'}),
+    );
+    fs.writeFileSync(
+      path.join(accidental, 'astryx.integration.mjs'),
+      `export default {templates: './templates'};\n`,
+    );
+    writeTemplate(accidental, 'CardCallout', {
+      kind: 'block',
+      body: `export default {type: 'block', name: 'ZZZ accidental block', description: 'collision'};\n`,
+      source:
+        'export default function Accidental() { return <span>Accidental block</span>; }\n',
+    });
+    fs.writeFileSync(
+      path.join(tmpDir, 'astryx.config.mjs'),
+      `export default { integrations: ['@acme/widgets', '@acme/extra'] };\n`,
+    );
+
+    const result = await layoutExpand('C{card-callout}', {
+      name: 'ReplacementLayout',
+      cwd: tmpDir,
+    });
+
+    expect(result.data.code).toContain('Acme replacement block');
+    expect(result.data.code).not.toContain('Accidental block');
+    expect(result.data.blocksReferenced).toEqual(
+      expect.arrayContaining([
+        expect.objectContaining({name: 'CardCallout', mode: 'splice'}),
+      ]),
+    );
+  }, 30_000);
+
+  it('resolves chained block aliases consistently in template and layout', async () => {
     const [firstTarget, secondTarget] = (await discoverCoreTemplates()).filter(
       candidate => candidate.type === 'block',
     );
@@ -545,12 +592,29 @@ describe('integration template discovery', () => {
       cwd: tmpDir,
     });
     expect(firstTemplate.data.source).toContain('First chain replacement');
+    const layoutId = id =>
+      id.replace(/([a-z0-9])([A-Z])/gu, '$1-$2').toLowerCase();
+    const firstLayout = await layoutExpand(
+      `C{${layoutId(firstTarget.dirName)}}`,
+      {
+        cwd: tmpDir,
+      },
+    );
+    expect(firstLayout.data.code).toContain('First chain replacement');
+    expect(firstLayout.data.code).not.toContain('Final chain replacement');
 
     const secondTemplate = await template(secondTarget.dirName, {
       show: true,
       cwd: tmpDir,
     });
     expect(secondTemplate.data.source).toContain('Final chain replacement');
+    const secondLayout = await layoutExpand(
+      `C{${layoutId(secondTarget.dirName)}}`,
+      {
+        cwd: tmpDir,
+      },
+    );
+    expect(secondLayout.data.code).toContain('Final chain replacement');
   }, 30_000);
 
   it('keeps the old discovery behavior when no replacement is declared', async () => {

package/api/template/template.mjs

@@ -8,7 +8,7 @@
  * the requested one, and routes to a leaf (list/show/skeleton/copy/cdn). The shared
  * discovery/IO + cross-command helpers live in `foundation/discovery/template-adapter.mjs` and are
  * RE-EXPORTED here so external import paths (`api/template/template.mjs`) —
- * used by component, search, init, discover, Doctor integration, and
+ * used by component, layout, search, init, discover, Doctor integration, and
  * lib/project — keep resolving unchanged.
  *
  * @position api/template — the template dispatcher + barrel; leaves live under

package/api/theme/add/add.mjs

@@ -15,7 +15,6 @@ import {
 import {AstryxError} from '../../error.mjs';
 import {ERROR_CODES} from '../../../foundation/response/error-codes.mjs';
 import {listAvailableThemes, findTheme} from '../_adapter.mjs';
-import {applyWrites} from '../../integration/add-helpers.mjs';
 // Scaffolded files must not carry our repo boilerplate into a consumer's tree.
 import {stripCopyrightHeader} from '../../../foundation/text/copyright-header.mjs';
 
@@ -137,28 +136,37 @@ export async function themeAdd(slug, options = {}) {
     }
   }
 
-  // One transaction: every file publishes, or every replaced file gets its
-  // previous bytes back and every created file is removed. mkdir is inside the
-  // try so a failure (e.g. an ancestor is a file → EEXIST/ENOTDIR) surfaces as
-  // a stable ERR_WRITE_FAILED rather than leaking a raw fs errno + absolute path.
+  // Stage to temp files then rename, rolling back partials on failure so a
+  // failed write never leaves a half-written theme. mkdir is inside the try so
+  // a failure (e.g. an ancestor is a file → EEXIST/ENOTDIR) surfaces as a
+  // stable ERR_WRITE_FAILED rather than leaking a raw fs errno + absolute path.
+  const staged = [];
   try {
     fs.mkdirSync(resolvedDir, {recursive: true});
-    const plans = writes.map(w => {
-      fs.mkdirSync(path.dirname(w.dest), {recursive: true});
-      // Confine again once the directories exist: one may have been swapped
-      // for a link since the destination was first checked.
+    for (const w of writes) {
       const dest = assertWithin(w.name, resolvedDir, {
         label: `theme destination for ${w.name}`,
       });
-      return {
-        path: dest,
-        contents: scaffoldContents(fs.readFileSync(w.src)),
-        createOnly: !overwrite,
-      };
-    });
-    applyWrites(plans);
+      fs.mkdirSync(path.dirname(dest), {recursive: true});
+      // The staging file is an output path too: confine it, and never write
+      // through an entry that already exists under its name.
+      const tmp = assertWithin(`${w.name}.${process.pid}.tmp`, resolvedDir, {
+        label: `theme staging file for ${w.name}`,
+      });
+      fs.writeFileSync(tmp, scaffoldContents(fs.readFileSync(w.src)), {flag: 'wx'});
+      staged.push({tmp, dest});
+    }
+    for (const s of staged) {
+      fs.renameSync(s.tmp, s.dest);
+    }
   } catch (err) {
-    if (err instanceof AstryxError) throw err;
+    for (const s of staged) {
+      try {
+        fs.rmSync(s.tmp, {force: true});
+      } catch {
+        /* best-effort */
+      }
+    }
     if (err instanceof PathSafetyError) {
       throw new AstryxError(
         err.message,