Skip to content

feat(option): update optnix dependency#142

Merged
water-sucks merged 1 commit intonix-community:mainfrom
water-sucks:update-optnix-dep
Dec 11, 2025
Merged

feat(option): update optnix dependency#142
water-sucks merged 1 commit intonix-community:mainfrom
water-sucks:update-optnix-dep

Conversation

@water-sucks
Copy link
Copy Markdown
Collaborator

@water-sucks water-sucks commented Dec 11, 2025 •
edited by coderabbitai bot
Loading

This PR updates optnix to v0.3.1.

v0.3.1 renames the module to snare.dev/optnix and adds a regex search mode to the TUI.

Summary by CodeRabbit

  • Chores
    • Switched the upstream source for a core dependency (no functional or API changes).
    • Updated package checksum used by the build system to ensure reproducible builds.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 11, 2025 •
edited
Loading

Walkthrough

This pull request migrates the optnix dependency from github.com/water-sucks/optnix to snare.dev/optnix: Go import paths and go.mod updated, and the nix package vendorHash recalculated. No functional code changes detected.

Changes

Cohort / File(s) Summary
Go import updates
cmd/option/completion.go, cmd/option/option.go		 Replaced imports from github.com/water-sucks/optnix/... and github.com/water-sucks/optnix/tui with snare.dev/optnix/... and snare.dev/optnix/tui; no logic changes.
Module dependency
go.mod		 Removed github.com/water-sucks/optnix v0.3.0 and added snare.dev/optnix v0.3.1 in the require block.
Nix build config
nix/package.nix		 Updated vendorHash value to the new sha256 checksum.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Focus review on:
    • Ensuring all import paths were correctly replaced and compile with the new module path.
    • Verifying go.mod and go.sum consistency (not shown in diff).
    • Confirming the updated vendorHash corresponds to the new dependency set.

Pre-merge checks and finishing touches

❌ 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%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main objective of updating the optnix dependency from water-sucks/optnix to snare.dev/optnix (v0.3.1), which is the primary change across all modified files.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2360305 and aa763b8.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (4)
  • cmd/option/completion.go (1 hunks)
  • cmd/option/option.go (1 hunks)
  • go.mod (1 hunks)
  • nix/package.nix (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (3)
  • nix/package.nix
  • cmd/option/completion.go
  • cmd/option/option.go
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Build/Test
🔇 Additional comments (1)
go.mod (1)

27-27: Import path migration from github.com/water-sucks/optnix to snare.dev/optnix is complete.

All Go files have been properly updated to use the new module path. No remnants of the old import path remain, and the new imports are correctly used in cmd/option/completion.go and cmd/option/option.go. The go.sum file is also properly updated with snare.dev/optnix v0.3.1 hashes.

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.

@water-sucks water-sucks merged commit a201978 into nix-community:main Dec 11, 2025
2 checks passed
@water-sucks water-sucks deleted the update-optnix-dep branch December 11, 2025 03:27
@coderabbitai coderabbitai bot mentioned this pull request Dec 21, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@water-sucks