@gjsify/cli 0.62.0 → 0.63.0

package/lib/utils/install-backend-native.js

@@ -63,7 +63,7 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import * as os from 'node:os';
 import { Range, SemVer, maxSatisfying, satisfies } from '@gjsify/semver';
-import { DEFAULT_REGISTRY, fetchPackument, fetchPackumentConditional, fetchTarball, parseNpmrc, registryFor, } from '@gjsify/npm-registry';
+import { DEFAULT_REGISTRY, fetchPackument, fetchPackumentConditional, fetchTarball, PackageNotFoundError, parseNpmrc, registryFor, } from '@gjsify/npm-registry';
 import { extractTarball } from '@gjsify/tar';
 import { atomicWriteStrict } from './install-cache-fs.js';
 import { acquireInstallLock } from './install-lock.js';
@@ -229,7 +229,7 @@ async function installPackagesNativeLocked(opts, npmrc, log) {
         // tree and the same recorded declarations on every machine (header note,
         // invariant 1), and giving it nothing target-shaped to read is what keeps
         // that true.
-        const resolveResult = await resolveDeps(opts.specs, npmrc, log, opts.overrides, opts.skipDeps, opts.signal, progress, preferred, opts.workspaceNames, opts.specOrigins, opts.optionalSpecs);
+        const resolveResult = await resolveDeps(opts.specs, npmrc, log, opts.overrides, opts.skipDeps, opts.signal, progress, preferred, opts.workspaceNames, opts.specOrigins, opts.optionalSpecs, opts.linkedNames);
         nodes = resolveResult.nodes;
         skippedEdges = resolveResult.skippedEdges;
         resolved = true;
@@ -507,7 +507,17 @@ specOrigins,
  * so a resolve failure is skipped rather than fatal. The FINAL flag still comes
  * from {@link computeOptionalFlags}, not from this walk.
  */
-optionalSpecs) {
+optionalSpecs, 
+/**
+ * Names provided by a `gjsify link` override — see
+ * {@link NativeInstallOptions.linkedNames}. Read ONLY in the failure path: a
+ * linked name the registry does not have at all is provided by the local
+ * checkout, so the edge is tolerated instead of fatal. Deliberately NOT used to
+ * skip the edge up front the way `workspaceNames` is, which is what keeps the
+ * designed case — a link over a PUBLISHED package — resolving into the same tree
+ * and hence the same lockfile.
+ */
+linkedNames) {
     progress?.beginPhase('resolve', specs.length);
     const applyOverride = (name, range) => {
         if (!overrides)
@@ -746,6 +756,35 @@ optionalSpecs) {
                 }
             }
             catch (e) {
+                // A dev-linked name the registry does not have AT ALL is provided by
+                // the local checkout, and an UNPUBLISHED package is a normal thing to
+                // link — a consumer developing against a private sibling repo has
+                // nothing on the registry to resolve. Before this branch the resolve
+                // died with "Package not found in registry: <name>" and no install
+                // could run at all, however correct the symlinks were.
+                //
+                // Tolerated HERE and not skipped before the fetch, which is the whole
+                // difference from `workspaceNames`: in the designed case — a link over
+                // a published package — the resolve still succeeds, the node still
+                // enters the tree, and the consumer's committed lockfile stays
+                // byte-identical (utils/dev-link.ts).
+                //
+                // A 404 ONLY. A timeout or a dropped connection is not an answer about
+                // what the registry holds, and treating it as one would write a
+                // lockfile missing the whole linked subtree over a network blip.
+                //
+                // NOT recorded in `skippedEdges`: that set is re-judged by
+                // `assertRequiredEdgesResolved`, which asks whether a MISSING package
+                // is required — and this one is not missing. What is genuinely absent
+                // is the linked package's own dependency subtree, since the walk never
+                // saw a manifest for it; the consumer declares what it needs itself
+                // (for `@girs/*` that is the peer-plus-dev-pin shape in lotse's widget
+                // package), and a real gap surfaces as an unresolved import, not as a
+                // silently incomplete tree.
+                if (linkedNames?.has(edge.name) && e instanceof PackageNotFoundError) {
+                    log('resolve: dev-linked %s not published — provided by the local checkout', edge.name);
+                    continue;
+                }
                 // Optional deps that fail to resolve are skipped (yarn/npm behaviour);
                 // required deps re-throw.
                 if (!edge.required) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gjsify/cli",
-    "version": "0.62.0",
+    "version": "0.63.0",
     "description": "CLI for Gjsify",
     "type": "module",
     "main": "lib/index.js",
@@ -142,22 +142,22 @@
         "cli"
     ],
     "dependencies": {
-        "@gjsify/blueprint": "^0.62.0",
-        "@gjsify/buffer": "^0.62.0",
-        "@gjsify/create-app": "^0.62.0",
-        "@gjsify/node-globals": "^0.62.0",
-        "@gjsify/node-polyfills": "^0.62.0",
-        "@gjsify/npm-registry": "^0.62.0",
-        "@gjsify/resolve-npm": "^0.62.0",
-        "@gjsify/rolldown-plugin-gjsify": "^0.62.0",
-        "@gjsify/vite-plugin-blueprint": "^0.62.0",
-        "@gjsify/rolldown-plugin-pnp": "^0.62.0",
-        "@gjsify/semver": "^0.62.0",
-        "@gjsify/tar": "^0.62.0",
-        "@gjsify/tsc": "^0.62.0",
-        "@gjsify/utils": "^0.62.0",
-        "@gjsify/web-polyfills": "^0.62.0",
-        "@gjsify/workspace": "^0.62.0",
+        "@gjsify/blueprint": "^0.63.0",
+        "@gjsify/buffer": "^0.63.0",
+        "@gjsify/create-app": "^0.63.0",
+        "@gjsify/node-globals": "^0.63.0",
+        "@gjsify/node-polyfills": "^0.63.0",
+        "@gjsify/npm-registry": "^0.63.0",
+        "@gjsify/resolve-npm": "^0.63.0",
+        "@gjsify/rolldown-plugin-gjsify": "^0.63.0",
+        "@gjsify/vite-plugin-blueprint": "^0.63.0",
+        "@gjsify/rolldown-plugin-pnp": "^0.63.0",
+        "@gjsify/semver": "^0.63.0",
+        "@gjsify/tar": "^0.63.0",
+        "@gjsify/tsc": "^0.63.0",
+        "@gjsify/utils": "^0.63.0",
+        "@gjsify/web-polyfills": "^0.63.0",
+        "@gjsify/workspace": "^0.63.0",
         "acorn": "^8.17.0",
         "cosmiconfig": "^9.0.2",
         "get-tsconfig": "^4.14.0",
@@ -166,13 +166,13 @@
         "yargs": "^18.0.0"
     },
     "devDependencies": {
-        "@gjsify/unit": "^0.62.0",
+        "@gjsify/unit": "^0.63.0",
         "@types/yargs": "^17.0.35",
         "typescript": "^6.0.3"
     },
     "peerDependencies": {
-        "@gjsify/oxfmt-native": "^0.62.0",
-        "@gjsify/rolldown-native": "^0.62.0"
+        "@gjsify/oxfmt-native": "^0.63.0",
+        "@gjsify/rolldown-native": "^0.63.0"
     },
     "peerDependenciesMeta": {
         "@gjsify/oxfmt-native": {