[PR #8710] Migrate lines range syntax from # to : #12836

Open
opened 2026-02-16 18:17:43 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/anomalyco/opencode/pull/8710

State: open
Merged: No


What does this PR do?

Moves lines range syntax from # to :.
Addresses https://github.com/anomalyco/opencode/issues/2129

How did you verify your code works?

Unfortunately only manually, proof: https://opncd.ai/share/vCAqcqDv

  1. I don't know why, but I couldn't reproduce if from the first attempt, sometimes in the thread it doesn't show the read tool call, so it may create an observation like it reads the entire file, e.g. https://opncd.ai/share/g5iytPsd
  2. Pls advise me how to tweak the unit tests on this case if the change itself is good

Note
It may require updating docs adding clearly the line numbers start from 0, not 1 as usually we see in the text editors.

**Original Pull Request:** https://github.com/anomalyco/opencode/pull/8710 **State:** open **Merged:** No --- ### What does this PR do? Moves lines range syntax from # to :. Addresses https://github.com/anomalyco/opencode/issues/2129 ### How did you verify your code works? Unfortunately only manually, proof: https://opncd.ai/share/vCAqcqDv 1. I don't know why, but I couldn't reproduce if from the first attempt, sometimes in the thread it doesn't show the read tool call, so it may create an observation like it reads the entire file, e.g. https://opncd.ai/share/g5iytPsd 2. Pls advise me how to tweak the unit tests on this case if the change itself is good **Note** It may require updating docs adding clearly the line numbers start from 0, not 1 as usually we see in the text editors.
yindo added the pull-request label 2026-02-16 18:17:43 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#12836