Skip to content

Bun tests - #13

Merged
igoramadas merged 2 commits into
masterfrom
bun-tests
Oct 8, 2026
Merged

igoramadas merged 2 commits into
masterfrom
bun-tests

Conversation

@igoramadas

Copy link
Copy Markdown
Owner

Summary

  • Replace capture-console with a local output capture helper that also hooks the console, so the existing Mocha suite runs under Bun, where console output does not go through process.stdout.write.
  • Add a test-bun script and declare Bun 1.3 as a supported engine.
  • Allow the GitHub publish workflow to be started manually, falling back to the current commit when there is no merge commit.

Made with Cursor

igoramadas and others added 2 commits October 8, 2026 20:04
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@greptile-apps

greptile-apps Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Replaces console capture library with custom implementation.

The PR appears safe to merge; no actionable defects were found.

What we checked:

  • Publishing an existing release again: The workflow checks for the release and skips publishing when it exists.

Reviews (1) · Last reviewed commit: "Allow the publish workflow to be started..." · Reviewed by Greptile

@igoramadas
igoramadas marked this pull request as ready for review October 8, 2026 18:27

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 10 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread .github/workflows/publish-github.yml
Comment thread test/capture.ts
@igoramadas
igoramadas merged commit 181fe62 into master Oct 8, 2026
7 checks passed
@igoramadas
igoramadas deleted the bun-tests branch October 8, 2026 18:49
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.

1 participant