summaryrefslogtreecommitdiffhomepage
path: root/packages/kernel/src/logging/logger.test.ts
diff options
context:
space:
mode:
authorAdam Malczewski <[email protected]>2026-06-27 01:12:40 +0900
committerAdam Malczewski <[email protected]>2026-06-27 01:12:40 +0900
commit98b0638838a8e754927d8c030ce8bded18d63e7d (patch)
tree0d4e21c3d4792fcd77e1040373b260e38efa34ce /packages/kernel/src/logging/logger.test.ts
parentd92a4af6191d7d20acf861adf605ad0227b6b287 (diff)
parent61e45e60d699ed1ca46f94a8f181c92a940317c6 (diff)
downloaddispatch-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.ts64
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();
+ });
});