Skip to content

Commit cc5716b

Browse files
committed
chore(deps): clear the vulnerable dependency tree and drop the overrides
The overrides block never reached anyone installing the CLI - copy-assets deletes it from the published manifest - so it protected this checkout and nothing else. Two of its six entries named packages that are not in the tree at all, three were obsolete, and the uuid entry actively pinned the whole tree down to a version the advisory covers. Fix the versions directly instead: jimp, lodash and uuid clear their own advisories, ts-morph 28 takes minimatch ^10 as a declared range rather than a semver-violating override, and sinon 22 clears the last dev-side one. universal-analytics goes with the GA4 migration, and with it the only bundleDependencies entry and the node_modules trees copy-assets mirrored into dist for it. Refreshing the lockfile clears eight further advisories - sigstore, its two helpers, xmldom, flatted, form-data, ip-address and picomatch - that were stale pins rather than outdated ranges. No lockfile ships, so these only ever affected development and CI. dotenv loads .env for the release scripts, and .gitignore keeps .env and the local Claude worktrees out of the repository.
1 parent 1a3c1f0 commit cc5716b

6 files changed

Lines changed: 470 additions & 588 deletions

File tree

.env.example

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
# Environment used to build and release the CLI.
2+
#
3+
# Copy to .env and fill in what you need; .env is gitignored, .env.example is not.
4+
# The build scripts load .env automatically, but a value already exported in the
5+
# environment always wins - CI secrets are never shadowed by a stray local file.
6+
7+
8+
# --- Analytics -------------------------------------------------------------
9+
# Measurement Protocol api secret, paired with the measurement id in
10+
# scripts/set-ga-id.js. Create one under
11+
# GA4 Admin -> Data Streams -> <stream> -> Measurement Protocol API secrets
12+
#
13+
# Deliberately not committed: this repository is public, so a committed secret
14+
# would make every contributor's `npm run pack.release` report into the
15+
# production property. Leave it unset locally - builds then report nothing.
16+
#
17+
# Set as the GA_API_SECRET repository secret for releases.
18+
GA_API_SECRET=
19+
20+
# Same, for the dev measurement id. Unused until a dev id is set in
21+
# scripts/set-ga-id.js.
22+
GA_API_SECRET_DEV=
23+
24+
25+
# --- Publishing ------------------------------------------------------------
26+
# Only needed when the USE_NPM_TOKEN repository variable is "true". The default
27+
# path is OIDC trusted publishing, which needs no token at all.
28+
NPM_PUBLISH_TOKEN=
29+
30+
# Used by the OpenSSF Scorecard workflow to read branch protection rules.
31+
SCORECARD_TOKEN=
32+
33+
# GITHUB_TOKEN is provided by Actions automatically and never needs setting here.

.gitignore

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,3 +92,13 @@ config/test-deps-versions-generated.json
9292

9393
# build output
9494
/dist
95+
96+
# local secrets, filled in from .env.example
97+
.env
98+
.env.*
99+
!.env.example
100+
101+
# Claude Code local state - worktrees and per-user settings. Ignoring the
102+
# contents rather than the directory keeps shared settings.json committable.
103+
.claude/*
104+
!.claude/settings.json

0 commit comments

Comments
 (0)