Skip to content

feat(setup): expose the rs setup command - #83

Merged
chenjiahan merged 1 commit into
mainfrom
chenjiahan/feat-rs-setup-command
Jul 28, 2026
Merged

feat(setup): expose the rs setup command#83
chenjiahan merged 1 commit into
mainfrom
chenjiahan/feat-rs-setup-command

Conversation

@chenjiahan

Copy link
Copy Markdown
Member

This PR exposes the default Git hook installer through rs setup, making the installer available from the CLI without loading Rstack config or creating user hooks. It adds command-specific help, strict option validation, and concise output and exit-code handling for installed, unchanged, skipped, and failed results. The command remains non-interactive and introduces no runtime dependencies.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying rstack-cli with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8b2ff5c
Status: ✅  Deploy successful!
Preview URL: https://66c13ba2.rstack-cli.pages.dev
Branch Preview URL: https://chenjiahan-feat-rs-setup-com.rstack-cli.pages.dev

View logs

@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7046089f-7fdb-4245-8eb1-9aba34097efa

📥 Commits

Reviewing files that changed from the base of the PR and between d8964a6 and 8b2ff5c.

📒 Files selected for processing (3)
  • packages/rstack/src/cli/commands.ts
  • packages/rstack/src/setup/index.ts
  • packages/rstack/tests/cli/setup/index.test.ts

📝 Walkthrough

Walkthrough

Adds an rstack setup command with help output, strict option parsing, and Git hook installation status handling. The command reports installed, unchanged, skipped, and error outcomes. CLI dispatch and help text are updated. Tests cover help aliases, unknown options, hook installation and idempotence, configuration isolation, non-Git directories, and missing Git executables.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding a setup command to the CLI.
Description check ✅ Passed The description matches the implemented setup command, help text, validation, and output behavior.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chenjiahan/feat-rs-setup-command

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8b2ff5c93c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/rstack/src/setup/index.ts
@chenjiahan
chenjiahan merged commit e8373a5 into main Jul 28, 2026
4 checks passed
@chenjiahan
chenjiahan deleted the chenjiahan/feat-rs-setup-command branch July 28, 2026 14:23
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.

1 participant