docs: add messageFilters config reference + examples (EN/ZH) - #381
Open
ranxianglei wants to merge 3 commits into
Open
docs: add messageFilters config reference + examples (EN/ZH)#381ranxianglei wants to merge 3 commits into
ranxianglei wants to merge 3 commits into
Conversation
Issue #369 showed the default-on OMO message filters are undiscoverable. Add the missing messageFilters documentation: - CONFIGURATION.md / CONFIGURATION.zh-CN.md: new messageFilters section (enabled, filters, builtin filter table, example) + a tuning/disabling recipe in Common Config Recipes - README.md / README.zh-CN.md: messageFilters entry in the default-config block Docs only — no code changes. Verified defaults against source (lib/config.ts, lib/messages/filter/builtin/*). typecheck/build/test pass (1112/1112).
📦 Built Plugin ArtifactBranch: Option A — Install from npm PR tag (recommended)opencode plugin opencode-acp@pr-381 --globalEach push to this PR publishes a new version under the Option B — Install from GitHubopencode plugin "github:ranxianglei/opencode-acp#2026-09-09_message-filters-docs" --globalOption C — Download artifact
tar xzf opencode-acp-pr381.tgz
cp -r package/dist ~/.cache/opencode/packages/opencode-acp@latest/node_modules/opencode-acp/dist
This comment is automatically updated on each push. |
This was referenced Sep 9, 2026
Closed
Owner
Author
|
解决冲突 |
…ters + storagePath recipes)
Owner
Author
冲突已解决，已推送到分支 冲突原因：master（v1.15.0 / v1.16.0 storagePath 功能）和本 PR 都在 CONFIGURATION.md / CONFIGURATION.zh-CN.md 的 "Common Config Recipes" 一节、紧跟 "Protect sensitive files" 之后追加了新 recipe——本 PR 加的是 OMO messageFilters recipe，master 加的是 解决方式：两边内容都保留（OMO 过滤器 recipe 在前，"Relocate session state storage" / "自定义会话状态存储位置" 在后），无内容丢弃。 合并后重新验证：
WORKLOG 已补充冲突解决记录。PR 现在 CI 跑完后即可由你合并（按规范合并操作只由人工执行）。 |
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.
Closes #369
Issue #369 asked how to safely remove oh-my-opencode's redundant injected prompts. The answer is that ACP already ships 5 built-in OMO message filters (enabled by default since v1.14.8) — but the
messageFiltersconfig surface was completely undocumented, so the feature was undiscoverable.Changes (docs only, no code)
### messageFilterssection:messageFilters.enabled(master switch, defaulttrue)messageFilters.filters(per-filterenabled+keepLast)keepLast, disable one filter, or turn the subsystem offmessageFiltersentry added to the Default Configuration block (EN + ZH comments)Verification
lib/config.ts(defaults),lib/messages/filter/builtin/*.ts(filter names/versions/keepLast),lib/messages/filter/apply.ts:115-116(keepLastresolution)npm run typecheck✅,npm run build✅, full test suite 1112/1112 pass ✅Notes
qualityGate.algorithms.rouge-recall-v1.layer1MinRetentionPctis documented as5.0in the CONFIGURATION.md table but1.0in the README default-config block. Worth a separate fix.devlog/2026-09-09_message-filters-docs/(REQ + WORKLOG)