Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(auto-edit): add basic webview debug-panel #7304

Merged
merged 6 commits into from
Mar 5, 2025

Conversation

valerybugakov
Copy link
Member

@valerybugakov valerybugakov commented Mar 4, 2025

  • Integrates the basic auto-edit debug panel. This includes all the infrastructure:
    • Build config updates.
    • Command to open the panel.
    • HTML and CSS files used to render the panel.
    • Integration with the analytics logger used to pipe auto-edit request updates to the webview.
  • Actual content will be added to a follow-up PR.
  • Built on top of chore(auto-edit): integrate model response metadata with analytics logger #7303

Test plan

  • CI
  • Build VS Code extension locally
  • Ensure that auto-edit is enabled
  • Trigger auto-edit several times
  • Open the command palette and search for "Debug Auto-Edit"
  • Execute the command and ensure that a webview appears showing a number of the executed auto-edit requests
Screenshot 2025-03-04 at 12 19 10

Base automatically changed from vb/analytics-logger-plus-adapters-metadata to main March 5, 2025 02:57
@valerybugakov valerybugakov merged commit 5ee987e into main Mar 5, 2025
23 of 25 checks passed
@valerybugakov valerybugakov deleted the vb/basic-auto-edit-debug-panel branch March 5, 2025 03:03
valerybugakov added a commit that referenced this pull request Mar 5, 2025
- Implements UI for the auto-edit debug panel based on the data we
already have in store. I'll share a demo video in Slack later today.
- Built on top of #7304
- There are several minor issues with the UI. E.g., context summary
shows the total number of items as 0 while it's definitely not. The
side-by-side diff view highlighting is inaccurate for some inline
changes. We address such problems in follow-ups or keep them as is
because this UI is for internal use, so we should not spend much extra
time on it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants