Slipstream/auto_review
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases 27 Packages Activity Actions

chore(dev): add just command interface #224

Merged
jwilger merged 1 commit from chore/216-just-command-interface into main 2026-05-16 15:11:37 -07:00
Conversation 1 Commits 1 Files changed 10 +77 -52
jwilger commented 2026-05-16 14:29:50 -07:00
Owner
Copy link

Summary

  • Adds a Justfile with fmt, clippy, test, deny, build, and aggregate ci recipes.
  • Adds just to the Nix dev shell while keeping recipes as direct tool invocations.
  • Updates contributor/agent guidance, PR template, verification command docs, and lefthook to use the new just command surface.

Closes #216.

Verification

  • nix develop --command just --list
  • nix develop --command just fmt
  • nix develop --command just clippy
  • nix develop --command just test
  • nix develop --command just deny
  • nix develop --command just build
  • nix develop --command just ci

Note: host shell does not currently have just on PATH, so local verification used the Nix dev shell to prove nix develop provides just and the recipes work there.

## Summary - Adds a `Justfile` with `fmt`, `clippy`, `test`, `deny`, `build`, and aggregate `ci` recipes. - Adds `just` to the Nix dev shell while keeping recipes as direct tool invocations. - Updates contributor/agent guidance, PR template, verification command docs, and lefthook to use the new `just` command surface. Closes #216. ## Verification - `nix develop --command just --list` - `nix develop --command just fmt` - `nix develop --command just clippy` - `nix develop --command just test` - `nix develop --command just deny` - `nix develop --command just build` - `nix develop --command just ci` Note: host shell does not currently have `just` on PATH, so local verification used the Nix dev shell to prove `nix develop` provides `just` and the recipes work there.
chore(dev): add just command interface
All checks were successful
CI / Verify PR with nix flake check (pull_request) Successful in 3m42s
Details
CI / Request auto_review semantic review (pull_request) Successful in 2s
Details
CI / Build PR artifacts (no token) (pull_request) Successful in 3s
Details
CI / Publish PR artifact packages (pull_request) Successful in 2s
Details
auto_review auto_review: no findings
801a86cf34
 
Adopt Justfile recipes for routine checks, expose just in the Nix dev shell, and point contributor/agent guidance plus hooks at the new command surface.\n\nVerifications:\n- nix develop --command just --list\n- nix develop --command just fmt\n- nix develop --command just clippy\n- nix develop --command just test\n- nix develop --command just deny\n- nix develop --command just build\n- nix develop --command just ci
auto-review approved these changes 2026-05-16 14:33:41 -07:00
auto-review left a comment
Copy link

This PR introduces a Justfile to streamline development commands and updates documentation to reflect this change. The integration appears well-executed, with updates to various documentation files and the Nix shell configuration. The changes seem safe to merge.

Walkthrough

  • Justfile: Introduces a new Justfile with recipes for common tasks like fmt, clippy, test, deny, build, and ci. This simplifies command execution for developers.
  • Documentation Updates: Updates to .forgejo/pull_request_template.md, .opencode/commands/verify.md, .opencode/rules/toolchain.md, .opencode/rules/verification.md, .opencode/skills/rust-workspace-engineering/SKILL.md, AGENTS.md, and CONTRIBUTING.md to reflect the new just command usage.
  • Nix Configuration: Modifies flake.nix to include just in the development environment, ensuring that the Justfile is part of the build process.
  • Lefthook Configuration: Updates lefthook.yml to use just ci for pre-push checks, aligning with the new command interface.
This PR introduces a `Justfile` to streamline development commands and updates documentation to reflect this change. The integration appears well-executed, with updates to various documentation files and the Nix shell configuration. The changes seem safe to merge. ## Walkthrough - **Justfile**: Introduces a new `Justfile` with recipes for common tasks like `fmt`, `clippy`, `test`, `deny`, `build`, and `ci`. This simplifies command execution for developers. - **Documentation Updates**: Updates to `.forgejo/pull_request_template.md`, `.opencode/commands/verify.md`, `.opencode/rules/toolchain.md`, `.opencode/rules/verification.md`, `.opencode/skills/rust-workspace-engineering/SKILL.md`, `AGENTS.md`, and `CONTRIBUTING.md` to reflect the new `just` command usage. - **Nix Configuration**: Modifies `flake.nix` to include `just` in the development environment, ensuring that the `Justfile` is part of the build process. - **Lefthook Configuration**: Updates `lefthook.yml` to use `just ci` for pre-push checks, aligning with the new command interface.
jwilger deleted branch chore/216-just-command-interface 2026-05-16 15:11:37 -07:00
Sign in to join this conversation.
Reviewers
No reviewers
auto-review
Labels
Clear labels   
bug

   
discussion

   
enhancement
  
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
discussion
enhancement
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Slipstream/auto_review!224
No description provided.