Skip to content

fix: qwen hook failback fail closed#79

Open
josstei wants to merge 4 commits into
mainfrom
fix/qwen-hook-fallback-fail-closed
Open

fix: qwen hook failback fail closed#79
josstei wants to merge 4 commits into
mainfrom
fix/qwen-hook-fallback-fail-closed

Conversation

@josstei
Copy link
Copy Markdown
Owner

@josstei josstei commented Apr 29, 2026

Summary

  • Make the shared hook runner exit with deny semantics when adapter errors hit the fallback path.
  • Update the Qwen adapter to emit a blocking fallback response instead of allowing execution through.
  • Regenerate detached runtime surfaces and align Qwen docs with the current hook/output contract.
  • Add unit and integration coverage for the Qwen invalid-stdin fallback path.

Testing

  • npm run generate
  • node --test tests/unit/platform-adapters.test.js tests/integration/hook-entrypoints.test.js
  • npm test

@josstei josstei marked this pull request as ready for review May 2, 2026 22:11
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