continue icon indicating copy to clipboard operation
continue copied to clipboard

fix: do not delete docs indexes on resync

Open shssoichiro opened this issue 5 months ago • 3 comments
trafficstars

Description

When Continue loads, it takes several seconds for it to fetch the remote assistant configs. Until then, it defaults back to the local assistant config. If the user has their docs set up in the remote assistant but not in the local assistant, then this code to delete all non-existent docs from the index will cause all doc indexes to be cleared out on every restart, which is the behavior I am seeing in #6139. This would also be an issue for users who have multiple assistants with different docs in each one.

Fixes #6139.

Checklist

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

Tests

Manually verified that the fix works. There were no automated tests covering this functionality, and it is only deleting code.

shssoichiro avatar Jun 16 '25 15:06 shssoichiro

Deploy Preview for continuedev canceled.

Name Link
Latest commit f0a7e2675d309968e836dfd0fcd3a67a216ab1ef
Latest deploy log https://app.netlify.com/projects/continuedev/deploys/68648d4db72d1200081ea98c

netlify[bot] avatar Jun 16 '25 15:06 netlify[bot]

✨ No issues found! Your code is sparkling clean! ✨

recurseml[bot] avatar Jun 16 '25 15:06 recurseml[bot]

As far as I can tell, the test failure doesn't appear to be related to this change.

shssoichiro avatar Jun 17 '25 04:06 shssoichiro

This issue causes severe CPU utilization every time my IDE is restarted, to the point that I've had to disable Continue. It would be great to have this reviewed please.

shssoichiro avatar Jun 30 '25 01:06 shssoichiro

:tada: This PR is included in version 1.1.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Jul 22 '25 05:07 sestinj