@pulse-editor/cli 0.1.13 → 0.1.15

package/dist/components/commands/build.js

@@ -1,62 +1,17 @@
 import { Fragment as _Fragment, jsx as _jsx } from "react/jsx-runtime";
 import { useEffect } from 'react';
-import { execa } from 'execa';
-import fs from 'fs';
-import { cleanDist } from '../../lib/execa-utils/clean.js';
-import { webpackCompile } from '../../lib/webpack/compile.js';
+import { buildProd } from '../../lib/build-prod.js';
 export default function Build({ cli }) {
     useEffect(() => {
-        async function buildProd() {
+        (async () => {
             const buildTarget = cli.flags.target;
-            console.log(`🚧 Building for target: ${buildTarget || 'both client and server'}`);
-            // Move node_modules/@pulse-editor/cli/dist/lib/server to node_modules/.pulse/server
-            if (buildTarget === 'server' || !buildTarget) {
-                if (fs.existsSync('node_modules/.pulse/server')) {
-                    // Remove existing directory
-                    if (process.platform === 'win32') {
-                        await execa('rmdir /S /Q node_modules\\.pulse\\server', {
-                            shell: true,
-                        });
-                    }
-                    else {
-                        await execa('rm -rf node_modules/.pulse/server', {
-                            shell: true,
-                        });
-                    }
-                }
-                // Create node_modules/.pulse/server directory
-                if (!fs.existsSync('node_modules/.pulse/server')) {
-                    if (process.platform === 'win32') {
-                        await execa('mkdir node_modules\\.pulse\\server', {
-                            shell: true,
-                        });
-                    }
-                    else {
-                        await execa('mkdir -p node_modules/.pulse/server', {
-                            shell: true,
-                        });
-                    }
-                }
-                if (process.platform === 'win32') {
-                    await execa('xcopy /E /I node_modules\\@pulse-editor\\cli\\dist\\lib\\server node_modules\\.pulse\\server', {
-                        shell: true,
-                    });
-                }
-                else {
-                    await execa('cp -r node_modules/@pulse-editor/cli/dist/lib/server/* node_modules/.pulse/server/', {
-                        shell: true,
-                    });
-                }
-            }
-            await cleanDist();
             try {
-                await webpackCompile('production', buildTarget);
+                await buildProd(buildTarget);
             }
             catch (err) {
                 console.error('❌ Webpack build failed', err);
             }
-        }
-        buildProd();
+        })();
     }, []);
     return _jsx(_Fragment, {});
 }

package/dist/components/commands/publish.js

@@ -6,6 +6,7 @@ import Spinner from 'ink-spinner';
 import fs from 'fs';
 import { $ } from 'execa';
 import { publishApp } from '../../lib/backend/publish-app.js';
+import { buildProd } from '../../lib/build-prod.js';
 export default function Publish({ cli }) {
     const [isInProjectDir, setIsInProjectDir] = useState(false);
     const [isCheckingAuth, setIsCheckingAuth] = useState(true);
@@ -51,17 +52,32 @@ export default function Publish({ cli }) {
             if (cli.flags.build) {
                 setIsBuilding(true);
                 try {
-                    await $ `npm run build`;
+                    // Run the same in-process build pipeline as `pulse build`. We
+                    // deliberately do NOT shell out to `npm run build` here — that
+                    // would resolve a different `pulse` binary (project-local vs
+                    // global) and potentially load a different webpack/MF tree.
+                    await buildProd(cli.flags.target);
                 }
                 catch (error) {
                     setIsBuildingError(true);
-                    setFailureMessage('Build failed. Please run `npm run build` to see the error.');
+                    setFailureMessage(error?.stack || error?.message || String(error));
                     return;
                 }
                 finally {
                     setIsBuilding(false);
                 }
             }
+            // Halt if the build did not actually produce a dist/ directory —
+            // this catches cases where the build script exited 0 but emitted
+            // nothing (or where --no-build was passed without a prior build).
+            if (!fs.existsSync('dist') || fs.readdirSync('dist').length === 0) {
+                setIsBuildingError(true);
+                setFailureMessage('No build output found at `dist/`. ' +
+                    (cli.flags.build
+                        ? 'The build script completed but did not produce any output.'
+                        : 'Run `pulse build` first, or omit `--no-build`.'));
+                return;
+            }
             setIsZipping(true);
             // Zip the dist folder
             try {
@@ -108,5 +124,5 @@ export default function Publish({ cli }) {
             buildExtension();
         }
     }, [isAuthenticated]);
-    return (_jsx(_Fragment, { children: !isInProjectDir ? (_jsx(Text, { color: 'redBright', children: "\u26D4 The current directory does not contain a Pulse Editor project." })) : isCheckingAuth ? (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Checking authentication..." })] })) : !isAuthenticated ? (_jsxs(Text, { children: ["You are not authenticated or your access token is invalid. Publishing to Extension Marketplace is in Beta access. Please visit", _jsx(Text, { color: 'blueBright', children: " https://palmos.ai/beta " }), "to apply for Beta access."] })) : (_jsxs(_Fragment, { children: [isBuilding && (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Building..." })] })), isBuildingError && (_jsx(Text, { color: 'redBright', children: "\u274C Error building the extension. Please run `npm run build` to see the error." })), isZipping && (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Compressing build..." })] })), isZippingError && (_jsxs(Text, { color: 'redBright', children: ["\u274C Error zipping the build output. ", failureMessage] })), isPublishing && (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Publishing..." })] })), isPublishingError && (_jsxs(_Fragment, { children: [_jsx(Text, { color: 'redBright', children: "\u274C Failed to publish extension." }), failureMessage && (_jsxs(Text, { color: 'redBright', children: ["Error: ", failureMessage] }))] })), isPublished && (_jsx(Text, { color: 'greenBright', children: "\u2705 Extension published successfully." }))] })) }));
+    return (_jsx(_Fragment, { children: !isInProjectDir ? (_jsx(Text, { color: 'redBright', children: "\u26D4 The current directory does not contain a Pulse Editor project." })) : isCheckingAuth ? (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Checking authentication..." })] })) : !isAuthenticated ? (_jsxs(Text, { children: ["You are not authenticated or your access token is invalid. Publishing to Extension Marketplace is in Beta access. Please visit", _jsx(Text, { color: 'blueBright', children: " https://palmos.ai/beta " }), "to apply for Beta access."] })) : (_jsxs(_Fragment, { children: [isBuilding && (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Building..." })] })), isBuildingError && (_jsxs(_Fragment, { children: [_jsx(Text, { color: 'redBright', children: "\u274C Error building the extension." }), failureMessage && (_jsx(Text, { color: 'redBright', children: failureMessage }))] })), isZipping && (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Compressing build..." })] })), isZippingError && (_jsxs(Text, { color: 'redBright', children: ["\u274C Error zipping the build output. ", failureMessage] })), isPublishing && (_jsxs(Box, { children: [_jsx(Spinner, { type: "dots" }), _jsx(Text, { children: " Publishing..." })] })), isPublishingError && (_jsxs(_Fragment, { children: [_jsx(Text, { color: 'redBright', children: "\u274C Failed to publish extension." }), failureMessage && (_jsxs(Text, { color: 'redBright', children: ["Error: ", failureMessage] }))] })), isPublished && (_jsx(Text, { color: 'greenBright', children: "\u2705 Extension published successfully." }))] })) }));
 }

package/dist/lib/build-prod.d.ts

@@ -0,0 +1,6 @@
+/**
+ * Run the production build pipeline in-process. Throws on failure so callers
+ * (build command, publish command) can react. This is the single source of
+ * truth for what `pulse build` does — do not duplicate this logic elsewhere.
+ */
+export declare function buildProd(buildTarget?: 'client' | 'server'): Promise<void>;

package/dist/lib/build-prod.js

@@ -0,0 +1,48 @@
+import { execa } from 'execa';
+import fs from 'fs';
+import { cleanDist } from './execa-utils/clean.js';
+import { webpackCompile } from './webpack/compile.js';
+/**
+ * Run the production build pipeline in-process. Throws on failure so callers
+ * (build command, publish command) can react. This is the single source of
+ * truth for what `pulse build` does — do not duplicate this logic elsewhere.
+ */
+export async function buildProd(buildTarget) {
+    console.log(`🚧 Building for target: ${buildTarget || 'both client and server'}`);
+    // Stage the server runtime under node_modules/.pulse/server so the server
+    // build has the helpers it needs at the expected path.
+    if (buildTarget === 'server' || !buildTarget) {
+        if (fs.existsSync('node_modules/.pulse/server')) {
+            if (process.platform === 'win32') {
+                await execa('rmdir /S /Q node_modules\\.pulse\\server', {
+                    shell: true,
+                });
+            }
+            else {
+                await execa('rm -rf node_modules/.pulse/server', {
+                    shell: true,
+                });
+            }
+        }
+        if (!fs.existsSync('node_modules/.pulse/server')) {
+            if (process.platform === 'win32') {
+                await execa('mkdir node_modules\\.pulse\\server', {
+                    shell: true,
+                });
+            }
+            else {
+                await execa('mkdir -p node_modules/.pulse/server', {
+                    shell: true,
+                });
+            }
+        }
+        if (process.platform === 'win32') {
+            await execa('xcopy /E /I node_modules\\@pulse-editor\\cli\\dist\\lib\\server node_modules\\.pulse\\server', { shell: true });
+        }
+        else {
+            await execa('cp -r node_modules/@pulse-editor/cli/dist/lib/server/* node_modules/.pulse/server/', { shell: true });
+        }
+    }
+    await cleanDist();
+    await webpackCompile('production', buildTarget);
+}

package/dist/lib/webpack/configs/utils.js

@@ -49,7 +49,16 @@ export async function loadPulseConfig() {
         recursive: true,
         force: true,
     });
-    return mod.default;
+    const config = mod.default;
+    // Always use the version from package.json as the source of truth
+    const pkgJsonPath = path.join(projectDirName, "package.json");
+    if (existsSync(pkgJsonPath)) {
+        const pkg = JSON.parse(await fs.readFile(pkgJsonPath, "utf-8"));
+        if (pkg.version) {
+            config.version = pkg.version;
+        }
+    }
+    return config;
 }
 export function getLocalNetworkIP() {
     const interfaces = networkInterfaces();

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@pulse-editor/cli",
-	"version": "0.1.13",
+	"version": "0.1.15",
 	"license": "MIT",
 	"bin": {
 		"pulse": "dist/cli.js"