Manage dev environment with Nix flake and switch to aube#21
Merged
Conversation
Add a Nix flake providing Node.js 22 and aube (a fast Node.js package manager) so the dev toolchain is pinned via flake.lock. Update AGENTS.md to document `nix develop` and replace `npm run` commands with `aube run`. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
There was a problem hiding this comment.
Pull request overview
Adds a Nix flake–based development shell for this browser-based ROS bag analyzer, pinning toolchain inputs via flake.lock, and updates agent-facing documentation to use aube for dependency install / script execution.
Changes:
- Introduce
flake.nix+flake.lockto provide a pinned dev shell with Node.js 22 andaube. - Package
aubevia a Nix derivation that fetches a prebuilt release artifact. - Update
AGENTS.mdto documentnix developandaube-based commands.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| flake.nix | Defines the Nix flake outputs: aube package + default dev shell with Node.js 22 and aube. |
| flake.lock | Pins nixpkgs and flake-utils inputs for reproducible dev environments. |
| AGENTS.md | Updates developer/agent instructions to enter the Nix shell and use aube commands. |
Drop aube from flake.nix and pin it via mise.toml instead, matching how aube is typically managed in the wider ecosystem. The flake now only provides Node.js 22, which is the conventional Nix pattern for npm-based projects. npm remains a first-class option since aube is package-lock compatible. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
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
flake.locknpmtoaube(compatible with existingpackage-lock.json)nix developsetup andaubecommandsTest plan
nix developenters the dev shell with Node.js 22 and aube availableaube installinstalls dependenciesaube run build,aube run lint,aube run testall succeed🤖 Generated with Claude Code