Skip to content

build: TS6 + oxc.rs - #106

Merged
dsanders11 merged 5 commits into
mainfrom
oxcrs
Apr 16, 2026
Merged

build: TS6 + oxc.rs#106
dsanders11 merged 5 commits into
mainfrom
oxcrs

Conversation

@erickzhao

@erickzhao erickzhao commented Apr 7, 2026

Copy link
Copy Markdown
Member
  • This repo didn't have ESLint configured at all. I instantiated a clean oxlint instance here.
  • Since we're linting against oxlint-tsgolint, let's upgrade to TypeScript 6 to bridge the gap to v7.
  • Ignoring Markdown in the config because we previously wanted to do that with Prettier cc @dsanders11

@erickzhao
erickzhao requested a review from a team as a code owner April 7, 2026 21:37
@socket-security

socket-security Bot commented Apr 7, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedoxfmt@​0.44.0901008996100
Updatedtypescript@​5.8.3 ⏵ 6.0.2100 +110090 +19990
Addedoxlint@​1.59.0991009196100
Updatedtypedoc@​0.25.13 ⏵ 0.28.181001009894 -1100
Addedoxlint-tsgolint@​0.20.01001009796100

View full report

@socket-security

socket-security Bot commented Apr 7, 2026

Copy link
Copy Markdown

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 High
High CVE: xmldom: XML injection via unsafe CDATA serialization allows attacker-controlled markup insertion in npm @xmldom/xmldom

CVE: GHSA-wh4c-j3r5-mjhp xmldom: XML injection via unsafe CDATA serialization allows attacker-controlled markup insertion (HIGH)

Affected versions: < 0.8.12; >= 0.9.0 < 0.9.9

Patched version: 0.8.12

From: ?npm/@electron/universal@3.0.1npm/@xmldom/xmldom@0.8.10

ℹ Read more on: This package | This alert | What is a 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 high severity 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/@xmldom/xmldom@0.8.10. 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.

Warn High
High CVE: Vite Vulnerable to Arbitrary File Read via Vite Dev Server WebSocket

CVE: GHSA-p9ff-h696-f583 Vite Vulnerable to Arbitrary File Read via Vite Dev Server WebSocket (HIGH)

Affected versions: >= 8.0.0 < 8.0.5; >= 7.0.0 < 7.3.2; >= 6.0.0 < 6.4.2

Patched version: 8.0.5

From: ?npm/vitest@4.1.2npm/vite@8.0.3

ℹ Read more on: This package | This alert | What is a 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 high severity 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/vite@8.0.3. 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.

Warn High
High CVE: Vite: server.fs.deny bypassed with queries

CVE: GHSA-v2wj-q39q-566r Vite: server.fs.deny bypassed with queries (HIGH)

Affected versions: >= 8.0.0 < 8.0.5; >= 7.1.0 < 7.3.2

Patched version: 8.0.5

From: ?npm/vitest@4.1.2npm/vite@8.0.3

ℹ Read more on: This package | This alert | What is a 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 high severity 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/vite@8.0.3. 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.

Warn Medium
Deprecated by its maintainer: npm @xmldom/xmldom

Reason: this version has critical issues, please update to the latest version

From: ?npm/@electron/universal@3.0.1npm/@xmldom/xmldom@0.8.10

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

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: Research the state of the package and determine if there are non-deprecated versions that can be used, or if it should be replaced with a new, supported solution.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@xmldom/xmldom@0.8.10. 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.

Ignoring alerts on:

  • entities@4.5.0
  • markdown-it@14.1.1

View full report

Comment thread .oxfmtrc.json Outdated
Comment thread package.json Outdated
@erickzhao erickzhao changed the title build: TypeScript 6/7 + use oxc.rs for linting and formatting build: TypeScript 6 + use oxc.rs for linting and formatting Apr 7, 2026
@erickzhao

Copy link
Copy Markdown
Member Author

@SocketSecurity ignore npm/entities@4.5.0
@SocketSecurity ignore npm/markdown-it@14.1.1

@erickzhao erickzhao changed the title build: TypeScript 6 + use oxc.rs for linting and formatting build: TS6 + oxc.rs Apr 14, 2026
@dsanders11
dsanders11 merged commit 5864276 into main Apr 16, 2026
8 checks passed
@dsanders11
dsanders11 deleted the oxcrs branch April 16, 2026 05:18
@electron-npm-package-publisher

Copy link
Copy Markdown

🎉 This PR is included in version 2.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants