Skip to content

fix(review): clean up dynamic review copy - #1724

Merged
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/review-copy-cleanup
Jul 24, 2026
Merged

fix(review): clean up dynamic review copy#1724
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/review-copy-cleanup

Conversation

@limityan

Copy link
Copy Markdown
Collaborator

Summary

  • remove retired fixed-reviewer presentation resources and keep the UI aligned with dynamic, evidence-driven review checks
  • replace internal or misleading copy with plain, count-neutral wording across English, Simplified Chinese, and Traditional Chinese
  • localize managed review batch cards without changing persisted packet identities or historical report compatibility
  • add locale, fallback, task-card, consent, and report-label regression coverage

This is a focused follow-up to #1723. Historical reviewer IDs and report coverage categories remain supported; only current user-facing presentation is cleaned up.

Validation

  • 110 focused Web UI tests passed
  • Rust review-team contract tests passed (3/3)
  • pnpm run type-check:web
  • pnpm run lint:web
  • pnpm run i18n:contract:test
  • pnpm run i18n:audit (0 warnings)
  • pnpm run theme:color-audit:all
  • cargo check -p bitfun-agent-runtime
  • pnpm run check:repo-hygiene
  • git diff --check

Full Web UI run completed with 1972/1973 tests passing locally; the sole failure is an unchanged RemoteConnectDialog source-text contract that assumes LF line endings and fails on this Windows CRLF worktree.

@limityan
limityan merged commit 0d0ec18 into GCWing:main Jul 24, 2026
5 checks passed
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.

1 participant