@codefast/cli 0.8.1 → 0.9.0

package/dist/pack-slim/domain/transform.js

@@ -1,8 +1,23 @@
-/** Pure transforms that strip a package's source lane from a publish artifact. */
+/** Pure transforms that slim a package's manifest and dist for publish. */
 /**
  * The emitted-file extensions that may carry a `sourceMappingURL` directive.
  */
 const MAP_ANNOTATED_EXTENSIONS = new Set([".js", ".mjs", ".cjs", ".d.ts", ".d.mts", ".d.cts"]);
+/**
+ * The script names npm and pnpm run on their own, on a consumer's install or on this package's publish.
+ */
+const LIFECYCLE_SCRIPT_STEMS = new Set([
+    "install",
+    "prepare",
+    "prepublish",
+    "prepublishOnly",
+    "pack",
+    "publish",
+]);
+/**
+ * The characters that make a `files` entry a glob rather than a path.
+ */
+const GLOB_CHARACTERS = /[*?[\]{}]/;
 /**
  * Whether a dist entry is a source-map sidecar (`*.map`).
  *
@@ -25,11 +40,26 @@ export function isMapAnnotatedFile(fileName) {
     return false;
 }
 /**
- * Produces a publish manifest with `src` dropped from `files` and every `source` export/import condition removed.
+ * Whether a script name is an install or publish lifecycle hook, with or without its `pre`/`post` prefix.
+ *
+ * @remarks Publish hooks stay because `pnpm publish` runs them right after pack-slim; install hooks stay because a
+ * consumer's package manager runs them.
+ *
+ * @since 0.9.0
+ */
+export function isLifecycleScript(name) {
+    if (LIFECYCLE_SCRIPT_STEMS.has(name)) {
+        return true;
+    }
+    const stem = name.replace(/^(?:pre|post)/, "");
+    return stem !== name && LIFECYCLE_SCRIPT_STEMS.has(stem);
+}
+/**
+ * Produces a publish manifest carrying only what a consumer's `tsc` and Node read.
  *
- * @remarks Operates on a structural clone, so the caller's manifest is left intact. A consumer never enables the
- * `source` condition and resolves `#/` through `types`/`default` to `dist`, so dropping the source lane and the shipped
- * sources it points at leaves the published surface whole.
+ * @remarks Operates on a structural clone, so the caller's manifest is left intact. A consumer never enables `source`
+ * and resolves `#/` through `types`/`default` to `dist`, so dropping the source lane, the `imports` left pointing
+ * outside `files`, the scripts that are not lifecycle hooks, and `devDependencies` leaves the published surface whole.
  *
  * @since 0.8.1
  */
@@ -38,10 +68,21 @@ export function slimPublishManifest(manifest) {
     const filesSrcRemoved = removeSrcFromFiles(draft);
     const exportsSourceRemoved = deleteSourceConditions(draft.exports);
     const importsSourceRemoved = deleteSourceConditions(draft.imports);
-    const changed = filesSrcRemoved || exportsSourceRemoved > 0 || importsSourceRemoved > 0;
+    const importsUnshippedRemoved = deleteUnshippedImports(draft);
+    const scriptsRemoved = deleteDevOnlyScripts(draft);
+    const devDependenciesRemoved = deleteDevDependencies(draft);
+    const changed = JSON.stringify(draft) !== JSON.stringify(manifest);
     return {
         manifest: draft,
-        report: { filesSrcRemoved, exportsSourceRemoved, importsSourceRemoved, changed },
+        report: {
+            filesSrcRemoved,
+            exportsSourceRemoved,
+            importsSourceRemoved,
+            importsUnshippedRemoved,
+            scriptsRemoved,
+            devDependenciesRemoved,
+            changed,
+        },
     };
 }
 /**
@@ -69,8 +110,8 @@ function removeSrcFromFiles(manifest) {
     manifest.files = next;
     return true;
 }
-// Walks a conditions tree deleting every `source` key. A subpath key always starts with ".", so only real
-// condition objects match, and a source-only entry never occurs here, so no node is left empty.
+// Walks a conditions tree deleting every `source` key. A subpath key always starts with ".", so only real condition
+// objects match; an `imports` entry this leaves empty is what the unshipped pass removes.
 function deleteSourceConditions(node) {
     if (Array.isArray(node)) {
         let removed = 0;
@@ -79,17 +120,102 @@ function deleteSourceConditions(node) {
         }
         return removed;
     }
-    if (node === null || typeof node !== "object") {
+    if (!isRecord(node)) {
         return 0;
     }
-    const record = node;
     let removed = 0;
-    if ("source" in record) {
-        delete record.source;
+    if ("source" in node) {
+        delete node.source;
         removed += 1;
     }
-    for (const key of Object.keys(record)) {
-        removed += deleteSourceConditions(record[key]);
+    for (const key of Object.keys(node)) {
+        removed += deleteSourceConditions(node[key]);
     }
     return removed;
+}
+// Keeps an `imports` entry only while one of its targets lands inside a path `files` ships. `files` is read after the
+// `src` drop, so a lane still pointing at `./src/*` counts as unshipped; with no `files` at all npm ships everything.
+function deleteUnshippedImports(manifest) {
+    const imports = manifest.imports;
+    const files = manifest.files;
+    if (!isRecord(imports) || !Array.isArray(files)) {
+        return 0;
+    }
+    const shippedEntries = files.filter((entry) => typeof entry === "string");
+    let removed = 0;
+    for (const specifier of Object.keys(imports)) {
+        const targets = collectTargets(imports[specifier]);
+        if (targets.some((target) => isShippedTarget(target, shippedEntries))) {
+            continue;
+        }
+        delete imports[specifier];
+        removed += 1;
+    }
+    if (Object.keys(imports).length === 0) {
+        delete manifest.imports;
+    }
+    return removed;
+}
+function collectTargets(node, into = []) {
+    if (typeof node === "string") {
+        into.push(node);
+    }
+    else if (Array.isArray(node)) {
+        for (const item of node) {
+            collectTargets(item, into);
+        }
+    }
+    else if (isRecord(node)) {
+        for (const value of Object.values(node)) {
+            collectTargets(value, into);
+        }
+    }
+    return into;
+}
+// A glob entry is left to npm, so any target counts as shipped under it; a negated entry ships nothing.
+function isShippedTarget(target, shippedEntries) {
+    const relativeTarget = stripDotSlash(target);
+    return shippedEntries.some((entry) => {
+        if (entry.startsWith("!")) {
+            return false;
+        }
+        if (GLOB_CHARACTERS.test(entry)) {
+            return true;
+        }
+        const shipped = stripDotSlash(entry).replace(/\/+$/, "");
+        return relativeTarget === shipped || relativeTarget.startsWith(`${shipped}/`);
+    });
+}
+function stripDotSlash(filePath) {
+    return filePath.startsWith("./") ? filePath.slice(2) : filePath;
+}
+// Drops every script that is not a lifecycle hook, and the field itself once nothing is left in it.
+function deleteDevOnlyScripts(manifest) {
+    const scripts = manifest.scripts;
+    if (!isRecord(scripts)) {
+        return 0;
+    }
+    let removed = 0;
+    for (const name of Object.keys(scripts)) {
+        if (isLifecycleScript(name)) {
+            continue;
+        }
+        delete scripts[name];
+        removed += 1;
+    }
+    if (Object.keys(scripts).length === 0) {
+        delete manifest.scripts;
+    }
+    return removed;
+}
+function deleteDevDependencies(manifest) {
+    const devDependencies = manifest.devDependencies;
+    if (!isRecord(devDependencies)) {
+        return 0;
+    }
+    delete manifest.devDependencies;
+    return Object.keys(devDependencies).length;
+}
+function isRecord(value) {
+    return value !== null && typeof value === "object" && !Array.isArray(value);
 }

package/dist/pack-slim/output.js

@@ -10,7 +10,7 @@ export class PackSlimProgressPresenter {
         this.dryRun = options.dryRun;
     }
     onBanner() {
-        logger.out(this.dryRun ? "pack-slim (dry run) — no files will change" : "pack-slim — stripping source lane");
+        logger.out(this.dryRun ? "pack-slim (dry run) — no files will change" : "pack-slim — slimming publish artifacts");
     }
     onPackageComplete(stats, ordinal, total) {
         const position = `[${ordinal}/${total}]`;
@@ -37,6 +37,15 @@ export class PackSlimProgressPresenter {
         if (stats.importsSourceRemoved > 0) {
             parts.push(`imports:source×${stats.importsSourceRemoved}`);
         }
+        if (stats.importsUnshippedRemoved > 0) {
+            parts.push(`imports:unshipped×${stats.importsUnshippedRemoved}`);
+        }
+        if (stats.scriptsRemoved > 0) {
+            parts.push(`scripts×${stats.scriptsRemoved}`);
+        }
+        if (stats.devDependenciesRemoved > 0) {
+            parts.push(`devDependencies×${stats.devDependenciesRemoved}`);
+        }
         if (stats.mapFilesDeleted > 0) {
             parts.push(`maps×${stats.mapFilesDeleted}`);
         }

package/dist/pack-slim/sync.js

@@ -5,11 +5,11 @@ import { listWorkspacePackageDirectories } from "#/core/workspace/resolver";
 import { DIST_DIR, PACKAGE_JSON } from "#/mirror/domain/constants";
 import { isMapAnnotatedFile, isSourceMapFile, slimPublishManifest, stripSourceMappingComment, } from "#/pack-slim/domain/transform";
 /**
- * Strips the source lane from every published package's build output and returns the aggregate stats.
+ * Slims every published package's manifest and build output for publish and returns the aggregate stats.
  *
- * @remarks Meant to run on an ephemeral CI checkout right before `changeset publish`: it rewrites `package.json`, drops
- * `dist` source maps, and clears their now-dangling `sourceMappingURL` directives, so the tarball ships only `dist`
- * runtime and types. Private packages are skipped because `changeset publish` never publishes them.
+ * @remarks Meant to run on an ephemeral CI checkout right before `changeset publish`: it rewrites `package.json` down to
+ * what a consumer reads, drops `dist` source maps, and clears their now-dangling `sourceMappingURL` directives, so the
+ * tarball ships only `dist` runtime and types. Private packages are skipped because `changeset publish` never publishes them.
  *
  * @since 0.8.1
  */
@@ -66,6 +66,9 @@ async function slimWorkspacePackage(fs, packageDir, write) {
         filesSrcRemoved: false,
         exportsSourceRemoved: 0,
         importsSourceRemoved: 0,
+        importsUnshippedRemoved: 0,
+        scriptsRemoved: 0,
+        devDependenciesRemoved: 0,
         mapFilesDeleted: 0,
         sourceCommentsStripped: 0,
         changed: false,
@@ -94,6 +97,9 @@ async function slimWorkspacePackage(fs, packageDir, write) {
         pkgStats.filesSrcRemoved = report.filesSrcRemoved;
         pkgStats.exportsSourceRemoved = report.exportsSourceRemoved;
         pkgStats.importsSourceRemoved = report.importsSourceRemoved;
+        pkgStats.importsUnshippedRemoved = report.importsUnshippedRemoved;
+        pkgStats.scriptsRemoved = report.scriptsRemoved;
+        pkgStats.devDependenciesRemoved = report.devDependenciesRemoved;
         if (report.changed && write) {
             await fs.writeFile(packageJsonPath, `${JSON.stringify(slimmed, null, 2)}\n`, "utf8");
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@codefast/cli",
-  "version": "0.8.1",
-  "description": "Developer CLI for the Codefast monorepo (arrange, audit, mirror, tag)",
+  "version": "0.9.0",
+  "description": "Developer CLI for the Codefast monorepo (arrange, audit, mirror, pack-slim, tag)",
   "keywords": [
     "cli",
     "codefast",
@@ -15,7 +15,7 @@
     "url": "https://github.com/codefastlabs/codefast/issues"
   },
   "license": "MIT",
-  "author": "Vuong Phan <mr.thevuong@gmail.com>",
+  "author": "Vuong Phan <mr.thevuong@gmail.com> (https://github.com/thevuong)",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/codefastlabs/codefast.git",
@@ -26,19 +26,12 @@
   },
   "files": [
     "dist",
-    "LICENSE",
+    "CHANGELOG.md",
     "README.md",
-    "package.json"
+    "LICENSE"
   ],
   "type": "module",
   "imports": {
-    "#/tests/*": [
-      "./tests/*",
-      "./tests/*.ts",
-      "./tests/*.tsx",
-      "./tests/*/index.ts",
-      "./tests/*/index.tsx"
-    ],
     "#/*": {
       "default": "./dist/*.js"
     }
@@ -55,28 +48,7 @@
     "yaml": "^2.9.0",
     "zod": "^4.5.4"
   },
-  "devDependencies": {
-    "@types/node": "^26.4.1",
-    "@types/picomatch": "^4.0.3",
-    "@vitest/coverage-v8": "^5.0.0",
-    "typescript": "^7.0.2",
-    "vite": "^8.2.2",
-    "vitest": "^5.0.0",
-    "@codefast/typescript-config": "0.9.0"
-  },
   "engines": {
     "node": ">=24.0.0"
-  },
-  "scripts": {
-    "build": "rm -rf dist && tsc -p tsconfig.build.json",
-    "check-types": "tsc --noEmit",
-    "clean": "rm -rf dist",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
-    "test:e2e": "vitest run tests/e2e",
-    "test:integration": "vitest run tests/integration",
-    "test:type": "vitest run tests/types",
-    "test:unit": "vitest run tests/unit",
-    "test:watch": "vitest"
   }
 }