fix: restore Escape-to-back on skill detail page

Escape was dropped when SkillDetail moved from a slide-over panel
(Escape -> onClose) to full-page routing (onBack) in cf9cc31 and
never got wired back up. Also guards SkillFileViewer's existing
Escape handler against Radix dialogs (Cmd+K, Manage Folders) that
preventDefault but don't stopPropagation, which would otherwise
dismiss the dialog and navigate back in one keypress.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017LKjJ7DNYh5G575m1YzeYp

Author: sairithik9849

src/renderer/src/views/SkillDetail.tsx

@@ -1,4 +1,4 @@
-import { useMemo } from 'react'
+import { useEffect, useMemo } from 'react'
 import { useQuery } from '@tanstack/react-query'
 import { AlertTriangle, ArrowLeft, FolderOpen, Power } from 'lucide-react'
 import { LintFindingsPanel } from '@/components/LintFindingsPanel'
@@ -39,6 +39,15 @@ export function SkillDetail({
   const metadataEntries = useMemo(() => parseMetadataEntries(data?.skill.metadata_json), [data])
   const hookEvents = parseHookEvents(data?.skill.hook_events).join(', ')
 
+  useEffect(() => {
+    function handleKeyDown(event: KeyboardEvent): void {
+      if (event.key !== 'Escape' || event.defaultPrevented) return
+      onBack()
+    }
+    document.addEventListener('keydown', handleKeyDown)
+    return () => document.removeEventListener('keydown', handleKeyDown)
+  }, [onBack])
+
   if (isPending) {
     return (
       <div className="flex flex-1 items-center justify-center text-sm text-muted-foreground">

src/renderer/src/views/SkillFileViewer.tsx

@@ -42,7 +42,7 @@ export function SkillFileViewer({
 
   useEffect(() => {
     function handleKeyDown(event: KeyboardEvent): void {
-      if (event.key !== 'Escape') return
+      if (event.key !== 'Escape' || event.defaultPrevented) return
       if (searchQuery) {
         setSearchQuery('')
         return