QRun-IO/qbit-assistant
0
0 Fork 0
Code Issues 3 Pull Requests 7 Actions Packages Projects Releases 23 Wiki Activity

fix(engine): a stalled database can no longer hang a turn on 'Thinking' (develop) #95

Merged
james.maes merged 1 commits from fix/turn-watchdog-never-hangs-develop into develop 2026-11-09 16:07:07 +00:00
Conversation 0 Commits 1 Files Changed 3
+629 -32
james.maes commented 2026-11-09 15:55:02 +00:00
Owner
Copy link
Copy source

Same change as PR 94 on release/0.1 (TurnEngine.java is identical on both branches). Generated with Claude Code

Same change as PR 94 on release/0.1 (TurnEngine.java is identical on both branches). Generated with Claude Code
james.maes added 1 commit 2026-11-09 15:55:03 +00:00
fix(engine): a stalled database can no longer hang a turn on 'Thinking'
maven / build (push) Successful in 5m23s
Details
build / maven (push) Successful in 5m23s
Details
0c32c43f8c 
The watchdog ran the deadline logic after two database reads and heartbeat() was synchronized on the monitor that finish() needs, so one blocked database call stopped the deadline, the force-finalize and the final message.end. The deadline now runs first with no database access, housekeeping runs on a bounded abandonable worker, finish() has a guard that ends the stream with an error event, and every phase logs at INFO.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
james.maes merged commit f8e3122672 into develop 2026-11-09 16:07:07 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
architecture
Architecture review: coupling, cohesion, UI API
 bug
Something isn't working
 dependencies
documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 github_actions
Pull requests that update GitHub Actions code
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 java
Pull requests that update java code
 question
Further information is requested
 release:rc
Merging this develop PR cuts the next release candidate
 secuurity
ux
Admin UX / information architecture
 wontfix
This will not be worked on
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
forge-recovery (Forge Recovery) james.maes (James Maes)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: QRun-IO/qbit-assistant#95