@astryxdesign/cli 0.7.11 → 0.7.12-canary.ccb4a6d

package/api/integration/add-helpers.mjs

@@ -131,7 +131,10 @@ export function packageJsonFilesUpdate(
     }
   }
 
-  if ((pkg.exports !== undefined || createExports) && publicExports.length > 0) {
+  if (
+    (pkg.exports !== undefined || createExports) &&
+    publicExports.length > 0
+  ) {
     const authored = pkg.exports;
     /** @type {Record<string, unknown>} */
     let exportsMap;
@@ -483,7 +486,7 @@ export async function loadCurrentManifest(manifestFile, manifestExists) {
  * @param {string} manifestPath
  * @param {string[]} files
  * @param {boolean} dryRun
- * @returns {{type: 'integration.add', data: Omit<import('./integration-authoring.type.mjs').IntegrationAddData, 'kind'> & {kind: K, icons?: number, refreshed?: boolean, renamed?: Array<{from: string, to: string}>, mapped?: Array<{definition: string, icon: string}>, suggestions?: Array<{definition: string, icon: string, word: string}>, notes?: string[]}}}
+ * @returns {{type: 'integration.add', data: Omit<import('./integration-authoring.type.mjs').IntegrationAddData, 'kind'> & {kind: K, icons?: number, refreshed?: boolean, updated?: boolean, renamed?: Array<{from: string, to: string}>, moved?: Array<{id: string, from: string, to: string}>, licenseNoticeMissing?: {library: string, path: string}, mapped?: Array<{definition: string, icon: string}>, suggestions?: Array<{definition: string, icon: string, word: string}>, notes?: string[]}}}
  */
 export function receipt(kind, name, root, manifestPath, files, dryRun) {
   return {

package/api/integration/authoring-checks.d.mts

@@ -1,8 +1,6 @@
 // @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
 // DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
 
-/** @param {string} value */
-export function shellArg(value: string): string;
 /**
  * Find integration template ids that are also owned by Core.
  * @param {string} [pkg] Installed integration package; omit for the local package.

package/api/integration/authoring-checks.mjs

@@ -57,12 +57,7 @@ import {
 } from './validate-integration.mjs';
 import {rootProblem} from '../../foundation/integrations/validate-contributions.mjs';
 import {normalizeProviderId} from '../../foundation/identity/provider-identity.mjs';
-
-/** @param {string} value */
-export function shellArg(value) {
-  if (/^[A-Za-z0-9@/._-]+$/.test(value)) return value;
-  return `'${value.replace(/'/g, `'\\''`)}'`;
-}
+import {shellArg} from '../../foundation/text/shell-arg.mjs';
 
 /**
  * The provider ID a package without a name is checked under: a valid bare

package/api/logger.mjs

@@ -3,7 +3,8 @@
 /**
  * @file The one `Logger`. Source of truth: the `Logger` type is generated from
  * the JSDoc here, and `logger` is the single instance every side-effecting
- * command (`upgrade`, `init`, `themeBuild`) writes progress through.
+ * command (`upgrade`, `init`, `themeBuild`, the experimental icon commands)
+ * writes progress through.
  *
  * It is silent by default, so a programmatic caller (or `--json` mode) sees
  * nothing but the returned `{type, data}`. The CLI calls `logger.setSilent(false)`

package/api/swizzle/_adapter.d.mts

@@ -30,3 +30,4 @@ export function resolveCore(cwd: string): {
     components: string[];
 };
 export { CORE_PROVIDER_ID as CORE_PACKAGE } from "../../foundation/identity/providers.mjs";
+export { planCopy, relativeSpecifierBetween, renderFile } from "./_moduleGraph.mjs";

package/api/swizzle/_adapter.mjs

@@ -9,7 +9,8 @@
  * for its "component not found" suggestions. Resolving core + listing lives
  * here once so neither leaf re-walks the filesystem. Throws AstryxError
  * (ERR_CORE_NOT_FOUND) when core can't be located. The copy receipt's `gh`
- * lookup is here too, on the shared search-path walk.
+ * lookup is here too, on the shared search-path walk, and so is the copy's
+ * module graph (_moduleGraph.mjs), which reads the owner package's source.
  */
 
 import {executablesOnSearchPath} from '../../foundation/env/search-path.mjs';
@@ -18,6 +19,8 @@ import {resolveComponentReplacements} from '../../foundation/discovery/component
 import {ERROR_CODES} from '../../foundation/response/error-codes.mjs';
 import {AstryxError} from '../error.mjs';
 
+export {planCopy, relativeSpecifierBetween, renderFile} from './_moduleGraph.mjs';
+
 /**
  * Whether an executable `gh` is on `PATH` (with a `PATHEXT` extension on
  * Windows), found without running it. The copy receipt offers its

package/api/swizzle/_moduleGraph.d.mts

@@ -0,0 +1,171 @@
+// @generated by scripts/sync-api-types.mjs from the JSDoc in api/**/*.mjs.
+// DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
+
+/**
+ * A specifier that is either fixed (a public subpath) or computed from where
+ * two copied files land.
+ * @typedef {string | {from: string, to: string}} SpecifierChoice
+ *
+ * @typedef {object} PlannedEdit
+ * @property {number} start
+ * @property {number} end
+ * @property {(specifierOf: (choice: SpecifierChoice) => string) => string} render
+ *
+ * @typedef {object} CopyPlan
+ * @property {Map<string, {code: string, edits: PlannedEdit[]}>} files every
+ *   file the copy holds (absolute source path), with the edits its imports need
+ * @property {Map<string, Set<string>>} dependsOn file -> carried files its
+ *   computed specifiers point at
+ * @property {Record<string, string>} dependencies each npm package the copied
+ *   files import, other than the owner package and Node's built-in modules,
+ *   with the range the owner package declares
+ */
+/**
+ * Follow a component's module graph from its entry files and decide what the
+ * copy holds. A module inside the component folder is copied. A module
+ * outside it is reached through the owner package's public exports when every
+ * name taken from it is public there; otherwise the module that defines the
+ * name is copied, and imported by relative path. Tests, stories and docs are
+ * never reached, so they are never copied.
+ *
+ * @param {object} options
+ * @param {string} options.packageDir the owner package root
+ * @param {string} options.srcRoot its source root
+ * @param {string} options.componentDir
+ * @param {string} options.ownerPackage
+ * @param {string[]} options.entries absolute entry files of the component
+ * @returns {CopyPlan}
+ */
+export function planCopy({ packageDir, srcRoot, componentDir, ownerPackage, entries, }: {
+    packageDir: string;
+    srcRoot: string;
+    componentDir: string;
+    ownerPackage: string;
+    entries: string[];
+}): CopyPlan;
+/**
+ * The source of a planned file with its edits applied.
+ * @param {{code: string, edits: PlannedEdit[]}} planned
+ * @param {(choice: SpecifierChoice) => string} specifierOf
+ */
+export function renderFile(planned: {
+    code: string;
+    edits: PlannedEdit[];
+}, specifierOf: (choice: SpecifierChoice) => string): string;
+/**
+ * The relative specifier from one written file to another.
+ * @param {string} fromDest
+ * @param {string} toDest
+ */
+export function relativeSpecifierBetween(fromDest: string, toDest: string): string;
+export type ImportSpecifier = {
+    /**
+     * the exported name (`default` for a default import)
+     */
+    imported: string;
+    local: string;
+    typeOnly: boolean;
+};
+export type ModuleReference = {
+    kind: "named" | "namespace" | "sideEffect" | "dynamic" | "reexportAll" | "reexportNamed";
+    /**
+     * the specifier as written
+     */
+    source: string;
+    /**
+     * statement (or, for `import()`, string literal) start
+     */
+    start: number;
+    end: number;
+    /**
+     * the whole statement is `import type` / `export type`
+     */
+    typeOnly: boolean;
+    specifiers: ImportSpecifier[];
+    /**
+     * the local name of `* as X`
+     */
+    namespace?: string | undefined;
+};
+export type ParsedModule = {
+    code: string;
+    references: ModuleReference[];
+    /**
+     * names the module declares and exports
+     */
+    declares: Set<string>;
+};
+/**
+ * `value` reads runtime exports only;
+ *   `type` reads type-only exports too
+ */
+export type ExportKind = "value" | "type";
+/**
+ * every name a module exports, with where it is declared
+ */
+export type OriginsOf = (file: string, kind: ExportKind) => Map<string, {
+    file: string;
+    name: string;
+}>;
+/**
+ * `file#name` of a declaration -> the subpaths that export it, and as what
+ */
+export type ExportersByOrigin = Map<string, Array<{
+    subpath: string;
+    name: string;
+}>>;
+export type PublicSurface = {
+    /**
+     * the subpaths
+     * that export each declaration as a value, and as a value or a type
+     */
+    byOrigin: Record<ExportKind, ExportersByOrigin>;
+    /**
+     * source file -> the subpath that
+     * exports exactly that file
+     */
+    byFile: Map<string, string>;
+    exportsField: Record<string, unknown> | undefined;
+};
+/**
+ * A specifier that is either fixed (a public subpath) or computed from where
+ * two copied files land.
+ */
+export type SpecifierChoice = string | {
+    from: string;
+    to: string;
+};
+/**
+ * A specifier that is either fixed (a public subpath) or computed from where
+ * two copied files land.
+ */
+export type PlannedEdit = {
+    start: number;
+    end: number;
+    render: (specifierOf: (choice: SpecifierChoice) => string) => string;
+};
+/**
+ * A specifier that is either fixed (a public subpath) or computed from where
+ * two copied files land.
+ */
+export type CopyPlan = {
+    /**
+     * every
+     * file the copy holds (absolute source path), with the edits its imports need
+     */
+    files: Map<string, {
+        code: string;
+        edits: PlannedEdit[];
+    }>;
+    /**
+     * file -> carried files its
+     * computed specifiers point at
+     */
+    dependsOn: Map<string, Set<string>>;
+    /**
+     * each npm package the copied
+     * files import, other than the owner package and Node's built-in modules,
+     * with the range the owner package declares
+     */
+    dependencies: Record<string, string>;
+};