feat(gh): add gh-stack extension - #2086
Conversation
|
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe GitHub CLI Home Manager configuration now installs ChangesGitHub CLI configuration
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds the gh-stack extension to the GitHub CLI (gh) configuration in Home Manager. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
![mesa-dot-dev[bot]](https://avatars.githubusercontent.com/in/1050077?s=60&v=4){kind=small}
There was a problem hiding this comment.
Performed full review of 4e2b2f8...402330c
Analysis
• Hard dependency on pkgs.gh-stack availability in pinned nixpkgs revision—if the package doesn't exist, evaluation will fail immediately and break users on certain pins
• No cross-version compatibility safeguards—if supporting multiple nixpkgs versions/channels, lack of conditional inclusion pattern creates brittle dependency chain
• Testing limited to installation visibility without functional verification—no validation that the gh-stack command actually executes properly after installation
Tip
Help
Slash Commands:
/review- Request a full code review/review latest- Review only changes since the last review/describe- Generate PR description. This will update the PR body or issue comment depending on your configuration/help- Get help with Mesa commands and configuration options
0 files reviewed | 1 comments | Edit Agent Settings • Read Docs
| programs.gh = { | ||
| enable = true; | ||
| extensions = with pkgs; [ gh-markdown-preview ]; | ||
| extensions = with pkgs; [ gh-markdown-preview gh-stack ]; |
There was a problem hiding this comment.
Before merging, verify that pkgs.gh-stack exists in your pinned nixpkgs revision. If the package is not available in the current pin, this will cause an evaluation error for all users during home-manager switch. You can test this by running a local build/switch with your current nixpkgs pin.
Agent: 🏛 Architecture •
Prompt for Agent
Task: Address review feedback left on GitHub.
Repository: shunkakinoki/dotfiles#2086
File: home-manager/programs/gh/default.nix#L5
Action: Open this file location in your editor, inspect the highlighted code, and resolve the issue described below.
Feedback:
Before merging, verify that `pkgs.gh-stack` exists in your pinned nixpkgs revision. If the package is not available in the current pin, this will cause an evaluation error for all users during `home-manager switch`. You can test this by running a local build/switch with your current nixpkgs pin.
Mesa DescriptionTL;DRAdded the What changed?
Test plan
Description generated by Mesa. Update settings |
1 participant
Summary
gh-stack(stacked PRs) to declarative gh extensions in home-managerTest plan
make switchactivates successfullygh extension listshowsgh-stackSummary by cubic
Add
gh-stackto the declarativeghextensions inhome-manager. This enables creating and managing stacked PRs from the GitHub CLI.Written for commit 402330c. Summary will update on new commits.