Skip to content

Comments

Heavily simplify the VS Code backend code#165

Merged
jviotti merged 1 commit intomainfrom
simpler-backend
Feb 13, 2026
Merged

Heavily simplify the VS Code backend code#165
jviotti merged 1 commit intomainfrom
simpler-backend

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Feb 13, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@augmentcode
Copy link

augmentcode bot commented Feb 13, 2026

🤖 Augment PR Summary

Summary: This PR refactors the VS Code extension backend to reduce indirection by consolidating most logic into vscode/src/extension.ts.

Changes:

  • Removes the dedicated CommandExecutor, DiagnosticManager, PanelManager, and fileUtils modules.
  • Inlines CLI invocation, lint/metaschema parsing, and diagnostic creation helpers directly in the extension entrypoint.
  • Simplifies webview panel lifecycle management by working directly with vscode.WebviewPanel.
  • Keeps panel state construction centralized via a buildPanelState helper and continues caching the CLI version.
  • Updates the VS Code debug launch config to point extension tests at ../test/vscode.

Technical Notes: Lint, format-check, and metaschema commands are still executed in parallel and diagnostics are derived from parsed CLI position ranges.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 7 files

@jviotti
Copy link
Member Author

jviotti commented Feb 13, 2026

By being a little clever, the entire thing goes down to like 500 lines of code, which is great

@jviotti jviotti merged commit 2ee927c into main Feb 13, 2026
5 checks passed
@jviotti jviotti deleted the simpler-backend branch February 13, 2026 16:43
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.

1 participant