fix: restore dnt build compatibility

Author: vicary

CONTRIBUTING.md

@@ -3,6 +3,13 @@
 ## Development
 
 ```bash
+# Readiness check
+deno test -A
+deno run build
+deno task check
+deno task lint
+deno fmt --check
+
 # Format
 deno fmt
 

docs/ContextOverhaul.md

@@ -812,7 +812,7 @@ docs/ContextOverhaulTests.md    — add Suite N (pre-tool routing) test cases
 - [ ] Policies exist for `Read`, `WebFetch`, `Bash`, `Grep`, `Glob`, `Task`.
 - [ ] Hard deny uses thrown errors from `tool.execute.before`.
 - [ ] Guidance is emitted at most once per canonical session lineage per type.
-- [ ] `deno test` passes; `deno task check` passes.
+- [ ] `deno test` passes; `deno run build` passes; `deno task check` passes.
 
 ### Phase 2: Pre-tool hook wiring
 
@@ -834,7 +834,7 @@ docs/ContextOverhaulTests.md    — add Suite N (pre-tool routing) test cases
 - [ ] The hook fires for parent and child sessions.
 - [ ] `tool.execute.before` does not call FalkorDB or Graphiti.
 - [ ] Parent and child sessions share one guidance throttle namespace.
-- [ ] `deno test` passes; `deno task check` passes.
+- [ ] `deno test` passes; `deno run build` passes; `deno task check` passes.
 
 ### Phase 3: Heavy-tool policies
 
@@ -862,7 +862,7 @@ docs/ContextOverhaulTests.md    — add Suite N (pre-tool routing) test cases
       args.
 - [ ] `Glob` does not rely on unsupported exclusion args.
 - [ ] `Task` rewrites delegated prompt text with routing instructions.
-- [ ] `deno test` passes; `deno task check` passes.
+- [ ] `deno test` passes; `deno run build` passes; `deno task check` passes.
 
 ### Phase 4: Extraction tightening
 
@@ -882,7 +882,7 @@ docs/ContextOverhaulTests.md    — add Suite N (pre-tool routing) test cases
 - [ ] Denied tool calls produce a compact event with the denial reason.
 - [ ] Modified/context-guided tool calls produce a compact event noting the
       routing action.
-- [ ] `deno test` passes; `deno task check` passes.
+- [ ] `deno test` passes; `deno run build` passes; `deno task check` passes.
 
 ### Phase 5: Snapshot tightening
 
@@ -900,7 +900,7 @@ docs/ContextOverhaulTests.md    — add Suite N (pre-tool routing) test cases
 - [ ] Snapshot with 50+ events (including routing events) stays within budget.
 - [ ] P0/P1 content (last request, active tasks, decisions) is never dropped.
 - [ ] Routing denial events are classified as P2.
-- [ ] `deno test` passes; `deno task check` passes.
+- [ ] `deno test` passes; `deno run build` passes; `deno task check` passes.
 
 ### Phase 6: Integration validation + documentation
 
@@ -917,8 +917,8 @@ docs/ContextOverhaulTests.md    — add Suite N (pre-tool routing) test cases
 **Acceptance criteria:**
 
 - [ ] All §3.1.1 alignment criteria (A1–A8) are met.
-- [ ] `deno test` passes; `deno task check` passes; `deno lint` passes;
-      `deno fmt --check` passes.
+- [ ] `deno test` passes; `deno run build` passes; `deno task check` passes;
+      `deno lint` passes; `deno fmt --check` passes.
 - [ ] `README.md` documents the pre-tool routing behavior.
 - [ ] `AGENTS.md` lists `tool.execute.before` in the hot-path section.
 - [ ] `docs/ContextOverhaulTests.md` includes Suite N with ≥ 10 test cases.
@@ -955,6 +955,7 @@ Add to `docs/ContextOverhaulTests.md` as Suite N:
 Before merging any part of this plan:
 
 - `deno test`
+- `deno run build`
 - `deno task check`
 - `deno lint`
 - `deno fmt --check`

docs/ContextOverhaulTests.md

@@ -666,6 +666,7 @@ deno test --allow-net --allow-env --filter "suite-k" src/
 # Full run
 docker compose -f tests/docker-compose.yml up -d
 deno test --allow-net --allow-env src/
+deno run build
 ```
 
 ### 9.2 CI Artifacts to Collect

src/services/connection-manager.ts

@@ -124,10 +124,16 @@ const validateEndpoint = (endpoint: string): string => {
   try {
     new URL(normalized);
   } catch (cause) {
-    throw new Error(
+    const error = new Error(
       `Invalid Graphiti endpoint: ${JSON.stringify(normalized)}`,
-      { cause },
     );
+    Object.defineProperty(error, "cause", {
+      value: cause,
+      writable: true,
+      configurable: true,
+      enumerable: false,
+    });
+    throw error;
   }
 
   return normalized;

src/services/session-executor.ts

@@ -110,12 +110,37 @@ const clampTimeoutSeconds = (
     defaults.maxCommandTimeoutSeconds,
   );
 
+type DenoCommandOutput = {
+  code: number;
+  stdout: Uint8Array;
+  stderr: Uint8Array;
+};
+
+type DenoCommandInstance = {
+  output(): Promise<DenoCommandOutput>;
+};
+
+type DenoCommandConstructor = new (
+  command: string | URL,
+  options?: {
+    args?: string[];
+    cwd?: string;
+    stdin?: "null" | "piped" | "inherit";
+    stdout?: "null" | "piped" | "inherit";
+    stderr?: "null" | "piped" | "inherit";
+    signal?: AbortSignal;
+  },
+) => DenoCommandInstance;
+
 const defaultRunCommand: NonNullable<SessionExecutorOptions["runCommand"]> =
   async ({ command, cwd, signal }) => {
     const shell = Deno.build.os === "windows"
       ? { executable: "cmd", args: ["/d", "/s", "/c", command] }
       : { executable: "/bin/sh", args: ["-lc", command] };
-    const output = await new Deno.Command(shell.executable, {
+    const DenoWithCommand = Deno as typeof Deno & {
+      Command: DenoCommandConstructor;
+    };
+    const output = await new DenoWithCommand.Command(shell.executable, {
       args: shell.args,
       cwd,
       stdin: "null",
@@ -556,14 +581,15 @@ export const createSessionExecutor = (
         truncated: false,
       };
 
-      return await ensureBatchResponseWithinBudget(batchResponse, {
+      const bounded = await ensureBatchResponseWithinBudget(batchResponse, {
         root_session_id: request.root_session_id,
         commands: request.commands,
       }, {
         responseBudgetBytes,
         maxNormalizedIndexedBodyBytes,
         storeArtifact,
-      }) as SessionBatchExecuteResponse;
+      });
+      return bounded as unknown as SessionBatchExecuteResponse;
     },
   };
 };