Skip to content

Fix reviewed IPC, plugin lifecycle, and dev-shim findings#31

Merged
MerhiOPS merged 4 commits intomainfrom
fix/review-findings-severity-batches
Mar 16, 2026
Merged

Fix reviewed IPC, plugin lifecycle, and dev-shim findings#31
MerhiOPS merged 4 commits intomainfrom
fix/review-findings-severity-batches

Conversation

@MerhiOPS
Copy link
Contributor

Summary

  • harden IPC worker and runtime timeout handling so panics and queued requests do not leak capacity or strand workers
  • align dev shims with production capabilities, including �olt:plugins, permission enforcement, SSRF guards, and consistent dev error formatting
  • close the remaining plugin lifecycle, storage, deferred-queue, and grant-registry gaps from the review findings

Validation

  • pnpm --filter @voltkit/volt-cli test -- src/tests/dev-backend.test.ts
  • cargo test -p volt-runner process_task_releases_window_slot_when_dispatch_panics -- --nocapture
  • cargo test -p volt-runner queued_ipc_requests_consume_the_same_end_to_end_timeout_budget -- --nocapture
  • cargo test -p volt-runner panicking_lifecycle_subscribers_do_not_stop_other_handlers -- --nocapture
  • cargo test -p volt-runner drain_waiters_disconnects_pending_receivers -- --nocapture
  • cargo test -p volt-core test_revoke_grant_removes_plugin_delegations -- --nocapture
  • cargo test -p volt-core revoke_grant_everywhere_prunes_empty_plugin_entries -- --nocapture
  • cargo test -p volt-plugin-host send_request_rejects_when_deferred_queue_exceeds_limit -- --nocapture
  • cargo test -p volt-runner deactivation_during_spawning_cancels_startup_and_terminates_plugin -- --nocapture
  • cargo test -p volt-runner disabled_plugins_ignore_process_exit_transitions -- --nocapture
  • cargo test -p volt-runner subscribe_and_off_recover_after_mutex_poisoning -- --nocapture
  • cargo test -p volt-runner emit_recover_after_mutex_poisoning -- --nocapture

Commit layout

@MerhiOPS MerhiOPS force-pushed the fix/review-findings-severity-batches branch from 344c3a1 to ce3fb69 Compare March 16, 2026 11:25
@MerhiOPS MerhiOPS merged commit 399e1b4 into main Mar 16, 2026
16 checks passed
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