Skip to content

chore(release): v3.6.6 — Stabilization - #1241

Merged
diegosouzapw merged 86 commits into
mainfrom
release/v3.6.6
Apr 16, 2026
Merged

chore(release): v3.6.6 — Stabilization#1241
diegosouzapw merged 86 commits into
mainfrom
release/v3.6.6

Conversation

@diegosouzapw

@diegosouzapw diegosouzapw commented Apr 14, 2026

Copy link
Copy Markdown
Owner

[3.6.6] — 2026-04-15

✨ New Features

  • feat(storage): Add database backup cleanup controls, UI management, and customizable retention period env vars (feat(storage): add database backup cleanup controls #1304)
  • feat(providers): Add Freepik Pikaso image generation provider with support for cookie/subscription-based auth modes (feat: Add Freepik Pikaso image generation provider (cookie/subscription-based) #1277)
  • feat(providers): Add Perplexity Web (Session) Provider — Routes through Perplexity's internal SSE API using a session cookie, giving native proxy access without separate API costs to GPT-5.4, Claude Opus, Gemini 3.1 Pro, and Nemotron via preferences mapping (feat: Add Perplexity Web (Session) provider #1289)
  • feat(api): Sync Tokens & V1 WebSocket Bridge — Dedicated sync token storage, issuance, revocation, and bundle download routes backed by stable config bundle versioning with ETag support. Exposes /v1/ws WebSocket upgrade route and a custom Next.js server bridge (scripts/v1-ws-bridge.mjs) so OpenAI-compatible WebSocket traffic can be proxied through the gateway. Compliance auditing expanded with structured metadata, pagination, request context, auth/provider credential events, and SSRF-blocked validation logging. New migrations: 024_create_sync_tokens.sql. New modules: syncTokens.ts, src/lib/sync/bundle.ts, src/lib/sync/tokens.ts, src/lib/ws/handshake.ts, src/lib/apiBridgeServer.ts, src/lib/compliance/providerAudit.ts.
  • feat(models): GLM Thinking Preset & Hybrid Token Counting — GLM Thinking (glmt) registered as a first-class provider preset with shared GLM model metadata, pricing, per-connection usage sync, dashboard support, and maxTokens: 65536 / thinkingBudgetTokens: 24576 request defaults with 900s extended timeout. Provider-side /messages/count_tokens endpoint used when a Claude-compatible upstream supports it; gracefully falls back to estimation on missing models, missing credentials, or upstream failures. Startup seeding of default model aliases (src/lib/modelAliasSeed.ts) normalizes common cross-proxy model dialects so canonical slash-based model IDs are not misrouted. New file open-sse/config/glmProvider.ts.
  • feat(core): Hardened Outbound Provider Calls & Cooldown Retries — Guarded outbound fetch helpers (src/shared/network/safeOutboundFetch.ts, src/shared/network/outboundUrlGuard.ts) blocking private/local URLs with configurable retry, timeout normalisation, and route-level status propagation for provider validation and model discovery. Cooldown-aware chat retries (src/sse/services/cooldownAwareRetry.ts) with configurable requestRetry and maxRetryIntervalSec settings and model-scoped cooldown responses. Improved rate-limit learning from headers and error bodies so short upstream lockouts can recover automatically. Runtime environment validation (src/lib/env/runtimeEnv.ts) checks env at startup. Pollinations now requires an API key. Antigravity and Codex header handling aligned via open-sse/config/antigravityUpstream.ts and open-sse/config/codexClient.ts. Gemini tool names restored in translated responses; synthetic Claude text block injected when upstream SSE completes empty.
  • feat(logs): Add TPS (Tokens Per Second) metric to log details modal metadata grid ([Feature] add TPS metric #1182)
  • feat(memory+skills): Full-featured Memory & Skills systems with FTS5 SQLite search, dynamic UI pagination, backend observability, and extensive test coverage (feat(memory+skills): full-featured memory & skills systems with tests #1228)
  • feat(bailian-quota): Add Alibaba Coding Plan quota monitoring, multi-window quota extraction, and UI credential validation (feat(bailian-quota): add Alibaba Coding Plan quota monitoring #1235)
  • feat(storage): Call Log Storage Refactor — Extracted heavy request/response JSON payloads from the core SQLite database (storage.sqlite) into filesystem artifacts stored within DATA_DIR/call_logs. This massively reduces WAL bloat and eliminates SQLITE_FULL crashes on high-traffic nodes (Backport call log storage refactor to release/v3.6.6 #1307).
  • feat(providers): Add Grok Web (Subscription) Provider — Routes through the xAI web interface for subscription users via cookie session mapping (feat: Add Grok Web (Subscription) provider #1295).
  • feat(api): Advanced Media Support — Extends OpenAI generic proxy layer to natively support image, embeddings, audio-transcriptions, and audio-speech workflows (feat(api): support more OpenAI types #1297).
  • feat(cli-tools): Qwen Code CLI Integration — Full integration for Qwen Code local execution mapping, model resolution, and dynamic API key fetching (feat(cli-tools): add Qwen Code CLI integration #1266, fix(cli-tools): resolve API key resolution and model mapping bugs #1263).
  • feat(oauth): Supports cursor-agent CLI as a native Cursor credential source alongside the standard configuration (feat(oauth): support cursor-agent CLI as Cursor credential source #1258).
  • feat(models): Custom and imported models now merge correctly into filter lists for all available global providers ([Feature] filter for custom model #1191).

🐛 Bug Fixes

🔧 Internal Improvements

  • Compliance Audit Expansion: src/lib/compliance/index.ts expanded with structured metadata, pagination support, request context enrichment, and new providerAudit.ts module logging auth and provider credential events, SSRF-blocked validation attempts, and provider CRUD operations
  • Config Sync Bundle: src/lib/sync/bundle.ts exports buildConfigBundle() generating a versioned JSON snapshot of settings, provider connections, nodes, model aliases, combos, and API keys (passwords redacted) with ETag support for bandwidth-efficient polling
  • Codex Client Constants: Centralized CODEX_CLIENT_VERSION, CODEX_USER_AGENT_PLATFORM, and pattern-validated env overrides (CODEX_CLIENT_VERSION, CODEX_USER_AGENT) in open-sse/config/codexClient.ts
  • Antigravity Upstream Constants: open-sse/config/antigravityUpstream.ts consolidates all Antigravity base URLs and model/fetchAvailableModels discovery path builders
  • Model Alias Seed: src/lib/modelAliasSeed.ts seeds 30+ cross-proxy model dialect aliases (e.g. openai/gpt-5gpt-5, anthropic/claude-opus-4-6cc/claude-opus-4-6) at startup via idempotent upsert
  • Test Coverage: 15+ new unit test suites covering sync routes, WebSocket bridge, compliance index, GLM provider config, cooldown-aware retry, safe outbound fetch, stream utilities, Codex executor, provider validation branches, model cross-proxy compatibility, and model alias seeding
  • TypeScript Migration: Finalized migration of remaining JS tests (proxy-load and testFromFile) to TypeScript ES modules, ensuring a fully synchronized TS stack.
  • Reliability & Resilience: Added exponential backoff to models.dev auto-sync to combat transient network failures, raised interval floor to 1 hour, and added LKGP debug logging for enhanced observability during routing. (improve: models.dev retry backoff, sync interval floor, LKGP debug logging #1286)

diegosouzapw and others added 10 commits April 14, 2026 07:46
…\n\n artifacts

- Changed regex quantifier from ? to * in combo.ts, comboAgentMiddleware.ts,
  and contextHandoff.ts to greedily strip all JSON-escaped newline sequences
  surrounding <omniModel> tags in SSE streaming chunks
- Added \r to the character class for cross-platform robustness
- Fixed Playwright strict-mode violation in combo-unification.spec.ts
- Bumped OpenAPI version and CHANGELOG to 3.6.6
- fix(gemini): strip VS Code JSON Schema extensions from tool schemas (#1175)
  Add enumDescriptions, markdownDescription, markdownEnumDescriptions,
  enumItemLabels and tags to UNSUPPORTED_SCHEMA_CONSTRAINTS so the Gemini
  sanitizer removes them before forwarding. GitHub Copilot injects these
  non-standard fields into tool definitions, causing Gemini to reject with
  'Unknown name enumDescriptions at functionDeclarations[n].parameters'.

- fix(health-check): unwrap proxy config object before passing to getAccessToken (#1187 #1218)
  resolveProxyForConnection() returns { proxy, level, levelId } but the health
  check loop was passing the full wrapper to getAccessToken(), which expects the
  inner config object (.host, .port etc). The proxy dispatcher validated .host
  on the wrapper (undefined) and threw 'Context proxy host is required', silently
  marking every connection as unhealthy every sweep. Fix mirrors the pattern
  already used in chatHelpers.ts: proxyResult?.proxy || null.

- fix(ui): debounce models.dev sync interval slider to save only on release (#1202)
  The slider's onChange fired updateInterval() on every drag tick, sending a
  PATCH per pixel of movement. Rapid API responses overwrote UI state mid-drag.
  Introduce draftIntervalHours for smooth visual feedback; the PATCH fires
  on onMouseUp / onBlur once the user releases the control.
- fix(core): #1206 inject startup guard against app/ and src/app/ conflict
- fix(health): #1220 add HEALTHCHECK_STAGGER_MS to prevent token refresh bursting
- fix(proxy): #1231 prioritize HTTP 429 over quota body heuristics
- fix(sse): #1211 strip leading double-newlines in responses API stream
@github-actions

github-actions Bot commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: cancelled
  • PR test policy: success

Coverage artifact was not available for this run.

Comment thread tests/unit/bailian-quota-fetcher.test.ts Fixed
Comment thread tests/unit/bailian-quota-fetcher.test.ts Fixed
Comment thread tests/unit/bailian-quota-fetcher.test.ts Fixed
Comment thread tests/unit/bailian-quota-fetcher.test.ts Fixed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9949fad67b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

await this.pendingLoad;
return;
}
if (!this.isCacheStale()) return;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Scope skills cache staleness by apiKeyId

loadFromDatabase(apiKeyId) is now guarded by a single global freshness check, so once one API key loads skills, subsequent calls for a different API key within 60 seconds return early without querying that tenant's rows. Because handleChatCore now calls skillRegistry.loadFromDatabase(apiKeyInfo.id), this can cause cross-tenant misses where skills are not injected/executed for other API keys until the TTL expires.

Useful? React with 👍 / 👎.

Comment on lines +90 to +92
this._clientHeaders = input.clientHeaders ?? null;
try {
const result = await super.execute(input);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid shared client header state in GitHub executor

This stores request-specific headers on a mutable instance field before awaiting super.execute(). Executors are singletons from getExecutor, so concurrent GitHub requests can overwrite _clientHeaders and leak one request's x-initiator into another, causing incorrect upstream X-Initiator values (notably affecting billing semantics for agent vs user).

Useful? React with 👍 / 👎.


-- Step 6: Recreate triggers using memory_id (INTEGER rowid) instead of id (UUID TEXT)
CREATE TRIGGER IF NOT EXISTS memory_fts_ai AFTER INSERT ON memories BEGIN
INSERT INTO memory_fts(rowid, content, key) VALUES (new.memory_id, new.content, new.key);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Set memory_id before indexing new rows in memory_fts

The migration backfills memory_id only once for existing rows, but this trigger indexes future inserts using new.memory_id even though normal inserts do not populate that column. New memories therefore keep memory_id = NULL, so FTS entries get unrelated rowids and JOIN memory_fts ... ON m.memory_id = f.rowid silently drops newly created memories from semantic/hybrid FTS results.

Useful? React with 👍 / 👎.

Comment on lines 25 to +29
const apiKeyId = searchParams.get("apiKeyId") || undefined;
const type = (searchParams.get("type") as any) || undefined;
const sessionId = searchParams.get("sessionId") || undefined;
const limitParams = searchParams.get("limit");
const offsetParams = searchParams.get("offset");

const memories = await listMemories({
const result = await listMemories({

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Apply q filter when listing memories

The dashboard now sends q for search and removed client-side filtering, but this route never reads q or forwards any text filter to listMemories. As a result, typing in the memory search box does not narrow results, which is a functional regression in the new paginated flow.

Useful? React with 👍 / 👎.

Comment thread src/lib/memory/retrieval.ts Outdated
Comment on lines +246 to +247
rows = db.prepare(ftsQuery).all(...ftsParams) as MemoryRow[];
if (rows.length === 0) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep FTS matches in hybrid memory retrieval

In the hybrid branch, FTS query results are assigned to rows, but the merge step unions ftsRows with keyword rows and ftsRows is never populated. This discards all FTS-ranked hits and effectively degrades hybrid retrieval to the fallback chronological/keyword path.

Useful? React with 👍 / 👎.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates OmniRoute to version 3.6.6, introducing a new quota fetcher for Alibaba Coding Plan (Bailian), updating Xiaomi MiMo models, and implementing pagination across memory and skill management interfaces. It also fixes streaming artifacts and adds FTS5-based semantic search for memories. However, several critical issues were identified: a race condition in the GithubExecutor when handling request headers, logic and SQL syntax errors in the hybrid memory retrieval strategy, and a cache isolation bug in the skill registry that could lead to data missing in multi-tenant environments. Additionally, a change in error classification may cause quota exhaustion errors to be misidentified as rate limits, leading to unnecessary retries.

result.response = new Response(payload, { status, statusText, headers });
return result;
}
this._clientHeaders = input.clientHeaders ?? null;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

Storing request-specific headers in a class property (this._clientHeaders) is unsafe if the GithubExecutor instance is shared across concurrent requests (which is typical for executor singletons in this architecture). This creates a race condition where one request's headers can overwrite another's before buildHeaders is called. Consider using AsyncLocalStorage or passing the headers through the call stack if the base class allows it.

Comment on lines +220 to +258
case "hybrid": {
let ftsRows: MemoryRow[] = [];
if (config.query && ftsAvailable) {
const ftsQuery =
`SELECT m.* FROM ${tableName} m ` +
`JOIN memory_fts f ON m.memory_id = f.rowid ` +
`WHERE f.memory_fts MATCH ? AND m.${columns.apiKeyId} = ? ` +
`AND (m.${columns.expiresAt} IS NULL OR datetime(m.${columns.expiresAt}) > datetime('now'))` +
(normalizedConfig.scope === "session" && config.sessionId
? ` AND m.${columns.sessionId} = ?`
: "") +
(normalizedConfig.retentionDays > 0
? ` AND datetime(m.${columns.createdAt}) >= datetime(?)`
: "") +
` ORDER BY f.rank LIMIT 100`;
const ftsParams: any[] = [config.query, apiKeyId];
if (normalizedConfig.scope === "session" && config.sessionId) {
ftsParams.push(config.sessionId);
}
if (normalizedConfig.retentionDays > 0) {
const cutoff = new Date(
Date.now() - normalizedConfig.retentionDays * 24 * 60 * 60 * 1000
).toISOString();
ftsParams.push(cutoff);
}
try {
rows = db.prepare(ftsQuery).all(...ftsParams) as MemoryRow[];
if (rows.length === 0) {
query += ` ORDER BY ${columns.createdAt} DESC LIMIT 100`;
rows = db.prepare(query).all(...params) as MemoryRow[];
}
} catch {
query += ` ORDER BY ${columns.createdAt} DESC LIMIT 100`;
rows = db.prepare(query).all(...params) as MemoryRow[];
}
}
// Get chronological results for keyword scoring
query += ` ORDER BY ${columns.createdAt} DESC LIMIT 100`;
const keywordRows = db.prepare(query).all(...params) as MemoryRow[];

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The hybrid strategy implementation has two significant issues:\n1. Result Assignment Bug: At line 246, the FTS query results are assigned to the shared rows variable instead of ftsRows. Since ftsRows remains empty, the FTS results are ignored during the union at line 263.\n2. SQL Syntax Error Risk: If the FTS query fails or returns no results, the code appends an ORDER BY clause to the query string (lines 249 or 253). However, line 257 appends the same clause again unconditionally. This will result in a malformed SQL statement with duplicate ORDER BY clauses (e.g., SELECT ... ORDER BY ... ORDER BY ...), causing a database error.

Comment on lines +234 to +238
if (this.pendingLoad) {
await this.pendingLoad;
return;
}
if (!this.isCacheStale()) return;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The loadFromDatabase method uses a global lastLoaded timestamp to manage cache TTL, but the loading logic can be filtered by apiKeyId. If the first call to this method is for a specific API key, the registry will only contain skills for that key, and subsequent calls for other keys (or for all skills) will be skipped until the TTL expires. This will cause missing skills for other users in a multi-tenant environment. The cache state should be tracked per apiKeyId, or the method should always perform a full load.

Comment thread open-sse/services/errorClassifier.ts Outdated
Comment on lines 99 to 101
if (statusCode === 429) {
return PROVIDER_ERROR_TYPES.RATE_LIMITED;
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Moving the statusCode === 429 check above the creditsExhausted check changes the behavior for providers that use 429 to signal quota exhaustion (e.g., OpenAI). Previously, these were correctly classified as QUOTA_EXHAUSTED (a terminal error). Now, they will be classified as RATE_LIMITED, which may trigger futile retries by the router. It is generally better to check the response body for specific error codes before falling back to status-code-based classification.

Update package versions for the electron app and open-sse package.
Sync llm.txt metadata and feature headings with the 3.6.6 release.
Add guarded outbound fetch helpers with private/local URL blocking,
controlled retries, timeout normalization, and route-level status
propagation for provider validation and model discovery.

Introduce cooldown-aware chat retries with configurable
requestRetry and maxRetryIntervalSec settings, model-scoped cooldown
responses, and improved rate-limit learning from headers and error
bodies so short upstream lockouts can recover automatically.

Also align Antigravity and Codex header handling, require API keys
for Pollinations, validate web runtime env at startup, restore
sanitized Gemini tool names in translated responses, and inject a
synthetic Claude text block when upstream SSE completes empty.
Introduce GLM Thinking as a first-class provider preset with shared GLM
model metadata, pricing, usage sync, dashboard support, and provider
request defaults for higher token budgets and longer timeouts.

Use provider-side /messages/count_tokens when a Claude-compatible
upstream supports it, while preserving estimated fallback behavior for
missing models, missing credentials, and upstream failures.

Also add startup seeding for default model aliases and normalize common
cross-proxy model dialects so canonical slashful model ids do not get
misrouted during resolution.
Add dedicated sync token storage, issuance, revocation, and bundle
download routes backed by stable config bundle versioning and ETag
support.

Expose the v1 websocket handshake route and custom Next server bridge so
OpenAI-compatible websocket traffic can be upgraded and proxied through
the dashboard and API bridge.

Expand compliance auditing with structured metadata, pagination, request
context, auth and provider credential events, and SSRF-blocked
validation logging.
- CHANGELOG: Add WebSocket bridge, GLM Thinking preset, safe outbound
  fetch/SSRF guard, cooldown-aware retries, compliance audit v2, model
  alias seeding, and all Internal Improvements for the 3 new commits
- README: Expand v3.6.x highlights table with 10 new features; add
  SafeOutboundFetch, CooldownAwareRetry, SSRF guard, TPS metric, sync
  tokens, WebSocket bridge to Resilience/Observability/Deployment tables
- ARCHITECTURE: Bump date; add new modules to executive summary, API
  routes, SSE core services, Auth/Security section; add SSRF/Outbound
  guard failure mode (section 6); expand module mapping
- ENVIRONMENT: Add OMNIROUTE_CRYPT_KEY/OMNIROUTE_API_KEY_BASE64 legacy
  aliases, OUTBOUND_SSRF_GUARD_ENABLED, CODEX_CLIENT_VERSION, and
  REQUEST_RETRY/MAX_RETRY_INTERVAL_SEC cooldown retry settings
- FEATURES: Add 6 new feature sections — V1 WebSocket Bridge, Sync
  Tokens & Config Bundle, GLM Thinking Preset, Safe Outbound Fetch &
  SSRF Guard, Cooldown-Aware Retries, Compliance Audit v2
Comment thread tests/unit/usage-fetcher-antigravity.test.ts Fixed
Comment thread tests/unit/usage-service-hardening.test.ts Fixed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 935c36f089

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +70 to +74
const credentials = await getProviderCredentials(
modelInfo.provider,
null,
null,
modelInfo.model

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Enforce API-key scope before provider token counting

This route now performs real upstream count_tokens calls, but it only checks isValidApiKey and then requests credentials with getProviderCredentials(..., null, null, ...). Passing null for allowedConnections bypasses API-key connection/model restrictions, so any valid key can trigger provider-side counting on accounts it should not access, which can leak model availability and consume restricted upstream quota. Resolve the caller's key metadata and enforce policy constraints before selecting credentials.

Useful? React with 👍 / 👎.

Comment on lines +93 to 96
const result = await listMemories({
apiKeyId: args.apiKeyId,
type: args.type as MemoryType | undefined,
});

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Fetch all matching memories before clear operation

omniroute_memory_clear now deletes only result.data from a single listMemories call, but listMemories is paginated by default (50 rows). For API keys with more than 50 matching memories, this tool reports success after partially clearing only the first page, leaving remaining rows undeleted. The clear path should iterate through pages or request an unbounded result set before deletion.

Useful? React with 👍 / 👎.

mgarmash and others added 5 commits April 15, 2026 00:25
Integrated into release/v3.6.6 — IPv6 proxy test fix
#1256)

Integrated into release/v3.6.6 — Gemini custom model picker fix
…ests (#1246)

Integrated into release/v3.6.6 — OAuth client_id default fallbacks
…n Chat→Responses translator (#1245)

Integrated into release/v3.6.6 — max_tokens → max_output_tokens Responses API translation + unit tests
)

Integrated into release/v3.6.6 — cursor-agent CLI credential source support

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: da9af4af59

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/lib/ws/handshake.ts
Comment on lines +77 to +79
const settings = await getSettings().catch(() => ({}));
return {
wsAuth: settings.wsAuth === true,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Fail closed when loading WebSocket auth settings

If getSettings() throws (for example during transient SQLite failures like SQLITE_BUSY), this code falls back to {} and forces wsAuth to false. In authorizeWebSocketHandshake, that immediately authorizes the handshake without any credential check, so a deployment that has WebSocket auth enabled can become temporarily unauthenticated during DB read failures. This should default to a fail-closed behavior (or return an error) when settings cannot be loaded.

Useful? React with 👍 / 👎.

Comment thread scripts/v1-ws-bridge.mjs
Comment on lines +534 to +546
this.closed = true;

clearInterval(this.pingTimer);
for (const active of this.activeRequests.values()) {
active.abortController.abort();
}
this.activeRequests.clear();

const reasonBuffer = Buffer.from(reason, "utf8");
const payload = Buffer.allocUnsafe(2 + reasonBuffer.length);
payload.writeUInt16BE(code, 0);
reasonBuffer.copy(payload, 2);
this.sendFrame(0x8, payload);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Send WebSocket close frame before setting closed flag

close() sets this.closed = true before calling sendFrame(0x8, payload), but sendFrame returns early when this.closed is true. That means no close frame is ever sent, so clients can observe abnormal shutdowns (e.g., missing close code/reason) when the server times out or actively closes a session. Reordering this so the close frame is written before flipping the flag avoids protocol-level disconnect issues.

Useful? React with 👍 / 👎.

…eout behavior (#1257)

Integrated into release/v3.6.6 — CC-compatible upstream SSE restore + stream timeout fix + README table repair

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a17b7234fe

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +112 to +113
if (isPrivateHost(url.hostname)) {
throw new OutboundUrlGuardError(PROVIDER_URL_BLOCKED_MESSAGE, {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Resolve DNS before treating provider URL as public

The new public-only guard only checks the literal hostname string, so any domain name that resolves to a private IP (e.g. internal DNS, nip.io-style hosts, or DNS rebinding) bypasses the SSRF protection. Because provider validation/model probes now rely on this guard, a user can still reach loopback/private services through a seemingly public hostname. Add DNS resolution (and post-redirect revalidation) instead of hostname-only checks.

Useful? React with 👍 / 👎.

Comment thread scripts/v1-ws-bridge.mjs
Comment on lines +288 to +289
this.buffer = Buffer.concat([this.buffer, chunk]);
const parsed = decodeClientFrames(this.buffer);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Enforce maximum WebSocket frame size

Incoming WebSocket data is appended with Buffer.concat without any size cap, and frame decoding accepts payload lengths up to Number.MAX_SAFE_INTEGER. A client can send very large or never-finished fragmented frames to force unbounded buffer growth and exhaust memory. This is especially risky on deployments with wsAuth disabled (the new default) or any reachable authenticated WS client. Reject oversized frames and close the socket early.

Useful? React with 👍 / 👎.

Comment thread scripts/run-next.mjs
Comment on lines +53 to +55
const wsBridge = createOmnirouteWsBridge({
baseUrl: `http://127.0.0.1:${dashboardPort}`,
});

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Handshake bridge against the configured host

The custom server listens on configurable hostname, but the WS bridge always performs its internal handshake against 127.0.0.1. If HOST is set to a specific non-loopback interface, the handshake fetch targets an address the server is not bound to, so WS upgrades fail even though HTTP serving is healthy. Build the bridge base URL from the actual bound host (or bind to loopback in addition).

Useful? React with 👍 / 👎.

…riptions, audio-speech) (#1297)

Integrated into release/v3.6.6 — adds embeddings, audio-transcriptions, audio-speech, and images-generations support for custom OpenAI-compatible providers, plus Pollinations image registry

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

return path.join(process.cwd(), "src", "lib", "db", "migrations");

P1 Badge Fail fast when migration path fallback is not package-rooted

When fileURLToPath(import.meta.url) fails, this fallback points migrations to process.cwd()/src/lib/db/migrations. In global installs or service launches from a different working directory, that folder usually does not exist, so getMigrationFiles() returns an empty list and runMigrations() silently skips all pending migrations. The process can then start against an out-of-date schema and fail later at runtime. This path resolution should be anchored to the installed package location (or throw if migrations cannot be found) instead of relying on CWD.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Integrated into release/v3.6.6

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cc867aac7f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +901 to +904
response: new Response(JSON.stringify(cached), {
headers: {
"Content-Type": "application/json",
"Access-Control-Allow-Origin": getCorsOrigin(),

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Preserve SSE contract for cached stream requests

When stream=true, this cache-hit branch returns a JSON payload with Content-Type: application/json instead of an SSE stream. Any OpenAI-compatible client/SDK expecting text/event-stream will fail to parse the response or hang waiting for stream events, so cached responses break the same requests that work on cache misses. The cache path should emit SSE chunks (or bypass cache for streaming requests) to keep response semantics consistent.

Useful? React with 👍 / 👎.


// Support wildcard matching (e.g. 192.168.* or *.local)
if (patternHost.includes("*")) {
const regexStr = "^" + patternHost.replace(/\./g, "\\.").replace(/\*/g, ".*") + "$";
@diegosouzapw diegosouzapw changed the title chore(release): v3.6.6 — skills registry stability & memory fts5 chore(release): v3.6.6 — Stabilization Apr 16, 2026
try {
const serialized = JSON.stringify(artifact, null, 2);
const sizeBytes = Buffer.byteLength(serialized);
const sha256 = crypto.createHash("sha256").update(serialized).digest("hex");

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3d6178870a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/lib/usage/callLogs.ts
@@ -486,18 +726,13 @@ export function rotateCallLogs() {
if (!CALL_LOGS_DIR || !fs.existsSync(CALL_LOGS_DIR)) return;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Run DB call-log retention even without artifact directory

rotateCallLogs() now performs SQLite retention (deleteCallLogsBefore and trimCallLogsToMaxRows), but it returns immediately when call_logs/ does not exist. In this release, summary rows are still inserted into call_logs even when no artifact is written (for example noLog traffic or artifact write failures), so this guard skips all DB pruning and allows unbounded table growth until that directory happens to be created.

Useful? React with 👍 / 👎.

Comment thread src/lib/db/core.ts
Comment on lines +919 to +921
db.prepare("INSERT OR IGNORE INTO _omniroute_migrations (version, name) VALUES (?, ?)").run(
"026",
"call_logs_cache_source"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Avoid reserving migration version 026 in applied table

This inserts a synthetic 026 migration row even though there is no 026_*.sql file. The migration runner keys applied state by version only, so when a real 026 migration is added later it will be treated as already applied and skipped, causing schema drift on upgrade. Use a non-conflicting marker strategy (or reserve with an actual migration file) instead of pre-populating version 026.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 103a781536

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

return NextResponse.json({ skills });
const allSkills = skillRegistry.list();
const url = request?.url || "http://localhost/api/skills";
const params = parsePaginationParams(new URL(url).searchParams);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Return 400 for invalid skills pagination parameters

parsePaginationParams throws on malformed page/limit values (for example GET /api/skills?page=abc), but this handler catches all errors and returns 500, which misclassifies client input errors as server failures. This regression was introduced with the new pagination path and will surface as false backend errors in dashboards/clients sending bad query params.

Useful? React with 👍 / 👎.

const executions = skillExecutor.listExecutions();
return NextResponse.json({ executions });
const url = new URL(request.url);
const params = parsePaginationParams(url.searchParams);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Handle malformed executions pagination as client errors

The executions list endpoint now parses pagination via parsePaginationParams, which throws on invalid query values (e.g. page=0, limit=foo), but the surrounding catch returns 500 for all exceptions. That makes simple request-validation failures look like internal outages and breaks expected API semantics for bad pagination input.

Useful? React with 👍 / 👎.

- Migrated legacy .mjs scripts to .ts (bin, prepublish, policies)
- Resolved pre-commit strict lint (t11 budget) errors in combo.ts
- Explicitly typed all TS bindings in pack-artifact policies
- Updated package.json commands to run Node via tsx/esm internally
- Hardened CI/CD with explicit node version 22.22.2 checks
- Completed stage validations for v3.6.6 final release
- Migrate nodeRuntimeSupport to TS interfaces avoiding implicit any
- Increase visibility timeouts in skills-marketplace E2E test to 15s to bypass CI flakiness
- Complete migration of .mjs scripts to .ts ensuring type safety
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@diegosouzapw
diegosouzapw merged commit 9e45baa into main Apr 16, 2026
5 of 6 checks passed
@diegosouzapw
diegosouzapw deleted the release/v3.6.6 branch April 16, 2026 08:26

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e5c16fb196

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +390 to +392
if (mb.progress === "DONE") {
fullAnswer = chunks.join("");
} else {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Emit final Perplexity content before ending stream

When markdown_block.progress === "DONE", this branch only assigns fullAnswer and does not emit a delta. In the streaming path, buildStreamingResponse() breaks on chunk.done without sending chunk.answer, so requests where Perplexity emits only a final DONE block (or puts new trailing text only in that block) return an empty/truncated assistant message even though upstream produced a valid answer.

Useful? React with 👍 / 👎.

Poid-ZA pushed a commit to Poid-ZA/OmniRoute that referenced this pull request Aug 5, 2026
* fix(streaming): diegosouzapw#1211 greedy strip omniModel tags to prevent literal \n\n artifacts

- Changed regex quantifier from ? to * in combo.ts, comboAgentMiddleware.ts,
  and contextHandoff.ts to greedily strip all JSON-escaped newline sequences
  surrounding <omniModel> tags in SSE streaming chunks
- Added \r to the character class for cross-platform robustness
- Fixed Playwright strict-mode violation in combo-unification.spec.ts
- Bumped OpenAPI version and CHANGELOG to 3.6.6

* fix: 3 bugs found during issue triage (diegosouzapw#1175, diegosouzapw#1187/diegosouzapw#1218, diegosouzapw#1202)

- fix(gemini): strip VS Code JSON Schema extensions from tool schemas (diegosouzapw#1175)
  Add enumDescriptions, markdownDescription, markdownEnumDescriptions,
  enumItemLabels and tags to UNSUPPORTED_SCHEMA_CONSTRAINTS so the Gemini
  sanitizer removes them before forwarding. GitHub Copilot injects these
  non-standard fields into tool definitions, causing Gemini to reject with
  'Unknown name enumDescriptions at functionDeclarations[n].parameters'.

- fix(health-check): unwrap proxy config object before passing to getAccessToken (diegosouzapw#1187 diegosouzapw#1218)
  resolveProxyForConnection() returns { proxy, level, levelId } but the health
  check loop was passing the full wrapper to getAccessToken(), which expects the
  inner config object (.host, .port etc). The proxy dispatcher validated .host
  on the wrapper (undefined) and threw 'Context proxy host is required', silently
  marking every connection as unhealthy every sweep. Fix mirrors the pattern
  already used in chatHelpers.ts: proxyResult?.proxy || null.

- fix(ui): debounce models.dev sync interval slider to save only on release (diegosouzapw#1202)
  The slider's onChange fired updateInterval() on every drag tick, sending a
  PATCH per pixel of movement. Rapid API responses overwrote UI state mid-drag.
  Introduce draftIntervalHours for smooth visual feedback; the PATCH fires
  on onMouseUp / onBlur once the user releases the control.

* fix(providers): update Xiaomi MiMo token-plan endpoints (diegosouzapw#1238)

Integrated into release/v3.6.6

* fix(cc-compatible): trim beta flags and preserve cache passthrough (diegosouzapw#1230)

Integrated into release/v3.6.6

* feat(memory+skills): full-featured memory & skills systems with tests (diegosouzapw#1228)

Integrated into release/v3.6.6

* fix: forward client x-initiator header to GitHub Copilot upstream (diegosouzapw#1227)

Integrated into release/v3.6.6

* feat(bailian-quota): add Alibaba Coding Plan quota monitoring (diegosouzapw#1235)

* fix: resolve v3.6.6 backlog bugs (diegosouzapw#1206, diegosouzapw#1211, diegosouzapw#1220, diegosouzapw#1231)

- fix(core): diegosouzapw#1206 inject startup guard against app/ and src/app/ conflict
- fix(health): diegosouzapw#1220 add HEALTHCHECK_STAGGER_MS to prevent token refresh bursting
- fix(proxy): diegosouzapw#1231 prioritize HTTP 429 over quota body heuristics
- fix(sse): diegosouzapw#1211 strip leading double-newlines in responses API stream

* fix(tests): resolve memory migration and skills route pagination bugs from PR overlaps

* docs: Update CHANGELOG.md with v3.6.6 features (diegosouzapw#1182, diegosouzapw#1165, diegosouzapw#1177)

* chore(release): bump version to 3.6.6

Update package versions for the electron app and open-sse package.
Sync llm.txt metadata and feature headings with the 3.6.6 release.

* feat(core): harden outbound provider calls and add cooldown retries

Add guarded outbound fetch helpers with private/local URL blocking,
controlled retries, timeout normalization, and route-level status
propagation for provider validation and model discovery.

Introduce cooldown-aware chat retries with configurable
requestRetry and maxRetryIntervalSec settings, model-scoped cooldown
responses, and improved rate-limit learning from headers and error
bodies so short upstream lockouts can recover automatically.

Also align Antigravity and Codex header handling, require API keys
for Pollinations, validate web runtime env at startup, restore
sanitized Gemini tool names in translated responses, and inject a
synthetic Claude text block when upstream SSE completes empty.

* feat(models): add glmt preset and hybrid token counting

Introduce GLM Thinking as a first-class provider preset with shared GLM
model metadata, pricing, usage sync, dashboard support, and provider
request defaults for higher token budgets and longer timeouts.

Use provider-side /messages/count_tokens when a Claude-compatible
upstream supports it, while preserving estimated fallback behavior for
missing models, missing credentials, and upstream failures.

Also add startup seeding for default model aliases and normalize common
cross-proxy model dialects so canonical slashful model ids do not get
misrouted during resolution.

* feat(api): add sync tokens and v1 websocket bridge

Add dedicated sync token storage, issuance, revocation, and bundle
download routes backed by stable config bundle versioning and ETag
support.

Expose the v1 websocket handshake route and custom Next server bridge so
OpenAI-compatible websocket traffic can be upgraded and proxied through
the dashboard and API bridge.

Expand compliance auditing with structured metadata, pagination, request
context, auth and provider credential events, and SSRF-blocked
validation logging.

* docs: Update all documentation for v3.6.6

- CHANGELOG: Add WebSocket bridge, GLM Thinking preset, safe outbound
  fetch/SSRF guard, cooldown-aware retries, compliance audit v2, model
  alias seeding, and all Internal Improvements for the 3 new commits
- README: Expand v3.6.x highlights table with 10 new features; add
  SafeOutboundFetch, CooldownAwareRetry, SSRF guard, TPS metric, sync
  tokens, WebSocket bridge to Resilience/Observability/Deployment tables
- ARCHITECTURE: Bump date; add new modules to executive summary, API
  routes, SSE core services, Auth/Security section; add SSRF/Outbound
  guard failure mode (section 6); expand module mapping
- ENVIRONMENT: Add OMNIROUTE_CRYPT_KEY/OMNIROUTE_API_KEY_BASE64 legacy
  aliases, OUTBOUND_SSRF_GUARD_ENABLED, CODEX_CLIENT_VERSION, and
  REQUEST_RETRY/MAX_RETRY_INTERVAL_SEC cooldown retry settings
- FEATURES: Add 6 new feature sections — V1 WebSocket Bridge, Sync
  Tokens & Config Bundle, GLM Thinking Preset, Safe Outbound Fetch &
  SSRF Guard, Cooldown-Aware Retries, Compliance Audit v2

* fix: use api64 for proxy test (diegosouzapw#1255)

Integrated into release/v3.6.6 — IPv6 proxy test fix

* fix(page): update custom models section to include all providers diegosouzapw#1200 (diegosouzapw#1256)

Integrated into release/v3.6.6 — Gemini custom model picker fix

* fix: provide default client_id fallbacks to prevent broken OAuth requests (diegosouzapw#1246)

Integrated into release/v3.6.6 — OAuth client_id default fallbacks

* fix: translate max_tokens/max_completion_tokens → max_output_tokens in Chat→Responses translator (diegosouzapw#1245)

Integrated into release/v3.6.6 — max_tokens → max_output_tokens Responses API translation + unit tests

* feat(oauth): support cursor-agent CLI as Cursor credential source (diegosouzapw#1258)

Integrated into release/v3.6.6 — cursor-agent CLI credential source support

* fix(cc-compatible): restore upstream SSE and correct stream/combo timeout behavior (diegosouzapw#1257)

Integrated into release/v3.6.6 — CC-compatible upstream SSE restore + stream timeout fix + README table repair

* fix(cli-tools): resolve API key resolution and model mapping bugs in CLI tools (diegosouzapw#1263)

Integrated into release/v3.6.6

* feat(cli-tools): add Qwen Code CLI integration (diegosouzapw#1266)

Integrated into release/v3.6.6

* fix(i18n): add missing zh-CN translations and fix logger imports (diegosouzapw#1269)

Integrated into release/v3.6.6

* fix(i18n): add Chinese i18n support to dashboard components (diegosouzapw#1274)

Integrated into release/v3.6.6

* feat: update Pollinations to require API key, remove free tier flag (diegosouzapw#1177)

* feat: friendly error messages for crypto/encryption failures (diegosouzapw#1165)

* feat: add TPS (tokens per second) metric column to request logs (diegosouzapw#1182)

* feat: merge custom/imported models into filter list for all providers (diegosouzapw#1191)

* feat(fallback): Fix provider-profile-driven lockouts (diegosouzapw#1267)

This integrates rdself's unify-provider-profile-locks PR manually to handle structural conflicts.

* fix(claude): proper Anthropic SDK integration (diegosouzapw#1271)

* fix(healthcheck): use correct proxy wrapper format for getAccessToken (diegosouzapw#1272)

* chore(release): v3.6.6 — skills registry stability fix + final integration

* fix(auth): harden bootstrap auth and memory dashboard behavior

Restrict unauthenticated writes to /api/settings/require-login to
the initial bootstrap window while keeping read-only checks public.
This prevents post-setup config changes without blocking first-run
login setup, and the onboarding flow now logs in immediately after
setting the password.

Restore memory API filtering and pagination behavior by supporting q
searches, honoring offset-based requests, and avoiding unrelated
fallback results when FTS misses. Update dashboard stats fallback to
use the response totals consistently.

Package the MCP server with explicit file entries and add regression
tests for bootstrap auth and memory route behavior

* fix(codex): remove max_output_tokens from body for compatibility

* chore(release): v3.6.6 — include PR 1274 fixes in changelog

* chore: exclude additional build artifacts and internal directories from npm package distribution

* fix: update Gemini OAuth test to match registry defaults + codex UI improvements

* fix: restore .mjs refs for scripts/ in test imports after ts migration

* fix: restore next.config.mjs ref in dev-origins test

* fix: implement db migration safety checks and codex config format

* fix: disable mass-migration abort during unit tests based on auto-backup flag

* fix: update script regex in auto-update tests to use .mjs

* feat: Add Perplexity Web (Session) provider (diegosouzapw#1289)

Integrated into release/v3.6.6

* fix(cli): resolve codex routing config parsing, standardize select model button positioning, and clarify oauth documentation

* docs(changelog): record recent cli, provider, and test updates

Document the latest fixes for Codex routing configuration parsing and
Lobehub provider icon fallback behavior.

Add the note that the remaining JavaScript test files were migrated to
TypeScript ES modules to reflect the completed test stack transition.

* chore(release): merge diegosouzapw#1286 minor improvements manually to avoid testing conflict

* chore(test): rename perplexity-web.test.mjs to .ts to maintain 100% TS codebase

* chore(docs): update CHANGELOG.md for perplexity-web provider

* fix(security): resolve CodeQL incomplete URL substring sanitization via URL parsing in test mocks

* fix: integrate compressContext() into chatCore.ts request pipeline

Proactively compress oversized contexts before sending to upstream providers,
preventing context_length_exceeded errors. Compression triggers at 85% of
model's context limit using the existing 3-layer compressContext() function.

- Import compressContext, estimateTokens, getTokenLimit from contextManager
- Add compression check after translation, before executor dispatch
- Estimate tokens and compare against 85% threshold of model's context limit
- Apply 3-layer compression (trim tools, compress thinking, purify history)
- Log compression events with before/after token counts and layers applied
- Audit compression events for observability
- Add unit tests verifying integration behavior

Closes diegosouzapw#1290

* fix(tests): align reasoning expectations with GLM thinking structure

* fix: prevent orphaned tool_result messages in purifyHistory()

When purifyHistory() drops oldest messages to fit context window, it can
split tool_use/tool_result pairs — keeping the tool_result but dropping
the tool_use that initiated it. This causes upstream providers to reject
the request with format errors.

Add fixToolPairs() that runs after each purification pass to remove:
- OpenAI format: orphaned role='tool' messages without matching tool_calls ID
- Claude format: orphaned tool_result content blocks without matching tool_use ID

Closes diegosouzapw#1291

* fix(tests): supply tool_use in mock so it is not dropped

* chore: convert remaining test to TypeScript

* fix(tests): restore compatibility with compressContext threshold test after tsx migration

* docs: finalize v3.6.6 release documentation

* fix(core): finalize provider removal, type issues, and codex API key config

* fix(dashboard): render Web/Cookie, Search, Audio provider sections and fix TypeScript errors

* fix: increase MCP web_search timeout to 60s (diegosouzapw#1278)

* fix: route combo testing properly for embedding models (diegosouzapw#1260)

* fix: accumulate excluded accounts in combo fallback loop (diegosouzapw#1233)

* fix: strip leading whitespace and newlines from first streaming chunk (diegosouzapw#1211)

* docs: clarify VPS and Docker settings for OAuth credentials (diegosouzapw#1204)

* fix: return real retry-after for pipeline gates (diegosouzapw#1301)

Integrated into release/v3.6.6 — returns real Retry-After values from pipeline gates

* feat: streaming semantic cache, Cursor auto-version detection, and call-log enhancements (diegosouzapw#1296)

Integrated into release/v3.6.6 — streaming semantic cache, Cursor auto-version detection, call-log cache_source tracking

* feat(api): support more OpenAI types (image, embeddings, audio-transcriptions, audio-speech) (diegosouzapw#1297)

Integrated into release/v3.6.6 — adds embeddings, audio-transcriptions, audio-speech, and images-generations support for custom OpenAI-compatible providers, plus Pollinations image registry

* deps: bump hono from 4.12.12 to 4.12.14 (diegosouzapw#1302)

Integrated into release/v3.6.6

* deps: bump hono from 4.12.12 to 4.12.14 (diegosouzapw#1306)

Integrated into release/v3.6.6

* chore: stabilization fixes for v3.6.6 (diegosouzapw#1298, diegosouzapw#1254, diegosouzapw#59, CI)

* fix(providers): match correct endpoint for Xiaomi MiMo, strip routing prefix for custom openai endpoints (diegosouzapw#1303, diegosouzapw#1261)

* feat(storage): add database backup cleanup controls

* chore(release): v3.6.6 — Final Stabilization Push

* Backport call log storage refactor to release/v3.6.6 (diegosouzapw#1307)

Integrated into release/v3.6.6

* deps: update dompurify to 3.4.0 to resolve CVE-XYZ (diegosouzapw#60)

* test: disable sqlite auto backup in CI to resolve E2E timeout (#24481475058)

* chore(docs): sync CHANGELOG for v3.6.6 with missing features and fixes

* chore(release): prep v3.6.6 infrastructure and type safety fixes

- Migrated legacy .mjs scripts to .ts (bin, prepublish, policies)
- Resolved pre-commit strict lint (t11 budget) errors in combo.ts
- Explicitly typed all TS bindings in pack-artifact policies
- Updated package.json commands to run Node via tsx/esm internally
- Hardened CI/CD with explicit node version 22.22.2 checks
- Completed stage validations for v3.6.6 final release

* chore: fix TS build errors and e2e timeouts in CI

- Migrate nodeRuntimeSupport to TS interfaces avoiding implicit any
- Increase visibility timeouts in skills-marketplace E2E test to 15s to bypass CI flakiness
- Complete migration of .mjs scripts to .ts ensuring type safety

* chore(release): sync package version 3.6.6 across workspaces

* test(e2e): universally increase UI component visibility timeouts from 5s to 15s to bypass CI starvation

* chore(build): inject baseUrl, paths, and types:node into MITM tsconfig within prepublish hook to fix missing types in CI check

---------

Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
Co-authored-by: Jack <5443152+hijak@users.noreply.github.com>
Co-authored-by: Randi <55005611+rdself@users.noreply.github.com>
Co-authored-by: Paijo <14921983+oyi77@users.noreply.github.com>
Co-authored-by: Samuel Cedric <ceds.sam@gmail.com>
Co-authored-by: Max Garmash <max@37bytes.com>
Co-authored-by: Markus Hartung <mail@hartmark.se>
Co-authored-by: Gi99lin <74502520+Gi99lin@users.noreply.github.com>
Co-authored-by: Payne <baboialex95@gmail.com>
Co-authored-by: Benson K B <bensonkbmca@gmail.com>
Co-authored-by: clousky2020 <33016567+clousky2020@users.noreply.github.com>
Co-authored-by: Ravi Tharuma <25951435+RaviTharuma@users.noreply.github.com>
Co-authored-by: oyi77 <oyi77@users.noreply.github.com>
Co-authored-by: Hdsje <vovan877@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: xiaoge1688 <moyekongling@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.