Skip to content

fix: clamp out-of-range byte offsets instead of failing the request#1404

Draft
pfitzseb wants to merge 1 commit into
mainfrom
sp/clamp-offset
Draft

fix: clamp out-of-range byte offsets instead of failing the request#1404
pfitzseb wants to merge 1 commit into
mainfrom
sp/clamp-offset

Conversation

@pfitzseb

Copy link
Copy Markdown
Member

A stale or out-of-bounds offset (e.g. a diagnostic or test-item range that lags a content edit) no longer throws and fails the whole request. It is clamped to the document bounds and a warning is emitted so the underlying inconsistency stays visible.

Adds a test covering the clamp/warn behavior and an AGENTS.md with JuliaWorkspaces development guidance.

A stale or out-of-bounds offset (e.g. a diagnostic or test-item range that
lags a content edit) no longer throws and fails the whole request. It is
clamped to the document bounds and a warning is emitted so the underlying
inconsistency stays visible.

Adds a test covering the clamp/warn behavior and an AGENTS.md with
JuliaWorkspaces development guidance.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant