@gjsify/cli 0.4.22 → 0.4.24

package/lib/commands/pack.d.ts

@@ -46,6 +46,14 @@ export interface PackWorkspaceOptions {
      * workflow has already produced the build artifacts.
      */
     lifecycleScripts?: readonly string[];
+    /**
+     * Stdio mode for lifecycle scripts. Default `'inherit'` — child output
+     * appears in the parent's terminal. Pass `'inherit-stderr'` to redirect
+     * the child's stdout → parent's stderr; used by `gjsify pack --json`
+     * and `gjsify publish` so the parent's stdout stays a clean
+     * machine-readable JSON stream.
+     */
+    lifecycleStdio?: 'inherit' | 'inherit-stderr' | 'pipe' | 'ignore';
 }
 /**
  * Programmatic equivalent of the `pack` command — used by `gjsify publish`

package/lib/commands/pack.js

@@ -65,6 +65,11 @@ export const packCommand = {
             destination: args['pack-destination'],
             dryRun: args['dry-run'] === true,
             lifecycleScripts: args['ignore-scripts'] ? [] : ['prepack'],
+            // When emitting machine-readable JSON on stdout, the prepack
+            // script's chatty log lines must NOT land on the parent's
+            // stdout — `JSON.parse(stdout)` callers would otherwise fail
+            // with `Unexpected token …`. Route lifecycle output to stderr.
+            lifecycleStdio: args.json ? 'inherit-stderr' : 'inherit',
         });
         if (args.json) {
             process.stdout.write(`${JSON.stringify([result], null, 2)}\n`);
@@ -99,7 +104,10 @@ export async function packWorkspace(wsDir, opts = {}) {
     // post-install. Matches `npm pack` / `npm publish` semantics.
     const lifecycleScripts = opts.lifecycleScripts ?? ['prepack'];
     for (const scriptName of lifecycleScripts) {
-        await runLifecycleScript(wsDir, pkg, scriptName, { optional: true });
+        await runLifecycleScript(wsDir, pkg, scriptName, {
+            optional: true,
+            stdio: opts.lifecycleStdio,
+        });
     }
     // Re-read package.json AFTER lifecycle scripts in case one of them
     // mutated it (e.g. a `prepack` that injects build metadata into

package/lib/commands/publish.js

@@ -159,6 +159,10 @@ export const publishCommand = {
         const packOpts = {
             dryRun: true,
             lifecycleScripts: ['prepublishOnly', 'prepack'],
+            // `gjsify publish --json` emits the publish summary on stdout.
+            // Lifecycle scripts must not pollute that stream with their
+            // own log lines; redirect their stdout → parent's stderr.
+            lifecycleStdio: args.json ? 'inherit-stderr' : 'inherit',
         };
         const packed = await packWorkspace(wsDir, packOpts);
         // We need the raw bytes — re-run with destination=null and capture.

package/lib/utils/run-lifecycle-script.d.ts

@@ -1,8 +1,15 @@
 export interface RunLifecycleScriptOptions {
     /** When true, do not throw on missing scripts — return `false` instead. */
     optional?: boolean;
-    /** Stdio inheritance. Defaults to `'inherit'` so output goes to the parent. */
-    stdio?: 'inherit' | 'pipe' | 'ignore';
+    /**
+     * Stdio inheritance for the spawned script. Default `'inherit'` so output
+     * appears in the parent's terminal. Pass `'inherit-stderr'` to mirror
+     * inheritance but redirect the child's stdout → parent's stderr — used by
+     * `gjsify pack --json` and `gjsify publish` so the parent's stdout stays
+     * a clean JSON stream (script log lines would otherwise corrupt the
+     * machine-readable output that callers `JSON.parse`).
+     */
+    stdio?: 'inherit' | 'inherit-stderr' | 'pipe' | 'ignore';
     /** Extra environment variables layered on top of the defaults. */
     env?: Record<string, string>;
 }

package/lib/utils/run-lifecycle-script.js

@@ -54,11 +54,20 @@ export async function runLifecycleScript(wsDir, pkg, name, opts = {}) {
         npm_package_version: pkg.version ?? '',
         ...(opts.env ?? {}),
     };
+    // `'inherit-stderr'` is our extension on top of node's stdio modes —
+    // child stdin inherits, child stdout → parent's stderr (fd 2), child
+    // stderr → parent's stderr. Used by `gjsify pack --json` so the
+    // prepack's log lines don't get interleaved with the JSON we emit on
+    // parent stdout. `spawn`'s `stdio` accepts numeric fds in array form
+    // and routes the child's matching stream to that fd.
+    const stdioConfig = opts.stdio === 'inherit-stderr'
+        ? ['inherit', 2, 2]
+        : (opts.stdio ?? 'inherit');
     await new Promise((resolveOk, reject) => {
         const child = spawn(literal, [], {
             cwd: wsDir,
             env: env,
-            stdio: opts.stdio ?? 'inherit',
+            stdio: stdioConfig,
             shell: true,
         });
         child.on('close', (code) => {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gjsify/cli",
-    "version": "0.4.22",
+    "version": "0.4.24",
     "description": "CLI for Gjsify",
     "type": "module",
     "main": "lib/index.js",
@@ -120,18 +120,18 @@
         "cli"
     ],
     "dependencies": {
-        "@gjsify/buffer": "^0.4.22",
-        "@gjsify/create-app": "^0.4.22",
-        "@gjsify/node-globals": "^0.4.22",
-        "@gjsify/node-polyfills": "^0.4.22",
-        "@gjsify/npm-registry": "^0.4.22",
-        "@gjsify/resolve-npm": "^0.4.22",
-        "@gjsify/rolldown-plugin-gjsify": "^0.4.22",
-        "@gjsify/rolldown-plugin-pnp": "^0.4.22",
-        "@gjsify/semver": "^0.4.22",
-        "@gjsify/tar": "^0.4.22",
-        "@gjsify/web-polyfills": "^0.4.22",
-        "@gjsify/workspace": "^0.4.22",
+        "@gjsify/buffer": "^0.4.24",
+        "@gjsify/create-app": "^0.4.24",
+        "@gjsify/node-globals": "^0.4.24",
+        "@gjsify/node-polyfills": "^0.4.24",
+        "@gjsify/npm-registry": "^0.4.24",
+        "@gjsify/resolve-npm": "^0.4.24",
+        "@gjsify/rolldown-plugin-gjsify": "^0.4.24",
+        "@gjsify/rolldown-plugin-pnp": "^0.4.24",
+        "@gjsify/semver": "^0.4.24",
+        "@gjsify/tar": "^0.4.24",
+        "@gjsify/web-polyfills": "^0.4.24",
+        "@gjsify/workspace": "^0.4.24",
         "cosmiconfig": "^9.0.1",
         "get-tsconfig": "^4.14.0",
         "pkg-types": "^2.3.1",
@@ -139,12 +139,12 @@
         "yargs": "^18.0.0"
     },
     "devDependencies": {
-        "@gjsify/unit": "^0.4.22",
+        "@gjsify/unit": "^0.4.24",
         "@types/yargs": "^17.0.35",
         "typescript": "^6.0.3"
     },
     "peerDependencies": {
-        "@gjsify/rolldown-native": "^0.4.22"
+        "@gjsify/rolldown-native": "^0.4.24"
     },
     "peerDependenciesMeta": {
         "@gjsify/rolldown-native": {