Skip to content

llm-wiki: init at 0.4.10 [LLM-assisted contribution]#521355

Draft
futile wants to merge 2 commits into
NixOS:masterfrom
futile:add-llm-wiki
Draft

llm-wiki: init at 0.4.10 [LLM-assisted contribution]#521355
futile wants to merge 2 commits into
NixOS:masterfrom
futile:add-llm-wiki

Conversation

@futile

@futile futile commented May 17, 2026

Copy link
Copy Markdown
Contributor

DISCLOSURE: THE COMMITS IN THIS PR WERE CREATED WITH LLM-ASSISTANCE.

I did not use any LLM whatsoever to write this PR description.
I have carefully reviewed the LLM output and understand what the new package is doing, and have done my best to make sure it's high-quality.
This includes having looked up things that weren't clear to me, and where necessary improved them and/or left comment(s) to clear up why things are being done.
I have (also using AI) looked up similar packages and existing conventions in this repo to determine the solutions and styles that are most idiomatic in nixpkgs.
I have done this just like I would have if I had written the code myself, or when doing a code review.

Please let me know if I missed something.
I am interested in learning, but this is the first new package I contribute to nixpkgs (I have contributed package updates etc. before), so I might have gotten something wrong.

I fully intend to take responsibility for this new package, and I don't intend to simply pass off reviewer feedback to an LLM or anything like that.

That said, let's move on to actual content of this PR.

Things done

This PR adds a new package, llm-wiki, which packages https://github.com/nashsu/llm_wiki, (mostly) building it from source.

I say "mostly" because the tool originally vendors a binary version of pdfium, which the package replaces with nixpkgs' pdfium-binary, but that is still a binary package (afaiu). That is the only binary part of the build (to the best of my knowledge).

I have only built this on x86_64-linux, but have made sure that it evals on all platforms the tool claims to support and that cargo-tauri.hook also supports.

This PR also adds myself to maintainer-list.nix, since I wasn't on it before.
The commit adding me to that is ordered before (older-than) the commit introducing the new package.

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
      • I did not think this change requires release notes, but please let me know if I should include them.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Since this an LLM-assisted contribution, I looked for guidance/policy around that, and found #514587. Following that (even though it's not yet merged, I thought it best to follow it nevertheless), I added the appropriate Assisted-by: git trailers, and am including the new checkbox that the PR suggests as well:

Assisted-by: Codex:gpt-5.5-medium
@nixpkgs-ci nixpkgs-ci Bot added 8.has: package (new) This PR adds a new package 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels May 17, 2026
Assisted-by: Codex:gpt-5.5-medium
@futile futile marked this pull request as draft May 18, 2026 16:32
@futile

futile commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

Actually, the app is not that great. While I created this derivation to use it myself, I'm not sure it's actually a good app that I want to be responsible for. And I have no idea if I'll even continue using it, so I'll put this PR into draft-mode for now, so hopefully nobody spends time on this until I decide what I want to do. At the same time, it's here if someone wants/needs it 🤷

@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant