can1357
a050474af7
- Migrated all wire protocol, schema definitions, and tools validation from Zod to ArkType across multiple packages. - Updated extension runtimes, custom tools loader, and TypeBox compatibility shim to expose and use ArkType instances. - Added a comprehensive ArkType migration guide, validation parity tests, and helper utilities. - Removed redundant PDF asset routing and parsing implementations from the read tool.
89 lines
3.0 KiB
TypeScript
89 lines
3.0 KiB
TypeScript
/**
|
||
* Contract: tool schema token estimation reflects the wire JSON Schema.
|
||
*
|
||
* Tools authored with arktype must be counted by the JSON Schema providers
|
||
* actually receive — not by stringifying the arktype instance's enumerable
|
||
* internals, which massively overcounts.
|
||
*/
|
||
import { describe, expect, it } from "bun:test";
|
||
import { arkToWireSchema } from "@oh-my-pi/pi-ai/utils/schema";
|
||
import {
|
||
type ContextBreakdown,
|
||
estimateToolSchemaTokens,
|
||
renderContextUsage,
|
||
} from "@oh-my-pi/pi-coding-agent/modes/utils/context-usage";
|
||
import { type } from "arktype";
|
||
|
||
describe("estimateToolSchemaTokens", () => {
|
||
it("counts arktype tool schemas by their wire JSON Schema, not arktype internals", () => {
|
||
const parameters = type({
|
||
"query /** search query */": "string",
|
||
"limit?": "number",
|
||
});
|
||
const arktypeEstimate = estimateToolSchemaTokens([
|
||
{ name: "web_search", description: "Searches the web.", parameters } as never,
|
||
]);
|
||
const wireEstimate = estimateToolSchemaTokens([
|
||
{ name: "web_search", description: "Searches the web.", parameters: arkToWireSchema(parameters) } as never,
|
||
]);
|
||
expect(arktypeEstimate).toBe(wireEstimate);
|
||
});
|
||
});
|
||
|
||
/**
|
||
* Contract: the /context panel surfaces estimated snapcompact wire savings —
|
||
* applied swaps show "saves" figures, inactive states say why.
|
||
*/
|
||
describe("renderContextUsage snapcompact section", () => {
|
||
const themeStub = {
|
||
fg: (_color: string, text: string) => text,
|
||
bold: (text: string) => text,
|
||
} as never;
|
||
|
||
function breakdownWith(snapcompact: ContextBreakdown["snapcompact"]): ContextBreakdown {
|
||
return {
|
||
model: { id: "test-model", name: "Test Model", contextWindow: 200000 } as never,
|
||
contextWindow: 200000,
|
||
categories: [],
|
||
usedTokens: 27929,
|
||
autoCompactBufferTokens: 0,
|
||
freeTokens: 172071,
|
||
snapcompact,
|
||
};
|
||
}
|
||
|
||
it("renders savings, skip reasons, and the wire total", () => {
|
||
const output = renderContextUsage(
|
||
breakdownWith({
|
||
visionCapable: true,
|
||
systemPrompt: {
|
||
applied: true,
|
||
scope: "all",
|
||
textTokens: 9768,
|
||
frames: 2,
|
||
imageTokens: 6600,
|
||
savedTokens: 3168,
|
||
},
|
||
toolResults: { total: 3, swapped: 0, textTokens: 0, frames: 0, imageTokens: 0, savedTokens: 0 },
|
||
savedTokens: 3168,
|
||
}),
|
||
themeStub,
|
||
);
|
||
expect(output).toContain("Snapcompact (estimated wire savings)");
|
||
expect(output).toContain("System prompt (all): saves ~3.2K (9.8K text → 2 frames ≈ 6.6K)");
|
||
expect(output).toContain("Tool results: none imaged (3 in history)");
|
||
// 27929 logical − 3168 saved ≈ 25K on the wire.
|
||
expect(output).toContain("Next request: ~25K tokens on the wire");
|
||
});
|
||
|
||
it("reports text-only models as inactive", () => {
|
||
const output = renderContextUsage(breakdownWith({ visionCapable: false, savedTokens: 0 }), themeStub);
|
||
expect(output).toContain("Snapcompact: inactive (model has no image input)");
|
||
});
|
||
|
||
it("omits the section entirely when no snapcompact setting is on", () => {
|
||
const output = renderContextUsage(breakdownWith(undefined), themeStub);
|
||
expect(output).not.toContain("Snapcompact");
|
||
});
|
||
});
|