Description
PR #4545 (bba7514) added #[non_exhaustive] to extensible pub enums across 14 crates. zeph-skills was only partially covered — scanner.rs was touched but the bulk of pub enums in the crate were missed.
Missing annotations
All of the following are in crates/zeph-skills/src/:
| Enum |
File |
Note |
MatchResult |
matcher.rs:82 |
New match modes expected |
SkillMatcherBackend |
matcher.rs:565 |
New backends (LLM, hybrid) expected |
PromotionRecommendation |
promoter.rs:30 |
Extensible verdict type |
SkillSource |
trust.rs:44 |
New trust sources expected |
FailureKind |
evolution.rs:36 |
New failure kinds likely |
SkillOutcome |
evolution.rs:137 |
New outcome types expected |
SkillVerdict |
evaluator.rs:112 |
Extensible verdict type |
MergeDecision |
merger.rs:51 |
New merge decisions may be added |
SkillEvent |
watcher.rs:44 |
New file-system events expected |
SkillError |
error.rs:24 |
Error enums should always be #[non_exhaustive] |
SkillRole |
group.rs:30 |
New roles likely |
GroupResult |
group.rs:59 |
New group result states expected |
Fix
Add #[non_exhaustive] to each listed enum. Follow the same pattern as bba7514.
Description
PR #4545 (bba7514) added
#[non_exhaustive]to extensible pub enums across 14 crates.zeph-skillswas only partially covered —scanner.rswas touched but the bulk of pub enums in the crate were missed.Missing annotations
All of the following are in
crates/zeph-skills/src/:MatchResultmatcher.rs:82SkillMatcherBackendmatcher.rs:565PromotionRecommendationpromoter.rs:30SkillSourcetrust.rs:44FailureKindevolution.rs:36SkillOutcomeevolution.rs:137SkillVerdictevaluator.rs:112MergeDecisionmerger.rs:51SkillEventwatcher.rs:44SkillErrorerror.rs:24#[non_exhaustive]SkillRolegroup.rs:30GroupResultgroup.rs:59Fix
Add
#[non_exhaustive]to each listed enum. Follow the same pattern as bba7514.