Address Grok adapter PR feedback

This commit is contained in:
Devin Foley committed 2026-05-15 17:51:55 -07:00
1 parent 9b3decfad6
commit 598cf919f5
4 files changed
+27 -2

No files matched your search

+1
View File
@@ -28,6 +28,7 @@ COPY packages/adapters/codex-local/package.json packages/adapters/codex-local/
COPY packages/adapters/cursor-cloud/package.json packages/adapters/cursor-cloud/
COPY packages/adapters/cursor-local/package.json packages/adapters/cursor-local/
COPY packages/adapters/gemini-local/package.json packages/adapters/gemini-local/
COPY packages/adapters/grok-local/package.json packages/adapters/grok-local/
COPY packages/adapters/openclaw-gateway/package.json packages/adapters/openclaw-gateway/
COPY packages/adapters/opencode-local/package.json packages/adapters/opencode-local/
COPY packages/adapters/pi-local/package.json packages/adapters/pi-local/
@@ -173,7 +173,7 @@ async function stageGrokProjectAssets(input: {
stagedInstructionsPath,
rulesFilePath,
cleanup: async () => {
for (const entry of cleanup.reverse()) {
for (const entry of [...cleanup].reverse()) {
if (entry.kind === "file") {
await fs.rm(entry.path, { force: true }).catch(() => undefined);
continue;
@@ -86,6 +86,23 @@ describe("grok_local testEnvironment", () => {
"grok_hello_probe_passed",
]),
);
expect(runProcessMock).toHaveBeenNthCalledWith(
2,
expect.any(String),
null,
"grok",
expect.arrayContaining([
"--output-format",
"streaming-json",
"--always-approve",
"--permission-mode",
"dontAsk",
"--disable-web-search",
"--single",
"Respond with exactly hello.",
]),
expect.any(Object),
);
});
it("downgrades auth failures to warnings", async () => {
@@ -241,7 +241,14 @@ export async function testEnvironment(
}
if (canRunProbe) {
const probeArgs = ["--output-format", "streaming-json", "--permission-mode", "dontAsk", "--disable-web-search", "--no-subagents", "--no-plan"];
const probeArgs = [
"--output-format",
"streaming-json",
"--always-approve",
"--permission-mode",
"dontAsk",
"--disable-web-search",
];
if (configuredModel && configuredModel !== DEFAULT_GROK_LOCAL_MODEL) {
probeArgs.push("--model", configuredModel);
}