Skip to content

feat: Signals API#939

Open
logan-gatlin wants to merge 5 commits intomainfrom
logan/signal-test
Open

feat: Signals API#939
logan-gatlin wants to merge 5 commits intomainfrom
logan/signal-test

Conversation

@logan-gatlin
Copy link
Collaborator

@logan-gatlin logan-gatlin commented Feb 25, 2026

Added the Signal struct which represents and interprets signals received by the runtime. Currently, signals require compiling with "panic=unwind", and are not supported in miniflare. I added the tests anyway, including a new test suite for panic=unwind tests as I presume we will want to write more of those in the future. The example code I added is deployed here: https://pi.logan.md/, confirming that the new API does work in production. We may want to wait for miniflare to catch up and signal support for panic=abort to land before merging this.

This PR bumps most of our dependencies by a significant amount which is something to keep in mind.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 4, 2026

Merging this PR will not alter performance

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

✅ 2 untouched benchmarks


Comparing logan/signal-test (a506eb4) with main (652cd4b)

Open in CodSpeed

@logan-gatlin logan-gatlin changed the title DO NOT MERGE - wasm signal test feat: Signals API Mar 9, 2026
@logan-gatlin logan-gatlin marked this pull request as ready for review March 9, 2026 21:33
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