@sasjs/cli 6.32.0 → 6.32.1

package/build/commands/build/internal/mockSite/assets/sw.js

@@ -37,6 +37,7 @@
  */
 importScripts('host/runtime-core.js')
 importScripts('host/api-browser.js')
+importScripts('host/lint-browser.js')
 
 const R = self.SASjsMockRuntime
 
@@ -967,19 +968,18 @@ const LINT_CONFIG_FILE = '.sasjslint'
  * The linter, bundled at build time from the installed @sasjs/api/code and
  * @sasjs/lint (see mockSite.ts).
  *
- * Imported on the first code route rather than by the worker at boot: it is
- * around 40KB of the worker's payload for two routes (`code/lint` and
- * `code/format`), and the install has to finish before the page may make its
- * first request - so every visit pays it, including the ones that never open the
- * editor. The build stamps the version into the URL below, so the copy fetched
- * here is cache-busted with the rest of the host.
+ * Imported with the rest of the host, not on the first code route: a service
+ * worker may only `importScripts` while its own script is being evaluated, so a
+ * call made from a request handler fails with "The script at ... failed to load"
+ * - the linter would be missing exactly where it is needed. What stays lazy is the
+ * service built from it: `createCodeService` is cheap, and only a code route
+ * calls it.
  */
 let codeService
 let linter
 
 const code = () => {
   if (codeService) return codeService
-  importScripts('host/lint-browser.js')
   linter = self.SASjsCode
   codeService = linter.createCodeService({ lintConfig })
   return codeService

package/build/commands/build/internal/mockSite/spec/sw.spec.js

@@ -1442,25 +1442,27 @@ describe('service worker host', function () {
             });
         }); });
         /**
-         * The linter is not part of the install.
+         * The linter is imported with the host, because it has to be.
          *
-         * The page cannot make its first request until the worker has installed, so
-         * anything the worker imports at boot is on the critical path of every
-         * visit. The linter answers two routes; it is fetched when one of them is
-         * called.
+         * A service worker may only `importScripts` while its own script is being
+         * evaluated: a call made from a request handler fails with "The script at ...
+         * failed to load", which is what a lazy import gave a real browser - the two
+         * code routes answered 500. So the import is at boot, and the assertion is
+         * that a code route answers *without* importing anything at request time.
          */
-        it('fetches the linter on the first code route, not at boot', function () { return __awaiter(void 0, void 0, void 0, function () {
-            var worker, response;
+        it('imports the linter at boot and answers a code route without importing', function () { return __awaiter(void 0, void 0, void 0, function () {
+            var worker, before, response;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
                         worker = seeded();
-                        expect(worker.imported.some(function (path) { return path.includes('lint-browser'); })).toBe(false);
+                        expect(worker.imported.some(function (path) { return path.includes('lint-browser'); })).toBe(true);
+                        before = worker.imported.length;
                         return [4 /*yield*/, worker.dispatch('/SASjsApi/code/lint', runRequest({ code: 'data a;\nrun;\n' }))];
                     case 1:
                         response = _a.sent();
                         expect(response.status).toBe(200);
-                        expect(worker.imported.some(function (path) { return path.includes('lint-browser'); })).toBe(true);
+                        expect(worker.imported.length).toBe(before);
                         return [2 /*return*/];
                 }
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sasjs/cli",
-  "version": "6.32.0",
+  "version": "6.32.1",
   "description": "Command line interface for SASjs",
   "engines": {
     "node": "^22.14.0 || >=24.10.0"