feat: VS Code extension for in-editor plan review#229
Conversation
…ode) Opens Plannotator plan reviews, code reviews, and annotations inside VS Code tabs instead of an external browser. Intercepts PLANNOTATOR_BROWSER via env var injection in integrated terminals, with cookie persistence and auto-close. Original repository: https://github.com/7tg/plannotator-vscode Closes #91 Co-Authored-By: Barbaros Gören <tayyipgoren@gmail.com>
- Update package.json: publisher → backnotprop, repo → plannotator, add private - Replace node -e URL encoding with curl --data-urlencode in router script - Simplify panel-manager tests to keep only behavioral tests - Add dev:vscode, build:vscode, package:vscode scripts to root - Add *.vsix to .gitignore - Update bun.lock with new workspace member Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Thank you |
Code reviewFound 3 issues:
plannotator/apps/vscode-extension/src/extension.ts Lines 61 to 65 in b70f709 The server-side code that will fail: plannotator/packages/server/browser.ts Lines 44 to 52 in b70f709
Lines 5 to 59 in b70f709
plannotator/apps/vscode-extension/package.json Lines 30 to 35 in b70f709 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
|
I'm addressing the issues. |
- Fix macOS browser: detect script paths in PLANNOTATOR_BROWSER and invoke directly instead of via 'open -a' (which only accepts app names/.app bundles) - Update CLAUDE.md: add vscode-extension to project structure, development, and build sections - Fix command title: 'Simple Browser' -> 'Open URL in Editor' - Add 1s debounce to panel opens: Claude Code fires ExitPlanMode hook twice, spawning two plannotator processes on different ports that both hit the IPC server simultaneously - Add .vscode/launch.json + tasks.json for F5 debugging
Code review fixes + duplicate panel issuePushed commit 626f356 addressing the 3 code review issues plus a new bug found during testing. Code review fixes:
Duplicate panel bug: This only happens via Claude Code — the test script ( Also added |
Code reviewNo issues found. Checked for bugs, CLAUDE.md compliance, git history context, previous PR comments, and code comment consistency. The previous review's 3 issues (macOS 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
Summary
Integrates the community VS Code extension from 7tg/plannotator-vscode (now archived) into the monorepo at
apps/vscode-extension/. When Claude Code runs in VS Code's integrated terminal, Plannotator opens inside a VS Code tab instead of an external browser — no context switching.plannotatorWebview.injectBrowsersettingCommit 1 is authored by @7tg (Barbaros Gören) to preserve contributor attribution.
Commit 2 adapts for the monorepo: updates publisher to
backnotprop, replacesnode -edependency in router script withcurl --data-urlencode, simplifies panel-manager tests, adds root build scripts.Closes #91
Test plan
bun installsucceeds with new workspace memberbun run build:vscodeproducesdist/extension.cjs(11.5kb)bun test --cwd apps/vscode-extension— 21 tests passbun run package:vscodeproduces.vsix(16KB).vsixin VS Code, run Claude Code in integrated terminal, trigger plan review — opens in VS Code tab🤖 Generated with Claude Code