Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
{
"name": "@openrouter/spawn",
"version": "0.2.57",
"version": "0.2.58",
"type": "module",
"bin": {
"spawn": "cli.js"
Expand Down
18 changes: 16 additions & 2 deletions cli/src/__tests__/cmdlist-integration.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -64,6 +64,7 @@ mock.module("@clack/prompts", () => ({

// Import after mock setup
const { cmdList, resolveDisplayName } = await import("../commands.js");
const { loadManifest, _resetCacheForTesting } = await import("../manifest.js");

// ── Test Setup ──────────────────────────────────────────────────────────────────

Expand All@@ -90,12 +91,15 @@ describe("cmdList integration", () => {
return consoleMocks.error.mock.calls.map((c: any[]) => c.join(" ")).join("\n");
}

beforeEach(() => {

beforeEach(async () => {
testDir = join(tmpdir(), `spawn-cmdlist-test-${Date.now()}-${Math.random()}`);
mkdirSync(testDir, { recursive: true });

originalEnv = { ...process.env };
process.env.SPAWN_HOME = testDir;
// Isolate disk cache so tests don't read/write the real ~/.cache/spawn
process.env.XDG_CACHE_HOME = join(testDir, "cache");

consoleMocks = createConsoleMocks();
mockLogError.mockClear();
Expand All@@ -106,6 +110,15 @@ describe("cmdList integration", () => {
mockSpinnerStop.mockClear();

originalFetch = global.fetch;

// Prime the manifest in-memory cache with mock data so tests don't
// depend on network availability or stale values from other test files.
global.fetch = mock(() =>
Promise.resolve({ ok: true, json: async () => mockManifest }) as any
);
await loadManifest(true);
global.fetch = originalFetch;

processExitSpy = spyOn(process, "exit").mockImplementation((() => {
throw new Error("process.exit");
}) as any);
Expand DownExpand Up@@ -258,7 +271,8 @@ describe("cmdList integration", () => {
it("should fall back to raw keys when manifest is unavailable", async () => {
writeHistory(sampleRecords);

// Mock fetch to fail
// Clear in-memory cache and mock fetch to fail
_resetCacheForTesting();
global.fetch = mock(() => Promise.reject(new Error("Network error")));

await cmdList();
Expand Down
49 changes: 39 additions & 10 deletions cli/src/__tests__/list-display.test.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -32,20 +32,26 @@ function formatTimestamp(iso: string): string {
}
}

// ── Exact replica of parseListFilters from index.ts lines 303-316 ───────────
// ── Exact replica of parseListFilters from index.ts ─────────────────────────

function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a" && args[i + 1] && !args[i + 1].startsWith("-")) {
if ((args[i] === "-a" || args[i] === "--agent") && args[i + 1] && !args[i + 1].startsWith("-")) {
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c" && args[i + 1] && !args[i + 1].startsWith("-")) {
} else if ((args[i] === "-c" || args[i] === "--cloud") && args[i + 1] && !args[i + 1].startsWith("-")) {
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}
return { agentFilter, cloudFilter };
}

Expand DownExpand Up@@ -205,9 +211,9 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBeUndefined();
});

it("should return no filters for unrelated args", () => {
const result = parseListFilters(["list", "extra"]);
expect(result.agentFilter).toBeUndefined();
it("should use first positional arg as agent filter", () => {
const result = parseListFilters(["claude"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBeUndefined();
});

Expand DownExpand Up@@ -246,11 +252,10 @@ describe("parseListFilters", () => {
expect(result.cloudFilter).toBe("sprite");
});

it("should not confuse -a and --agent or -c and --cloud", () => {
// Only short flags -a and -c are supported
it("should support --agent and --cloud long flags", () => {
const result = parseListFilters(["--agent", "claude", "--cloud", "sprite"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("sprite");
});

it("should handle value with hyphens (e.g., agent name with hyphen)", () => {
Expand All@@ -263,6 +268,30 @@ describe("parseListFilters", () => {
expect(result.agentFilter).toBe("x");
expect(result.cloudFilter).toBe("y");
});

it("should not use positional arg when -a flag is present", () => {
const result = parseListFilters(["-a", "aider", "extra"]);
expect(result.agentFilter).toBe("aider");
expect(result.cloudFilter).toBeUndefined();
});

it("should not use positional arg when -c flag is present", () => {
const result = parseListFilters(["-c", "sprite", "extra"]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBe("sprite");
});

it("should return no filters for empty args", () => {
const result = parseListFilters([]);
expect(result.agentFilter).toBeUndefined();
expect(result.cloudFilter).toBeUndefined();
});

it("should support mixing --agent long flag with -c short flag", () => {
const result = parseListFilters(["--agent", "claude", "-c", "hetzner"]);
expect(result.agentFilter).toBe("claude");
expect(result.cloudFilter).toBe("hetzner");
});
});
});

Expand Down
6 changes: 4 additions & 2 deletions cli/src/commands.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -1257,8 +1257,9 @@ ${pc.bold("USAGE")}
spawn <agent> Show available clouds for agent
spawn <cloud> Show available agents for cloud
spawn list Browse and rerun previous spawns
spawn list -a <agent> Filter spawn history by agent
spawn list -c <cloud> Filter spawn history by cloud
spawn list <filter> Filter history by agent or cloud name
spawn list -a <agent> Filter spawn history by agent (or --agent)
spawn list -c <cloud> Filter spawn history by cloud (or --cloud)
Aliases: ls, history
spawn matrix Full availability matrix (alias: m)
spawn agents List all agents with descriptions
Expand All@@ -1280,6 +1281,7 @@ ${pc.bold("EXAMPLES")}
spawn claude ${pc.dim("# Show which clouds support Claude")}
spawn hetzner ${pc.dim("# Show which agents run on Hetzner")}
spawn list ${pc.dim("# Browse history and pick one to rerun")}
spawn list claude ${pc.dim("# Filter history by agent name")}
spawn matrix ${pc.dim("# See the full agent x cloud matrix")}

${pc.bold("AUTHENTICATION")}
Expand Down
22 changes: 16 additions & 6 deletions cli/src/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -63,7 +63,7 @@ const KNOWN_FLAGS = new Set([
"--version", "-v", "-V",
"--prompt", "-p", "--prompt-file", "-f",
"--dry-run", "-n",
"-a", "-c",
"-a", "-c", "--agent", "--cloud",
]);

/** Check for unknown flags and show an actionable error */
Expand DownExpand Up@@ -293,29 +293,39 @@ function warnExtraArgs(filteredArgs: string[], maxExpected: number): void {
}
}

/** Parse -a <agent> and -c <cloud> filter flags from args */
/** Parse -a/--agent <agent> and -c/--cloud <cloud> filter flags from args.
* Also accepts a bare positional arg as a filter (e.g. "spawn list claude"). */
function parseListFilters(args: string[]): { agentFilter?: string; cloudFilter?: string } {
let agentFilter: string | undefined;
let cloudFilter: string | undefined;
const positional: string[] = [];
for (let i = 0; i < args.length; i++) {
if (args[i] === "-a") {
if (args[i] === "-a" || args[i] === "--agent") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-a")} requires an agent name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires an agent name`));
console.error(`\nUsage: ${pc.cyan("spawn list -a <agent>")}`);
process.exit(1);
}
agentFilter = args[i + 1];
i++;
} else if (args[i] === "-c") {
} else if (args[i] === "-c" || args[i] === "--cloud") {
if (!args[i + 1] || args[i + 1].startsWith("-")) {
console.error(pc.red(`Error: ${pc.bold("-c")} requires a cloud name`));
console.error(pc.red(`Error: ${pc.bold(args[i])} requires a cloud name`));
console.error(`\nUsage: ${pc.cyan("spawn list -c <cloud>")}`);
process.exit(1);
}
cloudFilter = args[i + 1];
i++;
} else if (!args[i].startsWith("-")) {
positional.push(args[i]);
}
}

// Support bare positional filter: "spawn list claude" or "spawn list hetzner"
if (!agentFilter && !cloudFilter && positional.length > 0) {
agentFilter = positional[0];
}

return { agentFilter, cloudFilter };
}

Expand Down
5 changes: 5 additions & 0 deletions cli/src/manifest.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -208,4 +208,9 @@ export function countImplemented(m: Manifest): number {
return count;
}

/** Clear the in-memory manifest cache (for testing only) */
export function _resetCacheForTesting(): void {
_cached = null;
}

export { RAW_BASE, REPO, CACHE_DIR };