feat(read): surface lines-read/total in Read transcript label#26
Closed
justin-carper wants to merge 1 commit into
Closed
feat(read): surface lines-read/total in Read transcript label#26justin-carper wants to merge 1 commit into
justin-carper wants to merge 1 commit into
Conversation
Cursor ReadArgs = { path } only — no offset/limit (cursor-sdk-shared
ReadArgsSchema is a strip object). Two same-path reads are re-reads of
identical content, never different sections.
Derive per-call detail from result instead: count content lines and
compare to totalLines. Title becomes e.g. "handlers.go (2000/5000 lines)"
so a reader can spot redundant re-reads and see truncation at a glance.
Also stores linesReturned + fileSize in block metadata, and documents
the no-range constraint in a comment above the adapter so the question
does not recur.
Merged
justin-carper
added a commit
that referenced
this pull request
Jun 17, 2026
* feat(read): surface lines-read/total in Read transcript label
Cursor ReadArgs = { path } only — no offset/limit (cursor-sdk-shared
ReadArgsSchema is a strip object). Two same-path reads are re-reads of
identical content, never different sections.
Derive per-call detail from result instead: count content lines and
compare to totalLines. Title becomes e.g. "handlers.go (2000/5000 lines)"
so a reader can spot redundant re-reads and see truncation at a glance.
Also stores linesReturned + fileSize in block metadata, and documents
the no-range constraint in a comment above the adapter so the question
does not recur.
* chore(deps-dev): bump the dev-dependencies group with 2 updates
Bumps the dev-dependencies group with 2 updates: [@opencode-ai/sdk](https://github.com/sst/opencode-sdk-js) and [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest).
Updates `@opencode-ai/sdk` from 1.17.3 to 1.17.7
- [Release notes](https://github.com/sst/opencode-sdk-js/releases)
- [Changelog](https://github.com/anomalyco/opencode-sdk-js/blob/main/CHANGELOG.md)
- [Commits](https://github.com/sst/opencode-sdk-js/commits)
Updates `vitest` from 4.1.8 to 4.1.9
- [Release notes](https://github.com/vitest-dev/vitest/releases)
- [Changelog](https://github.com/vitest-dev/vitest/blob/main/docs/releases.md)
- [Commits](https://github.com/vitest-dev/vitest/commits/v4.1.9/packages/vitest)
---
updated-dependencies:
- dependency-name: "@opencode-ai/sdk"
dependency-version: 1.17.7
dependency-type: direct:development
update-type: version-update:semver-patch
dependency-group: dev-dependencies
- dependency-name: vitest
dependency-version: 4.1.9
dependency-type: direct:development
update-type: version-update:semver-patch
dependency-group: dev-dependencies
...
Signed-off-by: dependabot[bot] <support@github.com>
* chore(deps): bump @cursor/sdk from 1.0.18 to 1.0.19
Bumps [@cursor/sdk](https://github.com/cursor/cursor) from 1.0.18 to 1.0.19.
- [Commits](https://github.com/cursor/cursor/commits)
---
updated-dependencies:
- dependency-name: "@cursor/sdk"
dependency-version: 1.0.19
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
* chore(deps): bump @opencode-ai/plugin from 1.17.3 to 1.17.7
Bumps @opencode-ai/plugin from 1.17.3 to 1.17.7.
---
updated-dependencies:
- dependency-name: "@opencode-ai/plugin"
dependency-version: 1.17.7
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
* chore(release): 0.4.1-next.0
Consolidate dependency updates and Read transcript feature for @next prerelease:
- feat(read): surface lines-read/total in Read transcript label (#26)
- chore(deps-dev): bump dev-dependencies group, 2 updates (#27)
- chore(deps): bump @cursor/sdk 1.0.18 -> 1.0.19 (#28)
- chore(deps): bump @opencode-ai/plugin 1.17.3 -> 1.17.7 (#29)
* refactor: remove obsolete sqlite3 native-binding self-heal
@cursor/sdk@1.0.19 (#28) drops the sqlite3 native addon in favor of Node's
built-in node:sqlite, so the self-heal that rebuilt sqlite3's binding under
Bun is no longer needed.
- Remove src/native-binding.ts and its tests
- Drop ensureSqliteBinding() from cursor-runtime loadCursorSdk and the
sidecar agent backend (direct SDK import / client delegation)
- Remove the now-obsolete sqlite troubleshooting note from README
Verified: clean npm ci has no sqlite3, @cursor/sdk@1.0.19 imports cleanly,
and the opencode integration smoke test lists Cursor models.
* chore(release): 0.4.1
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
readtool accepts only{ path }— nooffset/limit/line-range exists inReadArgsSchema. Two same-path reads in a transcript are re-reads of identical content, never different sections.contentlines vstotalLines. Title becomes e.g.handlers.go (2000/5000 lines).linesReturnedandfileSize.Test Plan
title: "/a.ts (2/2 lines)", metadata includeslinesReturned,fileSizetitle: "/a.ts (1/10 lines)"whencontentis shorter thantotalLines