@cratis/components.migrator 4.14.1 → 4.14.3

package/README.md

@@ -181,27 +181,16 @@ The transform never guesses through JSX spreads, dynamic legacy values, duplicat
 
 ## `change-handler`
 
-`cratis-components-change-handler` resolves affected Components-owned JSX identifiers imported with aliases or namespaces from `@cratis/components/Dropdown`, `@cratis/components/CommandForm`, and `@cratis/components/CommandForm/fields`. It rewrites only a structurally-proven single forwarding callback:
+`cratis-components-change-handler` resolves Components-owned JSX identifiers imported with aliases or namespaces from `@cratis/components/Dropdown`, `@cratis/components/CommandForm`, and `@cratis/components/CommandForm/fields`. It rewrites structurally-proven single forwarding callbacks on standalone `Dropdown` and CommandForm field `onValueChange` props:
 
 ```tsx
-// Safe rewrites
+// Before
 <Dropdown onChange={(event) => setRole(event.value)} />
-<InputTextField onChange={(event) => setName(event.target.value)} />
-<CheckboxField onChange={({ target: { checked } }) => setEnabled(checked)} />
-
-// become
+// After
 <Dropdown onChange={(value) => setRole(value)} />
-<InputTextField onChange={(value) => setName(value)} />
-<CheckboxField onChange={(value) => setEnabled(value)} />
 ```
 
-The affected CommandForm fields cover legacy `target`/`currentTarget` `value`, `checked`, and `valueAsNumber` payloads, plus `DropdownField`/`MultiSelectField` `event.value`. Already-semantic callbacks and callback references such as `onChange={setValue}` are unchanged.
-
-The transform changes a callback's shape, not where it lives. With Arc 22.16.0, `CommandForm` calls an
-`onChange` placed on a field with the new value at runtime, but the Components 4 CommandForm field
-types do not declare `onChange`, so type-checked TSX reports TS2322 on a field that keeps one. Move
-those side effects to `CommandForm`'s `onFieldChange` callback. The standalone `Dropdown` declares
-`onChange` and is not affected. The [Components 3 to 4 migration guide](https://cratis.io/components/migration/3-to-4/) covers this case.
+For Components 4 CommandForm fields, the codemod leaves `onChange` unchanged and reports an actionable diagnostic instead of changing its callback. Arc's `asCommandFormField` wrappers do not expose `onChange` as a field prop in Components 4, so keeping the attribute would produce TS2322 even if the callback were rewritten. Move its side effects to the enclosing `CommandForm`'s `onFieldChange(command, fieldName, oldValue, newValue, validationInfo?)` callback. Legacy `onValueChange` field callbacks with a proven event payload are rewritten to semantic values; already-semantic `onValueChange` callbacks are untouched. The [Components 3 to 4 migration guide](https://cratis.io/components/migration/3-to-4/) covers this case.
 
 Multi-statement, multi-use, wrong-payload, destructuring-with-default/rest, and native-event-dependent callbacks are refused, reported, and annotated once:
 

package/compat-manifest.json

@@ -68,7 +68,7 @@
         {
             "name": "@cratis/components",
             "role": "core",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
@@ -92,7 +92,7 @@
         {
             "name": "@cratis/eslint-plugin-components",
             "role": "eslint",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
@@ -105,7 +105,7 @@
         {
             "name": "@cratis/components.migrator",
             "role": "migrator",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
@@ -116,13 +116,13 @@
         {
             "name": "@cratis/components.conformance",
             "role": "conformance",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
             "packageAccess": "public",
             "peerDependencies": {
-                "@cratis/components": ">=4 <5",
+                "@cratis/components": "4.14.3",
                 "react": "^19.0.0",
                 "react-dom": "^19.0.0"
             },
@@ -132,13 +132,13 @@
         {
             "name": "@cratis/components.mui",
             "role": "renderer-adapter",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
             "packageAccess": "public",
             "peerDependencies": {
-                "@cratis/components": ">=4 <5",
+                "@cratis/components": "4.14.3",
                 "@emotion/react": ">=11.5 <12",
                 "@emotion/styled": ">=11.11 <12",
                 "@mui/material": ">=9 <10",
@@ -188,13 +188,13 @@
         {
             "name": "@cratis/components.primereact",
             "role": "renderer-adapter",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
             "packageAccess": "public",
             "peerDependencies": {
-                "@cratis/components": ">=4 <5",
+                "@cratis/components": "4.14.3",
                 "@primereact/core": ">=11 <12",
                 "@primereact/ui": ">=11 <12",
                 "@primeuix/themes": ">=3 <4",
@@ -256,13 +256,13 @@
         {
             "name": "@cratis/components.primereact10",
             "role": "renderer-adapter",
-            "version": "4.0.0",
+            "version": "4.14.3",
             "releaseMajorRange": ">=4 <5",
             "independentRelease": false,
             "private": false,
             "packageAccess": "public",
             "peerDependencies": {
-                "@cratis/components": ">=4 <5",
+                "@cratis/components": "4.14.3",
                 "primereact": ">=10.9.9 <11",
                 "react": "^19.0.0",
                 "react-dom": "^19.0.0"

package/lib/changeHandlerTransform.js

@@ -96,6 +96,14 @@ export function transformChangeHandlers(fileName, text, options = {}) {
             if (!ts.isJsxAttribute(attribute)) continue;
             const propName = attribute.name.getText(sourceFile);
             if (propName !== 'onChange' && propName !== 'onValueChange') continue;
+            if (target.group === 'CommandForm' && propName === 'onChange') {
+                report(
+                    attribute,
+                    target.component,
+                    "Components 4 field props do not accept onChange; move the handler to the enclosing CommandForm's onFieldChange(command, fieldName, oldValue, newValue, validationInfo?)",
+                );
+                continue;
+            }
             if (
                 !attribute.initializer ||
                 !ts.isJsxExpression(attribute.initializer) ||

package/lib/compatibility.js

@@ -98,10 +98,16 @@ export function validateBundledManifest(manifest, migratorVersion) {
     const packageEntry = manifest.packages?.find(
         ({ name }) => name === '@cratis/components.migrator',
     );
+    if (packageEntry?.version !== migratorVersion) {
+        throw new Error(
+            `Bundled compatibility manifest pins @cratis/components.migrator at ${packageEntry?.version ?? '<missing>'} but this package is ${migratorVersion}. ` +
+                'This is a packaging defect in this migrator release, not a problem with your project. ' +
+                'Report it at https://github.com/Cratis/Components/issues.',
+        );
+    }
     if (
-        packageEntry?.version !== migratorVersion ||
-        packageEntry?.independentRelease !== false ||
-        !semver.satisfies(migratorVersion, packageEntry?.releaseMajorRange ?? '')
+        packageEntry.independentRelease !== false ||
+        !semver.satisfies(migratorVersion, packageEntry.releaseMajorRange ?? '')
     ) {
         throw new Error(
             'Bundled compatibility manifest has stale migrator package metadata.',

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@cratis/components.migrator",
-    "version": "4.14.1",
+    "version": "4.14.3",
     "description": "Migration tooling for updating @cratis/components consumers from Components 3 to Components 4.",
     "author": "Cratis",
     "license": "MIT",