Skip to content

Redact recipient ID in missing-token warnings - #64

Closed
NewFuture with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-comments-in-review-thread
Closed

NewFuture with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-comments-in-review-thread

Conversation

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

变更说明

  • 使用 redactToken 脱敏缺少 contextToken 时的收件人 ID。
  • 增加回归断言,确保日志保留可诊断的脱敏标识但不包含完整 ID。

验收标准

  • 缺少 contextToken 时跳过发送。
  • 警告包含 oSYNTH…(len=38),不包含完整收件人 ID。

风险与不变量

  • 隐私脱敏:仅调整缺少 contextToken 分支的警告内容。
  • 发送行为、账号隔离和状态均不变。

验证方式

  • npm run test:unit -- src/messaging/error-notice.test.ts
  • npm run check
  • CodeQL 与敏感信息扫描

检查清单

  • 已在本地或 CI 中通过 npm run check
  • 已运行与本次改动直接相关的定向测试。
  • 行为变更已添加或更新测试。
  • 必要时已更新文档及中英文变更日志。
  • 未包含凭据、账号标识或私聊内容等敏感信息。
  • 我已审阅并对全部提交内容负责,包括 AI 辅助生成的内容。

Co-authored-by: NewFuture <6290356+NewFuture@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix code based on review comments Redact recipient ID in missing-token warnings Aug 11, 2026
Copilot AI requested a review from NewFuture August 11, 2026 07:28
@NewFuture NewFuture closed this Aug 11, 2026
@NewFuture
NewFuture deleted the copilot/fix-comments-in-review-thread branch August 17, 2026 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants