Skip to content

feat(nix): install completions#947

Open
betaboon wants to merge 1 commit intoFission-AI:mainfrom
betaboon:feat-nix-flake-completion
Open

feat(nix): install completions#947
betaboon wants to merge 1 commit intoFission-AI:mainfrom
betaboon:feat-nix-flake-completion

Conversation

@betaboon
Copy link
Copy Markdown

@betaboon betaboon commented Apr 10, 2026

Summary by CodeRabbit

  • New Features
    • Shell completion support for Bash, Fish, and Zsh shells is now available and automatically installed.

@betaboon betaboon requested a review from TabishB as a code owner April 10, 2026 17:47
Copy link
Copy Markdown

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 624338c1-348a-46f6-a479-1f7dc043a0a5

📥 Commits

Reviewing files that changed from the base of the PR and between af0b341 and d98d602.

📒 Files selected for processing (1)
  • flake.nix

📝 Walkthrough

Walkthrough

The flake.nix file is updated to add shell completion generation as part of the build process. The changes include adding installShellFiles to build inputs and implementing a postInstall hook that generates and installs Bash, Fish, and Zsh completions for the openspec command while disabling telemetry during installation.

Changes

Cohort / File(s) Summary
Build Configuration
flake.nix
Added installShellFiles to nativeBuildInputs and introduced a conditional postInstall hook to generate and install shell completions (Bash, Fish, Zsh) for the openspec command, with telemetry disabled during the build process.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Shell completions hop into place,
Bash, Fish, and Zsh join the race,
A postInstall hook so neat,
Makes command-line tips complete!
Telemetry silenced with care,
Building binaries with flair! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat(nix): install completions' accurately describes the main change—adding shell completion installation to the Nix build configuration.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

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