mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-07-26 09:52:11 +03:00
* fix(dashboard): resolve Kimi banner casing collision + shrink frozen test file (release tip) - Rename src/app/(dashboard)/dashboard/kimiSponsorBanner.ts to kimiSponsorBannerGate.ts so it no longer differs from KimiSponsorBanner.tsx only by the first letter's case (breaks next build on case-insensitive filesystems). Updates the sole importer (KimiSponsorBanner.tsx) and the two tests that reference it. - Extract the 8 Kimi/Moonshot featured-ordering tests out of the frozen tests/unit/providers-page-utils.test.ts (grown 3 lines past its 1294 cap by #8039's rebrand-comment update) into a new sibling file tests/unit/providers-page-utils-kimi.test.ts. No assertions dropped; both files pass in full (24 + 8 = 32 tests). * fix(sse): register PromptQlExecutor in the executor registry (release tip) getExecutor("promptql") had no entry in open-sse/executors/index.ts, so it silently fell through to DefaultExecutor's provider fallback, which issues a raw fetch() and returns the bare upstream Response instead of the executor wrapper shape {response, url, headers, transformedBody}. The real PromptQlExecutor class (open-sse/executors/promptql.ts) already honors the contract correctly — it was just never wired into the registry. Fixes tests/unit/executor-web-cookie-sweep.test.ts "promptql executor returns wrapper shape". * fix(i18n): backfill 2220 missing pt-BR keys to restore en.json parity (release tip) pt-BR.json fell behind after #7935 restored +2220 keys into en.json and vi.json but left pt-BR.json unmodified. Translated all missing entries to Brazilian Portuguese, preserving ICU/interpolation placeholders and existing terminology, and merged them mirroring en.json's key order so the diff is additions-only (the small comma-only deletions are pure JSON reformatting from new sibling keys). * fix(providers): repair 4 pre-existing catalog/registry reds on release tip - providers-constants-split.test.ts: APIKEY_PROVIDERS grew 182->187 (PR #7887 added 5 free-tier providers: ainative/aion/sealion/routeway/nara). Verified no dup/loss (6-family partition sums exactly to 187) and updated the stale expected count + comment trail to match. - cline registry: added the missing minimax/minimax-m3 free OpenRouter entry (#3321) and fixed the neighbouring nemotron-3-ultra-550b-a55b entry, which carried a stray ":free" id suffix and an imprecise 1_000_000 contextLength instead of the 1_048_576 the test (and every sibling 1M-context entry in this catalog) expects. - promptqlModels.ts / registry/promptql/index.ts: PROMPTQL_FALLBACK_MODELS's minimax-m3 entry was missing supportsVision, and the registry mapping dropped it entirely (only id/name were passed through) — it was the sole minimax-m3 entry across the whole registry not flagged multimodal, despite every other provider (minimax, minimax-cn, ollama-cloud, trae, bazaarlink, clinepass, codebuddy-cn, opencode-zen/go, synthetic, huggingchat, lmarena) agreeing MiniMax-M3 supports vision. Added the field to the PromptQlModel type and threaded it through. - tests/snapshots/provider/translate-path.json: regenerated the golden via UPDATE_GOLDEN=1. Diffed old vs new — zero providers removed, 5 added (ainative/aion/nara/routeway/sealion, matching #7887), and the only changed entry (cline) reflects the already-merged #7914 ClinePass header protocol change (Cline/<version> User-Agent + X-Task-ID) that a prior narrow golden touch-up missed capturing. * fix(docs): repair docs-sync/env-sync/repo-contract gates (release tip) Six pre-existing reds on release/v3.8.49, all "repo drifted from its own documented contract": - check-docs-counts-sync: free-tier headline was stale (~1.4B/~2.0B) vs the live catalog (~1.53B steady / ~2.15B first month, 43 pools). Updated README.md and docs/reference/FREE_TIERS.md to the live numbers and added a v3.8.49 correction note explaining the pool-count delta (39->43, #7840). Also fixed a soft executors-count drift in ARCHITECTURE.md (84->86, 268->271 providers) while touching that line. - release-green-docs-drift-7253: docs/proxy-subscriptions.md referenced a fabricated migration filename (123_proxy_subscriptions.sql); the real file is 131_proxy_subscriptions.sql. Fixed all 3 occurrences. - check-env-doc-sync + issue-7793-env-doc-sync-repro: OMNIROUTE_DATA_DIR (DATA_DIR fallback alias read by open-sse/executors/promptql/threadSticky.ts) was undocumented. Added to .env.example and docs/reference/ENVIRONMENT.md. - check-db-rules: src/lib/db/proxySubscriptions.ts (#7299) is a db-internal split of proxies.ts (kept under the frozen file-size cap) whose one export is already re-exported via proxies.ts -> localDb.ts. Added it to INTENTIONALLY_INTERNAL with the same db-internal justification used for identical split modules (apiKeyColumnFallbacks, providerNodeSelect, webSessionDedup) rather than a redundant direct re-export from localDb.ts. - mcp-server-hollow-dist-deps: the sanity test expected better-sqlite3 among the MCP bundle's static top-level external imports. That's been stale since the pre-#7878 migration to a cascading SqliteAdapter driver factory (createRequire()-based lazy require, not a static import); better-sqlite3 already has its own native-asset copy guarantee in assembleStandalone.mjs, unrelated to this test's EXTRA_MODULE_ENTRIES concern. Updated the assertion to a still-genuinely-static external (zod) with a comment explaining the change. No production runtime behavior changed — docs, .env.example, and a checker allowlist/test-expectation only. * fix(dashboard): repair stale UI component-shape test assertions (release tip) Two pre-existing reds in the dashboard UI component-contract cluster were caused by test assertions that had gone stale after intentional, correct refactors — not by real defects in the components: - quota-pool-wizard-multi.test.ts: the step-3 preview assertion required the literal single-line substring "connectionIds.map((cid)". Prettier (100-char width, project config) legitimately breaks the connectionIds.map(...).filter(...) chain across lines because of the multi-line callback body, so the literal never matches. PoolWizard.tsx still builds previewByProvider correctly by mapping over connectionIds; updated the assertion to a regex that tolerates the line break. - v388-phase1-screen-fixes.test.ts: the shared Select placeholder-guard assertion required the literal "!children && placeholder". An earlier, intentional i18n commit changed the hardcoded "Select an option" default to a translated fallback (`placeholder ?? t("selectOption")`), which requires parens around the ?? expression for operator precedence. The guard behavior is unchanged (still gated on !children); updated the assertion to match the current, correct guard shape. Both fixes are read-only test-file changes; no production behavior changed. review-reviews-v3814-fixes.test.ts still has one pre-existing, unrelated red (LEDGER-4: minimax-m3 registry entries missing supportsVision) that requires editing the promptql provider registry/catalog — out of this cluster's scope, left untouched and reported separately. * fix(providers): reconcile cline catalog contradictions + deterministic golden (release tip) The first tip-green pass introduced 3 regressions caught by CI on sibling guard tests: - clinepass-provider + cline-catalog-models-3321 encoded OPPOSITE expectations of the same cline model list (minimax presence, nvidia :free suffix). Reference upstream (OpenRouter free lineup) confirms nvidia/nemotron-3-ultra-550b-a55b:free (with :free, 1M ctx) is correct, so restore that id and fix #3321's stale no-:free assertion; add minimax/minimax-m3 (the real #3321 gap) to clinepass-provider's list. - check-db-rules-classification froze INTENTIONALLY_INTERNAL at 35; proxySubscriptions was the intentional 36th entry — add it + bump the count. - provider-translate-path golden stored a LITERAL Cline/3.8.49: clineAuth resolves the version from APP_CONFIG.version (stable), but the golden sanitizer collapsed only process.env.npm_package_version (unset under `node`, set under `npm run`) — so the golden was shard-dependent. Resolve APP_VERSION from APP_CONFIG.version like clineAuth and regenerate; now Cline/<APP> normalizes identically in every shard. * fix(services): type execFile signal/killed in classifyError + ratchet dashboard baseline (release tip) Pre-existing base-red on the tip's Fast Quality Gates (dashboard-typecheck), missed in the first inventory: - src/lib/services/installers/utils.ts TS2339 — `err.signal` was read off a value typed as NodeJS.ErrnoException, which @types/node does not declare `signal`/`killed` on (those belong to execFile's ExecFileException). Widen classifyError's param to type both, and drop the now-redundant `(err as … { killed })` cast. - Ratchet config/quality/dashboard-typecheck-baseline.json down: 5 baselined errors were fixed by already-merged PRs but never ratcheted (OAuthModal TS2769 4→3 / TS2345 4→3, CliproxyModelMappingEditor TS2339, CompressionPreviewAccordion TS4104, MonacoEditor TS2307). Baseline now 254, matching live — gate exits 0.
292 lines
16 KiB
JavaScript
292 lines
16 KiB
JavaScript
#!/usr/bin/env node
|
||
// scripts/check/check-db-rules.mjs
|
||
// Gate de convenções de banco (CLAUDE.md Hard Rules #2 e #5). Três verificações:
|
||
// (a) Todo módulo de domínio em src/lib/db/*.ts deve ser re-exportado por
|
||
// src/lib/localDb.ts (camada de compat). Um módulo db NOVO que não é
|
||
// re-exportado (e não está congelado) falha — força a decisão consciente
|
||
// de expor ou justificar (Hard Rule #2).
|
||
// (b) src/lib/localDb.ts é APENAS camada de re-export: nada de lógica
|
||
// (function/class/arrow de negócio). Mata o anti-padrão de "só uma
|
||
// funçãozinha aqui" que vira regra de negócio fora dos módulos db/.
|
||
// (c) Nenhum SQL cru em src/app/api/**/route.ts ou open-sse/handlers/*.ts.
|
||
// SQL deve viver em src/lib/db/ (Hard Rule #5). Ofensores pré-existentes
|
||
// são congelados; QUALQUER novo SQL cru em rota/handler falha.
|
||
// Stale-enforcement (6A.3): entradas em INTENTIONALLY_INTERNAL / EXTERNAL_DB_ALLOWED
|
||
// que não suprimem nenhuma violação real → gate falha com instrução de remoção.
|
||
import fs from "node:fs";
|
||
import path from "node:path";
|
||
import { pathToFileURL } from "node:url";
|
||
import { assertNoStale } from "./lib/allowlist.mjs";
|
||
|
||
const cwd = process.cwd();
|
||
const DB_DIR = path.join(cwd, "src/lib/db");
|
||
const LOCAL_DB = path.join(cwd, "src/lib/localDb.ts");
|
||
const API_DIR = path.join(cwd, "src/app/api");
|
||
const HANDLERS_DIR = path.join(cwd, "open-sse/handlers");
|
||
|
||
// (a) Módulos db/ que NÃO são re-exportados por localDb.ts por DESIGN (Hard Rule #2:
|
||
// "Never barrel-import from localDb.ts — import specific db/ modules instead").
|
||
// Cada entrada aqui foi auditada e é consumida via import direto de "@/lib/db/X"
|
||
// (estático ou dinâmico) pelos seus consumidores — exatamente o padrão correto.
|
||
// Re-exportar esses módulos via localDb.ts INCENTIVARIA o anti-padrão proibido.
|
||
// O gate ainda bloqueia QUALQUER módulo db/ NOVO que não seja re-exportado E não
|
||
// esteja nessa lista — mantendo a decisão consciente obrigatória (Hard Rule #2).
|
||
// Legenda de classificação:
|
||
// type-only = exporta apenas tipos (sem runtime API), não há o que re-exportar
|
||
// db-internal = importado apenas dentro de src/lib/db/ (coordenação interna)
|
||
// intentionally-internal = consumido por import direto fora de db/ (correto per Rule #2)
|
||
// DEAD? = zero importers encontrados na auditoria de 2026-06-11; não deletar
|
||
// sem investigação — pode ser reserva de schema ou F2 pendente
|
||
export const INTENTIONALLY_INTERNAL = new Set([
|
||
"_rowTypes", // type-only: 5 importers internos em db/ (AgentBridge/Inspector row types)
|
||
"accessTokens", // intentionally-internal: 4 rotas /api/cli/* (connect, whoami, tokens, tokens/[id]) + server/authz/accessTokenAuth.ts via import direto "@/lib/db/accessTokens" (Rule #2)
|
||
"apiKeyColumnFallbacks", // db-internal: importado só por db/apiKeys.ts (API_KEY_COLUMN_FALLBACKS — fallbacks de coluna split do apiKeys.ts)
|
||
"apiKeyUsageLimitFields", // db-internal: importado só por db/apiKeys.ts (helpers de campo de limite de uso split do apiKeys.ts; mig 101)
|
||
"caseMapping", // db-internal: importado só por db/core.ts (toSnakeCase/toCamelCase/objToSnake — column-mapping snake↔camel split do core.ts, #4947)
|
||
"cleanup", // intentionally-internal: 3 API routes (purge-quota-snapshots, purge-call-logs, purge-detailed-logs)
|
||
"cliToolState", // intentionally-internal: 14+ API routes em /api/cli-tools/*-settings
|
||
"comboForecast", // intentionally-internal: src/lib/usage/comboForecast.ts
|
||
"commandCodeAuth", // intentionally-internal: 5 API routes em /api/providers/command-code/auth/*
|
||
"compression", // intentionally-internal: 2 API routes (settings/compression, context/rtk/config)
|
||
"vacuumScheduler", // intentionally-internal: src/instrumentation-node.ts (dynamic import, lifecycle wiring per Rule #2)
|
||
"detailedLogs", // intentionally-internal: 3 callers (callLogs.ts, logs/detail route, embeddings handler)
|
||
"discovery", // DEAD?: 0 importers na auditoria de 2026-06-11; lib/discovery/index.ts não usa db/discovery
|
||
"domainState", // intentionally-internal: 5 callers (batchWriter, circuitBreaker, costRules, fallbackPolicy, lockoutPolicy)
|
||
"encryption", // intentionally-internal: 8+ callers (container, webhookDispatcher, cloudAgent/credentials, services/apiKey, 4+ routes, open-sse)
|
||
"healthCheck", // db-internal: importado por db/core.ts (runDbHealthCheck)
|
||
"jsonMigration", // intentionally-internal: src/app/api/settings/import-json/route.ts
|
||
"migrationRunner", // db-internal: importado por db/core.ts (runMigrations ao inicializar o DB)
|
||
"modelCapabilityOverrides", // intentionally-internal: src/app/api/model-capability-overrides/route.ts via import direto "@/lib/db/modelCapabilityOverrides" (#6727 — evita empurrar localDb.ts para o cap de 800 linhas)
|
||
"notion", // intentionally-internal: settings/notion API route + open-sse/mcp-server/tools/notionTools.ts
|
||
"obsidian", // intentionally-internal: src/lib/obsidianSync.ts + settings/obsidian route + MCP obsidianTools.ts
|
||
"optimizationSettings", // db-internal: imported by db/core.ts for SQLite PRAGMA application helpers that require the live adapter
|
||
"pluginMetrics", // DEAD? (production): write path não foi conectado ainda (documentado no cabeçalho do módulo); testado por tests/unit/plugins-metrics.test.ts
|
||
"prompts", // DEAD? (production): zero callers de produção encontrados; domínio domain/prompts.ts é independente; testado por tests/integration/proxy-pipeline.test.ts
|
||
"providerNodeSelect", // db-internal: importado só por db/providers.ts (selectProviderNodeForConnection — lógica pura de seleção de provider node split do providers.ts, #4421)
|
||
"providerStats", // intentionally-internal: src/app/api/provider-stats/route.ts
|
||
"proxyLatency", // intentionally-internal: imported directly by src/lib/db/proxies.ts (anti-barrel, #6798)
|
||
"proxySubscriptions", // db-internal: importado só por db/proxies.ts (addProxiesToScopePool — split do proxies.ts para ficar sob o cap de tamanho congelado, #7299); a função já é re-exportada por proxies.ts (que localDb.ts re-exporta)
|
||
"recovery", // intentionally-internal: bin/cli/runtime.mjs (import() dinâmico) + tests
|
||
"schemaColumns", // db-internal: importado só por db/core.ts (ensureProviderConnections/UsageHistory/CallLogsColumns + hasColumn/hasTable/getTableColumns — schema-column reconciliation split do core.ts, #4948)
|
||
"secrets", // intentionally-internal: src/instrumentation-node.ts (import() dinâmico na inicialização)
|
||
"serviceModels", // intentionally-internal: 3 callers (services/modelSync, services/bootstrap, /api/services/9router/models)
|
||
"stateReset", // db-internal: 3 callers dentro de src/lib/db/ (core, backup, apiKeys) para coordenação de reset
|
||
"stats", // intentionally-internal: src/app/api/settings/database/refresh-stats/route.ts
|
||
"tierConfig", // intentionally-internal: open-sse/services/tierResolver.ts (require() dinâmico)
|
||
"webSessionDedup", // db-internal: importado só por db/providers.ts (webSessionCredentialKey/parseProviderSpecificData — helpers puros de dedup de credencial web-session split do providers.ts, #3368 PR6)
|
||
]);
|
||
|
||
// Alias para retrocompatibilidade com os testes existentes que importam KNOWN_UNEXPORTED.
|
||
// O comportamento do gate é idêntico — só o nome e os comentários mudaram (#3499).
|
||
export const KNOWN_UNEXPORTED = INTENTIONALLY_INTERNAL;
|
||
|
||
// (c) Leituras de SQL contra bancos EXTERNOS, permitidas por design (#3500).
|
||
// Estas rotas NÃO consultam o DB do OmniRoute (getDbInstance) — elas abrem o
|
||
// SQLite de OUTRO aplicativo (Cursor / Kiro) para auto-importar credenciais.
|
||
// Por isso NÃO podem viver em src/lib/db/ (que é o domínio do DB do OmniRoute):
|
||
// são leituras read-only de um arquivo externo, com caminho/escopo próprios.
|
||
// Continuam no allowlist como exceção DOCUMENTADA — o gate ainda bloqueia
|
||
// QUALQUER novo SQL cru contra o DB do OmniRoute em rotas/handlers.
|
||
// Toda a dívida real da Hard Rule #5 (15 rotas internas) foi migrada para
|
||
// módulos src/lib/db/ nas slices do #3500; este set ficou só com as exceções.
|
||
const EXTERNAL_DB_ALLOWED = new Set([
|
||
"src/app/api/oauth/cursor/auto-import/route.ts", // read-only no itemTable do SQLite do Cursor (DB externo)
|
||
"src/app/api/oauth/kiro/auto-import/route.ts", // read-only no SQLite do Kiro (DB externo)
|
||
]);
|
||
|
||
// Alias de retrocompatibilidade (testes/consumidores que importam KNOWN_RAW_SQL).
|
||
// Comportamento do gate idêntico — só o nome e o enquadramento mudaram (#3500).
|
||
const KNOWN_RAW_SQL = EXTERNAL_DB_ALLOWED;
|
||
|
||
// Módulos sempre excluídos da checagem (a): não são domínio re-exportável.
|
||
const DB_MODULE_EXCLUDE = new Set(["core", "localDb", "index"]);
|
||
|
||
function walk(dir, acc = []) {
|
||
if (!fs.existsSync(dir)) return acc;
|
||
for (const e of fs.readdirSync(dir, { withFileTypes: true })) {
|
||
const p = path.join(dir, e.name);
|
||
if (e.isDirectory()) walk(p, acc);
|
||
else acc.push(p);
|
||
}
|
||
return acc;
|
||
}
|
||
|
||
// Lista os módulos de domínio em src/lib/db (top-level *.ts), excluindo
|
||
// core/localDb/index, *.d.ts e qualquer subdiretório (migrations/, adapters/, __tests__/).
|
||
export function collectDbModules(dbDir = DB_DIR) {
|
||
if (!fs.existsSync(dbDir)) return [];
|
||
return fs
|
||
.readdirSync(dbDir, { withFileTypes: true })
|
||
.filter((e) => e.isFile() && /\.ts$/.test(e.name) && !/\.d\.ts$/.test(e.name))
|
||
.map((e) => e.name.replace(/\.ts$/, ""))
|
||
.filter((name) => !DB_MODULE_EXCLUDE.has(name))
|
||
.sort();
|
||
}
|
||
|
||
// Extrai os nomes de módulo re-exportados de localDb.ts a partir de
|
||
// `... from "./db/X"` (cobre export {…}, export * e export type {…}).
|
||
export function extractReexportedModules(localDbSource) {
|
||
const re = /from\s+["']\.\/db\/([A-Za-z0-9_]+)["']/g;
|
||
const out = new Set();
|
||
let m;
|
||
while ((m = re.exec(localDbSource))) out.add(m[1]);
|
||
return out;
|
||
}
|
||
|
||
// (a) Módulos db/ que não são re-exportados e não estão na lista de
|
||
// intencionalmente-internos (INTENTIONALLY_INTERNAL). O gate falha para
|
||
// qualquer módulo NOVO que não seja re-exportado nem justificado.
|
||
export function findMissingReexports(dbModules, reexported, allowlist = INTENTIONALLY_INTERNAL) {
|
||
return dbModules.filter((mod) => !reexported.has(mod) && !allowlist.has(mod));
|
||
}
|
||
|
||
// (b) localDb.ts deve conter SOMENTE import/export + comentários (sem lógica).
|
||
// Remove comentários e strings, depois procura declarações de runtime.
|
||
export function hasLogic(localDbSource) {
|
||
const stripped = localDbSource
|
||
// comentários de bloco
|
||
.replace(/\/\*[\s\S]*?\*\//g, "")
|
||
// comentários de linha
|
||
.replace(/\/\/[^\n]*/g, "")
|
||
// template strings
|
||
.replace(/`(?:\\[\s\S]|[^\\`])*`/g, '""')
|
||
// strings simples/duplas (paths de import etc.)
|
||
.replace(/"(?:\\.|[^"\\])*"/g, '""')
|
||
.replace(/'(?:\\.|[^'\\])*'/g, '""');
|
||
|
||
// function/class declaradas, ou atribuição a função (const X = (…) =>, const X = function).
|
||
const logicPatterns = [
|
||
/(^|[^.\w])function\s+[A-Za-z_$]/, // function decl (não method .foo())
|
||
/(^|[^.\w])class\s+[A-Za-z_$]/, // class decl
|
||
/(?:const|let|var)\s+[A-Za-z_$][\w$]*\s*=\s*(?:async\s*)?\(/, // const X = (…) ... (arrow/call)
|
||
/(?:const|let|var)\s+[A-Za-z_$][\w$]*\s*=\s*(?:async\s+)?function\b/, // const X = function
|
||
];
|
||
return logicPatterns.some((rx) => rx.test(stripped));
|
||
}
|
||
|
||
// SQL cru é sempre uma STRING passada a db.prepare()/exec(): casamos os padrões
|
||
// SÓ dentro de literais de string (não em código JS — `import … from`, `.set(`,
|
||
// `new Set(`, `delete x` etc. são falsos positivos se varrermos o código todo).
|
||
const SQL_PATTERNS = [
|
||
/\bSELECT\b[\s\S]*?\bFROM\b/i, // SELECT … FROM (multi-linha)
|
||
/\bINSERT\s+INTO\b/i,
|
||
/\bUPDATE\b[\s\S]*?\bSET\b/i, // UPDATE … SET (multi-linha)
|
||
/\bDELETE\s+FROM\b/i,
|
||
/\bCREATE\s+TABLE\b/i,
|
||
];
|
||
|
||
// Remove comentários (linha // … e blocos /* */) — SQL em comentário não conta.
|
||
function stripComments(source) {
|
||
return source.replace(/\/\*[\s\S]*?\*\//g, "").replace(/\/\/[^\n]*/g, "");
|
||
}
|
||
|
||
// Extrai o conteúdo de todos os literais de string (template, aspas duplas, aspas
|
||
// simples) de um trecho de código já sem comentários. Retorna a concatenação dos
|
||
// corpos — é nesse corpo que SQL cru vive.
|
||
export function extractStringLiterals(code) {
|
||
const re = /`(?:\\[\s\S]|[^\\`])*`|"(?:\\.|[^"\\])*"|'(?:\\.|[^'\\])*'/g;
|
||
const out = [];
|
||
let m;
|
||
while ((m = re.exec(code))) {
|
||
// tira as aspas/crases delimitadoras
|
||
out.push(m[0].slice(1, -1));
|
||
}
|
||
return out.join("\n |