Ignore 😓 #2202
Ignore 😓 #2202
Conversation
API REQUEST BADGE input box auto approve setttings list
This reverts commit 53e08ab.
v1.8.0: Added Pear UI and functionality
* add-to-context-keybinding * add PearAI to context menu actions
Added auto-approve
|
dosubot
bot
added
the
size:XXL
|
This pull request is quite large, with 62 changed files, 2184 lines added, and 725 lines removed. It includes a variety of changes such as documentation updates, API integration, UI component updates, and configuration changes. To improve manageability and reviewability, I suggest splitting the pull request into smaller, more focused ones. Here are some suggestions for splitting:
This approach will help streamline the review process and ensure that each set of changes is thoroughly examined. Let me know if you have any questions or need further clarification! |
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
| for (const colorVar of VSC_THEME_COLOR_VARS) { | ||
| // Remove alpha channel from colors | ||
| const value = getComputedStyle(document.documentElement).getPropertyValue(colorVar) | ||
| if (colorVar.startsWith("#") && value.length > 7) { |
There was a problem hiding this comment.
In the loop over VSC_THEME_COLOR_VARS, the condition checks if the CSS variable name starts with # which will always be false. Consider checking the computed value (trimmed) instead.
| if (colorVar.startsWith("#") && value.length > 7) { | |
| if (value.trim().startsWith("#") && value.length > 7) { |
|
|
||
| // Validate the config using McpSettingsSchema | ||
| const result = McpSettingsSchema.safeParse(config) | ||
| console.log("IM HERE 10101", result) |
There was a problem hiding this comment.
The console.log message 'IM HERE 10101' might be leftover debugging code. Remove it to clean up the output.
| console.log("IM HERE 10101", result) |
| cacheWritesPrice: 3.75, | ||
| cacheReadsPrice: 0.3, | ||
| description: | ||
| "PearAI Model automatically routes you to the most best / most suitable model on the market. Recommended for most users.", |
There was a problem hiding this comment.
The description for PearAI within the pearai-model entry also uses the phrase 'most best / most suitable'. This should be revised to correct the redundancy, e.g. 'the best / most suitable' or a similar alternative.
| "PearAI Model automatically routes you to the most best / most suitable model on the market. Recommended for most users.", | |
| "PearAI Model automatically routes you to the best / most suitable model on the market. Recommended for most users.", |
| export const vscBadgeForeground = `var(${VSC_BADGE_FOREGROUND_VAR}, #fff)` | ||
| export const vscSidebarBorder = `var(${VSC_SIDEBAR_BORDER_VAR}, transparent)` | ||
|
|
||
| // cececd |
There was a problem hiding this comment.
Typographical Issue: The comment on line 63 reads // cececd, which appears to be a stray or unintended typo. Please clarify or remove it.
| // cececd |
| "didRead": "Roo read this file:", | ||
| "wantsToEdit": "Roo wants to edit this file:", | ||
| "wantsToCreate": "Roo wants to create a new file:" | ||
| "wantsToRead": "PearAI Agent wantsto read this file:", |
There was a problem hiding this comment.
Typographical error: "PearAI Agent wantsto read this file:" should be "PearAI Agent wants to read this file:".
| "wantsToRead": "PearAI Agent wantsto read this file:", | |
| "wantsToRead": "PearAI Agent wants to read this file:", |
2 participants
Context
Implementation
Screenshots
How to Test
Get in Touch
Important
Integrate PearAI into the application with new API handlers, UI updates, and configuration changes.
PearAiHandlerinpearai.tsto handle PearAI API interactions.buildApiHandlerinindex.tsto include PearAI.usePearAiModelshook to fetch PearAI models.ChatTextAreaandChatViewto include PearAI branding and functionality.TailandTail2intail.tsx.ExtensionStateContextto handle PearAI API configuration.package.jsonto reflect PearAI branding.api.tsandcheckExistApiConfig.ts.This description was created by
for b40e406. It will automatically update as commits are pushed.