Skip to content

fix(api): recover message model override - #7144

Merged
houko merged 8 commits into
mainfrom
fix/message-model-override-poison-recovery
Aug 14, 2026
Merged

houko merged 8 commits into
mainfrom
fix/message-model-override-poison-recovery

Conversation

@houko

@houko houko commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • route the agent message provider preflight through a poison-recovering default-model override reader
  • preserve provider, model, and API-key environment state while logging recovery and clearing poison
  • add a held-write-lock panic regression that verifies preserved fields and later ordinary reads and writes

Verification

  • cargo fmt --all -- --check
  • git diff --check
  • CARGO_INCREMENTAL=0 CARGO_TARGET_DIR=/tmp/librefang-message-model-target cargo test -p librefang-api message_model_override_recovers_after_held_write_lock_panic --lib
  • CARGO_INCREMENTAL=0 CARGO_TARGET_DIR=/tmp/librefang-message-model-target cargo clippy -p librefang-api --all-targets -- -D warnings
  • CARGO_INCREMENTAL=0 CARGO_TARGET_DIR=/tmp/librefang-message-model-target cargo check --workspace --lib

Out of scope

@github-actions github-actions Bot added the size/M 50-249 lines changed label Aug 13, 2026
@houko
houko enabled auto-merge (squash) August 14, 2026 09:39
@houko
houko merged commit 413b036 into main Aug 14, 2026
35 checks passed
@houko
houko deleted the fix/message-model-override-poison-recovery branch August 14, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M 50-249 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant