fix(nix): resolve build failures in the nix flake

[Frost-Phoenix]

This pr resolves two build failures when using the nix flake.

flake inputs

Some pkgs where outdated, thus causing a build failure.
Fixed by updating the flake.lock file.

Check phase

After building the pkg, the tests are ran in the check phase. This phase failed because it required some pkgs that the flake was not providing: zoxide and exiftool.
Fixed by adding those pkgs to the nativeCheckInputs.

Summary by CodeRabbit

• Chores

  • Updated build/check dependencies for the superfile package, adding additional tools to the validation environment. This affects development and CI only and does not alter runtime behavior or the user experience.

• Tests

  • Expanded the check environment with supplementary tools used during package checks. No user-facing changes.

[coderabbitai]

Walkthrough

Expands packages.superfile.nativeCheckInputs in flake.nix from a single hook to include zoxide and exiftool alongside writableTmpDirAsHomeHook, using a with pkgs; scope.

Changes

Cohort / File(s)	Summary of modifications
Nix flake configuration flake.nix	Broadened packages.superfile.nativeCheckInputs from [ pkgs.writableTmpDirAsHomeHook ] to with pkgs; [ zoxide exiftool writableTmpDirAsHomeHook ]

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant Nix as Nix Evaluator
  participant Build as superfile Build
  participant Ck as nativeCheckInputs

  Dev->>Nix: nix build .#packages.superfile
  Nix->>Build: Evaluate derivation
  Build->>Ck: Resolve nativeCheckInputs
  Note right of Ck: zoxide<br/>exiftool<br/>writableTmpDirAsHomeHook
  Ck-->>Build: Provide check-time tools/hooks
  Build-->>Dev: Build/check completes

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: outdated and broken nix flake #846 — Also modifies packages.superfile.nativeCheckInputs in flake.nix, adding writableTmpDirAsHomeHook; closely related to this expansion.

Suggested labels

awaiting pr review

Suggested reviewers

• yorukot

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The PR title "fix(nix): resolve build failures in the nix flake" is concise, directly describes the primary intent (fixing build failures in the Nix flake) and matches the changes in the PR (updating flake.lock and adding nativeCheckInputs such as zoxide and exiftool), so it clearly communicates the main change to reviewers.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

Poem

I hop through flakes with nimble might,
Adding tools to checks at night—
Zoxide, exif’s careful sight,
A writable home to make it right.
Thump-thump! The build now gleams so bright. 🐇✨

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

flake.nix (1)

52-66: Add test-only tools to the devShell for parity

Include zoxide and exiftool in the devShell so contributors can run tests locally without missing binaries.

   packages = with pkgs; [
     ## golang
     delve
     go-outline
     go
     golangci-lint
     gopkgs
     gopls
     gotools
     nix
     gomod2nix
     nixpkgs-fmt
+    zoxide
+    exiftool
   ];

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8b31697 and 3b764c1.

⛔ Files ignored due to path filters (1)

• flake.lock is excluded by !**/*.lock

📒 Files selected for processing (1)

• flake.nix (1 hunks)

⏰ 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 and Test (windows-latest)

🔇 Additional comments (1)

flake.nix (1)

34-38: LGTM: correct use of nativeCheckInputs for test-only tools

Adding zoxide and exiftool here is the right fix for the checkPhase; scoping with with pkgs; is fine and consistent.

[lazysegtree]

[lazysegtree]

@Frost-Phoenix I dont know much about nix.

Could you please provide an successful output of running the commands to build and spf via nix.

Just to keep as a reference befor merging this PR.

[Frost-Phoenix]

$ nix --version
nix (Nix) 2.28.5

First build failure

This one is caused by outdated packages.
fixed in f15f4e6

small output

$ nix build .
error: builder for '/nix/store/h7v6lq5079smvg8fhzqhdsdjn1bp2fch-superfile-1.3.3.drv' failed with exit code 1;
       last 10 log lines:
       > Running phase: unpackPhase
       > unpacking source archive /nix/store/7m040rv22mgk09bgk0y5nkrxnyj38ds3-bpi1s269raw1f6bwyk3h6a2drs7aqxw6-source
       > source root is bpi1s269raw1f6bwyk3h6a2drs7aqxw6-source
       > Running phase: patchPhase
       > Running phase: updateAutotoolsGnuConfigScriptsPhase
       > Running phase: configurePhase
       > Running phase: buildPhase
       > Building subPackage .
       > go: downloading go1.25.0 (linux/amd64)
       > go: download go1.25.0 for linux/amd64: toolchain not available
       For full logs, run:
         nix log /nix/store/h7v6lq5079smvg8fhzqhdsdjn1bp2fch-superfile-1.3.3.drv

Second build failure

This one is caused by missing pkgs (zoxide and exiftool) in the check phase.
fixed in 3b764c1

small output

$ nix build .
error: builder for '/nix/store/0mxaj9hbspg5dwx3pn1xyrzgh2yg1hpc-superfile-1.3.3.drv' failed with exit code 1;
       last 25 log lines:
       > time=2025-09-13T08:39:07.659Z level=ERROR msg="Error while read superfile data" error="open /build/.home/.local/share/superfile/pinned.json: no such file or directory"
       > === RUN   TestChooserFile/Open_with_file_editor_with_Invalid_chooser_file
       > time=2025-09-13T08:39:07.660Z level=ERROR msg="Error while read superfile data" error="open /build/.home/.local/share/superfile/pinned.json: no such file or directory"
       > time=2025-09-13T08:39:07.661Z level=INFO msg="Using default terminal cell size" os=linux
       > time=2025-09-13T08:39:07.661Z level=INFO msg="Terminal cell size detection" pixels_per_column=10 pixels_per_row=20
       > time=2025-09-13T08:39:07.661Z level=DEBUG msg="model.Update() called" msgType=tea.WindowSizeMsg
       > time=2025-09-13T08:39:07.661Z level=ERROR msg="Error while read superfile data" error="open /build/.home/.local/share/superfile/pinned.json: no such file or directory"
       > time=2025-09-13T08:39:07.662Z level=DEBUG msg="Submitting file preview render request" id=0 path=/build/spf_testdir/TestChooserFile/dir1/file1.txt
       > time=2025-09-13T08:39:07.662Z level=DEBUG msg="model.Update() called" msgType=tea.KeyMsg
       > time=2025-09-13T08:39:07.662Z level=DEBUG msg=model.handleKeyInput msg=e typestr=runes runes=[101] type=-1 paste=false alt=false
       > time=2025-09-13T08:39:07.662Z level=DEBUG msg="model.handleKeyInput. model info. " filePanelFocusIndex=0 filePanel.isFocused=true filePanel.panelMode=browserMode typingModal.open=false notifyModel.open=false promptModal.open=false fileModel.renaming=false searchBar.focussed=false helpMenu.open=false firstTextInput=false focusPanel=nonePanelFocus
       > time=2025-09-13T08:39:07.662Z level=ERROR msg="Error while writing to chooser file, continuing with open via file editor" error="open /build/spf_testdir/TestChooserFile/non_existent_dir/file.txt: no such file or directory"
       > time=2025-09-13T08:39:07.662Z level=ERROR msg="Error while read superfile data" error="open /build/.home/.local/share/superfile/pinned.json: no such file or directory"
       > --- PASS: TestChooserFile (0.02s)
       >     --- PASS: TestChooserFile/Open_with_default_app_with_valid_chooser_file (0.00s)
       >     --- PASS: TestChooserFile/Open_with_file_editor_with_valid_chooser_file (0.00s)
       >     --- PASS: TestChooserFile/Open_with_directory_editor_valid_chooser_file (0.00s)
       >     --- PASS: TestChooserFile/Open_with_file_editor_with_Blank_chooser_file (0.00s)
       >     --- PASS: TestChooserFile/Open_with_file_editor_with_Invalid_chooser_file (0.00s)
       > === RUN   TestZoxide
       >     model_test.go:258: zoxide initialization failed
       > --- FAIL: TestZoxide (0.00s)
       > FAIL
       > FAIL    github.com/yorukot/superfile/src/internal       0.846s
       > FAIL
       For full logs, run:
         nix log /nix/store/0mxaj9hbspg5dwx3pn1xyrzgh2yg1hpc-superfile-1.3.3.drv

New output

After the two fixes, the build is succesful.

output

$ nix build .
# no output, superfile builds successfully 

@lazysegtree