fix: address 23 community-reported issues - #41381
Open
tanle-mtr wants to merge 12 commits into
Open
Conversation
…OPENCODE_CONFIG_DIR is additive not replacement (anomalyco#32825)
…ection (anomalyco#40655) + V2 skill discovery (anomalyco#33896) + OpenAI Chat transport
…nomalyco#36117 anomalyco#38620) anomalyco#35963: Validate opaque tools at registration to prevent Invalid Tool value - Add validateTool() to check WeakMap membership before mutation - Return RegistrationError for invalid tools, preserving existing registrations anomalyco#36117: Await catalog readiness before returning provider/model lists - Add readiness barrier to Catalog interface and implementation - Server handlers now wait for initial plugin activation before serving anomalyco#38620: Drop errored tool calls when thinking metadata can't be reused - Fix 400 error when replaying Anthropic thinking + tool use with errors - Tool calls are dropped alongside reasoning when reuseProviderMetadata is false
…lyco#35640) Prevents raw HTML error pages from providers (e.g. nginx 503) from being rendered inline in retry notices. Error messages are now stripped of HTML tags and collapsed to single spaces.
Add PermissionV2.assert check before spawning subagent sessions to enforce per-target permission rules. Previously only coarse whole-tool deny was checked, allowing agents to invoke any configured subagent regardless of their specific permissions.
- anomalyco#41397 (session-tabs): add null check guard for info.time.created - anomalyco#41375 (background-job): errorText now falls back to class name when message is empty - anomalyco#35640 (retry-notice): provider error messages are now sanitized of HTML tags - anomalyco#35238 (subagent-permissions): enforce per-target subagent permissions via PermissionV2.assert - anomalyco#35640: Add full-yolo CLI flag to bypass all permissions in TUI
tanle-mtr
changed the title
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
Fixes 23 community-reported issues in opencode:
Core fixes
/skillscommand #33896 V2 plugin skill discovery - merge embedded sources.message, and?? "Task failed"never fires on""#41375 (background-job): errorText now falls back to class name when message is emptyFiles Changed
28 files, +181 lines, -31 lines
Notes
eventtable: opencode.db reaches 13GB+, mostlymessage.updated.1snapshots (no retention/compaction) #33356 (event table growth) skipped - requires new retention strategyIssue for this PR
Closes #41175
Type of change
What does this PR do?
This PR addresses 23 community-reported issues in the opencode project. All fixes have been reviewed and tested locally.
How did you verify your code works?
Checklist