@sparkelf/dsh-plus 0.1.0-rc.36 → 0.1.0-rc.38

package/lib/bin.js

@@ -2,7 +2,7 @@
 import { r as runApply } from "./apply-e0aMC4B4.js";
 import { createRequire } from "node:module";
 import { execFileSync, spawn, spawnSync } from "node:child_process";
-import { createWriteStream, existsSync, mkdirSync, readFileSync, readdirSync, rmSync, symlinkSync, writeFileSync } from "node:fs";
+import { createWriteStream, existsSync, mkdirSync, readFileSync, readdirSync, renameSync, rmSync, symlinkSync, writeFileSync } from "node:fs";
 import { dirname, join, posix, resolve, win32 } from "node:path";
 import { homedir } from "node:os";
 import semver from "semver";
@@ -364,13 +364,59 @@ function installProfilePackages(paths, consumerDirectory) {
 		return;
 	}
 	runPnpm(paths.profileDirectory, ["install", "--no-frozen-lockfile"], "pnpm install in the plus profile");
+	alignReplacedPackageNames(profileModules);
 	linkNestedBundles(paths, profileModules);
 }
-/** Run pnpm in one directory, inheriting its output. */
+/**
+* Make each replaced package declare the name of the location it occupies.
+*
+* \`overrides\` installs our build at the official path, but the manifest inside still
+* names our scope. The client module system resolves a loader entry's declared name and
+* then requires the manifest it finds to declare that same name
+* (\`client/modules\`: \`name === expectedPackageName\`); a mismatch makes the package own no
+* browser module at all. The symptom is silent — the packages install, the server starts,
+* and the panels those packages render simply never appear.
+*
+* The rewrite replaces the file rather than writing through it: pnpm hard-links a package
+* manifest into its content-addressed store, so an in-place write would edit every
+* profile sharing that store entry.
+*
+* @param profileModules - the profile's \`node_modules\` directory.
+*/
+function alignReplacedPackageNames(profileModules) {
+	const scoped = join(profileModules, "@deepseek-ai");
+	if (!existsSync(scoped)) return;
+	for (const entry of readdirSync(scoped)) {
+		const manifestPath = join(scoped, entry, "package.json");
+		if (!existsSync(manifestPath)) continue;
+		const declared = JSON.parse(readFileSync(manifestPath, "utf8"));
+		const expected = "@deepseek-ai/" + entry;
+		if (declared.name === expected) continue;
+		const replaced = {
+			...declared,
+			name: expected
+		};
+		const temporary = manifestPath + ".dsh-name";
+		writeFileSync(temporary, JSON.stringify(replaced, null, 2) + "\n");
+		renameSync(temporary, manifestPath);
+	}
+}
+/**
+* Run pnpm in one directory, inheriting its output.
+*
+* Windows resolves a command through its shell: \`spawnSync\` on a \`.cmd\` shim fails
+* with EINVAL, so the call goes through \`cmd.exe\` there. This matches the runner in
+* \`apply.ts\`, which the apply path has used on Windows since it shipped.
+*
+* @param cwd - the directory pnpm runs in.
+* @param args - pnpm arguments, without the executable.
+* @param label - the failing step's name, for the error.
+*/
 function runPnpm(cwd, args, label) {
-	const result = spawnSync(process.platform === "win32" ? "pnpm.cmd" : "pnpm", [...args], {
+	const result = spawnSync("pnpm", [...args], {
 		cwd,
-		stdio: "inherit"
+		stdio: "inherit",
+		shell: process.platform === "win32"
 	});
 	if (result.error !== void 0) throw result.error;
 	if (result.status !== 0) throw new Error(label + " failed with exit code " + String(result.status));

package/lib/types/standalone-profile.d.ts

@@ -77,6 +77,23 @@ export declare function readDistributionProfile(distributionDirectory: string):
  * @param consumerDirectory - directory whose `node_modules` holds the installation.
  */
 export declare function installProfilePackages(paths: StandalonePaths, consumerDirectory: string): void;
+/**
+ * Make each replaced package declare the name of the location it occupies.
+ *
+ * \`overrides\` installs our build at the official path, but the manifest inside still
+ * names our scope. The client module system resolves a loader entry's declared name and
+ * then requires the manifest it finds to declare that same name
+ * (\`client/modules\`: \`name === expectedPackageName\`); a mismatch makes the package own no
+ * browser module at all. The symptom is silent — the packages install, the server starts,
+ * and the panels those packages render simply never appear.
+ *
+ * The rewrite replaces the file rather than writing through it: pnpm hard-links a package
+ * manifest into its content-addressed store, so an in-place write would edit every
+ * profile sharing that store entry.
+ *
+ * @param profileModules - the profile's \`node_modules\` directory.
+ */
+export declare function alignReplacedPackageNames(profileModules: string): void;
 /** Resolve every path a command needs, without creating anything. */
 export declare function resolvePaths(anchor: string, env?: NodeJS.ProcessEnv): StandalonePaths;
 /**

package/lib/types/standalone-profile.js

@@ -9,7 +9,7 @@
  * bundle's own list.
  */
 import { spawnSync } from 'node:child_process';
-import { existsSync, mkdirSync, readdirSync, readFileSync, symlinkSync, writeFileSync } from 'node:fs';
+import { existsSync, mkdirSync, readdirSync, readFileSync, renameSync, symlinkSync, writeFileSync } from 'node:fs';
 import { createRequire } from 'node:module';
 import { homedir } from 'node:os';
 import { dirname, join, posix, resolve, win32 } from 'node:path';
@@ -160,12 +160,60 @@ export function installProfilePackages(paths, consumerDirectory) {
         return;
     }
     runPnpm(paths.profileDirectory, ['install', '--no-frozen-lockfile'], 'pnpm install in the plus profile');
+    alignReplacedPackageNames(profileModules);
     linkNestedBundles(paths, profileModules);
 }
-/** Run pnpm in one directory, inheriting its output. */
+/**
+ * Make each replaced package declare the name of the location it occupies.
+ *
+ * \`overrides\` installs our build at the official path, but the manifest inside still
+ * names our scope. The client module system resolves a loader entry's declared name and
+ * then requires the manifest it finds to declare that same name
+ * (\`client/modules\`: \`name === expectedPackageName\`); a mismatch makes the package own no
+ * browser module at all. The symptom is silent — the packages install, the server starts,
+ * and the panels those packages render simply never appear.
+ *
+ * The rewrite replaces the file rather than writing through it: pnpm hard-links a package
+ * manifest into its content-addressed store, so an in-place write would edit every
+ * profile sharing that store entry.
+ *
+ * @param profileModules - the profile's \`node_modules\` directory.
+ */
+export function alignReplacedPackageNames(profileModules) {
+    const scoped = join(profileModules, '@deepseek-ai');
+    if (!existsSync(scoped))
+        return;
+    for (const entry of readdirSync(scoped)) {
+        const manifestPath = join(scoped, entry, 'package.json');
+        if (!existsSync(manifestPath))
+            continue;
+        const declared = JSON.parse(readFileSync(manifestPath, 'utf8'));
+        const expected = '@deepseek-ai/' + entry;
+        if (declared.name === expected)
+            continue;
+        const replaced = { ...declared, name: expected };
+        const temporary = manifestPath + '.dsh-name';
+        writeFileSync(temporary, JSON.stringify(replaced, null, 2) + '\n');
+        renameSync(temporary, manifestPath);
+    }
+}
+/**
+ * Run pnpm in one directory, inheriting its output.
+ *
+ * Windows resolves a command through its shell: \`spawnSync\` on a \`.cmd\` shim fails
+ * with EINVAL, so the call goes through \`cmd.exe\` there. This matches the runner in
+ * \`apply.ts\`, which the apply path has used on Windows since it shipped.
+ *
+ * @param cwd - the directory pnpm runs in.
+ * @param args - pnpm arguments, without the executable.
+ * @param label - the failing step's name, for the error.
+ */
 function runPnpm(cwd, args, label) {
-    const command = process.platform === 'win32' ? 'pnpm.cmd' : 'pnpm';
-    const result = spawnSync(command, [...args], { cwd, stdio: 'inherit' });
+    const result = spawnSync('pnpm', [...args], {
+        cwd,
+        stdio: 'inherit',
+        shell: process.platform === 'win32',
+    });
     if (result.error !== undefined)
         throw result.error;
     if (result.status !== 0)

package/package.json

@@ -166,5 +166,5 @@
   },
   "type": "module",
   "types": "lib/types/index.d.ts",
-  "version": "0.1.0-rc.36"
+  "version": "0.1.0-rc.38"
 }