Skip to content

sync: upstream @just-bash/executor@1.0.2#12

Open
github-actions[bot] wants to merge 10 commits into
flowglad-mainfrom
sync-@just-bash/executor@1.0.2
Open

sync: upstream @just-bash/executor@1.0.2#12
github-actions[bot] wants to merge 10 commits into
flowglad-mainfrom
sync-@just-bash/executor@1.0.2

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions Bot commented May 13, 2026

Automated upstream sync to @just-bash/executor@1.0.2 has merge conflicts.

Check out this branch, resolve conflicts at TypeScript-source level, then run:

pnpm install
pnpm --filter just-bash build
pnpm --filter just-bash typecheck
pnpm --filter just-bash exec vitest run src/commands/sqlite3/sqlite3.test.ts src/commands/python3/python3.optin.test.ts
git add -f packages/just-bash/dist

Commit and push to update this PR.


View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

cramforce and others added 10 commits May 9, 2026 10:18
…-labs#209)

* feat: executor tool invocation for js-exec

* changeset

* Fix TS for sub packages
…s#233)

- Makes the issue visible in the type system
- Adds lots of tests
- We should still migrate to a typed array based internal representation in the future
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…OpenAPI, etc (vercel-labs#235)

* Upgrade executor because the original version was non-functional for OpenAPI, etc

* script name for CI run
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Remove network dependency from executor tests

* Turn off the tests in node 20
…ercel-labs#238)

The esbuild dynamic-require shim in the ESM Node bundles had no real
`require` to delegate to at chunk-init, so any runtime `require("tty")`
/ `require("os")` from the `file-type` → `debug` → `supports-color`
chain threw "Dynamic require of \"tty\" is not supported". This made
the `file` command crash for every consumer importing just-bash as ESM
and for the `just-bash` CLI binary.

Provide `createRequire(import.meta.url)` via the build banner for
`build:lib`, `build:cli`, and `build:shell`. CJS and browser bundles
are unchanged.

Add regression tests covering both the bundled CLI binary and the ESM
library bundle running the `file` command.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Conflict markers remain in the tree. Resolve locally before merging.
@github-actions github-actions Bot added the needs-conflict-resolution Auto-sync PR with merge conflicts label May 13, 2026
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 3 committers have signed the CLA.

❌ cramforce
❌ github-actions[bot]
❌ flowglad-sync[bot]


flowglad-sync[bot] seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Labels

needs-conflict-resolution Auto-sync PR with merge conflicts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants