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

Deep Research #933

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Deep Research #933

wants to merge 3 commits into from

Conversation

cte
Copy link
Collaborator

@cte cte commented Feb 11, 2025

Description

Embedded @dzhng's Deep Research (https://github.com/dzhng/deep-research) into Roo Code: "An AI-powered research assistant that performs iterative, deep research on any topic by combining search engines, web scraping, and large language models."

Demo: https://screen.studio/share/VtRtAGOX

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Checklist:

  • My code follows the patterns of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

Additional context

Related Issues

Reviewers


Important

Integrates 'Deep Research' feature for AI-powered research using search engines and language models, with new components, services, and tests.

  • Feature:
    • Integrates DeepResearchService to perform AI-powered research using search engines, web scraping, and language models.
    • Adds DeepResearch component in DeepResearch.tsx to manage research sessions and tasks.
    • Introduces GetStarted, History, Models, Providers, ResearchSession, and ResearchTask components for UI interactions.
  • Backend:
    • Implements DeepResearchService in DeepResearchService.ts for handling research logic and interactions with OpenAI and Firecrawl.
    • Defines types and schemas in types.ts for research-related data structures.
  • UI:
    • Updates App.tsx to include a new tab for Deep Research.
    • Modifies ChatInput.tsx, ChatMessage.tsx, and ChatMessages.tsx to support new message types and annotations.
  • Testing:
    • Adds tests for TextSplitter in TextSplitter.test.ts.
    • Updates App.test.tsx to test new Deep Research tab functionality.
  • Configuration:
    • Updates package.json and .eslintrc.json for new dependencies and configurations.

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

Copy link

changeset-bot bot commented Feb 11, 2025

⚠️ No Changeset found

Latest commit: c384fe9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@samhvw8
Copy link

samhvw8 commented Feb 11, 2025

cool <3

@cte cte force-pushed the cte/deep-research branch 3 times, most recently from d28239d to e856612 Compare February 18, 2025 07:12
Comment on lines +716 to +720
await provider.postMessageToWebview({
type: "invoke",
invoke: "setChatBoxMessage",
text: `@${mentionPath}`,
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe not necessary, but could be nice to switch into architect mode along with this

@cte cte changed the title [WIP] Deep Research Deep Research Feb 20, 2025
@cte cte force-pushed the cte/deep-research branch from ad244fd to 95d9c65 Compare February 20, 2025 16:47
@cte cte marked this pull request as ready for review February 20, 2025 16:48
@cte cte force-pushed the cte/deep-research branch from c8f16e3 to 4c19fbd Compare February 20, 2025 17:05
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Feb 22, 2025
@cte cte force-pushed the cte/deep-research branch 2 times, most recently from 8503e7b to 3cbe3d1 Compare February 23, 2025 06:15
const shadowDir = path.join(tmpDir, `shadow-${Date.now()}`)
const workspaceDir = path.join(tmpDir, `workspace-${Date.now()}`)

let t0 = Date.now()
Copy link

Choose a reason for hiding this comment

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

Remove or disable console.log used for timing during CheckpointServiceFactory creation.

cache: 'npm'
- name: Install dependencies
run: npm run install:all
# - name: Setup upterm session
Copy link

Choose a reason for hiding this comment

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

If the 'Setup upterm session' step is no longer needed, please remove the commented-out code. If it's being temporarily disabled, add a comment explaining why.

@cte cte force-pushed the cte/deep-research branch from 87f2557 to 4feafaf Compare February 23, 2025 08:12
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.

3 participants