🐛 fix(sidecar): 保留浏览器 RPC 具体错误码 - #265
Open
CavinHuang wants to merge 1 commit into
Open
Conversation
Desktop 已返回稳定浏览器错误码，Sidecar 反向 RPC 接收点此前将其覆盖为固定文本，导致 Broker 只能退化为 browser_internal_error。现在使用带 code 的错误对象贯穿接收点与 Broker，并保持未知载荷安全降级。 Constraint: 不改变 Browser RPC 签名与 MAC 校验格式 Tested: bun test apps/sidecar/src/rpc/browser-rpc-sequence.test.ts apps/sidecar/src/services/browser/browser-broker.test.ts Tested: bun run --filter @lume/sidecar typecheck
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
2 participants
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.
结果
修复 Desktop → Sidecar → BrowserBroker 链路中浏览器错误码被覆盖的问题。actionability_failed、stale_target 等稳定错误现在可以无损到达 Browser 调用结果，不再统一退化为 browser_internal_error。
改动
验证
Closes #252