Skip to content

ci: enable async fd leak detection in native test job - #22

Merged
paveg merged 1 commit into
mainfrom
ci/fd-leak-check
Jul 18, 2026
Merged

paveg merged 1 commit into
mainfrom
ci/fd-leak-check

Conversation

@paveg

@paveg paveg commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

Enable moonbitlang/async's file-descriptor leak detection in the native test job.
With MOONBIT_ASYNC_CHECK_FD_LEAK=1 (available since async 0.19.1),
any program that exits with unclosed files or network connections managed by the async runtime aborts with an error,
turning fd leaks into hard CI failures instead of silent resource bugs.

Verification

Ran locally against main with the flag enabled:
MOONBIT_ASYNC_CHECK_FD_LEAK=1 moon test --target native → 256/256 passed, no leaks detected,
so this check starts from a green baseline.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Jul 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f725039

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

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

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.59%. Comparing base (8fbb433) to head (f725039).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   63.59%   63.59%           
=======================================
  Files          16       16           
  Lines         739      739           
=======================================
  Hits          470      470           
  Misses        269      269           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@paveg
paveg merged commit 6b6fe77 into main Jul 18, 2026
4 checks passed
@paveg
paveg deleted the ci/fd-leak-check branch July 18, 2026 21:43
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