mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-07-31 20:32:20 +03:00
* fix(electron): bump electron 42→43 + build better-sqlite3 from source (ABI 148) (#6605) fix(electron): bump electron 42→43 + rebuild better-sqlite3 from source against the Electron ABI (148). Electron 43 raises NODE_MODULE_VERSION to 148; better-sqlite3@12.11.1 has no electron-v148 prebuild, so the packaged app died with 'Nenhum driver SQLite disponível'. prepare-electron-standalone now compiles better-sqlite3 from source against the electron headers into build/Release (where 'bindings' resolves it). Validated by Electron Package Smoke (green) + local (node_register_module_v148). Supersedes #6378. (--admin: the only reds are SonarQube/SonarCloud failing on a coverage-report artifact digest-mismatch — a GitHub Actions infra flake, not this diff; Sonar is green on main and the diff touches only the electron build.) * deps: bump the development group across 1 directory with 6 updates (#6588) deps: bump the development group (6 updates). Rebased onto current main; all checks green after the electron-smoke fix (#6605). * fix(proxy): force CONNECT tunnel for HTTP proxied requests (undici 8.7) + production deps bump (#6620) fix(proxy): force CONNECT tunnel for HTTP proxied requests (undici 8.7) + production deps bump. undici 8.6+ changed ProxyAgent to forward plain-HTTP via request-proxy instead of CONNECT, breaking OAuth refresh through a connection proxy (501). proxyDispatcher now passes proxyTunnel:true. Validated: Unit Tests 3/8 (the OAuth-proxy test) green, new regression test green (fails without the fix on undici 8.7), SonarQube green. Supersedes #6380. (--admin: the only red is Electron Package Smoke failing on a next-build artifact 'digest-mismatch' — a GitHub Actions infra flake corrupting the asar ('file data stream has unexpected number of bytes'); the better-sqlite3 rebuild itself succeeded (gyp ok) and the electron path is unchanged from #6605 which passed the smoke. Not this diff.) * fix(cli): detect WinGet Claude Code on Windows * chore(merge): drop unrelated main-drift from PR fork (deps/electron/proxy files belong to #6620, not this PR) Restores electron/package-lock.json, electron/package.json, package-lock.json, package.json, open-sse/utils/proxyDispatcher.ts, scripts/build/prepare-electron-standalone.mjs and tests/unit/proxy-dispatcher-family.test.ts to origin/release/v3.8.47's content. The PR fork branched from a state of main that already includes #6620 (proxy CONNECT tunnel fix + deps bump), which is not yet synced into release/v3.8.47 — the 3-way merge would otherwise silently carry that unrelated content into this doc-only PR. Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com> * chore(quality): rebaseline cliRuntime.ts file-size freeze for #6647 (1100->1110) The file was already exactly at the frozen 1100-line cap on release/v3.8.47. PR #6647's WinGet Claude Code detection path adds 10 lines (irreducible — the 62-char package folder name forces Prettier's 100-char width to break the path.join call across the same multi-line form used by every other long path in this function), tripping the Fast Quality Gates check:file-size job. Bumping the frozen cap to the file's real new size per the documented allowlist-with-justification policy (this is a pass/fail policy gate, not the ratchet metrics system). Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com> --------- Co-authored-by: Diego Rodrigues de Sa e Souza <8016841+diegosouzapw@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: quanturbo <faralechko@gmail.com> Co-authored-by: Diego Rodrigues de Sa e Souza <diegosouza.pw@gmail.com>
282 lines
10 KiB
TypeScript
282 lines
10 KiB
TypeScript
/**
|
|
* Tests for CLI tool detection: cross-platform known paths, size threshold,
|
|
* npm prefix deduplication, and env var overrides.
|
|
*/
|
|
|
|
import { describe, it, before, after } from "node:test";
|
|
import assert from "node:assert/strict";
|
|
import fs from "node:fs";
|
|
import os from "node:os";
|
|
import path from "node:path";
|
|
|
|
const { getCliRuntimeStatus, getKnownToolPaths, CLI_TOOL_IDS } =
|
|
await import("../../src/shared/services/cliRuntime.ts");
|
|
|
|
// ─── Helpers ──────────────────────────────────────────────────
|
|
|
|
function createTempDir() {
|
|
const testRoot = path.join(os.tmpdir(), "omniroute-test-tmp");
|
|
if (!fs.existsSync(testRoot)) {
|
|
fs.mkdirSync(testRoot, { recursive: true });
|
|
}
|
|
return fs.mkdtempSync(path.join(testRoot, "cli-test-"));
|
|
}
|
|
|
|
describe("Claude Code Windows known paths", () => {
|
|
it("should include the WinGet Anthropic.ClaudeCode install path", () => {
|
|
const localAppData = process.env.LOCALAPPDATA;
|
|
const expected = localAppData
|
|
? path.join(
|
|
localAppData,
|
|
"Microsoft",
|
|
"WinGet",
|
|
"Packages",
|
|
"Anthropic.ClaudeCode_Microsoft.Winget.Source_8wekyb3d8bbwe",
|
|
"claude.exe"
|
|
)
|
|
: null;
|
|
|
|
if (process.platform !== "win32" || !expected) return;
|
|
|
|
assert.ok(
|
|
getKnownToolPaths("claude").includes(expected),
|
|
"Claude Code installed by WinGet should be discoverable without CLI_CLAUDE_BIN"
|
|
);
|
|
});
|
|
});
|
|
|
|
function createFile(dir, name, content) {
|
|
const filePath = path.join(dir, name);
|
|
fs.writeFileSync(filePath, content);
|
|
if (process.platform !== "win32") {
|
|
fs.chmodSync(filePath, 0o755);
|
|
}
|
|
return filePath;
|
|
}
|
|
|
|
// ─── CLI_TOOL_IDS ─────────────────────────────────────────────
|
|
|
|
describe("CLI_TOOL_IDS", () => {
|
|
it("should include all expected tools from cliRuntime.ts (separate from CLI_TOOLS catalog)", () => {
|
|
// CLI_TOOL_IDS comes from cliRuntime.ts — a runtime-detection catalog that
|
|
// is SEPARATE from the UI catalog CLI_TOOLS in cliTools.ts.
|
|
// windsurf was removed from CLI_TOOLS (plan 14 D17) but may still be in
|
|
// cliRuntime.ts for binary detection purposes.
|
|
const expected = [
|
|
"claude",
|
|
"codex",
|
|
"droid",
|
|
"openclaw",
|
|
"cursor",
|
|
"cline",
|
|
"kilo",
|
|
"continue",
|
|
"opencode",
|
|
"qoder",
|
|
"qwen",
|
|
];
|
|
for (const id of expected) {
|
|
assert.ok(CLI_TOOL_IDS.includes(id), `Missing tool: ${id}`);
|
|
}
|
|
});
|
|
});
|
|
|
|
// ─── Size Threshold (30 bytes) ────────────────────────────────
|
|
|
|
describe("Size threshold — checkKnownPath", () => {
|
|
let tmpDir;
|
|
|
|
before(() => {
|
|
tmpDir = createTempDir();
|
|
});
|
|
|
|
after(() => {
|
|
fs.rmSync(tmpDir, { recursive: true, force: true });
|
|
});
|
|
|
|
it("should detect files >= 30 bytes via env var", async () => {
|
|
const prev = process.env.CLI_DROID_BIN;
|
|
// Create a valid 30-byte+ script (using spaces/comments for padding, NO \r on linux)
|
|
const content =
|
|
process.platform === "win32"
|
|
? "@echo off\r\necho 1.0.0\r\nREM PADDING_PADDIN\r\nexit 0\r\n"
|
|
: "#!/bin/sh\necho 1.0.0\n# PADDING_PADDING_PAD\nexit 0\n";
|
|
const script = createFile(tmpDir, "droid-valid", content);
|
|
// Verify it's at least 30 bytes
|
|
const stat = fs.statSync(script);
|
|
assert.ok(stat.size >= 30, `File should be >= 30 bytes, got ${stat.size}`);
|
|
|
|
process.env.CLI_DROID_BIN = script;
|
|
try {
|
|
const result = await getCliRuntimeStatus("droid");
|
|
assert.ok(result.installed, `Expected installed=true, got reason=${result.reason}`);
|
|
assert.ok(result.commandPath === script, `Expected commandPath=${script}`);
|
|
} finally {
|
|
if (prev !== undefined) process.env.CLI_DROID_BIN = prev;
|
|
else delete process.env.CLI_DROID_BIN;
|
|
}
|
|
});
|
|
|
|
it("should detect a valid CLI script (>= 30 bytes) via env var", async () => {
|
|
const prev = process.env.CLI_DROID_BIN;
|
|
// Ensure the size stays > 30 bytes without \r\n on bash
|
|
const content =
|
|
process.platform === "win32"
|
|
? "@echo off\r\necho 1.0.0\r\nREM PADDING_PAD\r\n"
|
|
: "#!/bin/sh\necho 1.0.0\n# PADDING_PADDING_PAD\n";
|
|
const script =
|
|
process.platform === "win32"
|
|
? createFile(tmpDir, "droid.cmd", content)
|
|
: createFile(tmpDir, "droid", content);
|
|
|
|
process.env.CLI_DROID_BIN = script;
|
|
try {
|
|
const result = await getCliRuntimeStatus("droid");
|
|
assert.ok(result.installed, `Expected installed=true, got reason=${result.reason}`);
|
|
assert.ok(
|
|
result.commandPath === script,
|
|
`Expected commandPath=${script}, got ${result.commandPath}`
|
|
);
|
|
} finally {
|
|
if (prev !== undefined) process.env.CLI_DROID_BIN = prev;
|
|
else delete process.env.CLI_DROID_BIN;
|
|
}
|
|
});
|
|
});
|
|
|
|
// ─── Healthcheck with --version ───────────────────────────────
|
|
|
|
describe("Healthcheck — checkRunnable", () => {
|
|
let tmpDir;
|
|
|
|
before(() => {
|
|
tmpDir = createTempDir();
|
|
});
|
|
|
|
after(() => {
|
|
fs.rmSync(tmpDir, { recursive: true, force: true });
|
|
});
|
|
|
|
it("should report runnable=true for a script that outputs version", async () => {
|
|
const prev = process.env.CLI_CLINE_BIN;
|
|
const script =
|
|
process.platform === "win32"
|
|
? createFile(tmpDir, "good.cmd", "@echo off\necho 1.0.0\n")
|
|
: createFile(tmpDir, "good", "#!/bin/sh\necho 1.0.0\n");
|
|
|
|
process.env.CLI_CLINE_BIN = script;
|
|
try {
|
|
const result = await getCliRuntimeStatus("cline");
|
|
assert.ok(result.installed, `Expected installed=true, got reason=${result.reason}`);
|
|
if (result.runnable) {
|
|
assert.ok(result.reason === null, `Expected no reason, got ${result.reason}`);
|
|
}
|
|
} finally {
|
|
if (prev !== undefined) process.env.CLI_CLINE_BIN = prev;
|
|
else delete process.env.CLI_CLINE_BIN;
|
|
}
|
|
});
|
|
|
|
it("should detect qodercli via env override and mark it runnable", async () => {
|
|
const prev = process.env.CLI_QODER_BIN;
|
|
const script =
|
|
process.platform === "win32"
|
|
? createFile(tmpDir, "qoder.cmd", "@echo off\necho qodercli 0.1.37\n")
|
|
: createFile(tmpDir, "qodercli", "#!/bin/sh\necho qodercli 0.1.37\n");
|
|
|
|
process.env.CLI_QODER_BIN = script;
|
|
try {
|
|
const result = await getCliRuntimeStatus("qoder");
|
|
assert.equal(result.installed, true);
|
|
assert.equal(result.runnable, true);
|
|
assert.equal(result.commandPath, script);
|
|
assert.equal(result.reason, null);
|
|
} finally {
|
|
if (prev !== undefined) process.env.CLI_QODER_BIN = prev;
|
|
else delete process.env.CLI_QODER_BIN;
|
|
}
|
|
});
|
|
});
|
|
|
|
// ─── Unknown tool ─────────────────────────────────────────────
|
|
|
|
describe("Unknown tool", () => {
|
|
it("should return unknown_tool for non-existent tool", async () => {
|
|
const result = await getCliRuntimeStatus("nonexistent-tool-xyz");
|
|
assert.equal(result.installed, false);
|
|
assert.equal(result.reason, "unknown_tool");
|
|
});
|
|
});
|
|
|
|
// ─── continue tool (requiresBinary: false) ────────────────────
|
|
|
|
describe("continue tool — no binary required", () => {
|
|
it("should report installed=true without checking binary", async () => {
|
|
const result = await getCliRuntimeStatus("continue");
|
|
assert.equal(result.installed, true);
|
|
assert.equal(result.reason, "not_required");
|
|
});
|
|
});
|
|
|
|
// Note: windsurf was removed from CLI_TOOLS in plan 14 D17 (MITM backlog plan 11).
|
|
// cliRuntime.ts may still have windsurf for binary detection (separate catalog).
|
|
// This test is skipped if windsurf is not registered in cliRuntime.ts.
|
|
describe("windsurf tool — guide-only integration (cliRuntime.ts)", () => {
|
|
it("should handle getCliRuntimeStatus for windsurf if it exists in cliRuntime catalog", async () => {
|
|
if (!CLI_TOOL_IDS.includes("windsurf")) {
|
|
// windsurf removed from runtime detection catalog too — skip
|
|
return;
|
|
}
|
|
const result = await getCliRuntimeStatus("windsurf");
|
|
assert.equal(result.installed, true);
|
|
assert.equal(result.runnable, true);
|
|
assert.equal(result.reason, "not_required");
|
|
});
|
|
});
|
|
|
|
// ─── resolveOpencodeConfigPath — cross-platform ─────────────────
|
|
|
|
const { resolveOpencodeConfigPath: resolveOpencodeConfigPathFn } =
|
|
await import("../../src/shared/services/cliRuntime.ts");
|
|
|
|
describe("resolveOpencodeConfigPath — cross-platform", () => {
|
|
it("should resolve on Linux with XDG_CONFIG_HOME", () => {
|
|
const result = resolveOpencodeConfigPathFn(
|
|
"linux",
|
|
{ XDG_CONFIG_HOME: "/tmp/xdg" },
|
|
"/home/dev"
|
|
);
|
|
assert.equal(result, path.join("/tmp/xdg", "opencode", "opencode.json"));
|
|
});
|
|
|
|
it("should resolve on Linux with default .config", () => {
|
|
const result = resolveOpencodeConfigPathFn("linux", {}, "/home/dev");
|
|
assert.equal(result, path.join("/home/dev", ".config", "opencode", "opencode.json"));
|
|
});
|
|
|
|
it("should resolve on Windows under ~/.config (XDG, NOT %APPDATA% — #3330)", () => {
|
|
// #3330: OpenCode reads its config from ~/.config/opencode on every
|
|
// platform, including Windows (%USERPROFILE%\.config). %APPDATA% is ignored.
|
|
const result = resolveOpencodeConfigPathFn(
|
|
"win32",
|
|
{ APPDATA: "C:\\Users\\dev\\AppData\\Roaming" },
|
|
"C:\\Users\\dev"
|
|
);
|
|
assert.equal(result, path.join("C:\\Users\\dev", ".config", "opencode", "opencode.json"));
|
|
});
|
|
|
|
it("should resolve on Windows under ~/.config without APPDATA (#3330)", () => {
|
|
const result = resolveOpencodeConfigPathFn("win32", {}, "C:\\Users\\dev");
|
|
assert.equal(result, path.join("C:\\Users\\dev", ".config", "opencode", "opencode.json"));
|
|
});
|
|
|
|
it("should honor XDG_CONFIG_HOME on Windows too (#3330)", () => {
|
|
const result = resolveOpencodeConfigPathFn(
|
|
"win32",
|
|
{ XDG_CONFIG_HOME: "D:\\xdg" },
|
|
"C:\\Users\\dev"
|
|
);
|
|
assert.equal(result, path.join("D:\\xdg", "opencode", "opencode.json"));
|
|
});
|
|
});
|