Skip to content

Support independent visualization lifecycles - #83

Merged
DecisionNerd merged 1 commit into
mainfrom
feature/81-visualization-instance-lifecycle
Aug 18, 2026
Merged

Support independent visualization lifecycles#83
DecisionNerd merged 1 commit into
mainfrom
feature/81-visualization-instance-lifecycle

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Closes #81

Summary

  • replace graph, artifact, and figure singleton panels with a shared instance registry
  • correlate host/webview work by privacy-safe instance ID and monotonic render generation
  • cancel superseded work, reject stale replies, dispose owned resources, and route coordination only through explicit groups
  • verify multiple unsaved visualization panels remain alive independently

Validation

  • npm run check
  • npm run test:unit (247 passing)
  • npm test (326 passing)
  • npm run verify:package
  • git diff --check

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your current included review allowance is based on your included PR review attempts over the past 7 days.

Next review available in: 23 minutes

Limit details: You’ve used all 2 included reviews currently available. Your 53 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3940ecd1-ce92-4054-816d-f96769770f43

📥 Commits

Reviewing files that changed from the base of the PR and between ba03ad4 and 0adda71.

📒 Files selected for processing (19)
  • docs/engineering/ARCHITECTURE.md
  • docs/engineering/TESTING.md
  • package.json
  • src/commands/projectArtifacts.ts
  • src/commands/visualize.ts
  • src/extension.ts
  • src/test/extension.test.ts
  • src/test/visualizationInstanceRegistry.test.ts
  • src/views/resultTableView.ts
  • src/webview/artifactVisualizationPanel.ts
  • src/webview/artifactVisualizationProtocol.ts
  • src/webview/figurePanel.ts
  • src/webview/figureSchema.ts
  • src/webview/protocol.ts
  • src/webview/resultGraphPanel.ts
  • src/webview/visualizationInstanceRegistry.ts
  • webview-ui/src/artifactVisualization/main.ts
  • webview-ui/src/figure/main.ts
  • webview-ui/src/resultGraph/main.ts

Comment @coderabbitai help to get the list of available commands.

@codspeed-hq

codspeed-hq Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 27 untouched benchmarks


Comparing feature/81-visualization-instance-lifecycle (0adda71) with main (ba03ad4)

Open in CodSpeed

@DecisionNerd
DecisionNerd merged commit 0d34718 into main Aug 18, 2026
8 checks passed
@DecisionNerd
DecisionNerd deleted the feature/81-visualization-instance-lifecycle branch August 18, 2026 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support multiple independent XYG visualizations with one lifecycle contract

1 participant