Skip to content

@bomb.sh/tools and OXC compat for IDEs - #8

Merged
jbolda merged 5 commits into
mainfrom
hoist-oxc-for-ide-usage
Aug 2, 2026
Merged

@bomb.sh/tools and OXC compat for IDEs#8
jbolda merged 5 commits into
mainfrom
hoist-oxc-for-ide-usage

Conversation

@jbolda

@jbolda jbolda commented Aug 1, 2026

Copy link
Copy Markdown
Member

What does this PR do?

IDEs seem to expect these bins at the root. Hoist them and pull in the config through root config files so our IDEs can pick up and match what our bsh commands output.

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

@socket-security

socket-security Bot commented Aug 1, 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
Addednpm/​@​bomb.sh/​tools@​0.6.1751007699100

View full report

@jbolda

jbolda commented Aug 1, 2026

Copy link
Copy Markdown
Member Author

Note: first test intentionally failed to confirm upstream changes did indeed pass the exit code properly.

@natemoo-re natemoo-re 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.

That works! I think making these devDependcies on the tools side would also work but then we'd lose the lockstep upgrades across repos (which might be good? open to pushback here)

@jbolda
jbolda merged commit 1219d65 into main Aug 2, 2026
5 checks passed
@jbolda
jbolda deleted the hoist-oxc-for-ide-usage branch August 2, 2026 03:10
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.

2 participants