codebase-memory-mcp: consume the kriswill nix fork instead of the sub-flake#19
Merged
Conversation
…-flake Replace the in-repo flakes/codebase-memory-mcp/ sub-flake (which fetched a pinned DeusData release and patched it at build time) with our kriswill/codebase-memory-mcp `nix` fork as a github input. The fork tracks upstream and carries the Nix work in source (nix symbols, flake-topology passes, cbm-with-ui + wrapper, darwin module), so we get upstream tracking + all upstream tooling + the cbm index in that repo. - flake.nix: input url -> github:kriswill/codebase-memory-mcp/nix (drop flake-parts follows) - remove flakes/codebase-memory-mcp/ (now lives in the fork) - packages/overlays/darwin module: comment refresh; attr names unchanged, wiring untouched Supersedes the patch-based approach (#18).
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the in-repo
flakes/codebase-memory-mcp/sub-flake (which fetched a pinned DeusData release tag and applied patches at build time) with our kriswill/codebase-memory-mcpnixfork consumed as a GitHub flake input.The fork branch tracks upstream (DeusData)
mainand carries all the Nix work directly in the source, which unlocks three wins: we track upstream changes viagit, we keep all of upstream's existing tooling + our Nix build advantages, and the cbm index now lives in that codebase so it can enhance work on itself.Supersedes #18 (the patch-based sub-flake approach).
The fork branch
github:kriswill/codebase-memory-mcp/nix(3 commits onupstream/main):source_expression→source_codefix +walk_nix_bindings(bindings → Variables named by attrpath) + Nix inwalk_defsdescend. (Upstream had already merged thefunction_expressionnaming, so only the still-missing half was ported.)pass_flakelock.c(flake.lock →Flake+DEPENDS_ON/FOLLOWS/MOUNTS, via the already-vendored yyjson, no Nix eval) andpass_nix_eval.c(nix flake show --json→FlakeOutput+PRODUCES, failure-tolerant + timeout-bounded).cbm-with-uibuild (graph-ui embedded),nix/git/coreutilswrapper,cbm-tools(cbm-ctl/cbm-daemon), and thedarwinModules.codebase-memory-mcplaunchd module — all in the fork'sflake.nix+nix/.What changed here
flake.nix: inputcodebase-memory-mcp.url→github:kriswill/codebase-memory-mcp/nix(dropped the now-absentflake-partsfollows).flakes/codebase-memory-mcp/(moved into the fork).modules/{packages,overlays}.nix+modules/darwin/codebase-memory-mcp.nix: comment refresh only — the attr names (packages.<sys>.codebase-memory-mcp,darwinModules.codebase-memory-mcp) are unchanged, so the re-export wiring is untouched.Verification
nix eval .#packages.aarch64-darwin.codebase-memory-mcp.name→codebase-memory-mcp-0.8.1-nixnix eval .#darwinConfigurations.k.config.kriswill.codebase-memory.enable→trueProgramArguments→…-cbm-tools-0.8.1-nix/bin/cbm-daemonnix build .#packages.aarch64-darwin.codebase-memory-mcp(via the fork input, under dotfiles' nixpkgs)Follow-up
CONFIGURESlinker) can now be developed in the fork with the cbm index available for the work.