Skip to content

feat(cli): implement init command with platform selection #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
CalvinAllen merged 1 commit into from
Jan 9, 2026
Merged

feat(cli): implement init command with platform selection #35

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
151 changes: 151 additions & 0 deletions .github/workflows/integration-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -225,6 +225,157 @@ jobs:
fi
echo "✅ Nested task delegation passed"

# ==================== Init Command Tests ====================

- name: "Test: init --current-platform-only"
shell: bash
run: |
INIT_DIR=$(mktemp -d)
cd "$INIT_DIR"
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --current-platform-only

# Verify files were created
if [ ! -f ".rnr/config.yaml" ]; then
echo "ERROR: .rnr/config.yaml not created"
exit 1
fi
if [ ! -d ".rnr/bin" ]; then
echo "ERROR: .rnr/bin directory not created"
exit 1
fi
if [ ! -f "rnr.yaml" ]; then
echo "ERROR: rnr.yaml not created"
exit 1
fi
if [ ! -f "rnr" ]; then
echo "ERROR: rnr wrapper not created"
exit 1
fi
if [ ! -f "rnr.cmd" ]; then
echo "ERROR: rnr.cmd wrapper not created"
exit 1
fi

# Verify config has exactly one platform
PLATFORM_COUNT=$(grep -c "^-" .rnr/config.yaml || echo "0")
if [ "$PLATFORM_COUNT" != "1" ]; then
echo "ERROR: Expected 1 platform, got $PLATFORM_COUNT"
exit 1
fi

echo "✅ init --current-platform-only passed"
rm -rf "$INIT_DIR"

- name: "Test: init --platforms with multiple platforms"
shell: bash
run: |
INIT_DIR=$(mktemp -d)
cd "$INIT_DIR"
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --platforms linux-amd64,macos-arm64,windows-amd64

# Verify config has the right platforms
if ! grep -q "linux-amd64" .rnr/config.yaml; then
echo "ERROR: linux-amd64 not in config"
exit 1
fi
if ! grep -q "macos-arm64" .rnr/config.yaml; then
echo "ERROR: macos-arm64 not in config"
exit 1
fi
if ! grep -q "windows-amd64" .rnr/config.yaml; then
echo "ERROR: windows-amd64 not in config"
exit 1
fi

# Verify binaries exist
if [ ! -f ".rnr/bin/rnr-linux-amd64" ]; then
echo "ERROR: rnr-linux-amd64 binary not created"
exit 1
fi
if [ ! -f ".rnr/bin/rnr-macos-arm64" ]; then
echo "ERROR: rnr-macos-arm64 binary not created"
exit 1
fi
if [ ! -f ".rnr/bin/rnr-windows-amd64.exe" ]; then
echo "ERROR: rnr-windows-amd64.exe binary not created"
exit 1
fi

echo "✅ init --platforms passed"
rm -rf "$INIT_DIR"

- name: "Test: init --add-platform and --remove-platform"
shell: bash
run: |
INIT_DIR=$(mktemp -d)
cd "$INIT_DIR"

# First init with one platform
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --platforms linux-amd64

# Add a platform
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --add-platform macos-arm64

if ! grep -q "macos-arm64" .rnr/config.yaml; then
echo "ERROR: macos-arm64 not added to config"
exit 1
fi
if [ ! -f ".rnr/bin/rnr-macos-arm64" ]; then
echo "ERROR: rnr-macos-arm64 binary not created"
exit 1
fi

# Remove a platform
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --remove-platform linux-amd64

if grep -q "linux-amd64" .rnr/config.yaml; then
echo "ERROR: linux-amd64 should have been removed from config"
exit 1
fi
if [ -f ".rnr/bin/rnr-linux-amd64" ]; then
echo "ERROR: rnr-linux-amd64 binary should have been removed"
exit 1
fi

echo "✅ init --add-platform and --remove-platform passed"
rm -rf "$INIT_DIR"

- name: "Test: init --show-platforms"
shell: bash
run: |
INIT_DIR=$(mktemp -d)
cd "$INIT_DIR"
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --platforms linux-amd64,windows-amd64

OUTPUT=$($GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --show-platforms 2>&1)
if ! echo "$OUTPUT" | grep -q "linux-amd64"; then
echo "ERROR: --show-platforms should list linux-amd64"
exit 1
fi
if ! echo "$OUTPUT" | grep -q "windows-amd64"; then
echo "ERROR: --show-platforms should list windows-amd64"
exit 1
fi

echo "✅ init --show-platforms passed"
rm -rf "$INIT_DIR"

- name: "Test: init refuses to remove last platform"
shell: bash
run: |
INIT_DIR=$(mktemp -d)
cd "$INIT_DIR"
$GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --platforms linux-amd64

# Try to remove the only platform - should fail
if $GITHUB_WORKSPACE/tests/${{ matrix.binary }} init --remove-platform linux-amd64 2>&1; then
echo "ERROR: Should not be able to remove last platform"
exit 1
fi

echo "✅ init correctly refuses to remove last platform"
rm -rf "$INIT_DIR"

# ==================== Error Cases ====================

- name: "Test: nonexistent task (should fail)"
Expand Down
4 changes: 4 additions & 0 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ thiserror = "1"
# Cross-platform support
dirs = "5"

# Interactive prompts
dialoguer = "0.11"
console = "0.15"

# HTTP client for init/upgrade
reqwest = { version = "0.12", features = ["blocking"], default-features = false, optional = true }

Expand Down
Loading