Skip to content

Conversation

@NobbZ
Copy link
Owner

@NobbZ NobbZ commented Oct 3, 2025

Summary by CodeRabbit

  • Chores

    • Updated pinned plugin sources to newer releases, bringing upstream fixes and improvements.
    • Cleaned up Nix inputs to simplify dependency resolution.
  • Refactor

    • Adjusted Neovide wrapper to use prepend flags for more consistent startup options.

NobbZ added 22 commits October 3, 2025 09:56
Flake lock file updates:

• Updated input 'gen-luarc':
    'github:mrcjkb/nix-gen-luarc-json/b36b69c4ded9f31b079523bc452e23458734cf00?narHash=sha256-Q4tgm8ZHAQUdvsNft86MqIbHQAm7OF7RT/wwYWXqSdY%3D' (2024-08-19)
  → 'github:mrcjkb/nix-gen-luarc-json/1865b0ebb753ae5324d7381b1fa8c98c04ec7509?narHash=sha256-xVKfjFwc0zMbLMjLTiHz%2B0llggkjs93SmHkhaa9S3M4%3D' (2025-08-16)
• Updated input 'nixpkgs':
    'github:nixos/nixpkgs/3fcbdcfc707e0aa42c541b7743e05820472bdaec?narHash=sha256-JBXbjJ0t6T6BbVc9iPVquQI9XSXCGQJD8c8SgnUquus%3D' (2025-05-17)
  → 'github:nixos/nixpkgs/dc704e6102e76aad573f63b74c742cd96f8f1e6c?narHash=sha256-O7eHcgkQXJNygY6AypkF9tFhsoDQjpNEojw3eFs73Ow%3D' (2025-10-02)
• Updated input 'parts':
    'github:hercules-ci/flake-parts/c621e8422220273271f52058f618c94e405bb0f5?narHash=sha256-hIshGgKZCgWh6AYJpJmRgFdR3WUbkY04o82X05xqQiY%3D' (2025-04-01)
  → 'github:hercules-ci/flake-parts/758cf7296bee11f1706a574c77d072b8a7baa881?narHash=sha256-wfG0S7pltlYyZTM%2BqqlhJ7GMw2fTF4mLKCIVhLii/4M%3D' (2025-10-01)
• Updated input 'wrapper-manager':
    'github:viperml/wrapper-manager/f2b0bec5140403cf24cae96f6764d97b5d59b0e2?narHash=sha256-XfF%2B5zjoWbAcAiGNb%2B%2Bog6yDGJRwWYpADr9P6WuieQA%3D' (2025-05-18)
  → 'github:viperml/wrapper-manager/801dd9c876fcada046af45543e8c7e0bbccf20ea?narHash=sha256-OGXsTE5jWhGiFfK6OwMvjksrYSobsIFUSUzKsexCDxY%3D' (2025-09-06)
• Removed input 'wrapper-manager/nixpkgs'
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 3, 2025

Walkthrough

Removes a nixpkgs follow for wrapper-manager in flake.nix, updates numerous pinned sources in npins/sources.json (including a version bump for nvim-augment), and switches Neovide wrapper config from flags to prependFlags in nvide.nix.

Changes

Cohort / File(s) Summary
Flake input tweak
flake.nix
Removed wrapper-manager.inputs.nixpkgs.follows = "nixpkgs" from inputs.
Pins refresh
npins/sources.json
Updated many plugin pins: new revisions, tarball URLs, and hashes; nvim-augment version bumped from v0.31.0 to v0.41.0. No logic changes.
Neovide wrapper flag rename
nvide.nix
Replaced wrappers.neovide.flags with wrappers.neovide.prependFlags.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • add neovide #44: Introduced the Neovide wrapper using wrappers.neovide.flags, which this PR updates to prependFlags.
  • update 2025-05-19 #65: Large npins update similar in scope, modifying revisions/URLs/hashes across many plugins.
  • update 2025-04-13 #57: Another mass update to npins/sources.json with overlapping pin entries.

Poem

A rabbit tweaks the flakes with care,
Pins hop ahead—new hashes everywhere.
Neovide’s flags now lead the run,
prepend, not pack—configuration done.
I thump my paw: “All set to go!”
Let code fields grow, then swiftly flow. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The title “update 2025-10-03” is too vague and only states a date without summarizing the content of the changes, which include broad plugin pin updates and nix wrapper adjustments, so it fails to meet clarity and specificity requirements. Please rename the pull request to a concise, descriptive phrase that summarizes the main changes, for example “Bump plugin pins to latest revisions and adjust Neovide wrapper flags,” so the purpose of the update is immediately clear.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-2025-10-03

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3719c9a and e5e740b.

⛔ Files ignored due to path filters (1)
  • flake.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • flake.nix (0 hunks)
  • npins/sources.json (20 hunks)
  • nvide.nix (1 hunks)
💤 Files with no reviewable changes (1)
  • flake.nix
🔇 Additional comments (3)
npins/sources.json (2)

1-462: LGTM! Routine dependency updates.

The pin updates follow a consistent pattern with updated revisions, URLs, and corresponding integrity hashes. The JSON structure is well-formed and all entries maintain the expected schema.


53-56: Confirm target repository: Is this update for the Augment Neovim plugin (augmentcode/augment.vim) or a different “nvim-augment” project? Once confirmed, I’ll fetch and compare the CHANGELOG between v0.31.0 and v0.41.0.

nvide.nix (1)

13-13: No semantic change: using prependFlags aligns with the new API
wrapper-manager deprecated flags in favor of prependFlags (with flags now an alias), so behavior remains unchanged.


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.

@NobbZ NobbZ merged commit 379188b into main Oct 3, 2025
3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Nov 9, 2025
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.

2 participants