Skip to content

build(deps): remediate all 67 security findings across root and test-project - #67

Merged
max-ostapenko merged 1 commit into
mainfrom
fix/security-remediations
Sep 11, 2026
Merged

build(deps): remediate all 67 security findings across root and test-project#67
max-ostapenko merged 1 commit into
mainfrom
fix/security-remediations

Conversation

@max-ostapenko

@max-ostapenko max-ostapenko commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes open Dependabot alerts and CodeQL alert (#6) across masthead-data/dataform-package, cleanly rebased on main following PR #65.

Key Changes

  1. Root Dependency Auditing:

    • Resolved all open alerts in package-lock.json (js-yaml, brace-expansion, browserslist, baseline-browser-mapping, @humanfs/node, @babel/core) via safe patch resolutions.
    • Verified root npm audit: 0 vulnerabilities.
  2. Test Project Upgrades:

    • Upgraded test-project/package.json to @dataform/core@^3.0.69 and @dataform/cli@3.0.69, eliminating critical vm2 sandbox breakouts, protobufjs, tmp, and uuid advisories.
    • Clean dependency management without using overrides.
  3. CI Workflow Permissions:

Verification

  • Rebased cleanly on main (incorporating Mike's builder interception from PR fix: skip reservation injection when preOps has an outer DECLARE #65).
  • Root npm audit: 0 vulnerabilities
  • npm run lint: Passed (ESLint 10.1.0)
  • npx jest: Passed (93/93 tests passed, 98.79% coverage)
  • ./scripts/test-matrix.sh: Passed across both Dataform 2.4.2 and 3.0.48

- Upgrade test-project dependencies to @dataform/core@^3.0.69 and @dataform/cli@3.0.69 without overrides
- Resolve open Dependabot security alerts across root and test-project lockfiles
- Add explicit contents: read permissions to CI workflow

Signed-off-by: Max Ostapenko <1611259+max-ostapenko@users.noreply.github.com>
@max-ostapenko
max-ostapenko force-pushed the fix/security-remediations branch from ad33eb5 to bbb6917 Compare September 11, 2026 10:47
@max-ostapenko
max-ostapenko merged commit da4432f into main Sep 11, 2026
7 checks passed
@max-ostapenko
max-ostapenko deleted the fix/security-remediations branch September 11, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant