Skip to content

Remove simple view functionality#6

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-3
Closed

Remove simple view functionality#6
Copilot wants to merge 2 commits intomainfrom
copilot/fix-3

Conversation

Copy link
Contributor

Copilot AI commented May 23, 2025

This PR removes the simple view functionality from the extension, which was only intended for debugging purposes.

The changes include:

  • Removed the doc-detective.simpleView command registration from the extension
  • Removed the simple view command from the package.json contributes section
  • Removed the simple view command from the activationEvents in package.json
  • Removed the menu entry for the simple view from the view/title section
  • Removed the getSimpleView() method from the DocDetectiveWebviewViewProvider class
  • Removed the getSimplifiedHtmlForWebview() method from the DocDetectiveWebviewViewProvider class
  • Updated the fallback mechanism in the updateWebview() method to use the error view instead of the simple view
  • Updated the test file to remove the assertion for the simpleView command
  • Updated JSDoc comments to remove references to the simple view

The extension now uses only the standard view without any simplified view fallback, which helps streamline the codebase and remove debugging functionality that wasn't intended for production use.

Fixes #3.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Co-authored-by: hawkeyexl <5209367+hawkeyexl@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove the simple view Remove simple view functionality May 23, 2025
Copilot AI requested a review from hawkeyexl May 23, 2025 01:53
@hawkeyexl hawkeyexl changed the base branch from main to rc May 23, 2025 02:01
@hawkeyexl hawkeyexl changed the base branch from rc to main May 23, 2025 02:11
@hawkeyexl hawkeyexl closed this May 23, 2025
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.

3 participants