Skip to content

Conversation

@chadgauth
Copy link
Contributor

@chadgauth chadgauth commented Feb 24, 2025

Important

Enhance chat interface with redesigned components and integrate Storybook for UI testing, including new stories and configurations.

  • Chat Interface Enhancements:
    • Redesigned chat window and updated start screen in ChatView.tsx.
    • Added ChatTextArea components with new layout, input, actions, and selections in ChatTextArea.tsx and related files.
    • Updated Announcement.tsx for new version announcement.
  • Storybook Integration:
    • Added Storybook configuration in .storybook/main.ts and .storybook/preview.tsx.
    • Created Storybook stories for components like Announcement, AutoApproveMenu, BrowserSessionRow, ChatRow, ChatTextArea, ChatView, ContextMenu, ReasoningBlock, TaskHeader, CodeAccordian, CodeBlock, and MarkdownBlock.
  • Testing and Mocking:
    • Added mock context and state in preview-context.tsx for Storybook.
    • Updated ChatTextArea.test.tsx to test new functionalities.
  • Miscellaneous:
    • Updated package.json to include new dependencies for Storybook and testing.
    • Added CSS styles in ChatTextArea.module.css and preview.css for consistent theming.

This description was created by Ellipsis for adbf283. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Feb 24, 2025

🦋 Changeset detected

Latest commit: adbf283

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 Minor

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

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Feb 24, 2025
@chadgauth chadgauth closed this Feb 24, 2025
highlightLayerRef.current.innerHTML = text
.replace(/\n$/, "\n\n")
.replace(/[<>&]/g, (c) => ({ "<": "&lt;", ">": "&gt;", "&": "&amp;" })[c] || c)
.replace(/[<>&]/g, (c) => ({ "<": "<", ">": ">", "&": "&amp;" })[c] || c)
Copy link
Contributor

Choose a reason for hiding this comment

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

HTML escaping is not performed correctly. The mapping returns '<' and '>' unescaped. Replace '<' with '<' and '>' with '>' to prevent potential XSS vulnerabilities.

Suggested change
.replace(/[<>&]/g, (c) => ({ "<": "<", ">": ">", "&": "&amp;" })[c] || c)
.replace(/[<>&]/g, (c) => ({ "<": "&lt;", ">": "&gt;", "&": "&amp;" })[c] || c)

}

/* VS Code scrollbar styles */
::-webkit-scrollbar {
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: The scrollbar pseudo-element selectors on lines 94, 99, 103, and 108 are incorrectly prefixed with three colons (e.g., ':::-webkit-scrollbar'). They should be written with two colons (e.g., '::-webkit-scrollbar').

Suggested change
::-webkit-scrollbar {
::-webkit-scrollbar {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant