summaryrefslogtreecommitdiffhomepage
path: root/packages/kernel/src/logging/logger.test.ts
diff options
context:
space:
mode:
authorAdam Malczewski <[email protected]>2026-06-11 21:12:03 +0900
committerAdam Malczewski <[email protected]>2026-06-11 21:12:03 +0900
commite7eada4802ceebd86c83bcd6e3eca70152e7f331 (patch)
tree447095fd60b43980358d1565506f3ae2430e5f29 /packages/kernel/src/logging/logger.test.ts
parent35937cee7f838e414eb8147c67205e01d85a4da0 (diff)
downloaddispatch-e7eada4802ceebd86c83bcd6e3eca70152e7f331.tar.gz
dispatch-e7eada4802ceebd86c83bcd6e3eca70152e7f331.zip
feat(lsp,cwd): LSP integration + per-conversation cwd; fix cache-warming cache bust
LSP + per-conversation CWD feature: - new bundled `lsp` extension: hand-rolled JSON-RPC codec (framing/rpc), lazy one-server-per-(serverID,root), per-cwd config resolution, on-demand `lsp` tool - `conversation-store`: getCwd/setCwd (cwdKey); `session-orchestrator` defaults a turn's cwd from the store - `transport-http`: cwd + lsp status endpoints; wire types in transport-contract - host-bin: register lsp; config wiring Cache-warming fix (the warm read 0% on the first reheat after a message): - warm assembled tools under a different cwd than the real turn (a reheat sends no cwd, and the warm service had no store fallback). The skills filter rewrites the cwd-sensitive `load_skill` description, so the tools block — the first bytes of the prompt-cache prefix — diverged and the cache missed entirely. Warm now resolves cwd as opts.cwd ?? conversationStore.getCwd(), mirroring handleMessage. - capture warm sends as `provider.request` spans flagged `warm:true` (thread a child logger into providerOpts) so warm vs real bodies are diffable (obs §3.1). - kernel logger: span-close now merges child-bound attrs like span-open, so a `warm:true` query finds the closed span (with usage/status), not just the open. Tests: warm forwards a warm-flagged logger; warm falls back to stored cwd; logger open/close attr consistency. Full suite green (873).
Diffstat (limited to 'packages/kernel/src/logging/logger.test.ts')
-rw-r--r--packages/kernel/src/logging/logger.test.ts45
1 files changed, 45 insertions, 0 deletions
diff --git a/packages/kernel/src/logging/logger.test.ts b/packages/kernel/src/logging/logger.test.ts
new file mode 100644
index 0000000..5d7bf45
--- /dev/null
+++ b/packages/kernel/src/logging/logger.test.ts
@@ -0,0 +1,45 @@
+import { describe, expect, it } from "vitest";
+import type { LogDeps, LogRecord, LogSink } from "../contracts/logging.js";
+import { createLogger } from "./logger.js";
+
+function harness() {
+ let idCounter = 0;
+ const deps: LogDeps = {
+ now: () => 1000 + idCounter * 10,
+ newId: () => `span-${++idCounter}`,
+ };
+ const records: LogRecord[] = [];
+ const sink: LogSink = { emit: (r) => records.push(r) };
+ return { logger: createLogger({ extensionId: "test" }, sink, deps), records };
+}
+
+describe("createLogger child-bound attributes", () => {
+ it("merges child-bound attrs into BOTH span-open and span-close records", () => {
+ const { logger, records } = harness();
+ // Bind `warm: true` via child() — mirrors the cache-warming capture path.
+ const warmLogger = logger.child({ conversationId: "c1", attrs: { warm: true } });
+
+ const span = warmLogger.span("provider.request", { model: "x" });
+ span.end({ attrs: { "usage.cacheReadTokens": 0 } });
+
+ const open = records.find((r) => r.kind === "span-open");
+ const close = records.find((r) => r.kind === "span-close");
+
+ // Open carries the bound attr (pre-existing behavior).
+ expect(open?.attributes?.warm).toBe(true);
+ // Close MUST carry it too, so a `warm = true` query finds the closed span
+ // (with its usage/status) — not just the open record.
+ expect(close?.attributes?.warm).toBe(true);
+ // Span-specific attrs from span()/end() are still present on close.
+ expect(close?.attributes?.model).toBe("x");
+ expect(close?.attributes?.["usage.cacheReadTokens"]).toBe(0);
+ });
+
+ it("omits attributes entirely when neither bound nor span attrs exist", () => {
+ const { logger, records } = harness();
+ const span = logger.span("bare");
+ span.end();
+ const close = records.find((r) => r.kind === "span-close");
+ expect(close?.attributes).toBeUndefined();
+ });
+});