@sps-woodland/codemods 8.53.8 → 8.53.9

package/migrate-wrapper-imports.mjs

@@ -10,9 +10,12 @@
 //
 // AST-based find-and-replace (via jscodeshift): rewrite imports of
 // @sps-woodland/<wrapper> packages (thin re-export shims) to import
-// directly from the @sps-woodland/core subpath they wrap. Does NOT touch
-// anything else: no @spscommerce/ds-react, no imported-name changes, no
-// JSX, no package.json/lockfile edits.
+// directly from the @sps-woodland/core subpath they wrap. Also rewrites
+// the matching CSS/SCSS/LESS "@sps-woodland/<wrapper>/style.css" (or
+// "/lib/style.css") @import paths, since those are just as real a wrapper
+// dependency as the JS imports and are otherwise invisible to a JS-only
+// codemod. Does NOT touch anything else: no @spscommerce/ds-react, no
+// imported-name changes, no JSX, no package.json/lockfile edits.
 //
 // This is intentionally NOT part of the main jscodeshift transform
 // pipeline (main.js / src/transform.ts) — that pipeline migrates
@@ -79,7 +82,8 @@ const WRAPPER_TO_CORE = {
 // @sps-woodland/codemods (these ship real, independent code, not re-exports).
 
 const SCOPE = "@sps-woodland";
-const EXTENSIONS = new Set([".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs"]);
+const JS_EXTENSIONS = new Set([".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs"]);
+const CSS_EXTENSIONS = new Set([".css", ".scss", ".sass", ".less"]);
 const SKIP_DIRS = new Set(["node_modules", ".git", "dist", "build", "lib", "coverage"]);
 
 function resolveNewSource(oldSource) {
@@ -159,6 +163,38 @@ function transformSource(source) {
   };
 }
 
+// Matches: @import [url(] ("|') @sps-woodland/<name><rest> (matching quote)
+// Only the two CSS subpath forms wrapper packages actually ship
+// ("/style.css" and "/lib/style.css") are rewritten — anything else for a
+// known package is left alone and flagged, since guessing wrong here fails
+// silently (no compiler to catch a bad CSS path).
+const CSS_IMPORT_RE =
+  /(@import\s+(?:url\(\s*)?)(["'])@sps-woodland\/([a-z-]+)((?:\/[^"']*)?)\2/g;
+
+function transformCssSource(source) {
+  const changes = []; // { oldSource, newSource }
+  const unknown = new Set();
+
+  const output = source.replace(CSS_IMPORT_RE, (match, prefix, quote, name, rest) => {
+    const mapping = WRAPPER_TO_CORE[name];
+    if (!mapping) {
+      unknown.add(name);
+      return match;
+    }
+    if (rest !== "/style.css" && rest !== "/lib/style.css") {
+      unknown.add(`${name}${rest}`);
+      return match;
+    }
+
+    const oldSource = `${SCOPE}/${name}${rest}`;
+    const newSource = `${SCOPE}/core/${mapping.cssSubpath}/style.css`;
+    changes.push({ oldSource, newSource });
+    return `${prefix}${quote}${newSource}${quote}`;
+  });
+
+  return { changes, unknown, output };
+}
+
 async function* walk(dir) {
   const entries = await readdir(dir, { withFileTypes: true });
   for (const entry of entries) {
@@ -166,7 +202,7 @@ async function* walk(dir) {
     const full = path.join(dir, entry.name);
     if (entry.isDirectory()) {
       yield* walk(full);
-    } else if (EXTENSIONS.has(path.extname(entry.name))) {
+    } else if (JS_EXTENSIONS.has(path.extname(entry.name)) || CSS_EXTENSIONS.has(path.extname(entry.name))) {
       yield full;
     }
   }
@@ -184,9 +220,10 @@ async function main() {
 
   for await (const file of walk(srcDir)) {
     const original = await readFile(file, "utf8");
+    const isCss = CSS_EXTENSIONS.has(path.extname(file));
     let result;
     try {
-      result = transformSource(original);
+      result = isCss ? transformCssSource(original) : transformSource(original);
     } catch (err) {
       console.error(`skipped (parse error): ${file} — ${err.message}`);
       continue;
@@ -215,8 +252,10 @@ async function main() {
   console.log(`${filesChanged} file(s), ${totalReplacements} import(s) ${write ? "rewritten" : "would be rewritten"}.`);
   if (unknownPackagesSeen.size > 0) {
     console.log(
-      `Note: found imports from @sps-woodland/${[...unknownPackagesSeen].join(", @sps-woodland/")} — ` +
-        `not in the wrapper map, left untouched. Add them to WRAPPER_TO_CORE if they're also re-export shims.`,
+      `Note: found @sps-woodland/${[...unknownPackagesSeen].join(", @sps-woodland/")} — ` +
+        `either not a wrapper package (e.g. core/tokens/illustrations, left untouched on purpose) ` +
+        `or a subpath this script doesn't recognize. Review manually, or add it to WRAPPER_TO_CORE ` +
+        `if it's a wrapper re-export this script should also handle.`,
     );
   }
   if (!write) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sps-woodland/codemods",
   "description": "SPS Woodland Design System codemods",
-  "version": "8.53.8",
+  "version": "8.53.9",
   "author": "SPS Commerce",
   "license": "UNLICENSED",
   "repository": "https://github.com/spscommerce/woodland/tree/master/packages/@sps-woodland/codemods",