fix(scripts): pass arguments through wrapper to install.sh

[pitzcarraldo]

Summary

Fix argument passing in the install wrapper script so that version specification works correctly when using bash -s VERSION.

Details

Problem: The wrapper script (scripts/install) was using bash -c "$(curl ...)" which doesn't forward arguments to the downloaded script. When users ran:

curl -fsSL https://clix.sh/install | bash -s v1.1.0

The version argument was lost and the latest version was installed instead.

Solution: Changed from:

bash -c "$(curl -fsSL https://clix.sh/install.sh)"

to:

bash <(curl -fsSL https://clix.sh/install.sh) "$@"

This properly forwards all arguments ($@) to install.sh.

Related Issues

Follow-up fix for #12

How to Validate

1. Test with local script:

   cat scripts/install | bash -s v1.1.0

2. Verify output shows "Installing Clix CLI v1.1.0..." (not "Fetching latest version...")
3. Test default behavior:

   cat scripts/install | bash

4. Verify output shows "Fetching latest version..."

Pre-Merge Checklist

Code Quality

• Code builds without errors (bun run build)
• Types check correctly (bun run typecheck)
• Linter passes (bun run lint)
• Tests pass (bun test)
• Added/updated tests for new functionality (if applicable)

Documentation

• Updated relevant documentation (if needed)
• Updated CLAUDE.md if architecture changed (if needed)

Commit Standards

• Commits follow Conventional Commits format
• No breaking changes, OR breaking changes are documented

Platform Validation

• macOS
• Linux

Summary by CodeRabbit

• Bug Fixes
  • Fixed the installation wrapper to properly forward command-line arguments to the install script, enabling version specification and other argument-based configurations during installation.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The install script wrapper in scripts/install is updated to use process substitution instead of embedding the command in a subshell. The new approach forwards all command-line arguments to the downloaded install script using "$@", changing from a bash -c invocation pattern to a direct bash execution with process substitution.

Changes

Cohort / File(s)	Change Summary
Install wrapper command scripts/install	Changed invocation from bash -c "$(curl ...)" to bash <(curl ...) "$@" to use process substitution and forward CLI arguments. Updated comment to reflect argument passing instead of environment variable emphasis.

Possibly related PRs

• feat(scripts): support version argument via bash -s #12: Updates install.sh to prioritize positional argument for CLIX_VERSION to work with the argument-forwarding mechanism introduced here.
• fix(scripts): export env vars to pass through piped execution #11: Modifies scripts/install wrapper to forward data from the wrapper into the downloaded install script with environment variables.
• fix(scripts): replace symlinks with wrapper scripts for install/uninstall #10: Previously introduced the curl-and-bash wrapper that exports CLIX_\* environment variables, which this PR refactors.

Suggested reviewers

• JeongwooYoo

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately and concisely summarizes the main change: passing arguments through the wrapper to install.sh.
Description check	✅ Passed	The pull request description is comprehensive and well-structured, including a clear problem statement, solution explanation, related issues, validation steps, and a completed pre-merge checklist.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}