@arcgis/codemod 5.2.0-next.117 → 5.2.0-next.119

package/README.md

@@ -12,11 +12,11 @@ and updating geometry operator usage.
 
 - Node.js 24
 - A project using the ArcGIS Maps SDK for JavaScript
-- This codemod operates exclusively on TypeScript files. Do not need to run this tool, if the project does not use TypeScript.
+- This codemod operates exclusively on TypeScript files. You do not need to run this tool if the project does not use TypeScript.
 
 ## Before you begin
 
-Ensure your project's dependencies are up to date, for example with `pnpm install`.
+Ensure your project's dependencies are up to date, for example by running `pnpm install`.
 
 If your project uses **ESLint, Prettier, or other formatters/linters**, run them after the codemod to ensure consistent formatting.
 
@@ -74,7 +74,7 @@ The codemod will emit warnings for cases it cannot safely rewrite, including:
 - Complex computed or dynamic type usage
 - Unresolvable symbols
 
-These will be reported in the console, manually review all changes before committing.
+These will be reported in the console. Manually review all changes before committing.
 
 ### Normalize Calcite icon names
 
@@ -104,9 +104,9 @@ npx @arcgis/codemod run normalize-calcite-icon-names [<target-directory>]
 
 ### Rename Geometry Operators
 
-At 5.2, geometry operator functions were renamed to improve consistency and discoverability.
+In version 5.2, geometry operator exports and option types were renamed to improve consistency and discoverability.
 The `geometry-operator-renames` codemod updates geometry operator calls to use the new names.
-It additionally inlines namespace imports, which are now unnecessary.
+It additionally inlines namespace imports, which are no longer necessary.
 
 Run it in the root of your project or with an explicit target directory:
 
@@ -116,7 +116,7 @@ npx @arcgis/codemod run geometry-operator-renames [<target-directory>]
 
 **Before:**
 
-````ts
+```ts
 import * as bufferOperator from "@arcgis/core/geometry/operators/bufferOperator";
 const options: bufferOperator.Options = { unit: "meters" };
 const buffered = bufferOperator.execute(polyline, 250, options);
@@ -128,10 +128,34 @@ const buffered = bufferOperator.execute(polyline, 250, options);
 import { bufferExecute, type BufferExecuteOptions } from "@arcgis/core/geometry/operators/bufferOperator";
 const options: BufferExecuteOptions = { unit: "meters" };
 const buffered = bufferExecute(polyline, 250, options);
-````
+```
+
+## Versioning
+
+The codemods are available until the corresponding deprecated feature is removed.
+If migrating from a much older release, the codemod might no longer be available in the most recent version of the `@arcgis/codemod` package.
+In such cases, run the codemod from an older version, matching the version in the release notes:
+
+```sh
+npx @arcgis/codemod@5.2 run refactor-out-esri-namespace
+```
+
+## Troubleshooting
+
+Some codemods rely on type information to perform more accurate migrations.
+On large codebases, this may require more RAM than the default Node.js limit.
+If you encounter an out-of-memory error (like `Allocation failed - JavaScript heap out of memory`), re-run the codemod with a higher RAM limit:
+
+Example on Linux/macOS:
+
+```sh
+NODE_OPTIONS="--max-old-space-size=10240" npx @arcgis/codemod run normalize-calcite-icon-names
+```
+
+[Windows command line](https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/set_1) and [PowerShell](https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_environment_variables?view=powershell-7.6) use a different environment variable syntax.
 
 ## License
 
 This package is licensed under the terms described in the `LICENSE.md` file,
 located in the root of the package, and at https://js.arcgis.com/5.2/LICENSE.txt.
-For third party notices, see https://js.arcgis.com/5.2/third-party-notices.txt.
+For third-party notices, see https://js.arcgis.com/5.2/third-party-notices.txt.

package/dist/{actions-CQZGPkk1.js → actions-pyK_bLYZ.js}

@@ -1,36 +1,31 @@
-import { path as O } from "@arcgis/node-toolkit/path";
+import { path as x } from "@arcgis/node-toolkit/path";
 import { SyntaxKind as I } from "ts-morph";
-import { g as D } from "./cli-BLMaWpqf.js";
-import { ModuleResolutionKind as P } from "typescript";
-import { capitalize as S } from "@arcgis/toolkit/string";
-import { log as j } from "@arcgis/toolkit/log";
-function K(t) {
-  const e = t.typeScriptConfig?.compilerOptions?.moduleResolution ?? P.Bundler;
-  return e >= P.Node16 && e <= P.NodeNext;
-}
-function A(t) {
-  return q(t.getFilePath());
-}
-function q(t) {
-  return O.relative(process.cwd(), t);
+import { g as D } from "./cli-C-HhqWfY.js";
+import { ModuleResolutionKind as K } from "typescript";
+import { a as w } from "./log-B76HBvov.js";
+import { capitalize as A } from "@arcgis/toolkit/string";
+import { log as P } from "@arcgis/toolkit/log";
+function j(t) {
+  const e = t.typeScriptConfig?.compilerOptions?.moduleResolution ?? K.Bundler;
+  return e >= K.Node16 && e <= K.NodeNext;
 }
 function V(t, e) {
   const s = {};
-  b(t);
-  const { specifiers: i, names: m, lastImportIndex: r } = C(t), l = [], n = {}, c = [];
-  let p = K(D.project.projectData);
-  for (const [o, d] of Object.entries(e)) {
+  q(t);
+  const { specifiers: i, names: m, lastImportIndex: r } = b(t), l = [], n = {}, c = [];
+  let p = j(D.project.projectData);
+  for (const [o, f] of Object.entries(e)) {
     let a = i.get(o) ?? (o.startsWith(".") ? i.get(
-      O.relative(t.getDirectoryPath(), O.resolve(D.project.projectData.cwd, o))
+      x.relative(t.getDirectoryPath(), x.resolve(D.project.projectData.cwd, o))
     ) : void 0);
     if (a === void 0) {
       const y = i.get(`${o}.js`);
       y !== void 0 && (p = !0, a === void 0 ? a = y : a = [...a, ...y]);
     }
-    for (const [y, g] of Object.entries(d)) {
+    for (const [y, g] of Object.entries(f)) {
       const v = Array.isArray(g) ? g : [g];
       for (const N of v)
-        N.do === "deprecate" ? R(y, o, s, a) : N.do === "add" ? E(o, y, N, a, l, n, m) : N.do === "rename" ? W(o, y, N, a, l, n, m) : N.do === "remove-namespace-import" && $(
+        N.do === "deprecate" ? R(y, o, s, a) : N.do === "add" ? C(o, y, N, a, l, n, m) : N.do === "rename" ? E(o, y, N, a, l, n, m) : N.do === "remove-namespace-import" && W(
           o,
           a,
           l,
@@ -40,12 +35,12 @@ function V(t, e) {
         );
     }
   }
-  const f = Object.entries(n);
+  const d = Object.entries(n);
   let u = r;
-  for (let o = 0; o < f.length; o += 1) {
-    const [d, a] = f[o], y = d.startsWith(".") ? U(
-      O.relative(t.getDirectoryPath(), O.resolve(D.project.projectData.cwd, d))
-    ) : d, g = p && y.startsWith("@arcgis/core") ? `${y}.js` : y, v = a.afterDeclaration === void 0 ? u : a.afterDeclaration.getChildIndex() + 1;
+  for (let o = 0; o < d.length; o += 1) {
+    const [f, a] = d[o], y = f.startsWith(".") ? M(
+      x.relative(t.getDirectoryPath(), x.resolve(D.project.projectData.cwd, f))
+    ) : f, g = p && y.startsWith("@arcgis/core") ? `${y}.js` : y, v = a.afterDeclaration === void 0 ? u : a.afterDeclaration.getChildIndex() + 1;
     if (a.named.length > 0) {
       let N = !0;
       for (let h = 0; h < a.named.length; h += 1)
@@ -71,18 +66,18 @@ function V(t, e) {
     }), u += 1);
   }
   if (Object.keys(s).length > 0) {
-    const o = Object.entries(s).map(([d, a]) => `${a.join(", ")} from ${d}`).join("; ");
-    j("warn", A(t), `Deprecated API usages found: ${o}`);
+    const o = Object.entries(s).map(([f, a]) => `${a.join(", ")} from ${f}`).join("; ");
+    P("warn", w(t), `Deprecated API usages found: ${o}`);
   }
   for (const o of c)
     o.remove();
   return l;
 }
-function b(t) {
+function q(t) {
   const e = t.getChildSyntaxList()?.getFirstChild();
-  if (w === e)
+  if (S === e)
     return;
-  w = e, x.clear();
+  S = e, O.clear();
   const s = /* @__PURE__ */ new Set();
   i(t);
   function i(m) {
@@ -99,11 +94,11 @@ function b(t) {
         if (n.isKind(I.QualifiedName) && n.getLeft() !== l || n.isKind(I.PropertyAccessExpression) && n.getExpression() !== l || (n.isKind(I.PropertyDeclaration) || n.isKind(I.PropertySignature) || n.isKind(I.PropertyAssignment)) && n.getNameNode() === l)
           return;
         const c = l.getText();
-        s.has(c) || x.add(l.getText());
+        s.has(c) || O.add(l.getText());
       });
   }
 }
-function C(t) {
+function b(t) {
   const e = /* @__PURE__ */ new Map(), s = /* @__PURE__ */ new Set(), i = t.getStatements();
   let m;
   for (let l = 0; l < i.length; l += 1) {
@@ -116,21 +111,21 @@ function C(t) {
     m = n;
     const c = n.getModuleSpecifierValue(), p = e.get(c);
     p === void 0 ? e.set(c, [n]) : p.push(n);
-    const f = n.getImportClause();
-    if (f === void 0)
+    const d = n.getImportClause();
+    if (d === void 0)
       continue;
-    const u = f.getDefaultImport()?.getText();
+    const u = d.getDefaultImport()?.getText();
     u !== void 0 && s.add(u);
-    const o = f.getNamedBindings();
+    const o = d.getNamedBindings();
     if (o !== void 0)
       if (o.isKind(I.NamedImports))
-        for (const d of o.getElements()) {
-          const a = d.getName();
+        for (const f of o.getElements()) {
+          const a = f.getName();
           s.add(a);
         }
       else {
-        const d = o.getName();
-        s.add(d);
+        const f = o.getName();
+        s.add(f);
       }
   }
   const r = m === void 0 ? 0 : m.getChildIndex() + 1;
@@ -163,17 +158,17 @@ function R(t, e, s, i) {
           }
   }
 }
-function E(t, e, s, i, m, r, l) {
+function C(t, e, s, i, m, r, l) {
   const n = e === "default";
   if (n || e === "*") {
     if (i !== void 0)
       for (const p of i) {
-        const f = n ? p.getDefaultImport() : p.getNamespaceImport();
-        if (f !== void 0) {
+        const d = n ? p.getDefaultImport() : p.getNamespaceImport();
+        if (d !== void 0) {
           s.isTypeOnly !== !0 && p.isTypeOnly() && p.setIsTypeOnly(!1), m.push({
             specifier: t,
             requestedName: e,
-            actualName: f.getText()
+            actualName: d.getText()
           });
           return;
         }
@@ -192,18 +187,18 @@ function E(t, e, s, i, m, r, l) {
     let c, p;
     if (i !== void 0) {
       let o = !1;
-      for (const d of i)
-        if (d.getNamespaceImport() === void 0) {
-          p ??= d, c ??= d.isTypeOnly() && d.getDefaultImport() === void 0 ? d : void 0;
-          for (const a of d.getNamedImports())
+      for (const f of i)
+        if (f.getNamespaceImport() === void 0) {
+          p ??= f, c ??= f.isTypeOnly() && f.getDefaultImport() === void 0 ? f : void 0;
+          for (const a of f.getNamedImports())
             if (a.getNameNode().getText() === e) {
               if (o = !0, s.isTypeOnly !== !0) {
                 if (a.isTypeOnly())
                   a.setIsTypeOnly(!1);
-                else if (d.isTypeOnly()) {
-                  for (const y of d.getNamedImports())
+                else if (f.isTypeOnly()) {
+                  for (const y of f.getNamedImports())
                     y !== a && y.setIsTypeOnly(!0);
-                  d.setIsTypeOnly(!1);
+                  f.setIsTypeOnly(!1);
                 }
               }
               m.push({
@@ -217,12 +212,12 @@ function E(t, e, s, i, m, r, l) {
       if (o)
         return;
     }
-    const f = T(
+    const d = T(
       s.preferAlias === !0 ? s.preferredAlias : e,
       s.preferAlias === !0 ? e : s.preferredAlias,
       t,
       l
-    ), u = e === f ? void 0 : f;
+    ), u = e === d ? void 0 : d;
     if (s.isTypeOnly && c !== void 0)
       c.addNamedImport({
         name: e,
@@ -250,37 +245,37 @@ function E(t, e, s, i, m, r, l) {
         alias: u,
         isTypeOnly: s.isTypeOnly === !0
       });
-    m.push({ specifier: t, requestedName: e, actualName: f });
+    m.push({ specifier: t, requestedName: e, actualName: d });
   }
 }
-function W(t, e, s, i, m, r, l) {
+function E(t, e, s, i, m, r, l) {
   if (i === void 0)
     return;
   const n = s.name ?? e, c = s.specifier ?? t, p = e === "default";
   if (p || e === "*") {
-    for (const f of i) {
-      const u = p ? f.getDefaultImport() : f.getNamespaceImport();
+    for (const d of i) {
+      const u = p ? d.getDefaultImport() : d.getNamespaceImport();
       if (u === void 0)
         continue;
-      if (s.isTypeOnly === !1 && f.isTypeOnly()) {
-        f.setIsTypeOnly(!1);
-        for (const d of f.getNamedImports())
-          d.setIsTypeOnly(!0);
+      if (s.isTypeOnly === !1 && d.isTypeOnly()) {
+        d.setIsTypeOnly(!1);
+        for (const f of d.getNamedImports())
+          f.setIsTypeOnly(!0);
       }
       let o = n;
       if (n !== e) {
-        const d = T(n, e, t, l);
-        n === d ? u.rename(n) : o = u.getText();
+        const f = T(n, e, t, l);
+        n === f ? u.rename(n) : o = u.getText();
       } else
         o = u.getText();
-      t !== c && (f.getNamedImports().length === 0 ? f.setModuleSpecifier(c) : (p ? f.removeDefaultImport() : f.removeNamespaceImport(), r[c] ??= {
+      t !== c && (d.getNamedImports().length === 0 ? d.setModuleSpecifier(c) : (p ? d.removeDefaultImport() : d.removeNamespaceImport(), r[c] ??= {
         afterDeclaration: void 0,
         named: [],
         default: void 0,
         "*": void 0
-      }, r[c].afterDeclaration ??= f, r[c][e] = {
+      }, r[c].afterDeclaration ??= d, r[c][e] = {
         name: o,
-        isTypeOnly: f.isTypeOnly()
+        isTypeOnly: d.isTypeOnly()
       })), m.push({
         specifier: c,
         requestedName: e,
@@ -290,46 +285,46 @@ function W(t, e, s, i, m, r, l) {
     }
     return;
   }
-  for (const f of i) {
-    for (const o of f.getNamedImports()) {
+  for (const d of i) {
+    for (const o of d.getNamedImports()) {
       if (o.getNameNode().getText() !== e)
         continue;
       if (s.isTypeOnly === !1) {
         if (o.isTypeOnly())
           o.setIsTypeOnly(!1);
-        else if (f.isTypeOnly()) {
-          f.setIsTypeOnly(!1);
-          for (const y of f.getNamedImports())
+        else if (d.isTypeOnly()) {
+          d.setIsTypeOnly(!1);
+          for (const y of d.getNamedImports())
             y !== o && y.setIsTypeOnly(!0);
         }
       }
-      let d = n;
+      let f = n;
       const a = o.getAliasNode();
       if (n !== e)
         if (a === void 0) {
           const y = T(n, e, t, l);
-          n === y ? o.getNameNode().asKind(I.Identifier)?.rename(n) : (o.setAlias(e), o.getNameNode().replaceWithText(n), d = e);
+          n === y ? o.getNameNode().asKind(I.Identifier)?.rename(n) : (o.setAlias(e), o.getNameNode().replaceWithText(n), f = e);
         } else
-          o.getNameNode().replaceWithText(n), d = a.getText();
+          o.getNameNode().replaceWithText(n), f = a.getText();
       if (c !== t) {
-        const g = (K(D.project.projectData) || f.getModuleSpecifierValue().endsWith(".js")) && c.startsWith("@arcgis/core") ? `${c}.js` : c;
-        f.getNamedImports().length === 1 && f.getDefaultImport() === void 0 ? f.setModuleSpecifier(g) : (r[g] ??= { named: [] }, r[g].afterDeclaration ??= i?.[0], r[g].named.push({
+        const g = (j(D.project.projectData) || d.getModuleSpecifierValue().endsWith(".js")) && c.startsWith("@arcgis/core") ? `${c}.js` : c;
+        d.getNamedImports().length === 1 && d.getDefaultImport() === void 0 ? d.setModuleSpecifier(g) : (r[g] ??= { named: [] }, r[g].afterDeclaration ??= i?.[0], r[g].named.push({
           name: n,
-          alias: n === d ? void 0 : d,
+          alias: n === f ? void 0 : f,
           isTypeOnly: o.isTypeOnly()
         }), o.remove());
       }
       m.push({
         specifier: c,
         requestedName: n,
-        actualName: a?.getText() ?? d
+        actualName: a?.getText() ?? f
       });
       return;
     }
-    const u = f.getNamespaceImport();
+    const u = d.getNamespaceImport();
     if (u)
       for (const o of u.findReferencesAsNodes().slice(1)) {
-        const d = o.getParentIfKind(I.PropertyAccessExpression), a = o.getParentIfKind(I.QualifiedName)?.getParentIfKind(I.TypeReference), y = d ?? a;
+        const f = o.getParentIfKind(I.PropertyAccessExpression), a = o.getParentIfKind(I.QualifiedName)?.getParentIfKind(I.TypeReference), y = f ?? a;
         if (!y)
           continue;
         const g = y.getText();
@@ -337,7 +332,7 @@ function W(t, e, s, i, m, r, l) {
       }
   }
 }
-function $(t, e, s, i, m, r) {
+function W(t, e, s, i, m, r) {
   if (e = e?.filter((u) => u.getNamespaceImport() != null), e === void 0 || e.length === 0)
     return;
   if (e.length !== 1)
@@ -346,21 +341,21 @@ function $(t, e, s, i, m, r) {
     );
   const [l] = e, n = l.getNamespaceImport(), c = {}, p = /* @__PURE__ */ new Set();
   for (const u of n.findReferencesAsNodes()) {
-    const o = u.getParentIfKind(I.PropertyAccessExpression), d = u.getParentIfKind(I.QualifiedName)?.getParentIfKind(I.TypeReference), a = o ?? d;
+    const o = u.getParentIfKind(I.PropertyAccessExpression), f = u.getParentIfKind(I.QualifiedName)?.getParentIfKind(I.TypeReference), a = o ?? f;
     if (!a)
       continue;
     const y = a.getText().split(".").pop().trim();
     c[y] ??= T(y, void 0, t, r);
     const g = c[y];
-    a.replaceWithText(g), d || p.add(g);
+    a.replaceWithText(g), f || p.add(g);
   }
   i[t] ??= {
     afterDeclaration: l,
     named: []
   };
-  const { named: f } = i[t];
+  const { named: d } = i[t];
   for (const [u, o] of Object.entries(c))
-    f.push({
+    d.push({
       name: u,
       alias: o === u ? void 0 : o,
       isTypeOnly: !p.has(o)
@@ -373,7 +368,7 @@ function $(t, e, s, i, m, r) {
 }
 function T(t, e, s, i) {
   if (t !== void 0) {
-    if (!i.has(t) && !x.has(t))
+    if (!i.has(t) && !O.has(t))
       return i.add(t), t;
     if (e !== void 0)
       return T(e, void 0, s, i);
@@ -382,21 +377,19 @@ function T(t, e, s, i) {
   const m = s.split("/");
   let r = t ?? "";
   for (let l = m.length - 1; l >= 0; l--) {
-    const n = m[l].replaceAll(M, "");
-    if (n !== r && (r = n + S(r), !i.has(r) && !x.has(r)))
+    const n = m[l].replaceAll($, "");
+    if (n !== r && (r = n + A(r), !i.has(r) && !O.has(r)))
       return i.add(r), r;
   }
   for (let l = 2; ; l++) {
     const n = r + l;
-    if (!i.has(n) && !x.has(n))
+    if (!i.has(n) && !O.has(n))
       return i.add(n), n;
   }
 }
-const M = /[^a-zA-Z0-9$_]/gu, U = (t) => t.startsWith(".") ? t : `./${t}`, x = /* @__PURE__ */ new Set();
-let w;
+const $ = /[^a-zA-Z0-9$_]/gu, M = (t) => t.startsWith(".") ? t : `./${t}`, O = /* @__PURE__ */ new Set();
+let S;
 export {
-  A as a,
-  q as f,
   V as p,
-  b as u
+  q as u
 };