#14469: feat: Add automatic pnpm install to shell hook
stale
size: S
Cluster:
Hooks Enhancements and Fixes
- Added 'pnpm install' command to the shellHook in flake.nix
- This ensures all project dependencies are automatically installed when entering the development environment
- Improves developer experience and ensures consistent setup
<!-- greptile_comment -->
<h2>Greptile Overview</h2>
<h3>Greptile Summary</h3>
This PR adds a new `flake.nix` that defines an OpenClaw `devShell` with Node/TypeScript/pnpm, and a `shellHook` that sets the npm registry and runs `pnpm install` when entering `nix develop`.
The main concern is that the change is much broader than the PR description: it introduces a full flake with additional (currently unused) inputs/modules (process-compose/services-flake/northwind) and a demo-style description. Additionally, the `shellHook` has persistent side effects (`npm set registry ...` writes to user config) and runs installs unconditionally on every shell entry, which can break offline/restricted environments and unexpectedly mutate the repo.
<h3>Confidence Score: 2/5</h3>
- This PR is not safe to merge as-is due to broad, likely unintended dev-environment changes and persistent side effects in the shell hook.
- The change introduces a full new flake with unrelated/demo inputs beyond the stated goal, and the dev shell hook persistently modifies user npm config and runs `pnpm install` on every entry (network + working-tree mutations). These are high-friction behaviors that would affect all Nix dev-shell users.
- flake.nix
<!-- greptile_other_comments_section -->
<sub>(2/5) Greptile learns from your feedback when you react with thumbs up/down!</sub>
**Context used:**
- Context from `dashboard` - CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=fd949e91-5c3a-4ab5-90a1-cbe184fd6ce8))
- Context from `dashboard` - AGENTS.md ([source](https://app.greptile.com/review/custom-context?memory=0d0c8278-ef8e-4d6c-ab21-f5527e322f13))
<!-- /greptile_comment -->
Most Similar PRs
#9914: fix(hooks): resolve bundled hook dist paths and packaging checks
by zimmra · 2026-02-05
75.7%
#22425: chore: make prepare git hooks setup cross-platform
by OldFineDev · 2026-02-21
74.0%
#6235: fix: from source development command
by witcxc · 2026-02-01
73.3%
#3973: fix: enhance npm package integrity checks and include critical files
by YeSuX · 2026-01-29
73.3%
#11951: Prepending Windows Node to PATH didn’t help
by tranhoangtu-it · 2026-02-08
73.3%
#11817: fix(build): compile bundled hook handlers into dist
by AnonO6 · 2026-02-08
73.0%
#13655: acceptAdd nixpacks configuration for Python and Node.js
by Manosage67 · 2026-02-10
72.7%
#12804: fix(daemon): use wrapper script for pnpm global installs in service...
by odinho · 2026-02-09
72.0%
#3006: fix(scripts): use local pnpm binary in run-node.mjs if global is mi...
by elliotsecops · 2026-01-27
71.0%
#7404: feat(slack): add Socket Mode bot + auto-join + bridge scaffolding; ad…
by zprager · 2026-02-02
71.0%