[remorses]

LSP diagnostics returned after file writes and edits were unbounded, which could easily fill up the context window when a project has many errors.

This adds limits:

• Max 10 diagnostics per file (sorted by severity - errors first)
• Max 5 files for project-wide diagnostics (in write tool)
• Shows "... and N more" suffix when truncated

Fixes #5259

I encountered this issue while working on a big codebase with many broken scripts. All the LSP issues were filling in the context. I even made a cli to investigate why just 1 file write was consuming so much context. The issue is that LSP diagnostics are unbounded. As you can see in the image a single file write was consuming 400k characters of context

[Copilot]

Pull request overview

This PR adds limits to LSP diagnostics returned after file writes and edits to prevent context window overflow when projects have many errors. The implementation sorts diagnostics by severity (errors first) and truncates the output with a clear indication of how many additional diagnostics exist.

• Adds configurable constants for limiting diagnostics per file (10) and project-wide files (5 in write tool)
• Implements sorting by severity with errors prioritized
• Adds "... and N more" suffix when diagnostics are truncated

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
packages/opencode/src/tool/write.ts	Adds MAX_DIAGNOSTICS_PER_FILE and MAX_PROJECT_DIAGNOSTICS_FILES constants; implements sorting and limiting for both file-specific and project-wide diagnostics with truncation messages
packages/opencode/src/tool/edit.ts	Adds MAX_DIAGNOSTICS_PER_FILE constant; replaces severity filtering (errors-only) with sorting and limiting that includes all diagnostic severities

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[rekram1-node]

/review

[github-actions]

lgtm