fix: drop explicit ui:install step
This commit is contained in:
@@ -157,7 +157,7 @@ export function handleControlUiHttpRequest(
|
||||
res.statusCode = 503;
|
||||
res.setHeader("Content-Type", "text/plain; charset=utf-8");
|
||||
res.end(
|
||||
"Control UI assets not found. Build them with `bun run ui:build` (or run `bun run ui:dev` during development).",
|
||||
"Control UI assets not found. Build them with `pnpm ui:build` (auto-installs UI deps), or run `pnpm ui:dev` during development.",
|
||||
);
|
||||
return true;
|
||||
}
|
||||
|
||||
@@ -76,7 +76,7 @@ export async function ensureControlUiAssetsBuilt(
|
||||
return {
|
||||
ok: false,
|
||||
built: false,
|
||||
message: `${hint}. Build them with \`bun run ui:build\`.`,
|
||||
message: `${hint}. Build them with \`pnpm ui:build\` (auto-installs UI deps).`,
|
||||
};
|
||||
}
|
||||
|
||||
@@ -94,27 +94,7 @@ export async function ensureControlUiAssetsBuilt(
|
||||
};
|
||||
}
|
||||
|
||||
runtime.log("Control UI assets missing; building (ui:build)…");
|
||||
|
||||
const ensureInstalled = !fs.existsSync(
|
||||
path.join(repoRoot, "ui", "node_modules"),
|
||||
);
|
||||
if (ensureInstalled) {
|
||||
const install = await runCommandWithTimeout(
|
||||
[process.execPath, uiScript, "install"],
|
||||
{
|
||||
cwd: repoRoot,
|
||||
timeoutMs: opts?.timeoutMs ?? 10 * 60_000,
|
||||
},
|
||||
);
|
||||
if (install.code !== 0) {
|
||||
return {
|
||||
ok: false,
|
||||
built: false,
|
||||
message: `Control UI install failed: ${summarizeCommandOutput(install.stderr) ?? `exit ${install.code}`}`,
|
||||
};
|
||||
}
|
||||
}
|
||||
runtime.log("Control UI assets missing; building (ui:build, auto-installs UI deps)…");
|
||||
|
||||
const build = await runCommandWithTimeout(
|
||||
[process.execPath, uiScript, "build"],
|
||||
|
||||
@@ -304,15 +304,6 @@ export async function runGatewayUpdate(
|
||||
timeoutMs,
|
||||
),
|
||||
);
|
||||
steps.push(
|
||||
await runStep(
|
||||
runCommand,
|
||||
"ui:install",
|
||||
managerScriptArgs(manager, "ui:install"),
|
||||
gitRoot,
|
||||
timeoutMs,
|
||||
),
|
||||
);
|
||||
steps.push(
|
||||
await runStep(
|
||||
runCommand,
|
||||
|
||||
Reference in New Issue
Block a user