Feat/mention large file

[samhvw8]

Context

Follow up with @1440 , this for @-mention but for read large file

Implementation

Screenshots

before	after

How to Test

Get in Touch

---

Important

Adds maxReadFileLine parameter to handle large file mentions by truncating content beyond a specified line count.

• Behavior:
  • Introduces maxReadFileLine parameter to limit lines read from files in Cline.ts, mentions/index.ts, and extract-text.ts.
  • Truncates file content if it exceeds maxReadFileLine, appending a notice about truncation.
• Functions:
  • Updates loadContext() in Cline.ts to pass maxReadFileLine to parseMentions().
  • Modifies parseMentions() in mentions/index.ts to handle maxReadFileLine.
  • Adjusts extractTextFromFile() in extract-text.ts to support line truncation.
• Misc:
  • Adds error handling for file access and content extraction in getFileOrFolderContent() in mentions/index.ts.

^{This description was created by for 3b7a3ca59c141d6efd76b3c3d9ea7e5dd4e7c12a. It will automatically update as commits are pushed.}

[changeset-bot]

⚠️ No Changeset found

Latest commit: 3588c53

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[mrubens]

This code seems kind of duplicative with what's in Cline.ts now. Should they all just use extractFromFile with the new parameter?

[samhvw8]

This code seems kind of duplicative with what's in Cline.ts now. Should they all just use extractFromFile with the new parameter?

let me try to refactor to make Cline and mention use the same function

[samhvw8]

@mrubens i make function to reuse cline & @ mention read file

[samhvw8]

@mrubens just update this pr, if you have time can you check it

[mrubens]

There's a lot changing in this PR. Could you try to break it down into smaller independent PRs? In particular I always like to do a refactor separate from any functionality changes, so you can keep the test cases the same.

[samhvw8]

There's a lot changing in this PR. Could you try to break it down into smaller independent PRs? In particular I always like to do a refactor separate from any functionality changes, so you can keep the test cases the same.

Yes, let me do it

[samhvw8]

@mrubens i just update pr, now it have smaller change

[daniel-lxs]

Hey @samhvw8,
Thank you for your contribution, We noticed this PR is stale and will be closed. If you plan to revisit this, please create an issue first as required by our issue-first approach before opening a new PR.