diff options
| author | Adam Malczewski <[email protected]> | 2026-06-27 01:12:40 +0900 |
|---|---|---|
| committer | Adam Malczewski <[email protected]> | 2026-06-27 01:12:40 +0900 |
| commit | 98b0638838a8e754927d8c030ce8bded18d63e7d (patch) | |
| tree | 0d4e21c3d4792fcd77e1040373b260e38efa34ce /packages/kernel/src/logging/logger.test.ts | |
| parent | d92a4af6191d7d20acf861adf605ad0227b6b287 (diff) | |
| parent | 61e45e60d699ed1ca46f94a8f181c92a940317c6 (diff) | |
| download | dispatch-98b0638838a8e754927d8c030ce8bded18d63e7d.tar.gz dispatch-98b0638838a8e754927d8c030ce8bded18d63e7d.zip | |
Merge branch 'dev' into feature/heartbeat
# Conflicts:
# packages/host-bin/package.json
# packages/host-bin/src/main.ts
# packages/session-orchestrator/src/orchestrator.ts
# packages/system-prompt/src/service.test.ts
# packages/system-prompt/src/service.ts
# packages/system-prompt/src/types.ts
# packages/transport-contract/package.json
# packages/transport-http/package.json
# packages/transport-http/src/app.test.ts
# packages/transport-http/src/app.ts
# packages/transport-http/src/extension.ts
# packages/transport-http/tsconfig.json
# tsconfig.json
Diffstat (limited to 'packages/kernel/src/logging/logger.test.ts')
| -rw-r--r-- | packages/kernel/src/logging/logger.test.ts | 64 |
1 files changed, 32 insertions, 32 deletions
diff --git a/packages/kernel/src/logging/logger.test.ts b/packages/kernel/src/logging/logger.test.ts index 5d7bf45..783d5af 100644 --- a/packages/kernel/src/logging/logger.test.ts +++ b/packages/kernel/src/logging/logger.test.ts @@ -3,43 +3,43 @@ 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 }; + 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 } }); + 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 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"); + 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); - }); + // 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(); - }); + 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(); + }); }); |
