Skip to content

fix(deps): resolve Zod and basic-ftp security advisories - #115

Merged
atldays merged 4 commits into
mainfrom
develop
Oct 27, 2026
Merged

atldays merged 4 commits into
mainfrom
develop

Conversation

@atldays

@atldays atldays commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

This removes the framework's Zod security advisory and the six high audit findings inherited through release tooling.

  • Migrate validation to Zod 4.6.7, preserving accepted settings, optional fields, defaults and Sandbox timeout values. Add parser and HTML callback regressions.
  • Override get-uri's basic-ftp dependency to 6.2.3 without changing other dependency versions.
  • Run regular CI, coverage and publication on Node 24; retain Linux/Windows × Node 22/24 for PRs to main and pre-release validation. Node 22 remains supported.

Standard Zod error wording may change; validation remains enforced and framework field context and hints are preserved.

Validation:

  • npm run check passed on Node 22 and 24: 217 suites / 2523 tests on each version.
  • Installed tarball consumers passed builds and strict declaration checks on both versions.
  • FTP/PAC/HTTP/HTTPS checks and read-only release previews passed on both versions.
  • Repository audit: 0 high, one existing low esbuild finding; production dependencies: 0 findings.
  • Develop CI: all 12 jobs passed, including Linux/Windows Node 24, Chrome, Firefox and coverage.
  • Full PR CI: all 22 jobs passed across Linux/Windows Node 22/24, Chrome/Firefox and coverage. Main PR source policy also passed.

github-actions Bot and others added 4 commits October 27, 2026 18:48
Upgrade Zod to 4.6.7 to resolve GHSA-mrhx-c7xc-m3w5.
Adapt object schemas, records and diagnostics while preserving optional
fields and accepted Sandbox timeout values.

Add parser and HTML callback regressions. npm run check passes 2523 tests
on Node 22 and 24. Installed tarballs pass consumer builds, strict
declaration checks and clean audits on both versions.
Override basic-ftp to 6.2.3 within get-uri to remove six high audit findings.

Verified proxy behavior, release previews and npm run check on Node 22 and 24.
Keep the full Linux/Windows Node 22/24 matrix for PRs to main and before release.

Move coverage producers, artifacts and publication to Node 24.
@atldays
atldays merged commit c80dc0d into main Oct 27, 2026
36 checks passed
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