Commit c0c1014
authored
fix: switch packageManager devEngines to warn + add minimumReleaseAge (#286)
Two related supply-chain hygiene changes:
**1. `devEngines.packageManager.onFail: error → warn`**
pnpm v10 still shells out to system npm for several subcommands (`pnpm
version`, `pnpm config`, etc.). This repo also has direct npm
invocations in CI:
- \`.github/workflows/tests.yaml:137\` — `npm install`
- \`.github/workflows/sync_branches.yaml:39\` — `npm i
@octokit/rest@release-19.x`
- \`.github/workflows/claude-md-maintenance.yml:31\` — `npm install -g
@anthropic-ai/claude-code`
With `onFail: error` those trip `EBADDEVENGINES`. `warn` keeps the
dev-visible signal without blocking. Also pins the version to `10.33.0`
for clarity.
**2. Add `minimumReleaseAge: 1440` to `pnpm-workspace.yaml`**
24-hour quarantine on new package versions. Mitigates compromised npm
packages that get discovered and yanked within the first day (shai-hulud
worm, nx self-replicator, etc.). Brings this repo in line with the rest
of the pnpm-migrated public repos.
Mirrors the rollout in apify/apify-client-js#895 + #896.1 parent ff1ff70 commit c0c1014
2 files changed
Lines changed: 8 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
| 14 | + | |
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
| 22 | + | |
| 23 | + | |
23 | 24 | | |
24 | 25 | | |
25 | 26 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
0 commit comments