Skip to content

fix(runtime): recover plugin hook lock poison - #7420

Merged
houko merged 3 commits into
mainfrom
fix/plugin-hook-lock-poison
Aug 15, 2026
Merged

houko merged 3 commits into
mainfrom
fix/plugin-hook-lock-poison

Conversation

@houko

@houko houko commented Aug 15, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • route the plugin state-file advisory-lock registry through a poison-recovery helper
  • route every persistent hook-process pool access through a second recovery helper
  • preserve active lock and process slots, log each recovery, and clear poison for subsequent ordinary access
  • cover both registries with real held-lock panic regressions, including the production health-check path
  • merge the latest origin/main before final verification

Verification

  • CARGO_TARGET_DIR=/private/tmp/librefang-target-pr7420-final cargo test -p librefang-runtime plugin_runtime::tests --lib — 28 tests passed
  • CARGO_TARGET_DIR=/private/tmp/librefang-target-pr7420-final cargo clippy -p librefang-runtime --all-targets -- -D warnings
  • cargo fmt --all -- --check
  • git diff --check origin/main...HEAD

Out of scope

  • cross-process state-file locking
  • hook subprocess lifecycle, retry policy, sandbox policy, and registry capacity
  • other poisoned-lock domains listed in the audit handoff

@github-actions github-actions Bot added size/M 50-249 lines changed area/runtime Agent loop, LLM drivers, WASM sandbox and removed size/M 50-249 lines changed labels Aug 15, 2026
@github-actions github-actions Bot added the size/M 50-249 lines changed label Aug 15, 2026
@houko
houko enabled auto-merge (squash) August 15, 2026 07:26
@houko
houko merged commit b3fa71e into main Aug 15, 2026
37 checks passed
@houko
houko deleted the fix/plugin-hook-lock-poison branch August 15, 2026 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/runtime Agent loop, LLM drivers, WASM sandbox size/M 50-249 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant