Repository navigation
chore(ci): refresh the smoke-test npm lockfile and enable Dependabot for it - #855
Conversation
…for it Lockfile-only bump of the transitive undici (7.29.0 -> 7.30.1) and source-map-js (1.2.1 -> 1.2.2) pins under .github/scripts, clearing all seven open Dependabot alerts; jsdom stays at 29.1.1. Adds an npm ecosystem entry for /.github/scripts so future advisories on this CI-only toolchain arrive as fix PRs instead of orphaned alerts.
Overall assessment: ✅ Looks goodNo P0, P1, P2, or P3 findings identified in the supplied diff. Executive summary
MethodologyNo findings. Checked scope against Code QualityNo findings. Dependabot YAML parses correctly, and the npm entry specifies a valid manifest directory, weekly schedule, and minor/patch group ( The manifest and lockfile agree. Updated versions satisfy PerformanceNo findings. The changes affect the CI dependency set and update automation; no library performance paths change. MaintainabilityNo findings. The npm configuration follows the existing ecosystem entries and correctly distinguishes version updates from security updates. Tech DebtNo findings. The diff introduces no apparent deferred implementation work. SecurityNo findings in the diff. Updated entries retain official npm registry URLs and valid SHA-512 integrity-field formats. The harness context supports the stated local-file exposure assessment. Clearance of all seven Dependabot alerts was not independently verified. Documentation/TestsNo findings. The fragment follows the repository convention and accurately describes the changes ( Verified:
A fresh install, npm audit, and behavioral smoke test were not run because the environment is read-only with restricted network access. Pytest is unavailable. |
Summary
.github/scripts/package-lock.json: transitiveundici7.29.0 -> 7.30.1 andsource-map-js1.2.1 -> 1.2.2. This clears all seven open Dependabot alerts (six undici advisories, one source-map-js advisory).jsdomstays at 29.1.1 andpackage.jsonis unchanged; both new versions satisfy the existing^7.25.0/^1.2.1ranges.npmecosystem entry for/.github/scriptsto.github/dependabot.yml(weekly, grouped minor/patch) so the smoke-test toolchain gets version-update PRs instead of drifting into advisories between manual refreshes. Added thejavascriptlabel it references.Security+Internal.Exposure context: this lockfile only pins the Node toolchain for the docs search-excerpt smoke test in
docs-tests.yml. The harness loads locally built HTML from disk and performs no network fetches, so the undici HTTP paths named in the advisories never execute. Nothing in the published wheel is affected.Methodology references (required if estimator / math changes)
Validation
npm ci --ignore-scriptsagainst the refreshed lock installs cleanly (39 packages) andnpm audit --package-lock-onlyreports 0 vulnerabilities.changelog_compile.py checkpasses;tests/test_changelog_fragments.pypasses.docs-tests.ymlpath filter includes.github/scripts/**, so the search-excerpt smoke test runs against the refreshed lock once CI is gated on.Security / privacy
Changelog