@speedkit/cli 4.4.0 → 4.4.2

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [4.4.2](https://gitlab.orestes.info/baqend/speed-kit-cli/compare/v4.4.1...v4.4.2) (2026-05-13)
+
+
+### Bug Fixes
+
+* **onboarding:** detect atomic file writes and shut down gracefully ([a62f5a9](https://gitlab.orestes.info/baqend/speed-kit-cli/commit/a62f5a9c134cd675ced842920e418b0a935f76fc))
+
+## [4.4.1](https://gitlab.orestes.info/baqend/speed-kit-cli/compare/v4.4.0...v4.4.1) (2026-05-13)
+
+
+### Bug Fixes
+
+* **sk-config:** mock methods ([d479dcf](https://gitlab.orestes.info/baqend/speed-kit-cli/commit/d479dcff73b0251ced9c160623b92831cc39648e))
+
 # [4.4.0](https://gitlab.orestes.info/baqend/speed-kit-cli/compare/v4.3.0...v4.4.0) (2026-05-13)
 
 

package/README.md

@@ -21,7 +21,7 @@ $ npm install -g @speedkit/cli
 $ sk COMMAND
 running command...
 $ sk (--version)
-@speedkit/cli/4.4.0 linux-x64 node-v22.22.2
+@speedkit/cli/4.4.2 linux-x64 node-v22.22.2
 $ sk --help [COMMAND]
 USAGE
   $ sk COMMAND

package/dist/commands/onboarding.js

@@ -13,6 +13,18 @@ export default class Onboarding extends Command {
     async run() {
         const { args: { customerPath }, flags: { configName, domain, local, verboseLevel, ignoreContentSecurityPolicy, useTestConfig, debuggingPort, headless, }, } = await this.parse(Onboarding);
         const service = await new OnboardingServiceFactory(new OnboardingContext(customerPath, configName, domain, local, ignoreContentSecurityPolicy, this.config.root, verboseLevel, useTestConfig, debuggingPort, headless), new CliConfig(this.config).load(customerPath)).getService();
-        await service.run();
+        const shutdown = new AbortController();
+        const onSignal = () => shutdown.abort();
+        // `once`: a second Ctrl+C falls through to Node's default handler and
+        // force-quits if cleanup hangs.
+        process.once("SIGINT", onSignal);
+        process.once("SIGTERM", onSignal);
+        try {
+            await service.run(shutdown.signal);
+        }
+        finally {
+            process.off("SIGINT", onSignal);
+            process.off("SIGTERM", onSignal);
+        }
     }
 }

package/dist/helpers/evaluate-speed-kit-config.js

@@ -56,10 +56,18 @@ function buildBrowserMock(origin, app) {
         serviceWorkerUrl: "",
         screen: { width: 0, height: 0 },
         devicePixelRatio: 1,
-        setTimeout: (callBack, time) => setTimeout(callBack, time),
-        clearTimeout: () => clearTimeout,
-        addEventListener: () => addEventListener,
-        removeEventListener: () => removeEventListener,
+        setTimeout: () => {
+            /* empty */
+        },
+        clearTimeout: () => {
+            /* empty */
+        },
+        addEventListener: () => {
+            /* empty */
+        },
+        removeEventListener: () => {
+            /* empty */
+        },
         navigator: {
             userAgent: "",
             connection: {},

package/dist/services/integration-api/integration-api-model.d.ts

@@ -70,7 +70,9 @@ export interface ConfigFileInterface extends FileInterface {
     updateFile(): Promise<void>;
     updateIfChanged(): Promise<void>;
     isUpdated(): boolean;
-    onChange(callback?: (file: FileInterface) => Promise<void>): void;
+    onChange(callback?: (file: FileInterface) => Promise<void>, options?: {
+        signal?: AbortSignal;
+    }): void;
     delete(): Promise<void>;
 }
 export interface ModuleInterface extends ConfigFileInterface, FileInterface {

package/dist/services/integration-api/struct/file.d.ts

@@ -14,6 +14,8 @@ export declare class File implements ConfigFileInterface, FileInterface {
     isUpdated(): boolean;
     updateFile(): Promise<void>;
     updateIfChanged(): Promise<void>;
-    onChange(callback: (file: FileInterface) => Promise<void>): Promise<void>;
+    onChange(callback: (file: FileInterface) => Promise<void>, options?: {
+        signal?: AbortSignal;
+    }): void;
     delete(): Promise<void>;
 }

package/dist/services/integration-api/struct/file.js

@@ -1,4 +1,5 @@
-import { readFile, unlink, watch, writeFile } from "node:fs/promises";
+import { readFile, unlink, writeFile } from "node:fs/promises";
+import { watch as chokidarWatch } from "chokidar";
 import { FILE_TYPE, } from "../integration-api-model.js";
 export class File {
     name;
@@ -41,15 +42,32 @@ export class File {
             await this.updateFile();
         }
     }
-    async onChange(callback) {
-        const watcher = watch(this.path, { encoding: "utf-8" });
-        for await (const info of watcher) {
-            if (info.eventType !== "change") {
-                continue;
+    onChange(callback, options = {}) {
+        const { signal } = options;
+        if (signal?.aborted) {
+            return;
+        }
+        // chokidar over node's fs.watch: handles atomic write-then-rename
+        // (Claude Code, VS Code's atomicSave, vim writebackup, …) which fs.watch
+        // misses because inotify is inode-bound and the new file gets a new inode.
+        const watcher = chokidarWatch(this.path, {
+            ignoreInitial: true,
+            atomic: true,
+            awaitWriteFinish: { stabilityThreshold: 200, pollInterval: 50 },
+        });
+        const handle = async () => {
+            const content = await readFile(this.path, "utf-8");
+            if (content === this.content) {
+                return;
             }
-            this.setContent(await readFile(this.path, "utf-8"));
+            this.setContent(content);
             await callback(this);
-        }
+        };
+        watcher.on("change", handle);
+        watcher.on("add", handle);
+        signal?.addEventListener("abort", () => void watcher.close(), {
+            once: true,
+        });
     }
     async delete() {
         await unlink(this.path);

package/dist/services/onboarding/file-events/file-watcher.d.ts

@@ -21,7 +21,7 @@ export declare class FileWatcher {
      * @param page
      * @param serviceWorkerReinstallCallback
      */
-    addCallbackOnFileChange(page: Page, serviceWorkerReinstallCallback: (file: ConfigFileInterface) => Promise<void>): void;
+    addCallbackOnFileChange(page: Page, serviceWorkerReinstallCallback: (file: ConfigFileInterface) => Promise<void>, signal: AbortSignal): void;
     /**
      * changes on these files will trigger
      * - a rebuild of the serviceWorker

package/dist/services/onboarding/file-events/file-watcher.js

@@ -26,11 +26,11 @@ export class FileWatcher {
      * @param page
      * @param serviceWorkerReinstallCallback
      */
-    addCallbackOnFileChange(page, serviceWorkerReinstallCallback) {
+    addCallbackOnFileChange(page, serviceWorkerReinstallCallback, signal) {
         const installResource = this.files.getByName(INTEGRATION_FILES.BUILD.INSTALL);
-        this.handleServiceWorkerDependencies(page, installResource, serviceWorkerReinstallCallback);
-        this.handleGenericConfigFiles(page, installResource);
-        this.handleDynamicStyles(page);
+        this.handleServiceWorkerDependencies(page, installResource, serviceWorkerReinstallCallback, signal);
+        this.handleGenericConfigFiles(page, installResource, signal);
+        this.handleDynamicStyles(page, signal);
     }
     /**
      * changes on these files will trigger
@@ -44,7 +44,7 @@ export class FileWatcher {
      * @param serviceWorkerReinstallCallback
      * @private
      */
-    handleServiceWorkerDependencies(page, installResource, serviceWorkerReinstallCallback) {
+    handleServiceWorkerDependencies(page, installResource, serviceWorkerReinstallCallback, signal) {
         // add watcher to serviceWorker related files
         const serviceWorkerDependencies = [
             INTEGRATION_FILES.CONFIG.SPEED_KIT,
@@ -65,7 +65,7 @@ export class FileWatcher {
                 if (result.success === false) {
                     this.cli.writeError(`error after fileChange: ${file.name}: ${result.error}`);
                 }
-            });
+            }, { signal });
         }
     }
     /**
@@ -78,7 +78,7 @@ export class FileWatcher {
      * @param installResource
      * @private
      */
-    handleGenericConfigFiles(page, installResource) {
+    handleGenericConfigFiles(page, installResource, signal) {
         // add watcher to configFiles
         const generalConfigFiles = [
             INTEGRATION_FILES.CONFIG.DYNAMIC_BLOCKS,
@@ -99,7 +99,7 @@ export class FileWatcher {
                 if (result.success === false) {
                     this.cli.writeError(`error after fileChange: ${file.name}: ${result.error}`);
                 }
-            });
+            }, { signal });
         }
     }
     /**
@@ -112,7 +112,7 @@ export class FileWatcher {
      * @param page
      * @private
      */
-    handleDynamicStyles(page) {
+    handleDynamicStyles(page, signal) {
         if (!this.files.hasFile(INTEGRATION_FILES.CONFIG.DYNAMIC_STYLES)) {
             return;
         }
@@ -124,7 +124,7 @@ export class FileWatcher {
             if (result.success === false) {
                 this.cli.writeError(`error after fileChange: ${file.name}: ${result.error}`);
             }
-        });
+        }, { signal });
     }
     /**
      * fileWatcherCallback that will reinstall the serviceWorker

package/dist/services/onboarding/onboarding-service.d.ts

@@ -17,9 +17,12 @@ export declare class OnboardingService {
     private reinstallOnNextNavigate;
     constructor(browserContext: BrowserContext, customerConfig: CustomerConfig, fetchEventHandler: FetchEventHandler, fileWatcher: FileWatcher, cli: CliService, developmentToolsMessages: DevtoolsExtensionApi);
     /**
-     * prepare puppeteerBrowser, add eventListeners and start
+     * Prepare the puppeteer browser, wire event listeners, navigate to the
+     * install page, then block until either the caller aborts the signal
+     * (e.g. SIGINT in the command) or the browser disconnects. On exit, close
+     * any still-open resources. File watchers close themselves via `signal`.
      */
-    run(): Promise<void>;
+    run(externalSignal: AbortSignal): Promise<void>;
     private prepareBrowser;
     private preparePage;
     private findPage;

package/dist/services/onboarding/onboarding-service.js

@@ -21,16 +21,26 @@ export class OnboardingService {
         this.developmentToolsMessages = developmentToolsMessages;
     }
     /**
-     * prepare puppeteerBrowser, add eventListeners and start
+     * Prepare the puppeteer browser, wire event listeners, navigate to the
+     * install page, then block until either the caller aborts the signal
+     * (e.g. SIGINT in the command) or the browser disconnects. On exit, close
+     * any still-open resources. File watchers close themselves via `signal`.
      */
-    async run() {
+    async run(externalSignal) {
         this.cli.startAction("ONBOARDING:PREPARE:BROWSER", `prepare browser ${this.cli.style.bold(this.browserContext.browserVersionString)}`);
-        const browser = await this.prepareBrowser();
-        const page = await this.preparePage(browser);
+        const localController = new AbortController();
+        const shutdownSignal = AbortSignal.any([
+            externalSignal,
+            localController.signal,
+        ]);
+        const browser = await this.prepareBrowser(localController);
+        const page = await this.preparePage(browser, shutdownSignal);
         this.cli.successAction("ONBOARDING:PREPARE:BROWSER");
         await this.navigate(page, `${this.customerConfig.scope}speed-kit-install.html?url=${this.customerConfig.scope}`);
+        await waitForAbort(shutdownSignal);
+        await safe(browser.close());
     }
-    async prepareBrowser() {
+    async prepareBrowser(localController) {
         const browser = await puppeteerExtra.default.launch(this.browserContext.getPuppeteerLaunchOptions());
         for (const target of browser.targets()) {
             if (target.type() === "browser") {
@@ -39,14 +49,14 @@ export class OnboardingService {
         }
         browser.on(BROWSER_EVENTS.DISCONNECTED, () => {
             this.cli.writeError("Browser disconnected, shutting down");
-            process.exit(-1);
+            localController.abort();
         });
         return browser;
     }
-    async preparePage(browser) {
+    async preparePage(browser, shutdownSignal) {
         const page = await this.findPage(browser);
         await safe(this.prepareCleanState(page));
-        this.registerFileWatcher(page);
+        this.registerFileWatcher(page, shutdownSignal);
         this.registerReinstallSpeedKitOnNavigate(page);
         this.developmentToolsMessages.addMessage({
             id: "cli",
@@ -106,10 +116,10 @@ export class OnboardingService {
         this.reinstallOnNextNavigate = false;
         await this.navigate(page, target);
     }
-    registerFileWatcher(page) {
+    registerFileWatcher(page, signal) {
         this.fileWatcher.addCallbackOnFileChange(page, async () => {
             this.reinstallOnNextNavigate = true;
-        });
+        }, signal);
     }
     registerReinstallSpeedKitOnNavigate(page) {
         page.on("framenavigated", async (frame) => {
@@ -127,3 +137,11 @@ export class OnboardingService {
         await safe(page.goto(goto.toString()));
     }
 }
+function waitForAbort(signal) {
+    if (signal.aborted) {
+        return Promise.resolve();
+    }
+    return new Promise((resolve) => {
+        signal.addEventListener("abort", () => resolve(), { once: true });
+    });
+}

package/oclif.manifest.json

@@ -751,5 +751,5 @@
       ]
     }
   },
-  "version": "4.4.0"
+  "version": "4.4.2"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@speedkit/cli",
   "description": "Speed Kit CLI",
-  "version": "4.4.0",
+  "version": "4.4.2",
   "author": {
     "name": "Baqend.com",
     "email": "info@baqend.com"
@@ -94,6 +94,7 @@
     "@oclif/plugin-warn-if-update-available": "^3.1.61",
     "baqend": "^4.2.5",
     "chalk": "^5.6.2",
+    "chokidar": "^5.0.0",
     "cli-progress": "^3.12.0",
     "clipboardy": "^5.3.1",
     "deepmerge": "^4.3.1",