@xstate/codemod 0.1.0-beta.1 → 0.1.0-beta.3

package/README.md

@@ -9,3 +9,5 @@ The transforms rename imports, wrap string targets, convert inline `types` decla
 
 <!-- helper binding and output validation from src/util.ts, src/transforms/types-to-schemas.ts and src/runner.ts -->
 The codemod supports named, namespace and type-only imports. It checks transformed syntax before saving. Run your application's typecheck and tests after migration, and review any reported manual migration steps.
+
+The `string-targets` transform also wraps inline `xstate/fsm` targets in `on` and `always`, including `setup(...).createFSM` and aliased `createFSM` imports. It preserves string `initial` values. Review separately stored configurations and string-returning transition functions manually.

package/dist/declarations/src/transforms/string-targets.d.ts

@@ -1,7 +1,7 @@
 import type { Transform } from "../types.js";
 /**
  * Converts bare string transition values inside `createMachine`/`setup().
- * createMachine`/`createStateConfig` config objects into `{ target: '...' }`.
+ * createMachine`/`createStateConfig`/`createFSM` config objects into `{ target: '...' }`.
  *
  * Does NOT touch `initial:` (still a string in v6) or existing `target:`
  * values.

package/dist/xstate-codemod.js

@@ -165,7 +165,7 @@ const DIRECT_TRANSITION_KEYS = new Set(['onDone', 'onError', 'always']);
  * Returns true if a change was made.
  */
 function wrapValue(node) {
-  if (Node.isStringLiteral(node)) {
+  if (Node.isStringLiteral(node) || Node.isNoSubstitutionTemplateLiteral(node)) {
     const raw = node.getText(); // includes quotes
     node.replaceWithText(`{ target: ${raw} }`);
     return true;
@@ -175,7 +175,7 @@ function wrapValue(node) {
     // Iterate over a snapshot; replacing elements is index-stable here since
     // we replace text in place without adding/removing elements.
     for (const el of node.getElements()) {
-      if (Node.isStringLiteral(el)) {
+      if (Node.isStringLiteral(el) || Node.isNoSubstitutionTemplateLiteral(el)) {
         el.replaceWithText(`{ target: ${el.getText()} }`);
         changed = true;
       }
@@ -267,18 +267,29 @@ function getKey$1(prop) {
 
 /**
  * Converts bare string transition values inside `createMachine`/`setup().
- * createMachine`/`createStateConfig` config objects into `{ target: '...' }`.
+ * createMachine`/`createStateConfig`/`createFSM` config objects into `{ target: '...' }`.
  *
  * Does NOT touch `initial:` (still a string in v6) or existing `target:`
  * values.
  */
 const stringTargets = {
   name: 'string-targets',
-  description: "Wrap bare string transition values (on/after/onDone/onError/always) into { target: '...' } inside machine config objects. Best-effort: matches createMachine/createStateConfig by call name.",
+  description: "Wrap bare string transition values (on/after/onDone/onError/always) into { target: '...' } inside machine config objects. Best-effort: matches createMachine/createStateConfig/createFSM by call name.",
   apply(sourceFile) {
     const notes = [];
     let total = 0;
-    for (const configObj of findMachineConfigObjects(sourceFile)) {
+    const callNames = ['createMachine', 'createStateConfig', 'createFSM'];
+    for (const declaration of sourceFile.getImportDeclarations()) {
+      if (declaration.getModuleSpecifierValue() !== 'xstate/fsm') {
+        continue;
+      }
+      for (const specifier of declaration.getNamedImports()) {
+        if (specifier.getName() === 'createFSM') {
+          callNames.push(specifier.getAliasNode()?.getText() ?? 'createFSM');
+        }
+      }
+    }
+    for (const configObj of findMachineConfigObjects(sourceFile, callNames)) {
       total += walkStateNode(configObj, notes);
     }
     return {
@@ -3567,7 +3578,7 @@ function toJson(value, stats, seen = new Set(), maxDepth = Infinity) {
   if (value === null) return null;
   const obj = value;
   if (isActorRef(obj)) return {
-    $actor: normalizeId(obj.id)
+    '@actor': normalizeId(obj.id)
   };
   if (obj instanceof Error) return String(obj);
   if (typeof obj.toJSON === 'function') return toJson(obj.toJSON(), stats, seen, maxDepth);
@@ -3667,7 +3678,7 @@ function mapDeep(value, fn) {
   return value;
 }
 const stripAnonSuffix = value => mapDeep(value, (key, item) => (key === 'id' || key === 'to') && typeof item === 'string' ? item.replace(/(:\d+|\[\d+\])$/, '').replace(/^(x:\d+|\d+\.)/, '') : item);
-const isActorMarker = item => !!item && typeof item === 'object' && '$actor' in item;
+const isActorMarker = item => !!item && typeof item === 'object' && '@actor' in item;
 const stripErrorPrefix = message => message.replace(/^(\w*Error:\s*)+/, '').trim();
 const INTENTIONAL_DELTAS = [{
   rule: 'anonymous-child-id',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xstate/codemod",
-  "version": "0.1.0-beta.1",
+  "version": "0.1.0-beta.3",
   "description": "Codemods for migrating XState source between versions (v5 → v6)",
   "keywords": [
     "xstate",
@@ -36,7 +36,7 @@
     "ts-morph": "^24.0.0",
     "tsx": "^4.23.13",
     "xstate5": "npm:xstate@^5.33.0",
-    "xstate": "^6.0.0-beta.1"
+    "xstate": "^6.0.0-beta.3"
   },
   "devDependencies": {
     "vitest": "^3.2.2"

package/skills/migrate-v5-to-v6/SKILL.md

@@ -50,3 +50,5 @@ Rules:
 - Never widen `--compare` to make a diff disappear.
 - Report every `unstubbed` entry to the user. Inline actor logic runs for real and is not stubbed.
 - Stop and ask the user after 5 repair rounds without exit 0.
+
+The `string-targets` transform also wraps inline `xstate/fsm` targets in `on` and `always`, including `setup(...).createFSM` and aliased `createFSM` imports. It preserves string `initial` values. Review separately stored configurations and string-returning transition functions manually.