continue icon indicating copy to clipboard operation
continue copied to clipboard

fix: Fix the issue where the chunk index might be duplicated

Open jacob-lcs opened this issue 5 months ago • 5 comments

Description

This pull request includes a small change to the chunkDocument function in core/indexing/chunk/chunk.ts. The change simplifies the incrementation of the index variable by directly incrementing it within the object assignment. Fix the issue where the chunk index might be duplicated.

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

Screenshots

[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]

jacob-lcs avatar Jun 09 '25 03:06 jacob-lcs

Your cubic subscription is currently inactive. Please reactivate your subscription to receive AI reviews and use cubic.

cubic-dev-ai[bot] avatar Jun 09 '25 03:06 cubic-dev-ai[bot]

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

github-actions[bot] avatar Jun 09 '25 03:06 github-actions[bot]

Deploy request for continuedev pending review.

Visit the deploys page to approve it

Name Link
Latest commit fa7bc17f36d9dbbe5ac7f9b1273f0d0e83fa1dc5

netlify[bot] avatar Jun 09 '25 03:06 netlify[bot]

I have read the CLA Document and I hereby sign the CLA

jacob-lcs avatar Jun 09 '25 03:06 jacob-lcs

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

recurseml[bot] avatar Jun 13 '25 20:06 recurseml[bot]

@sestinj I've updated the code, all CI checks have passed. Please take another look when you have time. If everything looks good, it can be merged. :)

jacob-lcs avatar Jun 21 '25 15:06 jacob-lcs

: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