Skip to content

Commit 886e34b

Browse files
committed
fix: Add explanatory comments to except clauses
Address Copilot AI review feedback: - Add comments explaining why exceptions are silently ignored
1 parent c71cea7 commit 886e34b

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

skills/skill-finder/scripts/search_skills.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -307,6 +307,8 @@ def update_source_skills(index: Dict, source_id: str, repo_full: str) -> None:
307307
print(" ❌ GitHub CLI (gh) not found.")
308308
break
309309
except Exception:
310+
# Silently ignore other errors (JSON parse errors, network issues, etc.)
311+
# and continue checking other paths
310312
pass
311313

312314
# If no skills/ directory found, check root for SKILL.md in subdirectories
@@ -340,6 +342,8 @@ def update_source_skills(index: Dict, source_id: str, repo_full: str) -> None:
340342
except subprocess.TimeoutExpired:
341343
print(" ⚠️ Timeout checking root")
342344
except Exception:
345+
# Silently ignore errors when checking root directory
346+
# (e.g., rate limits, permission issues)
343347
pass
344348

345349
# Add found skills to index

0 commit comments

Comments
 (0)