fix: replace silent catch blocks with warn messages

5 silent catch {} blocks across skill-loader, skill-discovery,
and gemini adapter now log the actual error. This matches the
CONTRIBUTING.md standard: 'Error messages should help the user
fix the problem.'

Also improved the OpenAI tool stub comment to identify which
tool needs implementation.

Changed:
  - skill-loader.ts: parseSkillMd/loadSkillMetadata failures now warn
  - skill-discovery.ts: skill discovery/load failures now warn
  - gemini.ts: hook config parse failure now warns via console.warn
  - openai.ts: tool stub comment names the specific tool

Author: vystartasv

src/adapters/gemini.ts

@@ -372,7 +372,8 @@ function buildHooksConfig(agentDir: string): Record<string, any> | null {
     }
 
     return Object.keys(geminiHooks).length > 0 ? geminiHooks : null;
-  } catch {
+  } catch (err) {
+    console.warn(`Hook config parse failed: ${(err as Error).message}`);
     return null;
   }
 }

src/adapters/openai.ts

@@ -30,7 +30,7 @@ export function exportToOpenAI(dir: string): string {
       const funcName = tool.name.replace(/-/g, '_');
       lines.push(`def ${funcName}(${tool.params}):`);
       lines.push(`    """${tool.description}"""`);
-      lines.push(`    # TODO: Implement tool logic`);
+      lines.push(`    # Tool stub — replace with actual ${tool.name} implementation`);
       lines.push(`    pass\n`);
     }
   }

src/utils/skill-discovery.ts

@@ -2,6 +2,7 @@ import { existsSync, readdirSync } from 'node:fs';
 import { join, resolve } from 'node:path';
 import { homedir } from 'node:os';
 import { loadSkillMetadata, loadSkillFull, type SkillMetadata, type ParsedSkill } from './skill-loader.js';
+import { warn } from './format.js';
 
 export interface DiscoveredSkill {
   name: string;
@@ -76,8 +77,8 @@ export function discoverSkills(options: DiscoveryOptions): DiscoveredSkill[] {
           directory: meta.directory,
           source,
         });
-      } catch {
-        // Skip unparseable skills
+      } catch (err) {
+        warn(`Skill discovery failed: ${skillMdPath} — ${(err as Error).message}`);
       }
     }
   }
@@ -96,8 +97,8 @@ export function discoverAndLoadSkills(options: DiscoveryOptions): ParsedSkill[]
     const skillMdPath = join(disc.directory, 'SKILL.md');
     try {
       skills.push(loadSkillFull(skillMdPath));
-    } catch {
-      // Skip skills that fail to load
+    } catch (err) {
+      warn(`Skill load failed: ${skillMdPath} — ${(err as Error).message}`);
     }
   }
 

src/utils/skill-loader.ts

@@ -1,6 +1,7 @@
 import { readFileSync, existsSync, readdirSync } from 'node:fs';
 import { join } from 'node:path';
 import yaml from 'js-yaml';
+import { warn } from './format.js';
 
 /**
  * Agent Skills standard frontmatter — matches agentskills.io spec exactly.
@@ -125,8 +126,8 @@ export function loadAllSkills(skillsDir: string): ParsedSkill[] {
 
     try {
       skills.push(parseSkillMd(skillMdPath));
-    } catch {
-      // Skip skills that fail to parse
+    } catch (err) {
+      warn(`Skill parse failed: ${skillMdPath} — ${(err as Error).message}`);
     }
   }
 
@@ -150,8 +151,8 @@ export function loadAllSkillMetadata(skillsDir: string): SkillMetadata[] {
 
     try {
       skills.push(loadSkillMetadata(skillMdPath));
-    } catch {
-      // Skip skills that fail to parse
+    } catch (err) {
+      warn(`Skill metadata load failed: ${skillMdPath} — ${(err as Error).message}`);
     }
   }