agent-dbg 0.1.3 → 0.1.4

package/src/main.ts

@@ -15,6 +15,7 @@ if (process.argv.includes("--daemon")) {
 	await import("./commands/pause.ts");
 	await import("./commands/run-to.ts");
 	await import("./commands/break.ts");
+	await import("./commands/break-fn.ts");
 	await import("./commands/break-rm.ts");
 	await import("./commands/break-ls.ts");
 	await import("./commands/logpoint.ts");

package/src/protocol/messages.ts

@@ -10,6 +10,7 @@ const LaunchRequest = z.object({
 		command: z.array(z.string()),
 		brk: z.optional(z.boolean()),
 		port: z.optional(z.number()),
+		runtime: z.optional(z.string()),
 	}),
 });
 
@@ -17,6 +18,7 @@ const AttachRequest = z.object({
 	cmd: z.literal("attach"),
 	args: z.object({
 		target: z.string(),
+		runtime: z.optional(z.string()),
 	}),
 });
 
@@ -65,6 +67,15 @@ const BreakRequest = z.object({
 		condition: z.optional(z.string()),
 		hitCount: z.optional(z.number()),
 		urlRegex: z.optional(z.string()),
+		column: z.optional(z.number()),
+	}),
+});
+
+const BreakFnRequest = z.object({
+	cmd: z.literal("break-fn"),
+	args: z.object({
+		name: z.string(),
+		condition: z.optional(z.string()),
 	}),
 });
 
@@ -271,6 +282,7 @@ export const DaemonRequestSchema = z.union([
 	PauseRequest,
 	RunToRequest,
 	BreakRequest,
+	BreakFnRequest,
 	BreakRmRequest,
 	BreakLsRequest,
 	LogpointRequest,

package/tests/fixtures/dap/hello.c

@@ -0,0 +1,8 @@
+#include <stdio.h>
+
+int main() {
+    int x = 42;
+    int y = x + 1;
+    printf("x=%d y=%d\n", x, y);
+    return 0;
+}

package/tests/fixtures/dap/hello.dSYM/Contents/Info.plist

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple Computer//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+	<dict>
+		<key>CFBundleDevelopmentRegion</key>
+		<string>English</string>
+		<key>CFBundleIdentifier</key>
+		<string>com.apple.xcode.dsym.hello</string>
+		<key>CFBundleInfoDictionaryVersion</key>
+		<string>6.0</string>
+		<key>CFBundlePackageType</key>
+		<string>dSYM</string>
+		<key>CFBundleSignature</key>
+		<string>????</string>
+		<key>CFBundleShortVersionString</key>
+		<string>1.0</string>
+		<key>CFBundleVersion</key>
+		<string>1</string>
+	</dict>
+</plist>

package/tests/fixtures/dap/hello.dSYM/Contents/Resources/Relocations/aarch64/hello.yml

@@ -0,0 +1,5 @@
+---
+triple:          'arm64-apple-darwin'
+binary-path:     '/Users/waltoss/Code/Theodo/OpenSource/agent-dbg/tests/fixtures/dap/hello'
+relocations:     []
+...

package/tests/fixtures/hotpatch-active-fn.js

@@ -0,0 +1,7 @@
+// Fixture: hotpatch while function is on the call stack
+function compute(x) {
+	debugger;
+	return x * 2;
+}
+const result = compute(21);
+console.log("result:", result);

package/tests/integration/daemon-logging.test.ts

@@ -1,9 +1,9 @@
 import { describe, expect, test } from "bun:test";
 import { existsSync, readFileSync, unlinkSync } from "node:fs";
-import { join } from "node:path";
 import { tmpdir } from "node:os";
-import { DaemonLogger } from "../../src/daemon/logger.ts";
+import { join } from "node:path";
 import type { DaemonLogEntry } from "../../src/daemon/logger.ts";
+import { DaemonLogger } from "../../src/daemon/logger.ts";
 import { getDaemonLogPath } from "../../src/daemon/paths.ts";
 import { DebugSession } from "../../src/daemon/session.ts";
 
@@ -104,9 +104,9 @@ describe("DaemonLogger integration", () => {
 		const logPath = getDaemonLogPath(sessionName);
 		try {
 			// Use echo (not node), which won't emit an inspector URL
-			await expect(
-				session.launch(["echo", "hello"], { brk: true }),
-			).rejects.toThrow("Failed to detect inspector URL");
+			await expect(session.launch(["echo", "hello"], { brk: true })).rejects.toThrow(
+				"Failed to detect inspector URL",
+			);
 
 			const entries = readEntries(logPath);
 			expect(hasEvent(entries, "child.spawn")).toBe(true);
@@ -128,10 +128,9 @@ describe("DaemonLogger integration", () => {
 			// Use process.exit() to force the child to terminate.
 			// Node.js waits for the debugger to disconnect before exiting,
 			// so we disconnect CDP after a delay to allow the natural exit.
-			await session.launch(
-				["node", "-e", "setTimeout(() => process.exit(0), 200)"],
-				{ brk: false },
-			);
+			await session.launch(["node", "-e", "setTimeout(() => process.exit(0), 200)"], {
+				brk: false,
+			});
 
 			// Disconnect CDP so Node.js can actually exit
 			// (Node prints "Waiting for the debugger to disconnect..." otherwise)

package/tests/integration/mutation.test.ts

@@ -165,6 +165,39 @@ describe("Mutation: hotpatch", () => {
 		}
 	});
 
+	test("hotpatch works when edited function is on the call stack", async () => {
+		const session = new DebugSession("test-hotpatch-active-fn");
+		try {
+			await session.launch(["node", "tests/fixtures/hotpatch-active-fn.js"], {
+				brk: true,
+			});
+			await waitForState(session, "paused");
+
+			// Continue past --brk to the `debugger;` inside compute()
+			await session.continue();
+			await waitForState(session, "paused");
+
+			// We're now paused inside compute() — it's on the call stack
+			expect(session.isPaused()).toBe(true);
+
+			// Get original source and modify compute to multiply by 3 instead of 2
+			const source = await session.getSource({ file: "hotpatch-active-fn.js", all: true });
+			const originalText = source.lines.map((l) => l.text).join("\n");
+			const modifiedSource = originalText.replace("x * 2", "x * 3");
+
+			// Hotpatch should succeed thanks to allowTopFrameEditing
+			const result = await session.hotpatch("hotpatch-active-fn.js", modifiedSource);
+			expect(result.status).toBe("Ok");
+
+			// Verify source was updated
+			const newSource = await session.getSource({ file: "hotpatch-active-fn.js", all: true });
+			const newText = newSource.lines.map((l) => l.text).join("\n");
+			expect(newText).toContain("x * 3");
+		} finally {
+			await session.stop();
+		}
+	});
+
 	test("hotpatch throws for unknown file", async () => {
 		const session = await launchAndPauseAtDebugger("test-hotpatch-unknown");
 		try {

package/tests/unit/daemon-logger.test.ts

@@ -1,9 +1,9 @@
 import { afterEach, describe, expect, test } from "bun:test";
 import { existsSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
 import { tmpdir } from "node:os";
-import { DaemonLogger } from "../../src/daemon/logger.ts";
+import { join } from "node:path";
 import type { DaemonLogEntry } from "../../src/daemon/logger.ts";
+import { DaemonLogger } from "../../src/daemon/logger.ts";
 
 const testDir = tmpdir();
 

package/bun.lock

@@ -1,60 +0,0 @@
-{
-  "lockfileVersion": 1,
-  "configVersion": 1,
-  "workspaces": {
-    "": {
-      "name": "ndbg",
-      "dependencies": {
-        "@jridgewell/trace-mapping": "^0.3.31",
-        "zod": "^4.0.0",
-      },
-      "devDependencies": {
-        "@biomejs/biome": "^2.3.14",
-        "@types/bun": "latest",
-        "devtools-protocol": "^0.0.1581282",
-      },
-      "peerDependencies": {
-        "typescript": "^5",
-      },
-    },
-  },
-  "packages": {
-    "@biomejs/biome": ["@biomejs/biome@2.3.14", "", { "optionalDependencies": { "@biomejs/cli-darwin-arm64": "2.3.14", "@biomejs/cli-darwin-x64": "2.3.14", "@biomejs/cli-linux-arm64": "2.3.14", "@biomejs/cli-linux-arm64-musl": "2.3.14", "@biomejs/cli-linux-x64": "2.3.14", "@biomejs/cli-linux-x64-musl": "2.3.14", "@biomejs/cli-win32-arm64": "2.3.14", "@biomejs/cli-win32-x64": "2.3.14" }, "bin": { "biome": "bin/biome" } }, "sha512-QMT6QviX0WqXJCaiqVMiBUCr5WRQ1iFSjvOLoTk6auKukJMvnMzWucXpwZB0e8F00/1/BsS9DzcKgWH+CLqVuA=="],
-
-    "@biomejs/cli-darwin-arm64": ["@biomejs/cli-darwin-arm64@2.3.14", "", { "os": "darwin", "cpu": "arm64" }, "sha512-UJGPpvWJMkLxSRtpCAKfKh41Q4JJXisvxZL8ChN1eNW3m/WlPFJ6EFDCE7YfUb4XS8ZFi3C1dFpxUJ0Ety5n+A=="],
-
-    "@biomejs/cli-darwin-x64": ["@biomejs/cli-darwin-x64@2.3.14", "", { "os": "darwin", "cpu": "x64" }, "sha512-PNkLNQG6RLo8lG7QoWe/hhnMxJIt1tEimoXpGQjwS/dkdNiKBLPv4RpeQl8o3s1OKI3ZOR5XPiYtmbGGHAOnLA=="],
-
-    "@biomejs/cli-linux-arm64": ["@biomejs/cli-linux-arm64@2.3.14", "", { "os": "linux", "cpu": "arm64" }, "sha512-KT67FKfzIw6DNnUNdYlBg+eU24Go3n75GWK6NwU4+yJmDYFe9i/MjiI+U/iEzKvo0g7G7MZqoyrhIYuND2w8QQ=="],
-
-    "@biomejs/cli-linux-arm64-musl": ["@biomejs/cli-linux-arm64-musl@2.3.14", "", { "os": "linux", "cpu": "arm64" }, "sha512-LInRbXhYujtL3sH2TMCH/UBwJZsoGwfQjBrMfl84CD4hL/41C/EU5mldqf1yoFpsI0iPWuU83U+nB2TUUypWeg=="],
-
-    "@biomejs/cli-linux-x64": ["@biomejs/cli-linux-x64@2.3.14", "", { "os": "linux", "cpu": "x64" }, "sha512-ZsZzQsl9U+wxFrGGS4f6UxREUlgHwmEfu1IrXlgNFrNnd5Th6lIJr8KmSzu/+meSa9f4rzFrbEW9LBBA6ScoMA=="],
-
-    "@biomejs/cli-linux-x64-musl": ["@biomejs/cli-linux-x64-musl@2.3.14", "", { "os": "linux", "cpu": "x64" }, "sha512-KQU7EkbBBuHPW3/rAcoiVmhlPtDSGOGRPv9js7qJVpYTzjQmVR+C9Rfcz+ti8YCH+zT1J52tuBybtP4IodjxZQ=="],
-
-    "@biomejs/cli-win32-arm64": ["@biomejs/cli-win32-arm64@2.3.14", "", { "os": "win32", "cpu": "arm64" }, "sha512-+IKYkj/pUBbnRf1G1+RlyA3LWiDgra1xpS7H2g4BuOzzRbRB+hmlw0yFsLprHhbbt7jUzbzAbAjK/Pn0FDnh1A=="],
-
-    "@biomejs/cli-win32-x64": ["@biomejs/cli-win32-x64@2.3.14", "", { "os": "win32", "cpu": "x64" }, "sha512-oizCjdyQ3WJEswpb3Chdngeat56rIdSYK12JI3iI11Mt5T5EXcZ7WLuowzEaFPNJ3zmOQFliMN8QY1Pi+qsfdQ=="],
-
-    "@jridgewell/resolve-uri": ["@jridgewell/resolve-uri@3.1.2", "", {}, "sha512-bRISgCIjP20/tbWSPWMEi54QVPRZExkuD9lJL+UIxUKtwVJA8wW1Trb1jMs1RFXo1CBTNZ/5hpC9QvmKWdopKw=="],
-
-    "@jridgewell/sourcemap-codec": ["@jridgewell/sourcemap-codec@1.5.5", "", {}, "sha512-cYQ9310grqxueWbl+WuIUIaiUaDcj7WOq5fVhEljNVgRfOUhY9fy2zTvfoqWsnebh8Sl70VScFbICvJnLKB0Og=="],
-
-    "@jridgewell/trace-mapping": ["@jridgewell/trace-mapping@0.3.31", "", { "dependencies": { "@jridgewell/resolve-uri": "^3.1.0", "@jridgewell/sourcemap-codec": "^1.4.14" } }, "sha512-zzNR+SdQSDJzc8joaeP8QQoCQr8NuYx2dIIytl1QeBEZHJ9uW6hebsrYgbz8hJwUQao3TWCMtmfV8Nu1twOLAw=="],
-
-    "@types/bun": ["@types/bun@1.3.8", "", { "dependencies": { "bun-types": "1.3.8" } }, "sha512-3LvWJ2q5GerAXYxO2mffLTqOzEu5qnhEAlh48Vnu8WQfnmSwbgagjGZV6BoHKJztENYEDn6QmVd949W4uESRJA=="],
-
-    "@types/node": ["@types/node@25.2.2", "", { "dependencies": { "undici-types": "~7.16.0" } }, "sha512-BkmoP5/FhRYek5izySdkOneRyXYN35I860MFAGupTdebyE66uZaR+bXLHq8k4DirE5DwQi3NuhvRU1jqTVwUrQ=="],
-
-    "bun-types": ["bun-types@1.3.8", "", { "dependencies": { "@types/node": "*" } }, "sha512-fL99nxdOWvV4LqjmC+8Q9kW3M4QTtTR1eePs94v5ctGqU8OeceWrSUaRw3JYb7tU3FkMIAjkueehrHPPPGKi5Q=="],
-
-    "devtools-protocol": ["devtools-protocol@0.0.1581282", "", {}, "sha512-nv7iKtNZQshSW2hKzYNr46nM/Cfh5SEvE2oV0/SEGgc9XupIY5ggf84Cz8eJIkBce7S3bmTAauFD6aysMpnqsQ=="],
-
-    "typescript": ["typescript@5.9.3", "", { "bin": { "tsc": "bin/tsc", "tsserver": "bin/tsserver" } }, "sha512-jl1vZzPDinLr9eUt3J/t7V6FgNEw9QjvBPdysz9KfQDD41fQrC2Y4vKQdiaUpFT4bXlb1RHhLpp8wtm6M5TgSw=="],
-
-    "undici-types": ["undici-types@7.16.0", "", {}, "sha512-Zz+aZWSj8LE6zoxD+xrjh4VfkIG8Ya6LvYkZqtUQGJPZjYl53ypCaUwWqo7eI0x66KBGeRo+mlBEkMSeSZ38Nw=="],
-
-    "zod": ["zod@4.3.6", "", {}, "sha512-rftlrkhHZOcjDwkGlnUtZZkvaPHCsDATp4pGpuOOMDaTdDDXF91wuVDJoWoPsKX/3YPQ5fHuF3STjcYyKr+Qhg=="],
-  }
-}