Skip to content

refactor: (slightly) decentralize type checking#64163

Merged
Sembauke merged 4 commits intofreeCodeCamp:mainfrom
ojeytonwilliams:refactor/overlapping-tsconfigs
Dec 1, 2025
Merged

refactor: (slightly) decentralize type checking#64163
Sembauke merged 4 commits intofreeCodeCamp:mainfrom
ojeytonwilliams:refactor/overlapping-tsconfigs

Conversation

@ojeytonwilliams
Copy link
Copy Markdown
Contributor

I've also made a start on making the helper scripts esm, but it's still a WIP

Checklist:

@ojeytonwilliams ojeytonwilliams requested a review from a team November 26, 2025 15:08
@github-actions github-actions bot added the scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc. label Nov 26, 2025
@socket-security
Copy link
Copy Markdown

socket-security bot commented Nov 26, 2025

@socket-security
Copy link
Copy Markdown

socket-security bot commented Nov 26, 2025

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Critical
Critical CVE: Babel vulnerable to arbitrary code execution when compiling specifically crafted malicious code in npm @babel/traverse

CVE: GHSA-67hx-6x53-jw92 Babel vulnerable to arbitrary code execution when compiling specifically crafted malicious code (CRITICAL)

Affected versions: < 7.23.2; >= 8.0.0-alpha.0 < 8.0.0-alpha.4

Patched version: 7.23.2

From: pnpm-lock.yamlnpm/@babel/traverse@7.23.0

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@babel/traverse@7.23.0. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

By the way, I have no idea why we used bson-objectid rather than bson, but the former doesn't seem to be terribly well maintained.

@ojeytonwilliams ojeytonwilliams force-pushed the refactor/overlapping-tsconfigs branch from bd849e9 to 6da1b89 Compare November 27, 2025 09:36
@ojeytonwilliams ojeytonwilliams requested a review from a team as a code owner November 27, 2025 09:36
@Sembauke Sembauke merged commit 37ce134 into freeCodeCamp:main Dec 1, 2025
11 checks passed
@ojeytonwilliams ojeytonwilliams deleted the refactor/overlapping-tsconfigs branch December 1, 2025 12:14
Ksound22 pushed a commit to Ksound22/freeCodeCamp that referenced this pull request Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants