fix: address PR review comments

- Remove redundant case 'api-keys' from switch (handled by default)
- Improve type safety by using SettingsViewId in NavigationItem interface
- Simplify onCheckedChange callback in AudioSection
- Import NAV_ITEMS from config instead of duplicating locally
- Update SettingsNavigation props to use SettingsViewId type

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

apps/app/src/components/views/settings-view/audio/audio-section.tsx

@@ -39,7 +39,7 @@ export function AudioSection({
           <Checkbox
             id="mute-done-sound"
             checked={muteDoneSound}
-            onCheckedChange={(checked) => onMuteDoneSoundChange(checked === true)}
+            onCheckedChange={onMuteDoneSoundChange}
             className="mt-1"
             data-testid="mute-done-sound-checkbox"
           />