Skip to content

fix(tools): increase read_file token limit and handle missing model c… - #13403

Open
rishabtiwari-dev wants to merge 1 commit into
continuedev:mainfrom
rishabtiwari-dev:fix/read-file-token-limit
Open

rishabtiwari-dev wants to merge 1 commit into
continuedev:mainfrom
rishabtiwari-dev:fix/read-file-token-limit

Conversation

@rishabtiwari-dev

Copy link
Copy Markdown

…ontext length

Description

[ What changed? Feel free to be brief. ]

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

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

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ Closes #12432.

  • Added a modern baseline context length fallback (64k tokens) if model.contextLength is missing or understated.
  • Increased the permissive token limit ratio from 50% to 75% of the model's context length in throwIfFileExceedsHalfOfContext.
  • Enhanced the error message to provide actionable guidance on checking configuration or line ranges. ]

@rishabtiwari-dev
rishabtiwari-dev requested a review from a team as a code owner October 19, 2026 23:27
@rishabtiwari-dev
rishabtiwari-dev requested review from sestinj and removed request for a team October 19, 2026 23:27
@github-actions

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


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


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Size limit for Read_File

1 participant