Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Conversation

aponcedeleonch
Copy link
Member

@aponcedeleonch aponcedeleonch commented Feb 5, 2025

Closes: #884

We used to have a generic CodeSnippetExtractor to detect code snippets and the corresponding filepaths. This PR extends it's functionality to be able to detect snippets coming from different clients. After this PR the supported and tested clients are:

  • Continue
  • Cline
  • Aider
  • Open Interpreter

The main reason behind this change is to be able to catch the filepaths coming through CodeGate, detect it's destination and mux according to them. The PR also adds the necessary in the router used for muxing to detect the filepaths from the request messages.

Since CodeSnippetExtractor is no longer a pipeline step it was moved out of the folder pipeline to it's own folder

@aponcedeleonch aponcedeleonch self-assigned this Feb 5, 2025
@aponcedeleonch aponcedeleonch force-pushed the extract-filepath-snippets branch 6 times, most recently from 57e088f to c344e3a Compare February 6, 2025 08:03
Copy link
Contributor

@jhrozek jhrozek left a comment

Choose a reason for hiding this comment

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

thank you, nice work, some comments and questions inline

We used to have a generic CodeSnippetExtractor to detect code snippets
and the corresponding filepaths. This PR extends it's functionality
to be able to detect snippets coming from different clients.
After this PR the supported and tested clients are:
- Continue
- Cline
- Aider
- Open Interpreter

The main reason behind this change is to be able to catch the filepaths
coming through CodeGate, detect it's destination and mux according
to them. The PR also adds the necessary in the router used for muxing
to detect the filepaths from the request messages.

Since CodeSnippetExtractor is no longer a pipeline step it was moved
out of the folder `pipeline` to it's own folder
@aponcedeleonch aponcedeleonch force-pushed the extract-filepath-snippets branch from c344e3a to ffea4b4 Compare February 6, 2025 10:09
@aponcedeleonch aponcedeleonch merged commit c93e106 into main Feb 6, 2025
9 checks passed
@aponcedeleonch aponcedeleonch deleted the extract-filepath-snippets branch February 6, 2025 12:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task]: Catch filenames from prompts
3 participants