xiaoxue_admin
/
orca
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(skills): make skippedReason switch exhaustive (#9594) 

The type-aware switch-exhaustiveness lint rule requires explicit cases
for 'current', 'duplicate', and undefined; they fall through to the
existing generic skipped message, so behavior is unchanged.

Co-authored-by: Yuris Auzins <zuz666@users.noreply.github.com>
This commit is contained in:
Zuz666 2026-07-23 08:04:41 +05:00 committed by GitHub
parent a2e440b308
commit ffe54a59a3
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/renderer/src/components/skills/skill-freshness-group.tsx
View File

@ -134,7 +134,11 @@ function skippedReason(locations: readonly SkillLocationRow[]): string {
'auto.components.skills.SkillFreshnessRow.skippedReasonBrokenLink',
'This copy is a shortcut to something that no longer exists, so Orca left it out — you can safely delete it.'
)
default:
// Why: 'current'/'duplicate' are non-blocking chips, and an empty priority
// list is possible; all fall through to the generic skipped message.
case 'current':
case 'duplicate':
case undefined:
return translate(
'auto.components.skills.SkillFreshnessRow.cantUpdateReason',
'Orca left this skill out of the update command.'