mirror of
https://github.com/larksuite/cli.git
synced 2026-08-03 08:32:46 +08:00
Review follow-ups on the member-types and chat-search changes. normalizeMemberTypes accepted any occurrence of "all" before validating the remaining values, so an invalid value alongside it (--member-types admin,all) was silently swallowed into "no filter". Validate every value first; "all" only widens the filter after the whole list is known to be well-formed, and the rejection message now names all three accepted spellings. The skill index and command descriptions for +chat-messages-list and +threads-messages-list advertised "sort" while the actual flag is --order; name the flag exactly so callers do not learn a spelling the command rejects. Test tightening from the same review: the canonical-precedence e2e now passes an alias value that would fail validation (--types p2p) alongside --chat-modes, proving an explicit canonical flag bypasses alias validation entirely; rejection-path e2e tests assert the structured validation metadata (error.type, error.subtype, param names) instead of message text alone.