mirror of
https://github.com/davila7/claude-code-templates.git
synced 2026-09-19 01:30:23 +08:00
a56202407d
* feat(cli-rust): add Rust port of the CLI core with npm distribution + CI Adds a new `cli-rust/` crate (binary `cct`) that ports the CORE of the claude-code-templates CLI to Rust, alongside the existing Node `cli-tool/`. Native (byte-parity with the Node CLI, verified via `diff -r`): - Component installation: --agent/--command/--mcp/--setting/--hook/--skill and --workflow YAML, replicating the exact GitHub raw/API URLs, target paths (flat dir, category dropped), .mcp.json/settings/hooks merge semantics, .py/.sh sidecars, 2-space JSON + trailing newline. - Fire-and-forget tracking (3 endpoints, detached threads, env opt-out). Delegated to the Node CLI for now (commands/delegate.rs forwards argv via CCT_NODE_BIN or `npx claude-code-templates@latest`): dashboards, sandbox, global agents, stats, health-check, interactive setup. Parity gotcha handled: serde_json Map::remove does swap-remove under preserve_order; switched to shift_remove to match JS `delete` key order. Distribution: npm shim (npm/cct/bin/cct.js) execs a prebuilt binary from optionalDependencies (@davila7/cct-<os>-<arch>, esbuild pattern) so `npx` keeps working; plus Homebrew, cargo-binstall, and install.sh from GitHub Releases. build-rust-cli.yml builds all 5 targets on tag `cli-rust-v*`. Tests: 23 offline unit/integration tests + 2 network integration tests (#[ignore]). fmt + clippy clean. CI: rust-ci.yml runs fmt + clippy + tests only when cli-rust/** changes and posts a sticky PR comment with the results. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * fix(cli-rust): address cubic review (config safety, opt-out, packaging, CI) Fixes from the cubic automated review on PR #640: - install.rs: fail safe on a corrupt/unreadable existing .mcp.json/settings file instead of silently overwriting it (MCP aborts; setting/hook skip the location), matching Node's throw-on-bad-config behavior. - tracking.rs: honor both `=1` and `=true` as opt-out values (the README documents `=1`); Node only checks `=true`, so this is more privacy-safe. - cli.rs/main.rs: implement --dry-run for the native install path (prints the plan, writes nothing) instead of parsing the flag and ignoring it. - main.rs: run_prompt surfaces a non-zero `claude` exit code (a missing `claude` stays a soft warning since the install already succeeded). - delegate.rs: propagate signal-based termination as 128+signal on Unix instead of collapsing to exit code 1. - github.rs: warn when a skill file fails to download (mirrors Node's "Could not download" log) instead of skipping silently. - npm/build-packages.mjs: accept the release `.tgz` artifacts (extract them) in addition to unpacked binaries, matching build-rust-cli.yml output. - npm/cct/package.json: add top-level os/cpu so npm rejects unsupported platforms at install time instead of failing at runtime. - build-rust-cli.yml: add `permissions: contents: write` (gh-release) and Swatinem/rust-cache for the 5-target matrix. - tests/integration.rs: tighten the 2-space-indent assertion. Not changed (intentional parity with the Node CLI): - merge.rs new-hook-type copy: Node copies the incoming value as-is for a previously-absent hook type (no old-format normalization); kept identical. fmt + clippy clean; 23 offline + 2 network tests pass. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * fix(cli-rust): abort install on corrupt existing settings/hook config Follow-up to the cubic review: a corrupt/unreadable existing settings or hook file now fails the whole component install (returns 0) instead of skipping just that location, matching Node — where readJson throws out of the per-location loop and fails the component. Prevents a multi-location run from reporting success despite a bad target location. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * chore(cli-rust): prep v0.1.0 release via GitHub Releases + cargo-binstall Wire up the two opt-in distribution channels that don't touch the existing `claude-code-templates` npm package: - Cargo.toml: fix cargo-binstall pkg-url to the `cli-rust-v<version>` tag (was `v<version>`), so `cargo binstall --git ...` resolves the release asset. - install.sh: resolve the latest `cli-rust-v*` release instead of the generic /releases/latest, which could otherwise pick a Node-CLI `vX.Y.Z` release. - README: install table now lists the available preview channels (curl|sh, cargo-binstall via --git, from source) and marks Homebrew/npm as planned; Release section documents the merge -> tag cli-rust-v0.1.0 -> verify flow. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
115 lines
3.8 KiB
YAML
115 lines
3.8 KiB
YAML
name: Rust CI
|
|
|
|
# Runs ONLY when the Rust implementation changes. Checks formatting, lints, and
|
|
# tests, then posts (and updates) a single sticky comment on the PR with the
|
|
# results. The job status (the required check) reflects fmt + clippy + the
|
|
# offline test suite; the network integration tests are informational only.
|
|
|
|
on:
|
|
pull_request:
|
|
paths:
|
|
- 'cli-rust/**'
|
|
- '.github/workflows/rust-ci.yml'
|
|
|
|
permissions:
|
|
contents: read
|
|
pull-requests: write
|
|
|
|
concurrency:
|
|
group: rust-ci-${{ github.event.pull_request.number }}
|
|
cancel-in-progress: true
|
|
|
|
jobs:
|
|
test:
|
|
name: fmt + clippy + test
|
|
runs-on: ubuntu-latest
|
|
defaults:
|
|
run:
|
|
working-directory: cli-rust
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
|
|
- name: Install Rust toolchain
|
|
uses: dtolnay/rust-toolchain@stable
|
|
with:
|
|
components: rustfmt, clippy
|
|
|
|
- name: Cache cargo build
|
|
uses: Swatinem/rust-cache@v2
|
|
with:
|
|
workspaces: cli-rust
|
|
|
|
- name: Format check
|
|
id: fmt
|
|
continue-on-error: true
|
|
run: cargo fmt --check
|
|
|
|
- name: Clippy
|
|
id: clippy
|
|
continue-on-error: true
|
|
run: cargo clippy --all-targets -- -D warnings
|
|
|
|
- name: Test (offline)
|
|
id: test
|
|
continue-on-error: true
|
|
shell: bash
|
|
run: |
|
|
set -o pipefail
|
|
cargo test --no-fail-fast 2>&1 | tee "$GITHUB_WORKSPACE/rust-test-output.txt"
|
|
|
|
- name: Test (network integration)
|
|
id: test_net
|
|
continue-on-error: true
|
|
shell: bash
|
|
run: |
|
|
set -o pipefail
|
|
cargo test --no-fail-fast -- --ignored 2>&1 | tee "$GITHUB_WORKSPACE/rust-test-net-output.txt"
|
|
|
|
- name: Build PR comment
|
|
if: always()
|
|
shell: bash
|
|
run: |
|
|
icon() { [ "$1" = "success" ] && echo "✅ Passed" || echo "❌ Failed"; }
|
|
{
|
|
echo "## 🦀 Rust CI"
|
|
echo ""
|
|
echo "| Check | Result |"
|
|
echo "|---|---|"
|
|
echo "| \`cargo fmt --check\` | $(icon '${{ steps.fmt.outcome }}') |"
|
|
echo "| \`cargo clippy -D warnings\` | $(icon '${{ steps.clippy.outcome }}') |"
|
|
echo "| \`cargo test\` (offline) | $(icon '${{ steps.test.outcome }}') |"
|
|
echo "| \`cargo test -- --ignored\` (network) | $(icon '${{ steps.test_net.outcome }}') |"
|
|
echo ""
|
|
echo "<details><summary>Test summary</summary>"
|
|
echo ""
|
|
echo '```'
|
|
grep -E "running [0-9]+ test|test result:" "$GITHUB_WORKSPACE/rust-test-output.txt" 2>/dev/null || echo "no offline output"
|
|
echo "--- network ---"
|
|
grep -E "running [0-9]+ test|test result:" "$GITHUB_WORKSPACE/rust-test-net-output.txt" 2>/dev/null || echo "no network output"
|
|
echo '```'
|
|
echo "</details>"
|
|
echo ""
|
|
echo "_Commit \`${GITHUB_SHA:0:7}\` • workflow run [#${GITHUB_RUN_ID}](${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID})_"
|
|
} > "$GITHUB_WORKSPACE/rust-ci-comment.md"
|
|
|
|
- name: Post sticky PR comment
|
|
if: always()
|
|
uses: marocchino/sticky-pull-request-comment@v2
|
|
with:
|
|
header: rust-ci
|
|
path: rust-ci-comment.md
|
|
|
|
- name: Enforce check status
|
|
if: always()
|
|
shell: bash
|
|
run: |
|
|
fmt='${{ steps.fmt.outcome }}'
|
|
clippy='${{ steps.clippy.outcome }}'
|
|
test='${{ steps.test.outcome }}'
|
|
echo "fmt=$fmt clippy=$clippy test=$test (network is informational)"
|
|
if [ "$fmt" != "success" ] || [ "$clippy" != "success" ] || [ "$test" != "success" ]; then
|
|
echo "::error::One or more required Rust checks failed."
|
|
exit 1
|
|
fi
|
|
echo "All required Rust checks passed."
|