argus-reviewer-e2e 0.3.0 → 0.4.0

package/dist/vision/openrouter.d.ts

@@ -21,6 +21,8 @@ export interface JsonSchema {
 export interface OpenRouterClientOptions {
     apiKey: string;
     fetch?: typeof fetch;
+    /** Per-request timeout in ms. Default 120_000. */
+    timeoutMs?: number;
     /**
      * Extra metadata sent on every request. `trace` is merged into the request
      * body `trace` field for cost attribution. `headers` are merged into the
@@ -44,10 +46,12 @@ export interface OpenRouterClientOptions {
 export declare class OpenRouterClient {
     private _apiKey;
     private _fetch;
+    private _timeoutMs;
     private _trace;
     private _headers;
     private _onCall;
     constructor(opts: OpenRouterClientOptions);
+    private _request;
     complete(opts: {
         model: string;
         messages: Message[];

package/dist/vision/openrouter.js

@@ -1,8 +1,16 @@
 import { debug } from '../debug.js';
+import { classifyHttpStatus, CliError, pickProviderError } from '../ui/errors.js';
 import { makeCallCost } from './cost.js';
+/**
+ * Per-request timeout for OpenRouter calls. A hung connection otherwise
+ * blocks the caller forever — observed in the wild pinning a self-hosted
+ * review job for 90+ minutes on one socket.
+ */
+const REQUEST_TIMEOUT_MS = 120_000;
 export class OpenRouterClient {
     _apiKey;
     _fetch;
+    _timeoutMs;
     _trace;
     _headers;
     _onCall;
@@ -12,10 +20,14 @@ export class OpenRouterClient {
         }
         this._apiKey = opts.apiKey;
         this._fetch = opts.fetch ?? globalThis.fetch;
+        this._timeoutMs = opts.timeoutMs ?? REQUEST_TIMEOUT_MS;
         this._trace = opts.trace;
         this._headers = opts.headers;
         this._onCall = opts.onCall;
     }
+    _request(url, init) {
+        return this._fetch(url, { ...init, signal: AbortSignal.timeout(this._timeoutMs) });
+    }
     async complete(opts) {
         const candidates = [opts.model, ...(opts.escalationModels ?? [])];
         const errors = [];
@@ -53,10 +65,23 @@ export class OpenRouterClient {
                 errors.push(err);
             }
         }
-        throw new Error(`OpenRouter completion failed for all candidate models: ${errors.map((e) => e.message).join('; ')}`);
+        const prefix = 'OpenRouter completion failed for all candidate models';
+        // When every candidate failed for a provider/account reason, keep the
+        // class (R15) so the CLI can name it and offer the matching fix.
+        const classified = pickProviderError(errors);
+        if (classified !== undefined) {
+            throw new CliError(classified.code, classified.message, {
+                cause: classified,
+                ...(classified.retryAfterSeconds !== undefined
+                    ? { retryAfterSeconds: classified.retryAfterSeconds }
+                    : {}),
+                ...(classified.httpStatus !== undefined ? { httpStatus: classified.httpStatus } : {}),
+            });
+        }
+        throw new Error(`${prefix}: ${errors.map((e) => e.message).join('; ')}`);
     }
     async reconcile(id) {
-        const res = await this._fetch(`https://openrouter.ai/api/v1/generation?id=${encodeURIComponent(id)}`, {
+        const res = await this._request(`https://openrouter.ai/api/v1/generation?id=${encodeURIComponent(id)}`, {
             method: 'GET',
             headers: { Authorization: `Bearer ${this._apiKey}` },
         });
@@ -98,13 +123,14 @@ export class OpenRouterClient {
             'X-OpenRouter-Metadata': 'enabled',
             ...(this._headers ?? {}),
         };
-        const res = await this._fetch('https://openrouter.ai/api/v1/chat/completions', {
+        const res = await this._request('https://openrouter.ai/api/v1/chat/completions', {
             method: 'POST',
             headers,
             body: JSON.stringify(body),
         });
         if (!res.ok) {
-            throw new Error(`OpenRouter request failed: ${res.status} ${res.statusText}`);
+            throw (classifyHttpStatus(res.status, res.headers) ??
+                new Error(`OpenRouter request failed: ${res.status} ${res.statusText}`));
         }
         return (await res.json());
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argus-reviewer-e2e",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "argus-reviewer — open-source vision-model E2E testing harness. Record, replay, heal, and assert with any OpenRouter model under a hard budget cap.",
   "type": "module",
   "license": "MIT",
@@ -34,6 +34,7 @@
   },
   "scripts": {
     "build": "tsc -p tsconfig.build.json",
+    "check:dist": "test -z \"$(git status --porcelain -- dist/)\" && test -z \"$(git ls-files --others --ignored --exclude-standard -- dist/)\"",
     "test": "vitest run",
     "test:watch": "vitest",
     "lint": "eslint .",
@@ -42,16 +43,24 @@
     "prepare": "npm run build && chmod +x dist/cli.js",
     "eval": "node evals/run.mjs",
     "watch": "node scripts/watch.mjs",
+    "smoke:dashboard": "node tests/e2e/dashboard-smoke.mjs",
     "app": "electron electron",
-    "demo": "node scripts/demo.mjs"
+    "demo": "node scripts/demo.mjs",
+    "demo:record": "node scripts/demo-record.mjs",
+    "tokens": "node scripts/build-tokens.mjs",
+    "brand": "node scripts/build-brand.mjs",
+    "render:brand": "node scripts/render-brand.mjs",
+    "check:contrast": "node scripts/check-contrast.mjs"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
+    "@resvg/resvg-js": "^2.6.2",
     "@types/node": "^26.6.2",
     "electron": "^44.2.0",
     "eslint": "^10.11.0",
     "globals": "^17.12.0",
     "prettier": "^3.0.0",
+    "svgo": "^4.1.0",
     "typescript-eslint": "^8.0.0",
     "vitest": "^5.0.1"
   },