test(#484): update go #308 bare-fn-bp e2e for auto-qualification; pin the rewrite

[debugmcpdev]

Summary

Test-only follow-up to #484 (issue #467): the #308 bare-fn-bp e2e test still set function: 'main' and asserted the program runs to completion — but #484 deliberately auto-qualifies a bare 'main' to 'main.main', so the breakpoint now binds and the session pauses (expected 'paused' to be 'stopped' at the state poll). The #484 commit even noted the behavior flip ("previously ran to completion"); the e2e just wasn't updated, and it isn't run in CI, so the miss surfaced only in a local full run.

Changes (all in tests/e2e/mcp-server-smoke-go.test.ts + .gitignore)

• Function breakpoints: no feedback when a name never binds (Go bare "main" silently never fires) #308 never-binds test now uses 'frobnicate' — not special-cased by normalizeFunctionBreakpointName (only 'main' is) and matching nothing in the binary's symbol table, so it genuinely never binds and the three-stage feedback loop (set-time hint → launch warning → post-exit explanation) keeps its coverage. Stage-1 assertion updated to 'main.frobnicate'. The name choice matters more than "absent from hello_world.go": Delve resolves a bare name against function base names across the whole binary, runtime included — a first attempt with 'compute' bound (verified: true) to runtime.metricReader.compute and never fired, which is exactly the ambiguity that historically made bare 'main' fail differently (ambiguous vs runtime.main). The test comment documents this trap.
• Stage 3 made robust: the "Never bound during this run" stamp only applies when the adapter left bp.message empty; the assertion now also accepts a Delve-supplied diagnostic, and the get_output warn-entry check is gated on the server stamp being the message that won.
• New companion test pinning the fix(#467): surface unbound breakpoints at exit; auto-qualify go's bare 'main' #484 rewrite: function: 'main' → warning matches /Auto-qualified/, requestedName: 'main', functionName: 'main.main'; the rewritten breakpoint binds, fires (lastStop.reason === 'function breakpoint'), and the program completes after continue.
• Cleanup ordering fixed: both tests now close the debug session before unlinking the compiled binary — previously the unlink ran first and silently EBUSY'd on Windows while the debuggee held the file (that's where the stray untracked hello_world_bare_test came from).
• .gitignore: added hello_world_bare_test / hello_world_fnbp_test / hello_world_autoqualify_test to the compiled-Go-binaries block.

Testing

✓ tests/e2e/mcp-server-smoke-go.test.ts (6 tests) — incl. the updated #308 test and the new #484 companion

git status clean after the run — no stray binaries.

🤖 Generated with Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!