mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-09-15 11:22:15 +03:00
* fix(ci): clear the base-reds the 2026-08-30 afternoon merge batch left on release/v3.8.51 (round 5)
- docs-counts / check-docs-counts-sync test: #12103 (Perplexity Agent) made it 352
providers; README, AGENTS.md, llm.txt (+42 i18n mirrors), package.json description
and the 4 README diagrams still said 351.
- api-route-typecheck: #11971 passes a third `{ featureEnabled }` argument to
appendNoThinkingVariants() that the helper never accepted (TS2554 — and the flag
silently did nothing); the helper now honours it. src/lib/skills/interception.ts
narrowed a mapped object with a `Record<string, string>` predicate (TS2677) —
predicate typed with the actual element shape.
Gates: check:docs-counts OK (test 28/28), check:docs-sync PASS, check:api-typecheck
OK (289 frozen). Refs #12103, #11971
* docs(env): document RATE_LIMIT_EXECUTION_MAX_WAIT_MS (#12027 added it to .env.example only)
* fix(ci): round 5b — freeze the react-hooks compiler-rule violations, align 7 tests to merged contracts
No new ESLint warnings: the exact CI command (lint:json --max-warnings 0) reports 278
problems on the tip — 226 from eslint-plugin-react-hooks 7 compiler rules
(set-state-in-effect 167, immutability 36, refs/static-components/purity/
preserve-manual-memoization) that were masked until the lockfile change of
dfc84ba030 invalidated the ESLint cache, plus 46 no-explicit-any in
tests/unit/call-log-cap.test.ts (#12026). Velocity phase: frozen with
`eslint --suppress-all` (+668 suppressions); the 5 now-unused
`eslint-disable react-hooks/immutability` directives and one unused import removed.
Verified: lint:json --max-warnings 0 → 0 problems.
Tests aligned to contracts merged this afternoon (all reproduced red on the pure tip):
- providers-constants-split: 235 → 236 (Perplexity Agent, #12103)
- sse-auth: a forced pin outside allowedConnections now yields no credential
instead of silently falling back (#12080)
- with-chat-admission-10786: withInjectionGuard(postHandler, { logger: null }) (#12117)
- hard-session-lease-bypass-inventory: classify src/app/api/oauth/codex/import/route.ts (#12116)
- usage-service-hardening: OpenCode Go official usage API shape (#12124)
- i18n placeholder parity: apiManager.restrictedToConnections rewritten as a plain
ICU plural (`{count, plural, one {# connection} other {# connections}}`) in en,
vi, pt-BR and the 40 __MISSING__ mirrors — the parity extractor counts every
`{word}` including the old literal `{s}`
Refs #12103, #12080, #12117, #12116, #12124, #12026
* fix(ci): run the ESLint warnings job on the box with an 8 GB heap; reserved-prefix set 398 → 400
The cold full lint with the react-hooks 7 compiler rules is killed on the 7 GB hosted
runner with no message (status null → exit 1, JSON never written) — it only looked
green while the ESLint cache was warm. tests/unit/provider-node-reserved-prefix.test.ts
aligned to the two prefixes the afternoon batch registered (#12103).
* test(ci): document the lint-guard runner exception; #9147 event-loop gap 400 → 800 ms
quality-rail-gate-membership pinned lint-guard to ubuntu-latest; the cold full lint is
OOM-killed there, so the job now runs on omni-light with an 8 GB heap — the test keeps
fast-gates pinned and asserts the documented exception. With the catalog at 352
providers the hosted shards measure 410–633 ms gaps on 9147-catalog-eventloop-yield
(3 runs); 800 ms still fails a true pin. Re-tighten with the v4.0 catalog split.
* chore(quality): summarize the ESLint report on failure — a red lint:json printed nothing
--format json --output-file swallows every problem; a red 'No new ESLint warnings' job
gave zero output (three blind debugging rounds in #12144), and a killed process (OOM,
status null) was equally silent. On any non-zero exit the runner now prints the problem
count and the first 60 'file:line rule — message' lines from the report.
* chore(lint): freeze react-hooks/immutability for the 5 UI test harnesses in the suppressions file
The rule fires for these files in CI but not locally (compiler analysis divergence),
so the inline eslint-disable directives read as 'unused directive' warnings locally.
A suppressions entry is symmetric: suppressed where the rule fires, tolerated as
unpruned (--pass-on-unpruned-suppressions) where it does not. Found via the new
lint:json failure summary.
298 lines
8.9 KiB
TypeScript
298 lines
8.9 KiB
TypeScript
// @vitest-environment jsdom
|
|
// tests/unit/ui/use-structured-output.test.tsx
|
|
// Runs via Vitest (vitest.config.ts)
|
|
// Uses React DOM directly (no @testing-library/dom dep required).
|
|
import React, { act, useRef } from "react";
|
|
import { createRoot } from "react-dom/client";
|
|
import { describe, it, expect } from "vitest";
|
|
import {
|
|
useStructuredOutput,
|
|
} from "../../../src/app/(dashboard)/dashboard/playground/hooks/useStructuredOutput";
|
|
|
|
// ─── Minimal hook test harness ────────────────────────────────────────────────
|
|
|
|
type HookResult<T> = { current: T };
|
|
|
|
function mountHook<T>(useHook: () => T): {
|
|
hookRef: HookResult<T>;
|
|
unmount: () => void;
|
|
} {
|
|
const hookRef: HookResult<T> = { current: undefined as unknown as T };
|
|
const container = document.createElement("div");
|
|
document.body.appendChild(container);
|
|
const root = createRoot(container);
|
|
|
|
function HookComponent() {
|
|
const captureRef = useRef<T>(undefined as unknown as T);
|
|
captureRef.current = useHook();
|
|
hookRef.current = captureRef.current;
|
|
return null;
|
|
}
|
|
|
|
act(() => {
|
|
root.render(React.createElement(HookComponent));
|
|
});
|
|
|
|
return {
|
|
hookRef,
|
|
unmount: () => {
|
|
act(() => root.unmount());
|
|
container.remove();
|
|
},
|
|
};
|
|
}
|
|
|
|
// ─── Test fixtures ────────────────────────────────────────────────────────────
|
|
|
|
const VALID_SCHEMA = {
|
|
name: "WeatherResponse",
|
|
schema: {
|
|
type: "object",
|
|
properties: {
|
|
temperature: { type: "number" },
|
|
condition: { type: "string" },
|
|
},
|
|
required: ["temperature", "condition"],
|
|
},
|
|
strict: true,
|
|
};
|
|
|
|
const VALID_SCHEMA_NO_REQUIRED = {
|
|
name: "FreeForm",
|
|
schema: {
|
|
type: "object",
|
|
properties: {
|
|
result: { type: "string" },
|
|
},
|
|
},
|
|
};
|
|
|
|
// ─── Tests ────────────────────────────────────────────────────────────────────
|
|
|
|
describe("useStructuredOutput", () => {
|
|
describe("initial state", () => {
|
|
it("starts with enabled=false, schema=null, error=null", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
expect(result.current.enabled).toBe(false);
|
|
expect(result.current.schema).toBeNull();
|
|
expect(result.current.error).toBeNull();
|
|
unmount();
|
|
});
|
|
});
|
|
|
|
describe("setEnabled()", () => {
|
|
it("sets enabled to true", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setEnabled(true);
|
|
});
|
|
|
|
expect(result.current.enabled).toBe(true);
|
|
unmount();
|
|
});
|
|
|
|
it("toggles enabled back to false", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setEnabled(true);
|
|
result.current.setEnabled(false);
|
|
});
|
|
|
|
expect(result.current.enabled).toBe(false);
|
|
unmount();
|
|
});
|
|
});
|
|
|
|
describe("setSchema()", () => {
|
|
it("accepts a valid schema and clears error", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA);
|
|
});
|
|
|
|
expect(result.current.schema).toEqual(VALID_SCHEMA);
|
|
expect(result.current.error).toBeNull();
|
|
unmount();
|
|
});
|
|
|
|
it("sets error when schema name is empty", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema({ name: "", schema: { type: "object" } });
|
|
});
|
|
|
|
expect(result.current.error).toBeTruthy();
|
|
expect(result.current.schema).toBeNull();
|
|
unmount();
|
|
});
|
|
|
|
it("sets error when name is too long (>64 chars)", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema({ name: "a".repeat(65), schema: { type: "object" } });
|
|
});
|
|
|
|
expect(result.current.error).toBeTruthy();
|
|
unmount();
|
|
});
|
|
|
|
it("clears error after previously invalid schema when valid one is set", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema({ name: "", schema: {} });
|
|
});
|
|
expect(result.current.error).toBeTruthy();
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA);
|
|
});
|
|
expect(result.current.error).toBeNull();
|
|
expect(result.current.schema).toEqual(VALID_SCHEMA);
|
|
unmount();
|
|
});
|
|
|
|
it("accepts schema with strict=false", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema({ ...VALID_SCHEMA, strict: false });
|
|
});
|
|
|
|
expect(result.current.schema?.strict).toBe(false);
|
|
expect(result.current.error).toBeNull();
|
|
unmount();
|
|
});
|
|
|
|
it("accepts schema without strict field", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA_NO_REQUIRED);
|
|
});
|
|
|
|
expect(result.current.schema?.strict).toBeUndefined();
|
|
expect(result.current.error).toBeNull();
|
|
unmount();
|
|
});
|
|
});
|
|
|
|
describe("validateResponse()", () => {
|
|
it("returns { valid: false, error } when no schema is set", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
const res = result.current.validateResponse({ temperature: 25, condition: "sunny" });
|
|
|
|
expect(res.valid).toBe(false);
|
|
expect(res.error).toBeTruthy();
|
|
unmount();
|
|
});
|
|
|
|
it("parses JSON string and validates correctly", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA);
|
|
});
|
|
|
|
const json = JSON.stringify({ temperature: 25, condition: "sunny" });
|
|
const res = result.current.validateResponse(json);
|
|
|
|
expect(res.valid).toBe(true);
|
|
unmount();
|
|
});
|
|
|
|
it("returns { valid: false } when content is not valid JSON string", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA);
|
|
});
|
|
|
|
const res = result.current.validateResponse("not-valid-json{{");
|
|
|
|
expect(res.valid).toBe(false);
|
|
expect(res.error).toContain("JSON");
|
|
unmount();
|
|
});
|
|
|
|
it("validates object directly (no JSON.parse needed)", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA);
|
|
});
|
|
|
|
const res = result.current.validateResponse({ temperature: 20, condition: "cloudy" });
|
|
expect(res.valid).toBe(true);
|
|
unmount();
|
|
});
|
|
|
|
it("returns { valid: false } when required field is missing", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA);
|
|
});
|
|
|
|
const res = result.current.validateResponse({ temperature: 25 }); // missing "condition"
|
|
expect(res.valid).toBe(false);
|
|
expect(res.error).toContain("condition");
|
|
unmount();
|
|
});
|
|
|
|
it("returns { valid: false } when content is null", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA_NO_REQUIRED);
|
|
});
|
|
|
|
const res = result.current.validateResponse(null);
|
|
expect(res.valid).toBe(false);
|
|
unmount();
|
|
});
|
|
|
|
it("returns { valid: false } when content is an array", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA_NO_REQUIRED);
|
|
});
|
|
|
|
const res = result.current.validateResponse([1, 2, 3]);
|
|
expect(res.valid).toBe(false);
|
|
unmount();
|
|
});
|
|
|
|
it("passes validation for schema without required field", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema(VALID_SCHEMA_NO_REQUIRED);
|
|
});
|
|
|
|
const res = result.current.validateResponse({});
|
|
expect(res.valid).toBe(true);
|
|
unmount();
|
|
});
|
|
|
|
it("passes validation for schema with no properties field", () => {
|
|
const { hookRef: result, unmount } = mountHook(() => useStructuredOutput());
|
|
|
|
act(() => {
|
|
result.current.setSchema({ name: "AnyObj", schema: { type: "object" } });
|
|
});
|
|
|
|
const res = result.current.validateResponse({ foo: "bar" });
|
|
expect(res.valid).toBe(true);
|
|
unmount();
|
|
});
|
|
});
|
|
});
|