Rebased main

[proggod]

Context

Implementation

Screenshots

before	after

How to Test

Get in Touch

---

Important

This PR adds a unified diff viewer, an API logger, and enhanced system prompt handling to improve code review and debugging capabilities.

• Unified Diff Viewer:
  • Added DiffApproveProvider in src/integrations/diff-approve/DiffApproveProvider.ts for diff viewing and approval.
  • Updated ChatTextArea.tsx and CheckpointMenu.tsx to integrate diff viewing.
• API Logger:
  • Introduced logApiRequest() in src/utils/api-logger.ts to log API requests.
  • Added API logging configuration in settings.json.
• System Prompt Enhancements:
  • Added SystemPromptTextArea in ChatView.tsx for dynamic prompt appending.
  • Updated Cline.ts to handle system prompt changes.
• Miscellaneous:
  • Updated experiments.ts and related test files to include new experiment IDs.
  • Modified webview-ui/package.json to include new dependencies.

^{This description was created by for 7c930a4. It will automatically update as commits are pushed.}

[changeset-bot]

🦋 Changeset detected

Latest commit: 7c930a4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
roo-cline	Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[ellipsis-dev]

This pull request is quite large, with 60 changed files, 13,906 lines added, and 9,171 lines removed. It includes a variety of changes such as new features, documentation updates, and refactoring across different files and components. To improve the review process and ensure each change gets the attention it deserves, I suggest splitting this pull request into smaller, more focused ones.

Here are some suggestions on how to split the changes:

1. Documentation Updates: Create a separate pull request for all documentation-related changes, such as those in DiffViewerButtonImplementation.md, ShadowCheckpoint.md, add_config_option.md, cline-merge-steps.md, completed-steps.md, merge-plan.md, pull_request.md, and tool-update-checklist.md.

2. New Features and Enhancements: Group all new features and enhancements into another pull request. This includes changes related to the diff viewer, API logging, and system prompt functionalities.

3. Refactoring and Fixes: Create a pull request for refactoring and fixes, such as those in jest.config.js, src/api/providers/__tests__/vscode-lm.test.ts, and other similar files.

By splitting the pull request, it will be easier to review and test each set of changes independently, ensuring a smoother integration process.

[ellipsis-dev]

Typo detected: 'Cline' in the sentence 'Replaces the previous savePointHash variable in the Cline class' may be a typographical error. Please verify if this should be 'Client' or another intended name.

Suggested change

	- Replaces the previous `savePointHash` variable in the `Cline` class
	- Replaces the previous `savePointHash` variable in the `Client` class

[ellipsis-dev]

There is an inconsistency in the naming of the API logging experiment. In one example, the experiment ID is "apiLogging", but in another, it is "api_logging". Please update the document to use a consistent naming convention, preferably 'apiLogging' as suggested in the Best Practices.