c06cc08453
`open` derives its own directory from import.meta.url at module scope. Webpack replaces that with the build machine's absolute path as a string literal, so a release built on macOS ships a file:///Users/... URL that fileURLToPath rejects on Windows (no drive letter), throwing on import. refreshXaiToken dynamic-imports the xAI OAuth service, which imports open eagerly, so every Grok token refresh silently failed and was swallowed by a catch that only logs a warning. Add open to serverExternalPackages so it keeps its real import.meta.url at runtime, and bundle it into the CLI package via ensureModuleInBundle (same guard already used for sql.js) since externalizing it means webpack no longer traces/copies it automatically.
39 lines
1.9 KiB
JavaScript
39 lines
1.9 KiB
JavaScript
import { describe, it, expect } from "vitest";
|
|
import { readFileSync } from "node:fs";
|
|
import { fileURLToPath } from "node:url";
|
|
import path from "node:path";
|
|
|
|
// Regression guard for the Windows xAI/Grok token refresh failure.
|
|
//
|
|
// `open` computes its own directory from `import.meta.url`. When it is bundled, webpack
|
|
// replaces that with the BUILD machine's absolute path as a string literal, so a release
|
|
// built on macOS ships `fileURLToPath("file:///Users/.../open/index.js")`. On Windows that
|
|
// throws ERR_INVALID_FILE_URL_PATH ("File URL path must be absolute" — no drive letter) at
|
|
// module scope, which kills every importer. `refreshXaiToken` imports the xAI OAuth
|
|
// service, which imports `open`, so no Grok refresh could ever reach auth.x.ai on Windows.
|
|
//
|
|
// Keeping the package external preserves the real `import.meta.url` at runtime.
|
|
const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../..");
|
|
|
|
describe("open must not be bundled into the server build", () => {
|
|
const config = readFileSync(path.join(repoRoot, "next.config.mjs"), "utf8");
|
|
|
|
it("is declared in serverExternalPackages", () => {
|
|
const match = config.match(/serverExternalPackages:\s*\[([^\]]*)\]/);
|
|
expect(match, "serverExternalPackages not found in next.config.mjs").toBeTruthy();
|
|
const packages = match[1].split(",").map((s) => s.trim().replace(/^["']|["']$/g, ""));
|
|
expect(packages).toContain("open");
|
|
});
|
|
|
|
it("stays a runtime dependency so the standalone output can resolve it", async () => {
|
|
const pkg = JSON.parse(readFileSync(path.join(repoRoot, "package.json"), "utf8"));
|
|
expect(pkg.dependencies?.open).toBeTruthy();
|
|
});
|
|
|
|
it("resolves its own directory from import.meta.url, which is why it must stay external", async () => {
|
|
const entry = path.join(repoRoot, "node_modules", "open", "index.js");
|
|
const source = readFileSync(entry, "utf8");
|
|
expect(source).toMatch(/import\.meta\.url/);
|
|
});
|
|
});
|