fix: move claudeArgs to end of args array for slash command support #139
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
Running
happy /helpor any slash command didn't work. Commands weren't being interpreted by Claude.Root Cause
Claude CLI expects:
claude [options] [prompt]— the prompt must come last.Happy was building args like this:
The
/helpwas placed before--settings, so Claude didn't recognize it as the prompt.Fix
Moved
claudeArgsto be added after all flags:Now
/helpis at the end, Claude parses it as the prompt, and interprets/as a slash command.Changes
src/claude/claudeLocal.ts— reordered whenclaudeArgsgets pushed to the args array (6 lines moved, no logic change)Testing