Skip to content

Harden release and documentation sanitization - #55

Merged
GGULBAE merged 1 commit into
masterfrom
codex/close-codeql-sanitization-alerts
Aug 13, 2026
Merged

GGULBAE merged 1 commit into
masterfrom
codex/close-codeql-sanitization-alerts

Conversation

@GGULBAE

@GGULBAE GGULBAE commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace suffix-based npm attestation URL matching with strict URL parsing and exact decoded package@version identity
  • reject wrong origin, userinfo, query/fragment, malformed encoding, double encoding, and prefixed identities without throwing
  • replace multi-character Markdown heading tag removal with a state machine that cannot synthesize a new tag opener
  • add scoped-package and adversarial heading/URL regression coverage

Why

This removes the actual fail-open behavior behind the repository's two open high-severity CodeQL alerts instead of dismissing them as false positives.

Verification

  • VITEST_MAX_WORKERS=2 pnpm verify: 59 files / 546 tests
  • coverage: 94.08% statements, 86.12% branches, 98.40% functions, 94.56% lines
  • pnpm example:typecheck
  • focused release/docs semantic tests: pass
  • independent adversarial reviews: approve, no P0/P1

@GGULBAE
GGULBAE merged commit 9da5ffa into master Aug 13, 2026
7 checks passed
@GGULBAE
GGULBAE deleted the codex/close-codeql-sanitization-alerts branch August 13, 2026 14:44
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