Skip to content

fix(ci): restore structured Feishu review cards - #492

Merged
SaladDay merged 1 commit into
mainfrom
codex/review-card-layout
Oct 7, 2026
Merged

SaladDay merged 1 commit into
mainfrom
codex/review-card-layout

Conversation

@SaladDay

@SaladDay SaladDay commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

The combined Code/Docs notification collapsed the previous report into free-form summaries. Restore a fixed Feishu card layout with PR details, behavior changes, code and repository rules, CI results, and documentation review, separated by dividers and headed by the aggregate review status.

Each reviewer now fills named fields so the notification renderer owns the layout. Both results still use one delivery, with existing signing, message limits and incomplete-report handling. The Actions summary follows the same sections; English and Chinese maintainer guidance is updated.

Validation: 57 CI script tests, 3 translation tests, and a fresh independent review passed. Feishu client rendering has not been exercised with a real message.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@SaladDay
SaladDay merged commit a02ff8e into main Oct 7, 2026
14 checks passed
@SaladDay
SaladDay deleted the codex/review-card-layout branch October 7, 2026 13:34
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