Skip to content

update oxc deps - #72

Open
jbolda wants to merge 1 commit into
mainfrom
update-oxc-libs-to-latest
Open

jbolda wants to merge 1 commit into
mainfrom
update-oxc-libs-to-latest

Conversation

@jbolda

@jbolda jbolda commented Oct 21, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Just bumping to latest! Don't see any changes that would break our wrapper expectations.

Closes #

Type of change

  • Bug fix
  • Feature
  • Refactor (no behavior change)
  • Documentation
  • Performance improvement
  • Tests
  • Chore (dependencies, CI, tooling)

Checklist

  • All tests pass (pnpm test)
  • Files are formatted (pnpm format)
  • I have added/updated tests for my changes (if applicable)
  • I have added a changeset

AI-generated code disclosure

  • This PR includes AI-generated code

@changeset-bot

changeset-bot Bot commented Oct 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5bf828e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bomb.sh/tools Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@socket-security

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
Updatedoxfmt@​0.47.0 ⏵ 0.74.087 +11008896100
Updatedoxlint@​1.62.0 ⏵ 1.89.099 +110091 +196100

View full report

@pkg-pr-new

pkg-pr-new Bot commented Oct 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@bomb.sh/tools@72

commit: 5bf828e

@dreyfus92 dreyfus92 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks jacob! ✌🏻

Comment thread rules/plugin.js
* Require JSDoc comments on exported functions and classes.
*
* Public APIs should have `/** ... *​/` documentation. Internal/unexported
* Public APIs should have /** ... */ documentation. Internal/unexported

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does this do?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We had a non-visible whitespace character there before to trick the lint, but they added a new rule to flag those (which seems reasonable).

I switched to an HTML character reference instead so it doesn't get flagged but still renders in the JSDoc.

image

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.

3 participants