mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-07-26 09:52:11 +03:00
* chore(release): open v3.8.35 development cycle
* fix db vacuum scheduler settings (#4726)
Scheduled VACUUM now follows Storage page settings (scheduledVacuum/vacuumHour) as single source of truth; env-flag control path removed. 11/11 vacuum-scheduler tests pass against release/v3.8.35 tip; no orphaned env refs. Integrated into release/v3.8.35.
* fix(tier): noAuth providers count as free; free filter returns empty … (#4753)
noAuth providers now classified free (union of legacy list + NOAUTH_PROVIDERS chat-tier derivation), -free arena_elo alias, and auto/<cat>:free returns an empty pool when no free candidate matches (opt-in legacy fallback via OMNIROUTE_AUTO_FREE_FALLBACK_TO_FULL_POOL). New env var documented in .env.example + ENVIRONMENT.md; CHANGELOG bullet added (maintainer co-author). 46/46 node + 56/56 vitest tests pass on release tip; env-doc-sync, docs-sync, typecheck:core, lint, file-size all green. Integrated into release/v3.8.35.
* refactor(chatCore): extrai 11 helpers de nível superior para 6 leaves puros (#3501) (#4571)
chatCore god-file decomposition (#3501): extract 6 pure leaves (cacheUsageMeta, executorClientHeaders, nonStreamingResponseBody, skillsFormat, streamErrorResult, streamFinalize) from chatCore.ts. Rebased onto release/v3.8.35 tip (resolved single chatCore.ts conflict — removed now-extracted inline buildExecutorClientHeaders). 265/265 chatcore tests, 26/26 new leaf tests, typecheck:core, cycles, file-size all green. Integrated into release/v3.8.35.
* refactor(chatCore): extrai resolveExecutorWithProxy + getExecutionCredentials para leaves (#3501) (#4646)
chatCore #3501: extract resolveExecutorWithProxy + getExecutionCredentials to leaves (executorProxy.ts, executionCredentials.ts). Clean cherry-pick onto release tip post-#4571. 12/12 new leaf tests, typecheck:core, cycles, file-size green. Integrated into release/v3.8.35.
* refactor(chatCore): extrai transforms de mensagens Claude p/ leaf (#3501) (#4708)
chatCore #3501: extract Claude upstream-message transforms to leaf (claudeUpstreamMessages.ts + claudeMessageTypes.ts). Clean cherry-pick post-#4646. 8/8 new leaf tests, typecheck/cycles/file-size green. Integrated into release/v3.8.35.
* refactor(chatCore): extrai persistAttemptLogs para leaf (#3501) (#4717)
chatCore #3501: extract persistAttemptLogs to leaf (attemptLogging.ts). Rebased onto release tip post-#4708 (resolved imports conflict: kept tip's resolveCompressionHeader from compression Phase 3, dropped now-unused logTruncation import moved into the leaf). 288/288 chatcore tests, typecheck/cycles/file-size green. Integrated into release/v3.8.35.
* refactor(chatCore): extrai stageTrace + compressionUsageReceipt para leaves (#3501) (#4721)
chatCore #3501: extract stageTrace + compressionUsageReceipt to leaves. Clean cherry-pick post-#4717. 6/6 new leaf tests, typecheck/cycles/file-size green. Integrated into release/v3.8.35.
* refactor(chatCore): extrai prepareUpstreamBody (1ª sub-fatia do executeProviderRequest, #3501) (#4730)
chatCore #3501: extract prepareUpstreamBody (first sub-slice of executeProviderRequest) to leaf (upstreamBody.ts). Clean cherry-pick post-#4721. 7/7 new leaf tests, full 301/301 chatcore suite, typecheck/cycles/file-size green. Completes the 6-PR chatCore decomposition stack into release/v3.8.35.
* fix(db): make db-backup import size cap configurable (#4719) (#4757)
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* chore(quality): expand check:release-green to the FULL release-PR gate set (#4758)
The release-green pre-flight (Solution C) previously covered only a subset of the
gates that run exclusively on the release PR (PR→main), so reds still accrued
silently on release/** and surfaced in ~40-min layers at release time (v3.8.34:
3 CI rounds — CodeQL sanitization, then the fail-fast Quality Ratchet revealing
openapi then cyclomatic-complexity one push at a time, plus zizmor/integration).
Now check:release-green reproduces the COMPLETE release-PR gate set and reports
EVERY red in one pass (collected, not fail-fast):
- New DRIFT ratchets (report-only, rebaselined at release, never block):
cyclomatic complexity, dead-code, type-coverage, compression-budget,
openapi-coverage, workflow-lint (zizmor), codeql-ratchet.
- New HARD gates (real defects): docs-all (fabricated-docs strict + i18n mirror
sync) and the integration test suite (gated behind !--quick).
The only release-PR gates it still cannot reproduce locally are GitHub-side CodeQL
semantic analysis and SonarQube/SonarCloud (external services).
The nightly-release-green workflow and /green-prs inherit the expanded coverage
automatically (they invoke this script), so cycle drift is now surfaced
continuously and the release PR is green on its first CI run.
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* fix(dashboard): add missing onboarding.tiers step title (#4698) (#4755)
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* feat(compression): Output Styles registry + D0 telemetry (Phase 4A) (#4694)
Phase 4A: Output Styles registry + D0 telemetry. Integrated into release/v3.8.35.
* feat(compression): SLM tier for ultra (Phase 4B) [stacked on #4694] (#4707)
Phase 4B: SLM tier for ultra. Integrated into release/v3.8.35.
* feat(compression): context-budget adaptive compression (Phase 4C) [stacked on #4707] (#4716)
Phase 4C: adaptive context-budget compression. Integrated into release/v3.8.35.
* feat(compression): offline evaluation harness (Phase 4 D1) [stacked on #4716] (#4720)
Phase 4 D1: offline evaluation harness. Integrated into release/v3.8.35.
* fix(sse): deepseek-web folds role:tool results into prompt transcript (#4712) (#4756)
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* fix(dashboard): remove dead unconditional useLiveRequests call in HomePageClient (#4759, #4745, #4596) (#4761)
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* fix(dashboard): dedupe provider nodes by id on compatible-provider add (#4746) (#4768)
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* chore(db): re-export compressionRunTelemetry from localDb to satisfy db-rules (#4775)
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
* docs(security): add canonical STRIDE-based threat model (#4783)
Canonical STRIDE threat model. Integrated into release/v3.8.35.
* test(dashboard): add smoke test for home client dashboard (#4793)
Smoke test guarding the dashboard home client render (regression #4745/#4759). Code fix already landed via #4761; this PR's jsdom smoke test is the net-new regression guard. Integrated into release/v3.8.35.
* fix(combos): auto-promote zeroLatencyOptimizationsEnabled so legacy configs (pre-3.8.33 fallbackCompressionMode="lite") round-trip on the first GUI edit (#4774)
Auto-promote zeroLatencyOptimizationsEnabled + strip v3.8.31-era removed keys so legacy combo configs round-trip through PUT /api/combos/{id} on first GUI edit (closes #4382 followup). Pre-merge: rewrote the now-stale reject test to assert auto-promotion + added passthrough/round-trip regression guards; reconciled combos/page.tsx file-size baseline. Integrated into release/v3.8.35.
* refactor(chatCore): extrai parse + usage-stats não-streaming do executeProviderRequest (#3501) (#4762)
chatCore #3501: extract parseNonStreamingResponseBody + recordNonStreamingUsageStats. Integrated into release/v3.8.35.
* refactor(chatCore): extrai recordContextEditingTelemetryHook (#3501) (#4779)
chatCore #3501: extract recordContextEditingTelemetryHook. Integrated into release/v3.8.35.
* refactor(chatCore): extrai recordCompressionCacheStats (#3501) (#4792)
chatCore #3501: extract recordCompressionCacheStats. Integrated into release/v3.8.35.
* refactor(chatCore): extrai writeCavemanOutputAnalytics (#3501) (#4794)
chatCore #3501: extract writeCavemanOutputAnalytics. Integrated into release/v3.8.35.
* refactor(chatCore): extrai scheduleQuotaShareConsumption (POST-hook não-streaming, #3501) (#4780)
chatCore #3501: extract scheduleQuotaShareConsumption (non-streaming POST-hook). Integrated into release/v3.8.35.
* refactor(chatCore): extrai emitRequestGamificationEvent (helper compartilhado DRY, #3501) (#4776)
chatCore #3501: extract emitRequestGamificationEvent (DRY streaming/non-streaming). Integrated into release/v3.8.35.
* refactor(chatCore): extrai runPluginOnResponseHook (#3501) (#4782)
chatCore #3501: extract runPluginOnResponseHook. Integrated into release/v3.8.35.
* refactor(chatCore): extrai scheduleStreamingQuotaShareConsumption (POST-hook streaming, #3501) (#4784)
chatCore #3501: extract scheduleStreamingQuotaShareConsumption (streaming POST-hook). Integrated into release/v3.8.35.
* refactor(chatCore): extrai recordStreamingUsageStats (analytics de usage streaming, #3501) (#4791)
chatCore #3501: extract recordStreamingUsageStats. Integrated into release/v3.8.35.
* refactor(chatCore): extrai recordStreamingCost (custo por-request streaming, #3501) (#4790)
chatCore #3501: extract recordStreamingCost (per-request streaming cost). Integrated into release/v3.8.35.
* docs(readme): credit ponytail + OmniCompress; restore env-doc-sync release-green (#4799)
README compression credits (ponytail/OmniCompress) + env-doc-sync ignore for eval-only OMNIROUTE_EVAL_CREDENTIALS (restores release-green after #4720). Integrated into release/v3.8.35.
* chore(quality): trim combo-config.test.ts comments under file-size cap (#4774 follow-up) (#4800)
Restore file-size release-green. Integrated into release/v3.8.35.
* feat(api-docs): Redoc-rendered /api/docs + consolidate OpenAPI spec to docs/openapi.yaml (#4781)
Redoc /api/docs + OpenAPI spec consolidated to docs/openapi.yaml (canonical 201-path complete spec; old path → legacy fallback). All refs/gates/tests/CI updated. Integrated into release/v3.8.35.
* docs(compression): declare Phase 4 layers — Output Styles, adaptive dial, per-request control (#4801)
The README compression section listed the 9 input engines but not the Phase 4
layers now in production:
- Output Styles (output-axis steering: terse-prose / less-code / terse-cjk, lite/full/ultra)
- adaptive context-budget dial (reserve-output|percentage|absolute · floor|replace-autotrigger|off)
- per-request x-omniroute-compression precedence + the offline eval harness
Also bumped the highlights range to v3.8.35, expanded the compression feature bullet,
and marked the GUIDE's Phase 4 row Shipped (was 'Planned' — it's merged on v3.8.35).
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* chore(release): finalize v3.8.35 CHANGELOG + docs reconciliation
- CHANGELOG: complete 3.8.35 section (all 35 commits since v3.8.34,
contributor attribution: @rdself @megamen32 @KooshaPari @JxnLexn)
- docs(security): align THREAT_MODEL.md refs with real code
(routeGuard.ts, tokenLimits.ts, /api/monitoring/health) — fabricated-docs gate
- check:fabricated-docs: skip docs/superpowers/specs (dated research reports)
- i18n: sync 3.8.35 section into 41 CHANGELOG mirrors (docs-sync size gate)
- ratchet rebaseline: cyclomatic 1916->1920, eslintWarnings 3907->3912
(inherited cycle drift; release-finalize diff is docs-only)
* fix(release): resolve inherited base-reds surfaced by v3.8.35 release CI
Cycle base-reds that only run on PR→main (not the PR→release fast-path):
- test(autoCombo): suffixComposition-4517 used node:test in a vitest-only dir
(#4753) → vitest found no suite. Switch to the vitest API. (Vitest job)
- test(agentSkills): openapiParser fixture wrote docs/reference/openapi.yaml;
parser reads docs/openapi.yaml since #4781 → point fixture at the new path.
(Unit/Coverage/Node24/Node26 shard 4)
- test(integration): proxy-pipeline source-scan expected inline streaming-cost
code that #4790/#3501 extracted to the recordStreamingCost leaf → assert the
delegation instead. (Integration 1/2)
- fix(chatCore): derive the log trace id from crypto, not Math.random
(CodeQL js/insecure-randomness — log-correlation id, not a secret).
- test(resilience): circuit-breaker invalid-cooldown fallback asserted t>29000,
flaking on slow CI where ~1.6s elapsed gave t=28401 → tolerate wall-clock
drift (t>25000). (Unit 6/8)
* fix(usage): derive pending-request id from crypto, not Math.random
CodeQL js/insecure-randomness (#669): the pending-request id generated in
trackPendingRequest (usageHistory.ts) flows into attempt logging and was flagged
as insecure randomness in a security context. It's a log-correlation id, not a
secret — switch to crypto RNG to clear the alert. Pairs with the chatCore traceId
fix in 37c49781a (same sink).
---------
Co-authored-by: Diego Rodrigues de Sa e Souza <souzamiriamrodrigues790@gmail.com>
Co-authored-by: Randi <55005611+rdself@users.noreply.github.com>
Co-authored-by: Demiurge The Single <megamen932@gmail.com>
Co-authored-by: KooshaPari <42529354+KooshaPari@users.noreply.github.com>
Co-authored-by: Jan Leon <Jan.gaschler@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
379 lines
14 KiB
TypeScript
379 lines
14 KiB
TypeScript
/**
|
|
* Proxy Pipeline Integration Tests — T-3
|
|
*
|
|
* Tests the proxy pipeline wiring: format detection, credential retry loop,
|
|
* circuit breaker integration, and the new Phase 2 modules (DI container,
|
|
* prompt versioning, plugin architecture, eval cleanup).
|
|
*
|
|
* @module tests/integration/proxy-pipeline.test.ts
|
|
*/
|
|
|
|
import { describe, it, beforeEach, afterEach } from "node:test";
|
|
import assert from "node:assert/strict";
|
|
import { readFileSync, existsSync } from "node:fs";
|
|
import { join, dirname } from "node:path";
|
|
import { fileURLToPath } from "node:url";
|
|
|
|
const __dirname = dirname(fileURLToPath(import.meta.url));
|
|
const ROOT = join(__dirname, "..", "..");
|
|
|
|
function readSrc(relPath) {
|
|
const full = join(ROOT, "src", relPath);
|
|
if (!existsSync(full)) return null;
|
|
return readFileSync(full, "utf8");
|
|
}
|
|
|
|
function readOpenSse(relPath) {
|
|
const full = join(ROOT, "open-sse", relPath);
|
|
if (!existsSync(full)) return null;
|
|
return readFileSync(full, "utf8");
|
|
}
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 1. Chat Handler Pipeline Wiring
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("Chat Pipeline — handleSingleModelChat decomposition", () => {
|
|
const src = readSrc("sse/handlers/chat.ts");
|
|
const helpersSrc = readSrc("sse/handlers/chatHelpers.ts");
|
|
const coreSrc = readOpenSse("handlers/chatCore.ts");
|
|
|
|
it("should define resolveModelOrError helper", () => {
|
|
assert.ok(helpersSrc, "chatHelpers.ts should exist");
|
|
assert.match(helpersSrc, /function\s+resolveModelOrError/);
|
|
});
|
|
|
|
it("should define checkPipelineGates helper", () => {
|
|
assert.match(helpersSrc, /function\s+checkPipelineGates/);
|
|
});
|
|
|
|
it("should define executeChatWithBreaker helper", () => {
|
|
assert.match(helpersSrc, /function\s+executeChatWithBreaker/);
|
|
});
|
|
|
|
it("should keep cost accounting in the core chat pipeline", () => {
|
|
assert.ok(coreSrc, "open-sse/handlers/chatCore.ts should exist");
|
|
assert.match(coreSrc, /calculateCost\(/);
|
|
assert.match(coreSrc, /recordCost\(/);
|
|
});
|
|
|
|
it("handleSingleModelChat should use resolveModelOrError", () => {
|
|
// Extract handleSingleModelChat body
|
|
assert.match(src, /resolveModelOrError\(\s*modelStr/);
|
|
});
|
|
|
|
it("handleSingleModelChat should use checkPipelineGates", () => {
|
|
assert.match(src, /checkPipelineGates\(provider/);
|
|
});
|
|
|
|
it("handleSingleModelChat should use executeChatWithBreaker", () => {
|
|
assert.match(src, /executeChatWithBreaker\(/);
|
|
});
|
|
|
|
it("chatCore should record cost for both non-streaming and streaming responses", () => {
|
|
// Non-streaming cost is still recorded inline; streaming cost was extracted to
|
|
// the recordStreamingCost leaf (open-sse/handlers/chatCore/streamingCost.ts,
|
|
// #4790 / #3501), so chatCore now delegates streaming cost to it.
|
|
assert.match(coreSrc, /if \(apiKeyInfo\?\.id && estimatedCost > 0\)/);
|
|
assert.match(coreSrc, /recordStreamingCost\(/);
|
|
});
|
|
});
|
|
|
|
describe("Chat Pipeline — combo fallback support", () => {
|
|
const src = readSrc("sse/handlers/chat.ts");
|
|
|
|
it("should import handleComboChat", () => {
|
|
assert.ok(src, "chat.ts should exist");
|
|
assert.match(src, /handleComboChat/);
|
|
});
|
|
|
|
it("should delegate to handleSingleModelChat for each combo model", () => {
|
|
assert.match(src, /handleSingleModel.*handleSingleModelChat/s);
|
|
});
|
|
|
|
it("should preflight provider credentials before attempting combo models", () => {
|
|
assert.match(src, /getProviderCredentialsWithQuotaPreflight/);
|
|
});
|
|
});
|
|
|
|
describe("Chat Pipeline — circuit breaker integration", () => {
|
|
const helpersSrc = readSrc("sse/handlers/chatHelpers.ts");
|
|
|
|
it("should import providerCircuitOpenResponse", () => {
|
|
assert.ok(helpersSrc, "chatHelpers.ts should exist");
|
|
assert.match(helpersSrc, /providerCircuitOpenResponse/);
|
|
});
|
|
|
|
it("should handle circuit-open responses with retry-after", () => {
|
|
assert.match(helpersSrc, /retryAfterMs/);
|
|
});
|
|
|
|
it("should reject requests when circuit is open via structured provider breaker response", () => {
|
|
assert.match(helpersSrc, /providerCircuitOpenResponse\(provider,\s*retryAfterSec\)/);
|
|
});
|
|
});
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 2. DI Container (A-5)
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("DI Container — container.ts", () => {
|
|
let container;
|
|
|
|
beforeEach(async () => {
|
|
const mod = await import("../../src/lib/container.ts");
|
|
container = mod.container;
|
|
});
|
|
|
|
afterEach(() => {
|
|
// Don't reset — keep default registrations
|
|
});
|
|
|
|
it("should export a container singleton", () => {
|
|
assert.ok(container);
|
|
assert.equal(typeof container.register, "function");
|
|
assert.equal(typeof container.resolve, "function");
|
|
assert.equal(typeof container.has, "function");
|
|
});
|
|
|
|
it("should register and resolve a custom service", () => {
|
|
container.register("testService", () => ({ greeting: "hello" }));
|
|
const svc = container.resolve("testService");
|
|
assert.deepEqual(svc, { greeting: "hello" });
|
|
});
|
|
|
|
it("should return cached singleton on repeated resolve", () => {
|
|
let count = 0;
|
|
container.register("counterService", () => ({ value: ++count }));
|
|
const a = container.resolve("counterService");
|
|
const b = container.resolve("counterService");
|
|
assert.strictEqual(a, b);
|
|
assert.equal(a.value, 1);
|
|
});
|
|
|
|
it("should throw on resolving unregistered service", () => {
|
|
assert.throws(() => container.resolve("nonExistent"), /No factory registered/);
|
|
});
|
|
|
|
it("should have default registrations", () => {
|
|
const names = container.list();
|
|
assert.ok(names.includes("settings"), "should have settings");
|
|
assert.ok(names.includes("db"), "should have db");
|
|
assert.ok(names.includes("encryption"), "should have encryption");
|
|
assert.ok(names.includes("policyEngine"), "should have policyEngine");
|
|
assert.ok(names.includes("circuitBreaker"), "should have circuitBreaker");
|
|
assert.ok(names.includes("telemetry"), "should have telemetry");
|
|
});
|
|
|
|
it("should support re-registration (overwrite)", () => {
|
|
container.register("testOverwrite", () => "v1");
|
|
assert.equal(container.resolve("testOverwrite"), "v1");
|
|
container.register("testOverwrite", () => "v2");
|
|
assert.equal(container.resolve("testOverwrite"), "v2");
|
|
});
|
|
});
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 3. Plugin Architecture (L-8) — hooks.ts registry
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("Plugin Architecture — plugins/hooks.ts", () => {
|
|
let hooks;
|
|
|
|
beforeEach(async () => {
|
|
hooks = await import("../../src/lib/plugins/hooks.ts");
|
|
hooks.resetHooks();
|
|
});
|
|
|
|
afterEach(() => {
|
|
hooks.resetHooks();
|
|
});
|
|
|
|
it("should register hooks for events", () => {
|
|
hooks.registerHook("onRequest", "test-logger", () => {}, 10);
|
|
const list = hooks.getHooks("onRequest");
|
|
assert.equal(list.length, 1);
|
|
assert.equal(list[0].pluginName, "test-logger");
|
|
assert.equal(list[0].priority, 10);
|
|
});
|
|
|
|
it("should sort hooks by priority", () => {
|
|
hooks.registerHook("onRequest", "low", () => {}, 200);
|
|
hooks.registerHook("onRequest", "high", () => {}, 1);
|
|
hooks.registerHook("onRequest", "mid", () => {}, 50);
|
|
|
|
const list = hooks.getHooks("onRequest");
|
|
assert.deepEqual(
|
|
list.map((r) => r.pluginName),
|
|
["high", "mid", "low"]
|
|
);
|
|
});
|
|
|
|
it("should run onRequest hooks in priority order", async () => {
|
|
const order = [];
|
|
hooks.registerHook(
|
|
"onRequest",
|
|
"first",
|
|
() => {
|
|
order.push("first");
|
|
},
|
|
1
|
|
);
|
|
hooks.registerHook(
|
|
"onRequest",
|
|
"second",
|
|
() => {
|
|
order.push("second");
|
|
},
|
|
2
|
|
);
|
|
|
|
const ctx = { requestId: "r1", body: {}, model: "test", metadata: {} };
|
|
await hooks.runOnRequest(ctx);
|
|
assert.deepEqual(order, ["first", "second"]);
|
|
});
|
|
|
|
it("should support request blocking via emitHookBlocking", async () => {
|
|
hooks.registerHook(
|
|
"onRequest",
|
|
"blocker",
|
|
() => ({
|
|
blocked: true,
|
|
response: { error: "denied" },
|
|
}),
|
|
1
|
|
);
|
|
hooks.registerHook(
|
|
"onRequest",
|
|
"never-runs",
|
|
() => {
|
|
throw new Error("should not run");
|
|
},
|
|
2
|
|
);
|
|
|
|
const ctx = { requestId: "r2", body: {}, model: "test", metadata: {} };
|
|
const result = await hooks.emitHookBlocking("onRequest", ctx);
|
|
assert.equal(result.blocked, true);
|
|
assert.deepEqual(result.response, { error: "denied" });
|
|
});
|
|
|
|
it("should unregister all hooks for a plugin", () => {
|
|
hooks.registerHook("onRequest", "removable", () => {});
|
|
hooks.registerHook("onResponse", "removable", () => {});
|
|
assert.equal(hooks.getHooks("onRequest").length, 1);
|
|
hooks.unregisterHooks("removable");
|
|
assert.equal(hooks.getHooks("onRequest").length, 0);
|
|
assert.equal(hooks.getHooks("onResponse").length, 0);
|
|
});
|
|
|
|
it("should run onResponse hooks", async () => {
|
|
hooks.registerHook("onResponse", "response-modifier", (payload) => ({
|
|
response: { ...payload.response, modified: true },
|
|
}));
|
|
|
|
const ctx = { requestId: "r3", body: {}, model: "test", metadata: {} };
|
|
const result = await hooks.runOnResponse(ctx, { data: "original" });
|
|
assert.equal(result.modified, true);
|
|
assert.equal(result.data, "original");
|
|
});
|
|
|
|
it("should fire onError hooks", async () => {
|
|
let caught = false;
|
|
hooks.registerHook("onError", "error-handler", () => {
|
|
caught = true;
|
|
});
|
|
|
|
const ctx = { requestId: "r4", body: {}, model: "test", metadata: {} };
|
|
await hooks.runOnError(ctx, new Error("test error"));
|
|
assert.equal(caught, true);
|
|
});
|
|
});
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 4. Prompt Template Versioning (L-6)
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("Prompt Template Versioning — prompts.ts module existence", () => {
|
|
it("prompts.ts should exist", () => {
|
|
const full = join(ROOT, "src", "lib", "db", "prompts.ts");
|
|
assert.ok(existsSync(full), "prompts.ts should exist");
|
|
});
|
|
|
|
it("should export CRUD functions", () => {
|
|
const src = readFileSync(join(ROOT, "src", "lib", "db", "prompts.ts"), "utf8");
|
|
assert.match(src, /export function savePrompt/);
|
|
assert.match(src, /export function getActivePrompt/);
|
|
assert.match(src, /export function getPromptVersion/);
|
|
assert.match(src, /export function listPromptVersions/);
|
|
assert.match(src, /export function listPrompts/);
|
|
assert.match(src, /export function rollbackPrompt/);
|
|
assert.match(src, /export function renderPrompt/);
|
|
});
|
|
|
|
it("should define PromptTemplate interface", () => {
|
|
const src = readFileSync(join(ROOT, "src", "lib", "db", "prompts.ts"), "utf8");
|
|
assert.match(src, /export interface PromptTemplate/);
|
|
});
|
|
|
|
it("should use content hashing for deduplication", () => {
|
|
const src = readFileSync(join(ROOT, "src", "lib", "db", "prompts.ts"), "utf8");
|
|
assert.match(src, /content_hash/);
|
|
assert.match(src, /sha256/);
|
|
});
|
|
});
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 5. Eval cleanup (Task 28)
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("Eval cleanup — orphaned scheduler module", () => {
|
|
it("scheduler.ts should remain deleted", () => {
|
|
const full = join(ROOT, "src", "lib", "evals", "scheduler.ts");
|
|
assert.equal(existsSync(full), false, "scheduler.ts should stay removed");
|
|
});
|
|
});
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 6. Migration Runner (E-5)
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("Migration System — files exist", () => {
|
|
it("migrationRunner.ts should exist", () => {
|
|
const full = join(ROOT, "src", "lib", "db", "migrationRunner.ts");
|
|
assert.ok(existsSync(full), "migrationRunner.ts should exist");
|
|
});
|
|
|
|
it("001_initial_schema.sql should exist", () => {
|
|
const full = join(ROOT, "src", "lib", "db", "migrations", "001_initial_schema.sql");
|
|
assert.ok(existsSync(full), "001_initial_schema.sql should exist");
|
|
});
|
|
|
|
it("core.ts should reference migration runner", () => {
|
|
const src = readSrc("lib/db/core.ts");
|
|
assert.ok(src);
|
|
assert.match(src, /runMigrations/);
|
|
assert.match(src, /_omniroute_migrations/);
|
|
});
|
|
});
|
|
|
|
// ═══════════════════════════════════════════════════
|
|
// 7. CORS Configuration (L-5)
|
|
// ═══════════════════════════════════════════════════
|
|
|
|
describe("CORS — centralized configuration", () => {
|
|
it("shared/utils/cors.ts should exist", () => {
|
|
const full = join(ROOT, "src", "shared", "utils", "cors.ts");
|
|
assert.ok(existsSync(full), "shared/utils/cors.ts should exist");
|
|
});
|
|
|
|
it("should export CORS_HEADERS without a wildcard origin", () => {
|
|
const src = readSrc("shared/utils/cors.ts");
|
|
assert.match(src, /CORS_HEADERS/);
|
|
// Extract the CORS_HEADERS object body (between { and }) to avoid matching JSDoc comments
|
|
const objMatch = src.match(/CORS_HEADERS\s*=\s*\{([^}]+)\}/);
|
|
assert.ok(objMatch, "CORS_HEADERS object should be found");
|
|
assert.doesNotMatch(objMatch[1], /Access-Control-Allow-Origin/);
|
|
});
|
|
});
|