Skip to content

feat: migrate devcontainer to Nix flake-based Docker image#58

Merged
sonesuke merged 1 commit intomainfrom
feat/nix-container
Apr 12, 2026
Merged

feat: migrate devcontainer to Nix flake-based Docker image#58
sonesuke merged 1 commit intomainfrom
feat/nix-container

Conversation

@sonesuke
Copy link
Copy Markdown
Owner

Summary

  • Replace .devcontainer/ (Dockerfile + devcontainer.json + post-create.sh) with a Nix flake-based Docker image (flake.nix) for reproducible builds
  • Add mise tasks (build, up, down, attach, setup) for container lifecycle management
  • Update scripts/setup.sh with git config, Claude CLI, MCP tools, and skill installation
  • Improve skill instructions: claim-analyzing (batch unmatched elements), evaluating (one-at-a-time claim decomposition), screening (require abstract fetch before judgment), search-feature (keyword matching)
  • Add Rust toolchain, Chromium compatibility symlink, and native build dependencies to the Nix flake

Test plan

  • mise run build succeeds and produces patent-kit:latest image
  • mise run up starts the container successfully
  • mise run setup completes without errors inside the container
  • mise run attach opens a working shell with all tools available
  • claude plugin validate . passes
  • Existing skill-bench tests pass (mise run test)

🤖 Generated with Claude Code

Switch nixpkgs to nixos-unstable, add rust-overlay for Rust toolchain,
symlink chromium as google-chrome, and include native build dependencies
(perl, gnumake, gcc, pkg-config, openssl.dev, lcov) for MCP tool compilation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sonesuke sonesuke force-pushed the feat/nix-container branch from d1e8361 to 067cc4f Compare April 12, 2026 07:23
@sonesuke sonesuke merged commit 53c7259 into main Apr 12, 2026
3 checks passed
@sonesuke sonesuke deleted the feat/nix-container branch April 12, 2026 07:25
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