Skip to content

Harden skills with CLI checks, timeouts, and parallel execution#1

Open
schmug wants to merge 1 commit intomainfrom
claude/improve-coding-skills-5FkOy
Open

Harden skills with CLI checks, timeouts, and parallel execution#1
schmug wants to merge 1 commit intomainfrom
claude/improve-coding-skills-5FkOy

Conversation

@schmug
Copy link
Owner

@schmug schmug commented Feb 6, 2026

  • Add CLI availability verification (command -v) as first step in all
    skills, with actionable install instructions on failure
  • Wrap all external CLI calls with timeout 120 to prevent indefinite hangs
  • Use mktemp for codex temp files instead of fixed /tmp paths to avoid
    race conditions with concurrent invocations
  • Explicitly instruct /second-opinion to run Gemini and Codex queries as
    parallel Bash tool calls for ~2x faster wall-clock time
  • Improve context gathering guidance: structured prompt format with
    labeled code blocks, stdin piping for large context
  • Handle missing config files gracefully (check existence before reading)
  • Add graceful degradation to /second-opinion when only one CLI is available

https://claude.ai/code/session_01PVhFi3d3ptmxtHPCCdxu8A

- Add CLI availability verification (command -v) as first step in all
  skills, with actionable install instructions on failure
- Wrap all external CLI calls with timeout 120 to prevent indefinite hangs
- Use mktemp for codex temp files instead of fixed /tmp paths to avoid
  race conditions with concurrent invocations
- Explicitly instruct /second-opinion to run Gemini and Codex queries as
  parallel Bash tool calls for ~2x faster wall-clock time
- Improve context gathering guidance: structured prompt format with
  labeled code blocks, stdin piping for large context
- Handle missing config files gracefully (check existence before reading)
- Add graceful degradation to /second-opinion when only one CLI is available

https://claude.ai/code/session_01PVhFi3d3ptmxtHPCCdxu8A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants