Fix method graph panel stability and column behavior#10
Open
Fix method graph panel stability and column behavior#10
Conversation
Replace the repo root README (which includes VS Code banner and full Taplo README) with a CLI-focused standalone README for the PyPI page. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Code blocks render as plain TOML on GitHub and have no syntax coloring on PyPI. Use a hosted screenshot instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Backfill CHANGELOG.md with all changes since v0.2.1 (0.3.0–0.3.2 + unreleased), annotate CLI-specific entries with plxt versions, and broaden the title to cover both extension and CLI. Add Changelog URL to pyproject.toml [project.urls] for PyPI sidebar. Add .claude/skills/release/ project skill with detection script, version map reference, and workflow instructions for automated version bumping and changelog updates via /release. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The fix was already shipped in plxt 0.1.4 / ext v0.3.2 but was mislabeled under [Unreleased] with an incorrect future version annotation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Integrate pipelex-agent CLI to provide on-save validation diagnostics for MTHDS files with source-located errors, and add a "Show Method Graph" command that renders the ReactFlow graph in a side panel. Includes CLI resolution (venv, PATH, uv fallback), validation types, source locator, and unit tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use undefined instead of ViewColumn.Beside when revealing an existing panel so it stays in its current column. Add retainContextWhenHidden to preserve webview content when backgrounded. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When a file opens in the graph panel's editor group (e.g. user clicks explorer while the graph has focus), close it there and re-open it in the main editor column to keep the graph group dedicated. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9942fec934
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
… path Address 6 PR review issues: extract shared processUtils (spawnCli, cancelInflight helpers), cancel all inflight graph jobs on file switch with staleness check, guard against infinite loop when panel is in column 1, add missing warning message in MethodGraphPanel, detect Windows .venv/Scripts path, and gate validator/graph behind child_process availability for browser hosts. Add cliResolver unit tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
registerNodeFeatures was called fire-and-forget, so the showMethodGraph
command could be missing at activation time and import failures were
unhandled. Make registerPipelexFeatures async, await the helper, and
surface errors as warnings.
Fix filename extraction using split('/') which broke on Windows
backslash paths — replace with a regex that handles both separators.
Add staleness guard after fs.promises.readFile so a file switch during
the read doesn't overwrite the current graph with stale HTML.
Add three new test files (processUtils, methodGraphPanel,
pipelexExtension) covering these bugs plus regression guards for
previous fixes.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
reveal(undefined)instead ofreveal(ViewColumn.Beside)so the panel stays in its current column when updatingretainContextWhenHidden: trueto preserve webview content when backgroundedTest plan
.mthdsfile, trigger "Show Method Graph".mthdsfiles — the same panel should update in place without jumping🤖 Generated with Claude Code
Note
Medium Risk
Adds new VS Code extension features that execute an external
pipelex-agentprocess on save and in a webview, introducing async/cancellation logic and potential platform/environment edge cases (Node vs web host, CLI resolution). Documentation/release tooling changes are low risk but broaden the operational surface area.Overview
Adds Node-host-only Pipelex IDE extension features for
.mthds: on-save validation viapipelex-agentdiagnostics plus a newPipelex: Show Method Graphcommand that renders a generated graph in a persistent webview, with CLI auto-resolution, timeouts, cancellation, and staleness guards.Updates extension configuration (
pipelex.validation.*) and activation toawaitPipelex feature registration, and adds focused unit tests for the graph panel behavior, CLI resolution, JSON extraction, error location mapping, and inflight-process cancellation.Separately updates release/docs plumbing: introduces a
/releaseskill with a change-detection script, refreshesCHANGELOG.md, makesmake checkrun tests, switches PyPI packaging to a dedicated README, and updates README branding/example rendering.Written by Cursor Bugbot for commit ad57bef. This will update automatically on new commits. Configure here.