You are now the owner of this repo.

5
93%
main: 93%

Ran 08 May 2026 09:23AM UTC

Files 1176

Run time 40s

Badge

Embed ▾

Committed 08 May 2026 09:10AM UTC coverage: 52.455%. Remained the same

Job # 25547290322.5

Build Type

push

github

Committed by

web-flow

Commit Message

fix(js): propagate append_only_caches to RunRequest for node_run_script/run_node_build_script (#23300)

# Problem

`pants run` with `node_run_script`/`run_node_build_script` always got a
fresh
ephemeral pnpm store because `RunRequest` was constructed without
`append_only_caches`. The pnpm `PackageManager` declares
`extra_caches={"pnpm_home": "._pnpm_home"}` which the Process carries,
but
`RunRequest` in run/rules.py dropped it. This caused pnpm to
reinstall all packages from scratch on every pants run invocation.
Similarly,
the yarn `PackageManager` declares:
`extra_caches={"yarn_cache": "._yarn_cache"}`.

# Solution

Propagate the `append_only_caches` value to `RunRequest` in
`run/rules.py`.

The `test` `package` and `lint` goals were unaffected — their code paths
correctly forward `append_only_caches` through Process -> named cache
machinery.

Adds a regression tests asserting pnpm_home/yarn_cache appears in
`RunRequest.append_only_caches` when a `node_run_script` or a
`node_build_script` target uses pnpm/yarn as the package manager.

# AI Disclosure

Claude Code was used to diagnose the problem, write the fix with my
input, and write
the regression tests

Coverage Stats

31987 of 60980 relevant lines covered (52.45%)

0.52 hits per line

pantsbuild / pants / 25547290322 / 5
93%
main: 93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job test_python_linux_arm64 - 25547290322.5

pantsbuild / pants / 25547290322 / 5 93% main: 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job test_python_linux_arm64 - 25547290322.5

pantsbuild / pants / 25547290322 / 5
93%
main: 93%

README BADGES
x