Skip to content

Conversation

Jazzcort
Copy link
Contributor

@Jazzcort Jazzcort commented Jul 2, 2025

Description

Due to the new retrieval mechanism, getting context items for codebase and docx is not working. This patch resolve the issue.

#6411

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screenshots

ScreenRecording2025-07-02at2.50.18PM-ezgif.com-video-speed.mov

Tests

Mention Docs and Codebase and you should get some context items back instead of nothing.


Summary by cubic

Fixed an issue where context items for codebase and docs were not being retrieved, so users now get relevant context as expected.

@Jazzcort Jazzcort requested a review from a team as a code owner July 2, 2025 18:54
@Jazzcort Jazzcort requested review from sestinj and removed request for a team July 2, 2025 18:54
Copy link

netlify bot commented Jul 2, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit ebeb661
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/68672b193c4eb60008bb416e

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 2, 2025
Copy link

github-actions bot commented Jul 2, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Due to the new retrieval mechanism, getting context items for codebase and
docx is not working. This patch resolve the issue.

continuedev#6411
@Jazzcort Jazzcort force-pushed the fix-context-retrieval-issue branch from 6442d4b to 377e515 Compare July 2, 2025 19:17
RomneyDa
RomneyDa previously approved these changes Jul 2, 2025
Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

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

Ensures unique id per context item

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jul 2, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 2, 2025
@RomneyDa
Copy link
Collaborator

RomneyDa commented Jul 3, 2025

Note, I broke this with #6264
I assumed itemIds were unique
This fixes by making them unique
#6429 fixes by removing item id assumption
@sestinj could do one or both

@sestinj sestinj merged commit 31d2c91 into continuedev:main Jul 4, 2025
38 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jul 4, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 4, 2025
@sestinj
Copy link
Contributor

sestinj commented Jul 22, 2025

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer released size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants