Skip to content

Conversation

@pitzcarraldo
Copy link
Contributor

@pitzcarraldo pitzcarraldo commented Jan 28, 2026

Summary

Replace symlinks with wrapper scripts for install and uninstall to fix web server compatibility issues. Previously, curl -fsSL https://clix.sh/install | bash returned just the symlink target filename instead of the actual script content.

Details

  • Removed symlinks: scripts/installinstall.sh, scripts/uninstalluninstall.sh
  • Created wrapper scripts that download and execute the main .sh files
  • Environment variables (CLIX_VERSION, CLIX_INSTALL_DIR, etc.) are passed through to the main script

Supports multiple URL patterns:

  • https://clix.sh/install
  • https://clix.sh/install.sh
  • https://cli.clix.so/install
  • https://cli.clix.so/install.sh

Related Issues

N/A (bug fix discovered during usage)

How to Validate

  1. After deployment, verify all URL patterns work:
    curl -fsSL https://clix.sh/install | head -5
    curl -fsSL https://clix.sh/install.sh | head -5
  2. Both should output valid bash script content (shebang line)
  3. Test actual installation: curl -fsSL https://clix.sh/install | bash

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

New Features

  • Added installation script with support for custom installation directories and configurable environment options.
  • Added uninstallation script for easy system cleanup and removal.

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

…tall

Symlinks were not working correctly when served via web server, causing
`curl -fsSL https://clix.sh/install | bash` to fail. Replace symlinks
with wrapper scripts that download and execute the main .sh files.

Supports multiple URL patterns:
- https://clix.sh/install
- https://clix.sh/install.sh
- https://cli.clix.so/install
- https://cli.clix.so/install.sh

Co-Authored-By: Claude (global.anthropic.claude-opus-4-5-20251101-v1:0) <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Jan 28, 2026

Walkthrough

Adds two new wrapper shell scripts that download and execute remote installers from https://clix.sh, forwarding environment variables to control the installation and uninstallation behavior of the CLIX tool.

Changes

Cohort / File(s) Summary
Installation and Uninstallation Wrappers
scripts/install, scripts/uninstall
New shell wrapper scripts that download and execute remote scripts via curl. scripts/install defines and exports four environment variables (CLIX_VERSION, CLIX_INSTALL_DIR, CLIX_NO_MODIFY_PATH, CLIX_SKIP_CHECKSUM) to configure the remote installer. scripts/uninstall forwards CLIX_INSTALL_DIR to the remote uninstall script. Both scripts support multiple URL patterns for the remote resources.
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: replacing symlinks with wrapper scripts for install/uninstall functionality.
Description check ✅ Passed The PR description is comprehensive and well-structured, covering all template sections including summary, details, validation steps, and 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.

@pitzcarraldo pitzcarraldo self-assigned this Jan 28, 2026
@pitzcarraldo pitzcarraldo merged commit 02a2a23 into main Jan 28, 2026
5 checks passed
@pitzcarraldo pitzcarraldo deleted the feat/install-symlink-support branch January 28, 2026 06:54
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