@aion0/forge 0.3.0 → 0.3.1

package/components/SkillsPanel.tsx

@@ -2,16 +2,22 @@
 
 import { useState, useEffect, useCallback } from 'react';
 
+type ItemType = 'skill' | 'command';
+
 interface Skill {
   name: string;
+  type: ItemType;
   displayName: string;
   description: string;
   author: string;
   version: string;
   tags: string[];
   score: number;
+  rating: number;
   sourceUrl: string;
   installedGlobal: boolean;
+  installedVersion: string;
+  hasUpdate: boolean;
   installedProjects: string[];
 }
 
@@ -26,6 +32,22 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
   const [syncing, setSyncing] = useState(false);
   const [loading, setLoading] = useState(true);
   const [installTarget, setInstallTarget] = useState<{ skill: string; show: boolean }>({ skill: '', show: false });
+  const [typeFilter, setTypeFilter] = useState<'all' | 'skill' | 'command' | 'local' | 'rules'>('all');
+  const [localItems, setLocalItems] = useState<{ name: string; type: string; scope: string; fileCount: number; projectPath?: string }[]>([]);
+  // Rules (CLAUDE.md templates)
+  const [rulesTemplates, setRulesTemplates] = useState<{ id: string; name: string; description: string; tags: string[]; builtin: boolean; isDefault: boolean; content: string }[]>([]);
+  const [rulesProjects, setRulesProjects] = useState<{ name: string; path: string }[]>([]);
+  const [rulesSelectedTemplate, setRulesSelectedTemplate] = useState<string | null>(null);
+  const [rulesEditing, setRulesEditing] = useState(false);
+  const [rulesEditId, setRulesEditId] = useState('');
+  const [rulesEditName, setRulesEditName] = useState('');
+  const [rulesEditDesc, setRulesEditDesc] = useState('');
+  const [rulesEditContent, setRulesEditContent] = useState('');
+  const [rulesEditDefault, setRulesEditDefault] = useState(false);
+  const [rulesShowNew, setRulesShowNew] = useState(false);
+  const [rulesBatchProjects, setRulesBatchProjects] = useState<Set<string>>(new Set());
+  const [searchQuery, setSearchQuery] = useState('');
+  const [collapsedLocalSections, setCollapsedLocalSections] = useState<Set<string>>(new Set());
   const [expandedSkill, setExpandedSkill] = useState<string | null>(null);
   const [skillFiles, setSkillFiles] = useState<{ name: string; path: string; type: string }[]>([]);
   const [activeFile, setActiveFile] = useState<string | null>(null);
@@ -33,16 +55,78 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
 
   const fetchSkills = useCallback(async () => {
     try {
-      const res = await fetch('/api/skills');
-      const data = await res.json();
+      const [registryRes, localRes] = await Promise.all([
+        fetch('/api/skills'),
+        fetch('/api/skills/local?action=scan&all=1'),
+      ]);
+      const data = await registryRes.json();
       setSkills(data.skills || []);
       setProjects(data.projects || []);
+      const localData = await localRes.json();
+      // Filter out items already in registry
+      const registryNames = new Set((data.skills || []).map((s: any) => s.name));
+      setLocalItems((localData.items || []).filter((i: any) => !registryNames.has(i.name)));
     } catch {}
     setLoading(false);
   }, []);
 
   useEffect(() => { fetchSkills(); }, [fetchSkills]);
 
+  const fetchRules = useCallback(async () => {
+    try {
+      const res = await fetch('/api/claude-templates?action=list');
+      const data = await res.json();
+      setRulesTemplates(data.templates || []);
+      setRulesProjects(data.projects || []);
+    } catch {}
+  }, []);
+
+  useEffect(() => { if (typeFilter === 'rules') fetchRules(); }, [typeFilter, fetchRules]);
+
+  const saveRule = async () => {
+    if (!rulesEditId || !rulesEditName || !rulesEditContent) return;
+    await fetch('/api/claude-templates', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ action: 'save', id: rulesEditId, name: rulesEditName, description: rulesEditDesc, tags: [], content: rulesEditContent, isDefault: rulesEditDefault }),
+    });
+    setRulesEditing(false);
+    setRulesShowNew(false);
+    fetchRules();
+  };
+
+  const deleteRule = async (id: string) => {
+    if (!confirm(`Delete template "${id}"?`)) return;
+    await fetch('/api/claude-templates', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ action: 'delete', id }),
+    });
+    if (rulesSelectedTemplate === id) setRulesSelectedTemplate(null);
+    fetchRules();
+  };
+
+  const toggleDefault = async (id: string, isDefault: boolean) => {
+    await fetch('/api/claude-templates', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ action: 'set-default', id, isDefault }),
+    });
+    fetchRules();
+  };
+
+  const batchInject = async (templateId: string) => {
+    const projects = [...rulesBatchProjects];
+    if (!projects.length) return;
+    await fetch('/api/claude-templates', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ action: 'inject', templateId, projects }),
+    });
+    setRulesBatchProjects(new Set());
+    fetchRules();
+  };
+
   const sync = async () => {
     setSyncing(true);
     await fetch('/api/skills', {
@@ -85,11 +169,17 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
     } catch { setSkillFiles([]); }
   };
 
-  const loadFile = async (skillName: string, filePath: string) => {
+  const loadFile = async (skillName: string, filePath: string, isLocalItem?: boolean, localType?: string, localProject?: string) => {
     setActiveFile(filePath);
     setFileContent('Loading...');
     try {
-      const res = await fetch(`/api/skills?action=file&name=${encodeURIComponent(skillName)}&path=${encodeURIComponent(filePath)}`);
+      let res;
+      if (isLocalItem) {
+        const projectParam = localProject ? `&project=${encodeURIComponent(localProject)}` : '';
+        res = await fetch(`/api/skills/local?action=read&name=${encodeURIComponent(skillName)}&type=${localType || 'command'}&path=${encodeURIComponent(filePath)}${projectParam}`);
+      } else {
+        res = await fetch(`/api/skills?action=file&name=${encodeURIComponent(skillName)}&path=${encodeURIComponent(filePath)}`);
+      }
       const data = await res.json();
       setFileContent(data.content || '(Empty)');
     } catch { setFileContent('(Failed to load)'); }
@@ -104,10 +194,37 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
     fetchSkills();
   };
 
-  // Filter skills if viewing a specific project
-  const filtered = projectFilter
-    ? skills.filter(s => s.installedGlobal || s.installedProjects.includes(projectFilter))
-    : skills;
+  // Filter by project, type, and search
+  const q = searchQuery.toLowerCase();
+  const filtered = typeFilter === 'local' ? [] : skills
+    .filter(s => projectFilter ? (s.installedGlobal || s.installedProjects.includes(projectFilter)) : true)
+    .filter(s => typeFilter === 'all' ? true : s.type === typeFilter)
+    .filter(s => !q || s.name.toLowerCase().includes(q) || s.displayName.toLowerCase().includes(q) || s.description.toLowerCase().includes(q));
+
+  const filteredLocal = localItems
+    .filter(item => typeFilter === 'local' || typeFilter === 'all' || item.type === typeFilter)
+    .filter(item => !q || item.name.toLowerCase().includes(q));
+
+  // Group local items by scope
+  const localGroups = new Map<string, typeof localItems>();
+  for (const item of filteredLocal) {
+    const key = item.scope;
+    if (!localGroups.has(key)) localGroups.set(key, []);
+    localGroups.get(key)!.push(item);
+  }
+
+  const toggleLocalSection = (section: string) => {
+    setCollapsedLocalSections(prev => {
+      const next = new Set(prev);
+      if (next.has(section)) next.delete(section);
+      else next.add(section);
+      return next;
+    });
+  };
+
+  const skillCount = skills.filter(s => s.type === 'skill').length;
+  const commandCount = skills.filter(s => s.type === 'command').length;
+  const localCount = localItems.length;
 
   if (loading) {
     return <div className="p-4 text-xs text-[var(--text-secondary)]">Loading skills...</div>;
@@ -118,8 +235,22 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
       {/* Header */}
       <div className="flex items-center justify-between px-4 py-2 border-b border-[var(--border)] shrink-0">
         <div className="flex items-center gap-2">
-          <span className="text-xs font-semibold text-[var(--text-primary)]">Skills</span>
-          <span className="text-[9px] text-[var(--text-secondary)]">{filtered.length} available</span>
+          <span className="text-xs font-semibold text-[var(--text-primary)]">Marketplace</span>
+          <div className="flex items-center bg-[var(--bg-tertiary)] rounded p-0.5">
+            {([['all', `All (${skills.length})`], ['skill', `Skills (${skillCount})`], ['command', `Commands (${commandCount})`], ['local', `Local (${localCount})`], ['rules', 'Rules']] as const).map(([value, label]) => (
+              <button
+                key={value}
+                onClick={() => setTypeFilter(value)}
+                className={`text-[9px] px-2 py-0.5 rounded transition-colors ${
+                  typeFilter === value
+                    ? 'bg-[var(--bg-secondary)] text-[var(--text-primary)] shadow-sm'
+                    : 'text-[var(--text-secondary)] hover:text-[var(--text-primary)]'
+                }`}
+              >
+                {label}
+              </button>
+            ))}
+          </div>
         </div>
         <button
           onClick={sync}
@@ -129,8 +260,18 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
           {syncing ? 'Syncing...' : 'Sync'}
         </button>
       </div>
+      {/* Search — hide on rules tab */}
+      {typeFilter !== 'rules' && <div className="px-3 py-1.5 border-b border-[var(--border)] shrink-0">
+        <input
+          type="text"
+          value={searchQuery}
+          onChange={e => setSearchQuery(e.target.value)}
+          placeholder="Search skills & commands..."
+          className="w-full px-2 py-1 bg-[var(--bg-tertiary)] border border-[var(--border)] rounded text-[10px] text-[var(--text-primary)] placeholder:text-[var(--text-secondary)] focus:outline-none focus:border-[var(--accent)]"
+        />
+      </div>}
 
-      {skills.length === 0 ? (
+      {typeFilter === 'rules' ? null : skills.length === 0 ? (
         <div className="flex-1 flex flex-col items-center justify-center gap-2 text-[var(--text-secondary)]">
           <p className="text-xs">No skills yet</p>
           <button onClick={sync} className="text-xs px-3 py-1 bg-[var(--accent)] text-white rounded hover:opacity-90">
@@ -141,6 +282,7 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
         <div className="flex-1 flex min-h-0">
           {/* Left: skill list */}
           <div className="w-56 border-r border-[var(--border)] overflow-y-auto shrink-0">
+            {/* Registry items */}
             {filtered.map(skill => {
               const isInstalled = skill.installedGlobal || skill.installedProjects.length > 0;
               const isActive = expandedSkill === skill.name;
@@ -155,40 +297,216 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
                   <div className="flex items-center gap-2">
                     <span className="text-[11px] font-semibold text-[var(--text-primary)] truncate flex-1">{skill.displayName}</span>
                     <span className="text-[8px] text-[var(--text-secondary)] font-mono shrink-0">v{skill.version}</span>
-                    {skill.score > 0 && (
-                      <span className="text-[8px] text-[var(--yellow)] shrink-0">{skill.score}pt</span>
+                    {skill.rating > 0 && (
+                      <span className="text-[8px] text-[var(--yellow)] shrink-0" title={`Rating: ${skill.rating}/5`}>
+                        {'★'.repeat(Math.round(skill.rating))}{'☆'.repeat(5 - Math.round(skill.rating))}
+                      </span>
+                    )}
+                    {skill.score > 0 && !skill.rating && (
+                      <span className="text-[8px] text-[var(--text-secondary)] shrink-0">{skill.score}pt</span>
                     )}
                   </div>
                   <p className="text-[9px] text-[var(--text-secondary)] mt-0.5 line-clamp-1">{skill.description}</p>
                   <div className="flex items-center gap-1.5 mt-1">
+                    <span className={`text-[7px] px-1 rounded font-medium ${
+                      skill.type === 'skill' ? 'bg-purple-500/20 text-purple-400' : 'bg-blue-500/20 text-blue-400'
+                    }`}>{skill.type === 'skill' ? 'SKILL' : 'CMD'}</span>
                     <span className="text-[8px] text-[var(--text-secondary)]">{skill.author}</span>
                     {skill.tags.slice(0, 2).map(t => (
                       <span key={t} className="text-[7px] px-1 rounded bg-[var(--bg-tertiary)] text-[var(--text-secondary)]">{t}</span>
                     ))}
-                    {isInstalled && <span className="text-[8px] text-[var(--green)] ml-auto">installed</span>}
+                    {skill.hasUpdate && <span className="text-[8px] text-[var(--yellow)] ml-auto">update</span>}
+                    {isInstalled && !skill.hasUpdate && <span className="text-[8px] text-[var(--green)] ml-auto">installed</span>}
                   </div>
                 </div>
               );
             })}
+            {/* Local items — collapsible by scope group */}
+            {(typeFilter === 'all' || typeFilter === 'local') && filteredLocal.length > 0 && (
+              <>
+                {/* Local section header — collapsible */}
+                {typeFilter !== 'local' && (
+                  <button
+                    onClick={() => toggleLocalSection('__local__')}
+                    className="w-full px-3 py-1 text-[8px] text-[var(--text-secondary)] uppercase bg-[var(--bg-tertiary)] border-b border-[var(--border)]/50 flex items-center gap-1 hover:text-[var(--text-primary)]"
+                  >
+                    <span>{collapsedLocalSections.has('__local__') ? '▸' : '▾'}</span>
+                    Local ({filteredLocal.length})
+                  </button>
+                )}
+                {(typeFilter === 'local' || !collapsedLocalSections.has('__local__')) && (
+                  <>
+                    {[...localGroups.entries()].sort(([a], [b]) => a === 'global' ? -1 : b === 'global' ? 1 : a.localeCompare(b)).map(([scope, items]) => (
+                      <div key={scope}>
+                        {/* Scope group header — collapsible */}
+                        <button
+                          onClick={() => toggleLocalSection(scope)}
+                          className="w-full px-3 py-1 text-[8px] text-[var(--text-secondary)] border-b border-[var(--border)]/30 flex items-center gap-1.5 hover:bg-[var(--bg-tertiary)]"
+                        >
+                          <span className="text-[7px]">{collapsedLocalSections.has(scope) ? '▸' : '▾'}</span>
+                          <span className={scope === 'global' ? 'text-green-400' : 'text-[var(--accent)]'}>{scope}</span>
+                          <span className="text-[var(--text-secondary)]">({items.length})</span>
+                        </button>
+                        {!collapsedLocalSections.has(scope) && items.map(item => {
+                          const key = `local:${item.name}:${item.scope}`;
+                          const isActive = expandedSkill === key;
+                          const projectParam = item.projectPath ? encodeURIComponent(item.projectPath) : '';
+                          return (
+                            <div
+                              key={key}
+                              className={`px-3 py-2 border-b border-[var(--border)]/50 cursor-pointer ${
+                                isActive ? 'bg-[var(--accent)]/10 border-l-2 border-l-[var(--accent)]' : 'hover:bg-[var(--bg-tertiary)] border-l-2 border-l-transparent'
+                              }`}
+                              onClick={() => {
+                                if (expandedSkill === key) { setExpandedSkill(null); return; }
+                                setExpandedSkill(key);
+                                setSkillFiles([]);
+                                setActiveFile(null);
+                                setFileContent('');
+                                const fetchUrl = `/api/skills/local?action=files&name=${encodeURIComponent(item.name)}&type=${item.type}${projectParam ? `&project=${projectParam}` : ''}`;
+                                fetch(fetchUrl)
+                                  .then(r => r.json())
+                                  .then(d => {
+                                    const files = (d.files || []).map((f: any) => ({ name: f.path.split('/').pop(), path: f.path, type: 'file' }));
+                                    setSkillFiles(files);
+                                    const first = files.find((f: any) => f.name?.endsWith('.md'));
+                                    if (first) {
+                                      setActiveFile(first.path);
+                                      fetch(`/api/skills/local?action=read&name=${encodeURIComponent(item.name)}&type=${item.type}&path=${encodeURIComponent(first.path)}${projectParam ? `&project=${projectParam}` : ''}`)
+                                        .then(r => r.json())
+                                        .then(rd => setFileContent(rd.content || ''))
+                                        .catch(() => {});
+                                    }
+                                  })
+                                  .catch(() => {});
+                              }}
+                            >
+                              <div className="flex items-center gap-2">
+                                <span className={`text-[7px] px-1 rounded font-medium ${
+                                  item.type === 'skill' ? 'bg-purple-500/20 text-purple-400' : 'bg-blue-500/20 text-blue-400'
+                                }`}>{item.type === 'skill' ? 'S' : 'C'}</span>
+                                <span className="text-[10px] text-[var(--text-primary)] truncate flex-1">{item.name}</span>
+                                <span className="text-[8px] text-[var(--text-secondary)]">{item.fileCount}</span>
+                              </div>
+                            </div>
+                          );
+                        })}
+                      </div>
+                    ))}
+                  </>
+                )}
+              </>
+            )}
           </div>
 
           {/* Right: detail panel */}
           <div className="flex-1 flex flex-col min-w-0">
             {expandedSkill ? (() => {
-              const skill = skills.find(s => s.name === expandedSkill);
-              if (!skill) return null;
-              const isInstalled = skill.installedGlobal || skill.installedProjects.length > 0;
+              const isLocal = expandedSkill.startsWith('local:');
+              // Key format: "local:<name>:<scope>" — extract name (could contain colons in scope)
+              const localParts = isLocal ? expandedSkill.slice(6).split(':') : [];
+              const itemName = isLocal ? localParts[0] : expandedSkill;
+              const localScope = isLocal ? localParts.slice(1).join(':') : '';
+              const skill = isLocal ? null : skills.find(s => s.name === expandedSkill);
+              const localItem = isLocal ? localItems.find(i => i.name === itemName && i.scope === localScope) : null;
+              if (!skill && !localItem) return null;
+              const isInstalled = skill ? (skill.installedGlobal || skill.installedProjects.length > 0) : true;
               return (
                 <>
                   {/* Skill header */}
                   <div className="px-4 py-2 border-b border-[var(--border)] shrink-0">
                     <div className="flex items-center gap-2">
-                      <span className="text-sm font-semibold text-[var(--text-primary)]">{skill.displayName}</span>
-                      <span className="text-[9px] text-[var(--text-secondary)] font-mono">v{skill.version}</span>
-                      {skill.score > 0 && <span className="text-[9px] text-[var(--yellow)]">{skill.score}pt</span>}
+                      <span className="text-sm font-semibold text-[var(--text-primary)]">{skill?.displayName || localItem?.name || itemName}</span>
+                      <span className={`text-[8px] px-1.5 py-0.5 rounded font-medium ${
+                        (skill?.type || localItem?.type) === 'skill' ? 'bg-purple-500/20 text-purple-400' : 'bg-blue-500/20 text-blue-400'
+                      }`}>{(skill?.type || localItem?.type) === 'skill' ? 'Skill' : 'Command'}</span>
+                      {isLocal && <span className="text-[7px] px-1 rounded bg-green-500/10 text-green-400">local</span>}
+                      {skill && <span className="text-[9px] text-[var(--text-secondary)] font-mono">v{skill.version}</span>}
+                      {skill?.installedVersion && skill.installedVersion !== skill.version && (
+                        <span className="text-[9px] text-[var(--yellow)] font-mono">installed: v{skill.installedVersion}</span>
+                      )}
+                      {skill && skill.rating > 0 && (
+                        <span className="text-[9px] text-[var(--yellow)]" title={`Rating: ${skill.rating}/5`}>
+                          {'★'.repeat(Math.round(skill.rating))}{'☆'.repeat(5 - Math.round(skill.rating))}
+                        </span>
+                      )}
+                      {skill && skill.score > 0 && <span className="text-[9px] text-[var(--text-secondary)]">{skill.score}pt</span>}
+
+                      {/* Update button */}
+                      {skill?.hasUpdate && (
+                        <button
+                          onClick={async () => {
+                            if (skill.installedGlobal) await install(skill.name, 'global');
+                            for (const pp of skill.installedProjects) await install(skill.name, pp);
+                          }}
+                          className="text-[9px] px-2 py-1 bg-[var(--yellow)]/20 text-[var(--yellow)] border border-[var(--yellow)]/50 rounded hover:bg-[var(--yellow)]/30 transition-colors"
+                        >
+                          Update
+                        </button>
+                      )}
 
-                      {/* Install dropdown */}
-                      <div className="relative ml-auto">
+                      {/* Local item actions: install to other projects, delete */}
+                      {isLocal && localItem && (
+                        <>
+                          <div className="relative ml-auto">
+                            <button
+                              onClick={() => setInstallTarget(prev =>
+                                prev.skill === itemName && prev.show ? { skill: '', show: false } : { skill: itemName, show: true }
+                              )}
+                              className="text-[9px] px-2 py-1 border border-[var(--accent)] text-[var(--accent)] rounded hover:bg-[var(--accent)] hover:text-white transition-colors"
+                            >
+                              Install to...
+                            </button>
+                            {installTarget.skill === itemName && installTarget.show && (
+                              <>
+                                <div className="fixed inset-0 z-40" onClick={() => setInstallTarget({ skill: '', show: false })} />
+                                <div className="absolute right-0 top-7 w-[200px] bg-[var(--bg-secondary)] border border-[var(--border)] rounded-lg shadow-xl z-50 py-1">
+                                  <button
+                                    onClick={async () => {
+                                      const res = await fetch('/api/skills/local', { method: 'POST', headers: { 'Content-Type': 'application/json' },
+                                        body: JSON.stringify({ action: 'install-local', name: itemName, type: localItem.type, sourceProject: localItem.projectPath, target: 'global', force: true }) });
+                                      const data = await res.json();
+                                      if (!data.ok) alert(data.error);
+                                      setInstallTarget({ skill: '', show: false });
+                                      fetchSkills();
+                                    }}
+                                    className="w-full text-left text-[10px] px-3 py-1.5 hover:bg-[var(--bg-tertiary)] text-[var(--text-primary)]"
+                                  >Global (~/.claude)</button>
+                                  <div className="border-t border-[var(--border)] my-0.5" />
+                                  {projects.map(p => (
+                                    <button
+                                      key={p.path}
+                                      onClick={async () => {
+                                        const res = await fetch('/api/skills/local', { method: 'POST', headers: { 'Content-Type': 'application/json' },
+                                          body: JSON.stringify({ action: 'install-local', name: itemName, type: localItem.type, sourceProject: localItem.projectPath, target: p.path, force: true }) });
+                                        const data = await res.json();
+                                        if (!data.ok) alert(data.error);
+                                        setInstallTarget({ skill: '', show: false });
+                                        fetchSkills();
+                                      }}
+                                      className="w-full text-left text-[10px] px-3 py-1.5 hover:bg-[var(--bg-tertiary)] text-[var(--text-primary)] truncate"
+                                      title={p.path}
+                                    >{p.name}</button>
+                                  ))}
+                                </div>
+                              </>
+                            )}
+                          </div>
+                          <button
+                            onClick={async () => {
+                              if (!confirm(`Delete "${itemName}" from ${localScope}?`)) return;
+                              await fetch('/api/skills/local', { method: 'POST', headers: { 'Content-Type': 'application/json' },
+                                body: JSON.stringify({ action: 'delete-local', name: itemName, type: localItem.type, project: localItem.projectPath }) });
+                              setExpandedSkill(null);
+                              fetchSkills();
+                            }}
+                            className="text-[9px] text-[var(--red)] hover:underline"
+                          >Delete</button>
+                        </>
+                      )}
+
+                      {/* Install dropdown — registry items only */}
+                      {skill && <div className="relative ml-auto">
                         <button
                           onClick={() => setInstallTarget(prev =>
                             prev.skill === skill.name && prev.show ? { skill: '', show: false } : { skill: skill.name, show: true }
@@ -228,22 +546,22 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
                             </div>
                           </>
                         )}
-                      </div>
+                      </div>}
                     </div>
-                    <p className="text-[10px] text-[var(--text-secondary)] mt-0.5">{skill.description}</p>
+                    <p className="text-[10px] text-[var(--text-secondary)] mt-0.5">{skill?.description || ''}</p>
                     {/* Installed indicators */}
-                    {isInstalled && (
+                    {skill && isInstalled && (
                       <div className="flex items-center gap-2 mt-1">
                         {skill.installedGlobal && (
                           <span className="flex items-center gap-1 text-[8px] text-[var(--green)]">
                             Global
-                            <button onClick={() => uninstall(skill.name, 'global')} className="text-[var(--text-secondary)] hover:text-[var(--red)]">x</button>
+                            <button onClick={() => { if (confirm(`Uninstall "${skill.name}" from global?`)) uninstall(skill.name, 'global'); }} className="text-[var(--text-secondary)] hover:text-[var(--red)]">x</button>
                           </span>
                         )}
                         {skill.installedProjects.map(pp => (
                           <span key={pp} className="flex items-center gap-1 text-[8px] text-[var(--accent)]">
                             {pp.split('/').pop()}
-                            <button onClick={() => uninstall(skill.name, pp)} className="text-[var(--text-secondary)] hover:text-[var(--red)]">x</button>
+                            <button onClick={() => { if (confirm(`Uninstall "${skill.name}" from ${pp.split('/').pop()}?`)) uninstall(skill.name, pp); }} className="text-[var(--text-secondary)] hover:text-[var(--red)]">x</button>
                           </span>
                         ))}
                       </div>
@@ -261,7 +579,7 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
                           f.type === 'file' ? (
                             <button
                               key={f.path}
-                              onClick={() => loadFile(skill.name, f.path)}
+                              onClick={() => loadFile(itemName, f.path, isLocal, localItem?.type, localItem?.projectPath)}
                               className={`w-full text-left px-2 py-1 text-[10px] truncate ${
                                 activeFile === f.path
                                   ? 'bg-[var(--accent)]/15 text-[var(--accent)]'
@@ -278,7 +596,7 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
                           )
                         ))
                       )}
-                      {skill.sourceUrl && (
+                      {skill?.sourceUrl && (
                         <div className="border-t border-[var(--border)] p-2">
                           <a
                             href={skill.sourceUrl.replace(/\/blob\/main\/.*/, '')}
@@ -315,6 +633,174 @@ export default function SkillsPanel({ projectFilter }: { projectFilter?: string
           </div>
         </div>
       )}
+
+      {/* Rules (CLAUDE.md Templates) — full-page view */}
+      {typeFilter === 'rules' && (
+        <div className="flex-1 flex min-h-0">
+          {/* Left: template list */}
+          <div className="w-56 border-r border-[var(--border)] overflow-y-auto shrink-0 flex flex-col">
+            <div className="px-3 py-1.5 border-b border-[var(--border)] flex items-center justify-between">
+              <span className="text-[9px] text-[var(--text-secondary)] uppercase">Rule Templates</span>
+              <button
+                onClick={() => { setRulesShowNew(true); setRulesEditing(true); setRulesEditId(''); setRulesEditName(''); setRulesEditDesc(''); setRulesEditContent(''); setRulesEditDefault(false); setRulesSelectedTemplate(null); }}
+                className="text-[9px] text-[var(--accent)] hover:underline"
+              >+ New</button>
+            </div>
+            <div className="flex-1 overflow-y-auto">
+              {rulesTemplates.map(t => {
+                const isActive = rulesSelectedTemplate === t.id;
+                return (
+                  <div
+                    key={t.id}
+                    className={`px-3 py-2 border-b border-[var(--border)]/50 cursor-pointer ${
+                      isActive ? 'bg-[var(--accent)]/10 border-l-2 border-l-[var(--accent)]' : 'hover:bg-[var(--bg-tertiary)] border-l-2 border-l-transparent'
+                    }`}
+                    onClick={() => { setRulesSelectedTemplate(t.id); setRulesEditing(false); setRulesShowNew(false); }}
+                  >
+                    <div className="flex items-center gap-1.5">
+                      <span className="text-[10px] text-[var(--text-primary)] truncate flex-1">{t.name}</span>
+                      {t.builtin && <span className="text-[7px] text-[var(--text-secondary)]">built-in</span>}
+                      <button
+                        onClick={(e) => { e.stopPropagation(); toggleDefault(t.id, !t.isDefault); }}
+                        className={`text-[7px] px-1 rounded ${t.isDefault ? 'bg-[var(--accent)]/20 text-[var(--accent)]' : 'bg-[var(--bg-tertiary)] text-[var(--text-secondary)] hover:text-[var(--text-primary)]'}`}
+                        title={t.isDefault ? 'Default: auto-applied to new projects' : 'Click to set as default'}
+                      >{t.isDefault ? 'default' : 'set default'}</button>
+                    </div>
+                    <p className="text-[8px] text-[var(--text-secondary)] mt-0.5 line-clamp-1">{t.description}</p>
+                  </div>
+                );
+              })}
+            </div>
+          </div>
+
+          {/* Right: template detail / editor / batch apply */}
+          <div className="flex-1 flex flex-col min-w-0">
+            {rulesShowNew || rulesEditing ? (
+              /* Edit / New form */
+              <div className="flex-1 flex flex-col overflow-hidden">
+                <div className="px-4 py-2 border-b border-[var(--border)] shrink-0">
+                  <div className="text-[11px] font-semibold text-[var(--text-primary)]">{rulesShowNew ? 'New Rule Template' : 'Edit Template'}</div>
+                </div>
+                <div className="flex-1 overflow-auto p-4 space-y-3">
+                  <div className="flex gap-2">
+                    <input
+                      type="text"
+                      value={rulesEditId}
+                      onChange={e => setRulesEditId(e.target.value.replace(/[^a-z0-9-]/g, ''))}
+                      placeholder="template-id (kebab-case)"
+                      disabled={!rulesShowNew}
+                      className="flex-1 px-2 py-1 bg-[var(--bg-tertiary)] border border-[var(--border)] rounded text-[10px] text-[var(--text-primary)] font-mono disabled:opacity-50"
+                    />
+                    <input
+                      type="text"
+                      value={rulesEditName}
+                      onChange={e => setRulesEditName(e.target.value)}
+                      placeholder="Display Name"
+                      className="flex-1 px-2 py-1 bg-[var(--bg-tertiary)] border border-[var(--border)] rounded text-[10px] text-[var(--text-primary)]"
+                    />
+                  </div>
+                  <input
+                    type="text"
+                    value={rulesEditDesc}
+                    onChange={e => setRulesEditDesc(e.target.value)}
+                    placeholder="Description"
+                    className="w-full px-2 py-1 bg-[var(--bg-tertiary)] border border-[var(--border)] rounded text-[10px] text-[var(--text-primary)]"
+                  />
+                  <textarea
+                    value={rulesEditContent}
+                    onChange={e => setRulesEditContent(e.target.value)}
+                    placeholder="Template content (markdown)..."
+                    className="w-full flex-1 min-h-[200px] p-2 bg-[var(--bg-tertiary)] border border-[var(--border)] rounded text-[10px] font-mono text-[var(--text-primary)] resize-none"
+                    spellCheck={false}
+                  />
+                  <div className="flex items-center gap-3">
+                    <label className="flex items-center gap-1.5 text-[10px] text-[var(--text-secondary)] cursor-pointer">
+                      <input type="checkbox" checked={rulesEditDefault} onChange={e => setRulesEditDefault(e.target.checked)} className="accent-[var(--accent)]" />
+                      Auto-apply to new projects
+                    </label>
+                    <div className="flex gap-2 ml-auto">
+                      <button onClick={saveRule} className="text-[10px] px-3 py-1 bg-[var(--accent)] text-white rounded hover:opacity-90">Save</button>
+                      <button onClick={() => { setRulesEditing(false); setRulesShowNew(false); }} className="text-[10px] text-[var(--text-secondary)] hover:text-[var(--text-primary)]">Cancel</button>
+                    </div>
+                  </div>
+                </div>
+              </div>
+            ) : rulesSelectedTemplate ? (() => {
+              const tmpl = rulesTemplates.find(t => t.id === rulesSelectedTemplate);
+              if (!tmpl) return null;
+              return (
+                <div className="flex-1 flex flex-col overflow-hidden">
+                  {/* Template header */}
+                  <div className="px-4 py-2 border-b border-[var(--border)] shrink-0">
+                    <div className="flex items-center gap-2">
+                      <span className="text-sm font-semibold text-[var(--text-primary)]">{tmpl.name}</span>
+                      {tmpl.builtin && <span className="text-[8px] px-1 rounded bg-[var(--bg-tertiary)] text-[var(--text-secondary)]">built-in</span>}
+                      <div className="ml-auto flex gap-1.5">
+                        <button
+                          onClick={() => { setRulesEditing(true); setRulesShowNew(false); setRulesEditId(tmpl.id); setRulesEditName(tmpl.name); setRulesEditDesc(tmpl.description); setRulesEditContent(tmpl.content); setRulesEditDefault(tmpl.isDefault); }}
+                          className="text-[9px] text-[var(--accent)] hover:underline"
+                        >Edit</button>
+                        {!tmpl.builtin && (
+                          <button onClick={() => deleteRule(tmpl.id)} className="text-[9px] text-[var(--red)] hover:underline">Delete</button>
+                        )}
+                      </div>
+                    </div>
+                    <p className="text-[9px] text-[var(--text-secondary)] mt-0.5">{tmpl.description}</p>
+                  </div>
+
+                  {/* Content + batch apply */}
+                  <div className="flex-1 flex min-h-0 overflow-hidden">
+                    {/* Template content */}
+                    <div className="flex-1 min-w-0 overflow-auto">
+                      <pre className="p-3 text-[11px] font-mono text-[var(--text-primary)] whitespace-pre-wrap break-words">
+                        {tmpl.content}
+                      </pre>
+                    </div>
+
+                    {/* Batch apply panel */}
+                    <div className="w-48 border-l border-[var(--border)] overflow-y-auto shrink-0 flex flex-col">
+                      <div className="px-2 py-1.5 border-b border-[var(--border)] text-[9px] text-[var(--text-secondary)] uppercase">Apply to Projects</div>
+                      <div className="flex-1 overflow-y-auto">
+                        {rulesProjects.map(p => (
+                          <label key={p.path} className="flex items-center gap-1.5 px-2 py-1 hover:bg-[var(--bg-tertiary)] cursor-pointer">
+                            <input
+                              type="checkbox"
+                              checked={rulesBatchProjects.has(p.path)}
+                              onChange={() => {
+                                setRulesBatchProjects(prev => {
+                                  const next = new Set(prev);
+                                  if (next.has(p.path)) next.delete(p.path); else next.add(p.path);
+                                  return next;
+                                });
+                              }}
+                              className="accent-[var(--accent)]"
+                            />
+                            <span className="text-[9px] text-[var(--text-primary)] truncate">{p.name}</span>
+                          </label>
+                        ))}
+                      </div>
+                      {rulesBatchProjects.size > 0 && (
+                        <div className="p-2 border-t border-[var(--border)]">
+                          <button
+                            onClick={() => batchInject(tmpl.id)}
+                            className="w-full text-[9px] px-2 py-1 bg-[var(--accent)] text-white rounded hover:opacity-90"
+                          >
+                            Apply to {rulesBatchProjects.size} project{rulesBatchProjects.size > 1 ? 's' : ''}
+                          </button>
+                        </div>
+                      )}
+                    </div>
+                  </div>
+                </div>
+              );
+            })() : (
+              <div className="flex-1 flex items-center justify-center text-[var(--text-secondary)]">
+                <p className="text-xs">Select a template or create a new one</p>
+              </div>
+            )}
+          </div>
+        </div>
+      )}
     </div>
   );
 }