@_linked/server 2.24.9 → 2.24.11

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @\_linked/server
 
+## 2.24.11
+
+### Patch Changes
+
+- [#175](https://github.com/linked-fw/server/pull/175) [`37318c4`](https://github.com/linked-fw/server/commit/37318c468ea9d18a33aa44dd441899e8fb0fbf8b) Thanks [@flyon](https://github.com/flyon)! - In dev, a source workspace's backend is found from `src/` (`src/backend.tsx`, `src/backend.ts`, or its `exports` target mapped from `lib/esm` back to `src`), the way linked's Vite resolver loads it. An unbuilt workspace, whose `exports` name only `lib/esm`, no longer has its `src/backend.ts` silently skipped. Published packages and production are unchanged.
+
+## 2.24.10
+
+### Patch Changes
+
+- [#173](https://github.com/linked-fw/server/pull/173) [`3181adf`](https://github.com/linked-fw/server/commit/3181adfdd7609f32c6866e3733580b8059c2ce9e) Thanks [@flyon](https://github.com/flyon)! - At boot the server no longer tries to import a workspace linked package that has no JavaScript entry (no `main`, `module` or root `exports`), such as the pure-CSS `@_linked/css`. Before, a pure-CSS package checked out as a workspace member logged `Error loading '…': Failed to resolve entry for package` on every start. The new `hasModuleEntry(packageJson)` in `utils/linkedDependencies` makes that decision.
+
 ## 2.24.9
 
 ### Patch Changes

package/lib/esm/shapes/LinkedServer.js

@@ -41,7 +41,7 @@ import { getShapeClass, getSuperShapesClasses, } from '@_linked/core/utils/Shape
 import path from 'path';
 import { pathToFileURL } from 'node:url';
 import { isFatalError } from '../utils/fatalError.js';
-import { discoverLinkedDependencies, isBundledBySsr, NODE_IMPORT_CONDITIONS, readInstalledPackage, resolveBackendEntry, ssrNoExternalOf, } from '../utils/linkedDependencies.js';
+import { discoverLinkedDependencies, hasModuleEntry, isBundledBySsr, NODE_IMPORT_CONDITIONS, readInstalledPackage, resolveBackendEntry, resolveDevBackendEntry, ssrNoExternalOf, } from '../utils/linkedDependencies.js';
 /**
  * `instanceof ShapeProvider`, tolerating a second copy of server-utils.
  *
@@ -1165,10 +1165,20 @@ let LinkedServer = class LinkedServer extends Shape {
         res.type(extension || 'bin').send(resized);
     }
     async indexLincdPackage(pkg, warnIfNotFound = false) {
-        var _a;
+        var _a, _b;
         if (pkg === this.package.name) {
             return;
         }
+        // A pure-CSS linked package has nothing to import whole.
+        const dir = (_a = this.linkedPackageDirs) === null || _a === void 0 ? void 0 : _a.get(pkg);
+        if (dir) {
+            try {
+                const json = JSON.parse(fsNative.readFileSync(path.join(dir, 'package.json'), 'utf-8'));
+                if (!hasModuleEntry(json))
+                    return;
+            }
+            catch (_c) { }
+        }
         try {
             // console.log(`🔍 Loading package: ${pkg}`);
             // console.log(
@@ -1182,7 +1192,7 @@ let LinkedServer = class LinkedServer extends Shape {
             // SECOND core copy (benign post-§P1, but a needless second instance).
             // Node import stays as the fallback when Vite isn't active (production /
             // Node-only CLI commands). Mirrors indexPackageBackendProviders.
-            const vite = (_a = this.config.server) === null || _a === void 0 ? void 0 : _a.vite;
+            const vite = (_b = this.config.server) === null || _b === void 0 ? void 0 : _b.vite;
             if (vite && typeof vite.ssrLoadModule === 'function') {
                 await vite.ssrLoadModule(pkg);
             }
@@ -1278,14 +1288,15 @@ let LinkedServer = class LinkedServer extends Shape {
             }
             if (dir && json) {
                 if (viteOwnsPackage) {
-                    // Vite resolves a bundled package with the `development` condition
-                    // (a workspace's `./backend` may point at src only there). It is
-                    // still loaded by its bare name, so it is the same module id the
+                    // A source package (one with src/) is a workspace to linked's Vite
+                    // config, whose resolver maps `<pkg>/backend` to src BEFORE
+                    // `exports` (resolveDevBackendEntry mirrors it). Checking only
+                    // `exports` missed an unbuilt workspace, whose `exports` name only
+                    // lib/esm: its src/backend.ts was never loaded. Otherwise Vite
+                    // resolves a bundled package with the `development` condition. It
+                    // is still loaded by its bare name, so it is the same module id the
                     // rest of the SSR graph imports.
-                    entry = resolveBackendEntry(dir, json, [
-                        'development',
-                        ...NODE_IMPORT_CONDITIONS,
-                    ]);
+                    entry = resolveDevBackendEntry(dir, json);
                     if (entry)
                         load = () => vite.ssrLoadModule(`${pkg}/backend`);
                 }