## Summary

Ports 5 active hook tests + 2 skipped divergences from
[`pnpm/test/install/hooks.ts`](https://github.com/pnpm/pnpm/blob/main/pnpm/test/install/hooks.ts)
into the new
[test/pnpm_install_hooks.bats](https://github.com/endevco/aube/blob/claude/pnpm-import-hooks/test/pnpm_install_hooks.bats),
following the pattern established by
[#416](#416) (tracked in
`test/PNPM_TEST_IMPORT.md`).

**Active tests (all passing):**
- async `readPackage` mutates a transitive's deps — pnpm hooks.ts:43
- async `afterAllResolved` runs and is awaited — hooks.ts:498
- syntax error in `.pnpmfile.cjs` fails the install — hooks.ts:292
- `require()` of a missing module fails the install — hooks.ts:303
- `readPackage` setting optional/peer/dev fields on a transitive —
hooks.ts:528

**Skipped — surfaced aube divergences (worth filing as Discussions
before un-skipping):**
- `readPackage` returning `undefined` should fail the install
(hooks.ts:68); aube continues with the original manifest
- `readPackage` mutating the root project's `.dependencies` should
install those deps (hooks.ts:551); aube does not run the hook on the
root manifest

The remaining 15 tests in hooks.ts depend on `@pnpm.e2e/*` fixtures
(Phase 0) or `addDistTag` helper (Phase 2) and are tracked in the TODO
doc.

## Test plan

- [x] `mise run test:bats test/pnpm_install_hooks.bats` — 5 ok, 2 skip,
0 fail.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk: changes are limited to test suite additions and a tracking
doc update, with no production code impact. Main risk is potential test
flakiness due to lockfile/path assertions and pnpmfile error-message
matching.
> 
> **Overview**
> Ports a first batch of `pnpm/test/install/hooks.ts` coverage into
aube’s bats suite by adding `test/pnpm_install_hooks.bats`.
> 
> The new tests exercise `.pnpmfile.cjs` hook behavior (async
`readPackage` mutation of transitives, async `afterAllResolved` being
awaited) and failure paths when the pnpmfile has a syntax error or
`require()`s a missing module; two additional cases are added but marked
`skip` to document current aube divergences.
> 
> Updates `test/PNPM_TEST_IMPORT.md` to track the hooks port progress,
list completed/skipped cases, and note divergences/fixture-dependent
tests still pending.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
10406fa. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

## Summary

Ports 5 active hook tests + 2 skipped divergences from
[`pnpm/test/install/hooks.ts`](https://github.com/pnpm/pnpm/blob/main/pnpm/test/install/hooks.ts)
into the new
[test/pnpm_install_hooks.bats](https://github.com/endevco/aube/blob/claude/pnpm-import-hooks/test/pnpm_install_hooks.bats),
following the pattern established by
[#416](#416) (tracked in
`test/PNPM_TEST_IMPORT.md`).

**Active tests (all passing):**
- async `readPackage` mutates a transitive's deps — pnpm hooks.ts:43
- async `afterAllResolved` runs and is awaited — hooks.ts:498
- syntax error in `.pnpmfile.cjs` fails the install — hooks.ts:292
- `require()` of a missing module fails the install — hooks.ts:303
- `readPackage` setting optional/peer/dev fields on a transitive —
hooks.ts:528

**Skipped — surfaced aube divergences (worth filing as Discussions
before un-skipping):**
- `readPackage` returning `undefined` should fail the install
(hooks.ts:68); aube continues with the original manifest
- `readPackage` mutating the root project's `.dependencies` should
install those deps (hooks.ts:551); aube does not run the hook on the
root manifest

The remaining 15 tests in hooks.ts depend on `@pnpm.e2e/*` fixtures
(Phase 0) or `addDistTag` helper (Phase 2) and are tracked in the TODO
doc.

## Test plan

- [x] `mise run test:bats test/pnpm_install_hooks.bats` — 5 ok, 2 skip,
0 fail.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk: changes are limited to test suite additions and a tracking
doc update, with no production code impact. Main risk is potential test
flakiness due to lockfile/path assertions and pnpmfile error-message
matching.
> 
> **Overview**
> Ports a first batch of `pnpm/test/install/hooks.ts` coverage into
aube’s bats suite by adding `test/pnpm_install_hooks.bats`.
> 
> The new tests exercise `.pnpmfile.cjs` hook behavior (async
`readPackage` mutation of transitives, async `afterAllResolved` being
awaited) and failure paths when the pnpmfile has a syntax error or
`require()`s a missing module; two additional cases are added but marked
`skip` to document current aube divergences.
> 
> Updates `test/PNPM_TEST_IMPORT.md` to track the hooks port progress,
list completed/skipped cases, and note divergences/fixture-dependent
tests still pending.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
10406fa. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

## Summary

Ports the relevant tests from
[pnpm/test/install/lifecycleScripts.ts](https://github.com/pnpm/pnpm/blob/main/pnpm/test/install/lifecycleScripts.ts)
into
[test/lifecycle_scripts.bats](https://github.com/endevco/aube/blob/claude/lifecycle-scripts-port/test/lifecycle_scripts.bats),
per the plan in
[test/PNPM_TEST_IMPORT.md](https://github.com/endevco/aube/blob/claude/elated-tharp-beb760/test/PNPM_TEST_IMPORT.md)
(#416), and closes the parity gaps surfaced by the port.

## Test ports

**3 green ports** — orthogonal stdout-visibility checks that complement
aube's existing filesystem-marker tests by proving the script's echoed
output reaches the user through aube's progress UI:
- preinstall echo (pnpm L43)
- postinstall echo (pnpm L56)
- prepare echo on argumentless install (pnpm L95)

**3 ports that previously documented parity gaps**, now green:
- `npm_config_user_agent` exported to lifecycle scripts (pnpm L29)
- root `postinstall` is NOT triggered when adding a named dep (pnpm L69)
- root `prepare` is NOT triggered when adding a named dep (pnpm L82)

**2 extra coverage tests for `aube remove` / `aube update`** — the same
pnpm contract extended to all chained-call commands (added in response
to PR review).

## Gap fixes

### 1. `npm_config_user_agent` exported

`crates/aube-scripts/src/lib.rs` — adds `aube_user_agent()` returning
`"aube/<version> <os> <arch>"` (mirrors pnpm's format) and sets it via
`apply_script_settings_env`, which covers root + dep, jailed +
non-jailed paths. Dep build scripts that sniff this env var to detect
the running PM (husky, unrs-resolver, node-pre-gyp) now recognize aube
instead of falling back to npm-mode.

### 2. Root hooks gated to argumentless `aube install` only

pnpm's contract: root lifecycle hooks fire only on the literal `pnpm
install` invocation (no package args). Every other entry point — `add`,
`remove`, `update`, `dedupe`, `dlx`, patch tooling, the
`ensure_installed` auto-install before `run`/`test`, nested git-prepare
installs — must skip them.

- `crates/aube/src/commands/install/mod.rs` — adds
`InstallOptions.skip_root_lifecycle: bool` and gates both
`run_root_lifecycle` blocks on it. The chained-call constructor
`with_mode()` defaults to `true` so every chained site (every command
except argumentless `install`) inherits the correct behavior
automatically. Argumentless `aube install` (via
`InstallArgs::into_options`) and `ci.rs` / `deploy.rs` (literal struct)
keep `false`.

## Test plan

- [x] `mise run test:bats test/lifecycle_scripts.bats` — 21 ok, no skips
- [x] `cargo test -p aube-scripts` — 29 ok
- [x] Combined run across install, add, remove, update,
lifecycle_scripts, dedupe, patch, ci suites — 141 ok, 0 failures
- [x] Pre-commit hooks (cargo-fmt, cargo-clippy, shfmt, shellcheck) —
green

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Changes when root lifecycle hooks run across multiple commands and
adds a new env var for all lifecycle scripts, which may affect projects
relying on prior hook behavior or script environment.
> 
> **Overview**
> Improves pnpm parity for lifecycle scripts by exporting
`npm_config_user_agent` (Node-style platform/arch) to all spawned
scripts so dependency postinstalls can correctly detect aube.
> 
> Updates the install pipeline to *skip root*
`preinstall`/`install`/`postinstall`/`prepare` hooks for chained
installs (e.g. `add`/`remove`/`update`), while keeping them enabled for
argumentless `aube install`, `ci`, `deploy`, and nested git-dep
`prepare` installs.
> 
> Ports/extends pnpm lifecycle tests in `test/lifecycle_scripts.bats`
and adds a unit test for the new user-agent formatting.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
3a16e90. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>