@astryxdesign/cli 0.6.3-canary.f22695a → 0.6.3

package/api/theme/theme.type.mjs

@@ -6,23 +6,23 @@
  * directly (functions own their types); the public `@astryxdesign/cli/api`
  * surface re-exports them via types/theme.d.ts, so consumers see the same names.
  *
- * Invocation                                  -> type discriminator
+ * Invocation                                 -> type discriminator
  * ------------------------------------------------------------------
- * astryx --json theme build <file>            -> theme.build
- * astryx --json theme build <file> --check    -> theme.build.check
- * astryx --json theme build <a> <b> …         -> theme.build.batch
- * astryx --json theme list                    -> theme.list
- * astryx --json theme add <slug>              -> theme.add
- * astryx --json theme template                -> theme.template
- * astryx --json theme targets [filter]        -> theme.targets
- * astryx --json theme palette generate <file> -> theme.palette.generate
- * (file not found / parse error)              -> CLIError
+ * xds --json theme build <file>             -> theme.build
+ * xds --json theme build <file> --check     -> theme.build.check
+ * xds --json theme build <a> <b> …          -> theme.build.batch
+ * xds --json theme list                     -> theme.list
+ * xds --json theme add <slug>               -> theme.add
+ * xds --json theme template                 -> theme.template
+ * xds --json theme targets [filter]         -> theme.targets
+ * xds --json theme palette generate <file>  -> theme.palette.generate
+ * (file not found / parse error)            -> CLIError
  *
  * @position api — colocated typedefs for api/theme/{theme,build,add,list,template,targets,_adapter}
  */
 
 /**
- * astryx --json theme build <file>
+ * xds --json theme build <file>
  * @typedef {object} ThemeBuildResponse
  * @property {'theme.build'} type
  * `warnings` are defects the theme author should fix. `notices` are advisories
@@ -32,14 +32,14 @@
  */
 
 /**
- * astryx --json theme build <file> --check
+ * xds --json theme build <file> --check
  * @typedef {object} ThemeBuildCheckResponse
  * @property {'theme.build.check'} type
  * @property {{name: string, upToDate: boolean, stale: Array<{path: string, reason: 'missing' | 'outdated'}>, checked: string[]}} data
  */
 
 /**
- * astryx --json theme build <a> <b> … — several themes in one invocation. Each
+ * xds --json theme build <a> <b> … — several themes in one invocation. Each
  * result carries the file as it was passed and the receipt a single-file build
  * would have returned (null when that theme produced no CSS). One file still
  * returns the bare theme.build / theme.build.check envelope.
@@ -59,21 +59,21 @@
  */
 
 /**
- * astryx --json theme list
+ * xds --json theme list
  * @typedef {object} ThemeListResponse
  * @property {'theme.list'} type
  * @property {ThemeListEntry[]} data
  */
 
 /**
- * astryx --json theme add <slug>
+ * xds --json theme add <slug>
  * @typedef {object} ThemeAddResponse
  * @property {'theme.add'} type
  * @property {{slug: string, displayName: string, maintained: boolean, package: string, outputDir: string, entry: string, exportName: string, files: string[]}} data
  */
 
 /**
- * astryx --json theme template
+ * xds --json theme template
  * `written: false` with `reason: 'exists'` is a success: the command is safe to
  * re-run, and an edited template is the consumer's file to keep.
  * @typedef {object} ThemeTemplateResponse
@@ -95,19 +95,19 @@
  */
 
 /**
- * astryx --json theme targets [filter]
+ * xds --json theme targets [filter]
  * @typedef {object} ThemeTargetsResponse
  * @property {'theme.targets'} type
  * @property {{filter: string | null, componentCount: number, targets: ThemeTargetEntry[]}} data
  */
 
 /**
- * A color an author pins at one stop of one mode, constraining generation.
+ * A generated palette candidate. The palette is still subject to author review
+ * and is not connected to runtime theme values.
  * @typedef {object} TonalPaletteAnchor
  * @property {'light' | 'dark'} mode Mode containing the anchored stop.
  * @property {number} stop Existing requested stop where the anchor applies.
- * @property {string} color sRGB hex the stop is pulled toward: three or six
- * digits, with or without `#`; the generator normalizes it.
+ * @property {string} color
  * @property {'exact' | 'bounded' | 'flexible'} policy `exact` preserves the
  * chosen color at that stop; `bounded` permits adjustment within `maxDeltaE`;
  * `flexible` treats the color as guidance and blends toward it.
@@ -116,16 +116,12 @@
  */
 
 /**
- * One requested family: a seed color and the constraints applied to its ramp.
  * @typedef {object} TonalPaletteFamilyInput
- * @property {string} id Lower-kebab-case key for the family in the generated
- * palette. `black` and `white` are reserved for the standalone values.
- * @property {string} seed sRGB hex the ramp is generated from: three or six
- * digits, with or without `#`; the generator normalizes it.
- * @property {string} [name] Display name for review artifacts; defaults to `id`.
- * @property {'chromatic' | 'neutral'} [kind] `neutral` derives the ramp from
- * `neutralProfile` instead of the seed hue; defaults to `chromatic`.
- * @property {TonalPaletteAnchor[]} [anchors] Colors pinned at specific stops.
+ * @property {string} id
+ * @property {string} seed
+ * @property {string} [name]
+ * @property {'chromatic' | 'neutral'} [kind]
+ * @property {TonalPaletteAnchor[]} [anchors]
  */
 
 /**
@@ -134,9 +130,6 @@
  * @property {number} [vibrancy] Chroma control from 0 (most muted) through 50
  * (default) to 100 (most vivid).
  * @property {'neutral-v1' | 'warm-v1' | 'cool-v1' | 'custom'} [neutralProfile]
- * Hue treatment for `neutral` families: `neutral-v1` is fully achromatic,
- * `warm-v1` and `cool-v1` add a slight tint, and `custom` derives the hue from
- * the family's own seed. Defaults to `neutral-v1`.
  * @property {'light-only' | 'dark-only' | 'light-and-dark'} [modeStrategy]
  * @property {number[]} [stops] Ordered stops shared by every requested family;
  * defaults to 0 through 100 in increments of 5. Decimal stops are supported,
@@ -144,8 +137,6 @@
  */
 
 /**
- * A generated palette candidate. The palette is still subject to author review
- * and is not connected to runtime theme values.
  * @typedef {object} TonalPaletteCandidate
  * @property {1} schemaVersion
  * @property {'candidate'} status
@@ -157,68 +148,10 @@
  */
 
 /**
- * Per-ramp evidence recorded for one family in one mode.
- * @typedef {object} TonalPaletteRampDiagnostics
- * @property {boolean} monotonic Whether luminance rises across every stop.
- * @property {number} minimumAdjacentDeltaE Smallest perceptual gap between
- * neighboring stops; a small value means two stops read as one color.
- * @property {number} maximumAdjacentDeltaE Largest gap between neighboring stops.
- * @property {number} maximumHueDrift Largest hue distance, in degrees, between
- * a stop and the family's reference hue.
- * @property {'blue-to-purple' | 'yellow-to-brown' | null} hueIdentityRisk Named
- * drift the ramp is at risk of, or `null`.
- * @property {number[]} gamutMappedStops Stops whose ideal color fell outside
- * sRGB and was mapped back into it.
- * @property {Array<TonalPaletteAnchor & {generatedColor: string, deltaE: number}>} anchors
- * Each anchor with the color the stop received after its policy was applied
- * and the perceptual distance that color still has from the requested target.
- */
-
-/**
- * Cross-family evidence for one mode, sampled at the stop nearest 50.
- * @typedef {object} TonalPaletteCoordinationDiagnostics
- * @property {'light' | 'dark'} mode Mode these samples come from.
- * @property {number} stop Sampled stop.
- * @property {[string, string] | null} closestFamilies The two chromatic
- * families hardest to tell apart, or `null` with fewer than two.
- * @property {number | null} minimumFamilyDeltaE Perceptual distance between them.
- * @property {string | null} strongestFamily Most saturated family at this stop.
- * @property {string | null} weakestFamily Least saturated family at this stop.
- * @property {number | null} chromaRatio Strongest chroma over weakest; a large
- * ratio means the families are unbalanced.
- */
-
-/**
- * The request as the generator resolved it, with every default filled in.
- * @typedef {object} TonalPaletteNormalizedRequest
- * @property {'astryx-oklch-v1'} recipe
- * @property {number} vibrancy
- * @property {'neutral-v1' | 'warm-v1' | 'cool-v1' | 'custom'} neutralProfile
- * @property {'light-only' | 'dark-only' | 'light-and-dark'} modeStrategy
- * @property {number[]} stops
- * @property {Array<{id: string, name: string, seed: string, kind: 'chromatic' | 'neutral', anchors: TonalPaletteAnchor[]}>} families
- */
-
-/**
- * The detached receipt written beside a candidate. It records what was asked
- * for and what the generator observed, so a candidate can be traced back to
- * its request without rerunning generation.
- * @typedef {object} TonalPaletteGenerationReceipt
- * @property {1} schemaVersion
- * @property {'astryx-oklch-v1'} recipe Recipe that produced the candidate.
- * @property {string} candidateSha256 SHA-256 over the candidate bytes as written.
- * @property {{version: string, sha256: string}} [preview] Present when preview
- * content was generated for the request, even if no file was written because
- * the target already exists.
- * @property {TonalPaletteNormalizedRequest} request
- * @property {{families: Record<string, {light?: TonalPaletteRampDiagnostics, dark?: TonalPaletteRampDiagnostics}>, coordination: TonalPaletteCoordinationDiagnostics[]}} diagnostics
- */
-
-/**
- * astryx --json theme palette generate <config>
+ * xds --json theme palette generate <config>
  * @typedef {object} ThemePaletteGenerateResponse
  * @property {'theme.palette.generate'} type
- * @property {{recipe: 'astryx-oklch-v1', status: 'candidate', familyCount: number, stopCount: number, modes: string[], output: string | null, receipt: string | null, preview: string | null, written: boolean, reason: 'exists' | null, candidate: TonalPaletteCandidate, generationReceipt: TonalPaletteGenerationReceipt}} data
+ * @property {{recipe: 'astryx-oklch-v1', status: 'candidate', familyCount: number, stopCount: number, modes: string[], output: string | null, receipt: string | null, preview: string | null, written: boolean, reason: 'exists' | null, candidate: TonalPaletteCandidate, generationReceipt: Record<string, unknown>}} data
  */
 
 // Make this a module so the @typedefs above are importable as types via

package/api/upgrade/_adapter.mjs

@@ -33,10 +33,7 @@ import {
 } from '../../foundation/agent-docs/agent-docs.mjs';
 import {formatCliCommand} from '../../foundation/env/package-manager.mjs';
 import {Project} from '../../foundation/config/project.mjs';
-import {
-  loadIntegrations,
-  markProviderConflicts,
-} from '../../foundation/integrations/integrations.mjs';
+import {loadIntegrations} from '../../foundation/integrations/integrations.mjs';
 import {warnOnIntegrationIssues} from '../../foundation/integrations/integration-warnings.mjs';
 import {logger} from '../logger.mjs';
 
@@ -311,15 +308,6 @@ export async function runCoreCodemods(versionManifests, {apply, path: srcPath, c
   });
 }
 
-/**
- * One installed package at one version; two specs that reach it share this.
- * @param {{name: string, version?: string}} integration
- * @returns {string}
- */
-function packageIdentity(integration) {
-  return `${integration.name}\u0000${integration.version ?? ''}`;
-}
-
 /**
  * Load the consumer project's config + integrations. Throws on invalid config;
  * the run leaf decides between the config_fixable preview and a hard abort.
@@ -333,65 +321,11 @@ export async function loadProjectContext(cwd, extraIntegrationSpecs = []) {
   // cache entry; ordinary Project discovery remains cached by default.
   const project = await Project.load(cwd, {fresh: true});
   const postCodemodHooks = project.config.hooks?.postCodemod ?? [];
-  const configuredSpecs = new Set(project.integrations ?? []);
-  // Configured packages come from Project, which has already resolved provider
-  // identity over configured, autolinked, and local packages. Loading them
-  // again here would let upgrade run codemods from a package Project set aside.
-  const configured = project.loadedIntegrations.filter(
-    integration =>
-      !integration.__autolinked && configuredSpecs.has(integration.__spec),
-  );
-  // A package that lists itself runs its installed copy's released codemods,
-  // never the work in progress being authored, so that copy stands in for it.
-  const local = project.loadedIntegrations.find(
-    integration => integration.__local,
-  );
-  const selfListed = local != null && configured.includes(local);
-  if (local != null && selfListed && project.configPath) {
-    const installed = await loadIntegrations([local.__spec], {
-      cwd: path.dirname(project.configPath),
-      resolveProviders: false,
-    });
-    configured.splice(configured.indexOf(local), 1, ...installed);
-  }
-  const extraSpecs = uniqueFiles(extraIntegrationSpecs ?? []).filter(
-    spec => !configuredSpecs.has(spec),
-  );
-  const extras =
-    extraSpecs.length === 0
-      ? []
-      : await loadIntegrations(extraSpecs, {resolveProviders: false});
-  // Naming the package being authored with --integration asks for its
-  // installed copy too, so the local package must not claim against it.
-  const localStandsIn =
-    local != null &&
-    (selfListed || extras.some(extra => extra.name === local.name));
-  // Autolinked packages and the package being authored join the pass only as
-  // claimants: every other command uses them for their provider IDs, but
-  // upgrade has never run their codemods and still does not. An extra that
-  // names one of them opts it in, so the extra stands in for that claimant.
-  const extraIdentities = new Set(extras.map(packageIdentity));
-  const claimantsOnly = project.loadedIntegrations.filter(
-    integration =>
-      (integration.__autolinked || (integration.__local && !localStandsIn)) &&
-      !extraIdentities.has(packageIdentity(integration)),
-  );
-  const resolved = markProviderConflicts([
-    ...configured,
-    ...claimantsOnly,
-    ...extras,
+  const integrationSpecs = uniqueFiles([
+    ...(project.integrations ?? []),
+    ...(extraIntegrationSpecs ?? []),
   ]);
-  // Claimants leave by package directory, not object identity: the pass can
-  // return one as a new conflict entry, and upgrade must not warn that a
-  // package every other command uses contributes nothing.
-  const claimantDirs = new Set(
-    claimantsOnly.map(integration => integration.__packageDir),
-  );
-  const integrations = resolved.filter(
-    integration =>
-      integration.__packageDir == null ||
-      !claimantDirs.has(integration.__packageDir),
-  );
+  const integrations = await loadIntegrations(integrationSpecs);
   return {postCodemodHooks, integrations};
 }
 

package/api/upgrade/upgrade.doc.mjs

@@ -16,9 +16,8 @@ export const doc = {
   description:
     'Migrates project source from a previous Astryx version to the currently ' +
     'installed one by running the registered codemods, and compares the fully ' +
-    'rendered managed agent-docs block on every migration path, including ' +
-    'same-Core integration guidance changes; list and registry-only modes do not ' +
-    'run migration reconciliation. Dry-run previews without writing; `apply` ' +
+    'rendered managed agent-docs block on every path, including same-Core ' +
+    'integration guidance changes. Dry-run previews without writing; `apply` ' +
     'writes the prepared block only after selected codemods and hooks succeed. ' +
     'Core codemods run before ' +
     'the config is loaded so a config codemod can repair an otherwise-invalid ' +
@@ -69,7 +68,7 @@ export const doc = {
       name: 'options.integration',
       type: 'string[]',
       description:
-        'Explicit integration specifiers to process. Resolved beneath node_modules; absolute paths and `.` or `..` segments are rejected.',
+        'Explicit integration package names / file paths to process.',
     },
     {
       name: 'options.path',

package/api/upgrade/upgrade.type.d.mts

@@ -2,7 +2,7 @@
 // DO NOT EDIT — run `pnpm sync:api-types` to regenerate.
 
 /**
- * astryx --json upgrade --list
+ * xds --json upgrade --list
  */
 export type UpgradeListResponse = {
     type: "upgrade.list";
@@ -121,14 +121,14 @@ export type RegistryCompositionSummary = {
     items: RegistryCompositionItemSummary[];
 };
 /**
- * astryx --json upgrade --registry [--apply]
+ * xds --json upgrade --registry [--apply]
  */
 export type UpgradeRegistryResponse = {
     type: "upgrade.registry";
     data: RegistryCompositionSummary;
 };
 /**
- * astryx --json upgrade [--apply]
+ * xds --json upgrade [--apply]
  */
 export type UpgradeRunResponse = {
     type: "upgrade.run";
@@ -150,7 +150,7 @@ export type UpgradeRunResponse = {
     };
 };
 /**
- * astryx --json upgrade — short-circuit status results.
+ * xds --json upgrade — short-circuit status results.
  *
  * - `up_to_date`: `--from` is >= installed target and `--force` was not passed.
  * - `no_codemods`: no codemods (core or integration) apply to the range.
@@ -211,7 +211,7 @@ export type UpgradeOptions = {
      */
     skipCodemod?: string[] | undefined;
     /**
-     * Explicit integration specifiers resolved beneath node_modules; absolute paths and `.` or `..` segments are rejected.
+     * Explicit integration package names / file paths.
      */
     integration?: string[] | undefined;
     /**

package/api/upgrade/upgrade.type.mjs

@@ -4,17 +4,17 @@
  * @file Colocated types for the `upgrade` command — source of truth for the
  * upgrade command JSON responses. Re-exported by `types/upgrade.d.ts`.
  *
- * Invocation                                   -> type discriminator
+ * Invocation                                 -> type discriminator
  * ------------------------------------------------------------------
- * astryx --json upgrade --list                 -> upgrade.list
- * astryx --json upgrade [--apply]              -> upgrade.run
- * astryx --json upgrade --registry [--apply]   -> upgrade.registry
- * astryx --json upgrade (status short-circuit) -> upgrade.status
- * (version detection failure)                  -> CLIError
+ * xds --json upgrade --list                 -> upgrade.list
+ * xds --json upgrade [--apply]              -> upgrade.run
+ * xds --json upgrade --registry [--apply]   -> upgrade.registry
+ * xds --json upgrade (status short-circuit) -> upgrade.status
+ * (version detection failure)               -> CLIError
  */
 
 /**
- * astryx --json upgrade --list
+ * xds --json upgrade --list
  * @typedef {object} UpgradeListResponse
  * @property {'upgrade.list'} type
  * @property {UpgradeListEntry[]} data
@@ -88,14 +88,14 @@
  */
 
 /**
- * astryx --json upgrade --registry [--apply]
+ * xds --json upgrade --registry [--apply]
  * @typedef {object} UpgradeRegistryResponse
  * @property {'upgrade.registry'} type
  * @property {RegistryCompositionSummary} data
  */
 
 /**
- * astryx --json upgrade [--apply]
+ * xds --json upgrade [--apply]
  * @typedef {object} UpgradeRunResponse
  * @property {'upgrade.run'} type
  * @property {object} data
@@ -112,7 +112,7 @@
  */
 
 /**
- * astryx --json upgrade — short-circuit status results.
+ * xds --json upgrade — short-circuit status results.
  *
  * - `up_to_date`: `--from` is >= installed target and `--force` was not passed.
  * - `no_codemods`: no codemods (core or integration) apply to the range.
@@ -135,7 +135,7 @@
  * @property {boolean} [force] Run codemods even if `from` >= installed.
  * @property {string} [codemod] Run a single named transform.
  * @property {string[]} [skipCodemod] Exclude named codemods (re-run past a failure).
- * @property {string[]} [integration] Explicit integration specifiers resolved beneath node_modules; absolute paths and `.` or `..` segments are rejected.
+ * @property {string[]} [integration] Explicit integration package names / file paths.
  * @property {string} [path] Source directory to scan (default `./src`).
  * @property {boolean} [installDeps] Auto-install jscodeshift without prompting.
  * @property {boolean} [registry] Reconcile only ShadCN-copied compositions; `from` is not required.

package/assets/codemods/integration-discovery.mjs

@@ -32,40 +32,6 @@ import {semverCompare} from '../../foundation/env/semver.mjs';
 /** File extensions recognized as codemod modules. */
 const CODEMOD_EXTENSIONS = ['.ts', '.mjs', '.js'];
 
-/**
- * Directories never walked for codemods: a test directory beside a transform is
- * the natural place to put its test, and every file found here is loaded and
- * validated as a codemod.
- */
-const SKIP_DIRS = new Set([
-  'node_modules',
-  '.git',
-  '__tests__',
-  '__fixtures__',
-]);
-
-/**
- * Whether a file name is a test or fixture rather than a codemod.
- *
- * The trap this closes: EVERY `.ts`/`.mjs`/`.js` under a version folder used to
- * be loaded as a codemod, so a test file colocated with its transform failed
- * validation and — because a definition error is a hard error — took every
- * codemod in that package's version with it. `astryx upgrade` then applied no
- * transforms and reported success, which is the worst shape a failure can take.
- *
- * Core's own codemods never hit this: they are enumerated in a registry, and
- * their colocated tests are simply not in it. Only integrations are discovered
- * by walking a directory, so only integrations carry the landmine — which is why
- * this is a loader fix and not a documentation one. It protects the integrations
- * that already exist, which no authoring tool can reach.
- *
- * @param {string} name a file's base name
- * @returns {boolean}
- */
-function isTestFile(name) {
-  return /\.(test|spec)\.[^.]+$/.test(name) || /\.fixture\.[^.]+$/.test(name);
-}
-
 /**
  * Recursively collect codemod module files under a version folder. Returns
  * entries of {id, file} where id is the extension-less relative path
@@ -83,18 +49,14 @@ function collectCodemodFiles(versionDir) {
     for (const entry of entries) {
       const full = path.join(dir, entry.name);
       if (entry.isDirectory()) {
-        if (SKIP_DIRS.has(entry.name)) continue;
+        if (entry.name === 'node_modules' || entry.name === '.git') continue;
         walk(full);
         continue;
       }
       const ext = path.extname(entry.name);
       if (!CODEMOD_EXTENSIONS.includes(ext)) continue;
-      if (isTestFile(entry.name)) continue;
       const rel = path.relative(versionDir, full);
-      const id = rel
-        .slice(0, rel.length - ext.length)
-        .split(path.sep)
-        .join('/');
+      const id = rel.slice(0, rel.length - ext.length).split(path.sep).join('/');
       out.push({id, file: full});
     }
   }

package/assets/codemods/integration-discovery.test.mjs

@@ -221,61 +221,3 @@ describe('integration codemod discovery', () => {
     ).rejects.toThrow(/across versions/i);
   });
 });
-
-describe('test files beside a codemod', () => {
-  // The incident this closes: every .ts/.mjs/.js under a version folder was
-  // loaded AND VALIDATED as a codemod, so a test file colocated with its
-  // transform failed validation — and because a definition error is a hard
-  // error, it took every codemod in that version with it. `astryx upgrade`
-  // then applied nothing and reported success. Core is immune because its own
-  // codemods are enumerated in a registry rather than discovered by walking a
-  // directory, so its colocated tests are simply never visited. Only
-  // integrations carry the landmine.
-  const TRANSFORM = `
-    export default {
-      type: 'code',
-      title: 'Drop foo',
-      transform: (file) => file.source.replace(/foo/g, 'bar'),
-    };
-  `;
-  // Not a codemod: no default export of the right shape. Loading it throws.
-  const A_TEST = `
-    import {describe, it, expect} from 'vitest';
-    describe('drop-foo', () => {
-      it('drops foo', () => expect(1).toBe(1));
-    });
-  `;
-
-  it.each([
-    ['a .test. sibling', '0.2.0/drop-foo.test.mjs'],
-    ['a .spec. sibling', '0.2.0/drop-foo.spec.mjs'],
-    ['a fixture sibling', '0.2.0/drop-foo.fixture.mjs'],
-    ['a __tests__ directory', '0.2.0/__tests__/drop-foo.mjs'],
-    ['a __fixtures__ directory', '0.2.0/__fixtures__/input.mjs'],
-  ])('ignores %s and still discovers the codemod', async (_label, testPath) => {
-    scaffold({'0.2.0/drop-foo.mjs': TRANSFORM, [testPath]: A_TEST});
-
-    const project = await Project.load(tmpDir);
-    const byVersion = await discoverIntegrationCodemods(
-      project.loadedIntegrations,
-    );
-
-    expect([...byVersion.keys()]).toEqual(['0.2.0']);
-    expect(byVersion.get('0.2.0').map(entry => entry.id)).toEqual(['drop-foo']);
-  });
-
-  it('a nested helper directory is still walked', async () => {
-    // Only test and fixture names are skipped. A package that organises its
-    // transforms into subdirectories keeps working.
-    scaffold({'0.2.0/imports/drop-foo.mjs': TRANSFORM});
-
-    const project = await Project.load(tmpDir);
-    const byVersion = await discoverIntegrationCodemods(
-      project.loadedIntegrations,
-    );
-
-    expect(byVersion.get('0.2.0').map(entry => entry.id)).toEqual([
-      'imports/drop-foo',
-    ]);
-  });
-});

package/assets/codemods/transforms/v0.3.0/__tests__/unwrap-authoring-factories.test.mjs

@@ -3,8 +3,7 @@
 import {describe, it, expect} from 'vitest';
 
 async function applyTransform(source, path = 'test.ts') {
-  const {default: transform} =
-    await import('../unwrap-authoring-factories.mjs');
+  const {default: transform} = await import('../unwrap-authoring-factories.mjs');
   const jscodeshift = (await import('jscodeshift')).default;
   const j = jscodeshift.withParser('tsx');
   const api = {jscodeshift: j, stats: () => {}, report: () => {}};
@@ -19,7 +18,7 @@ export default createConfig({integrations: ['@acme/widgets']});
 `;
     const output = await applyTransform(input);
     expect(output).not.toContain('createConfig');
-    expect(output).toContain('export default {');
+    expect(output).toContain("export default {");
     expect(output).toContain("integrations: ['@acme/widgets']");
     expect(output).not.toContain('type:');
   });
@@ -133,34 +132,13 @@ export default createComponentDoc();
     expect(output).toContain("type: 'component'");
   });
 
-  it('leaves same-named factories from unrelated packages unchanged', async () => {
-    const input = `import {createConfig} from '@acme/eslint';
-export default createConfig({strict: true});
-`;
-    expect(await applyTransform(input)).toBe(input);
-  });
-
-  it('keeps an unrelated same-named import when another factory is migrated', async () => {
-    const input = `import {createConfig as createLintConfig} from '@acme/eslint';
-import {createDoc} from '@astryxdesign/cli/doc';
-export const lintConfig = createLintConfig({strict: true});
-export const doc = createDoc({name: 'Theming', description: 'How theming works.'});
-`;
-    const output = await applyTransform(input);
-    expect(output).toContain(
-      "import {createConfig as createLintConfig} from '@acme/eslint'",
-    );
-    expect(output).toContain('createLintConfig({strict: true})');
-    expect(output).not.toContain('createDoc');
-    expect(output).toContain("type: 'generic'");
-  });
-
   it('is a no-op when no authoring factory is imported', async () => {
     const input = `import {Button} from '@astryxdesign/core';
 export default Button;
 `;
-    const {default: transform} =
-      await import('../unwrap-authoring-factories.mjs');
+    const {default: transform} = await import(
+      '../unwrap-authoring-factories.mjs'
+    );
     const jscodeshift = (await import('jscodeshift')).default;
     const j = jscodeshift.withParser('tsx');
     const api = {jscodeshift: j, stats: () => {}, report: () => {}};