fix(always-on): extract report from LLM text when report tool is not invoked

[Gucc111]

Summary

• When the report-phase agent produces text but does not call always_on_report, the run was incorrectly marked as failed with reason report_tool_not_invoked. Now the runtime extracts assistant_text_delta events from the report session, runs them through parseReportMarkdown for validation/completion, and persists the result — so the outcome reflects the actual execution success rather than a missed tool call.
• The existing tool-call path is completely unchanged; the new logic only activates as a fallback when reportCtx.report is unset and there is no reportError.

Changes

File: src/always-on/runtime/DiscoveryFire.ts

1. Added extractAssistantText() helper that concatenates all assistant_text_delta events into a single string.
2. In both fire() and run() methods, after drainTurn completes and before outcome determination: if the report tool was not called and there is no error, attempt to extract LLM text from events. If non-empty, parse it via parseReportMarkdown and write to disk, setting reportCtx.report so the outcome resolves to "executed".
3. If the LLM produced no text output either, the original writeFallbackReport path is preserved (outcome = "failed").

Priority: tool call > LLM text > fallback template

Scenario	Outcome
Tool called successfully	executed (unchanged)
Tool not called, LLM text present	executed (new)
Tool not called, no LLM text	failed (unchanged)

Closes #148

Made with Cursor

[Copilot]

Pull request overview

This PR updates the Always-On runtime to avoid incorrectly marking a run as failed when the report-phase agent produces a valid textual report but does not invoke the always_on_report tool. It adds a fallback path that reconstructs the report from streamed assistant_text_delta events, validates/completes it via parseReportMarkdown, and persists it so the run outcome reflects execution success.

Changes:

• Capture report-phase gateway events (reportEvents) and extract concatenated assistant text when the report tool was not invoked and no report error occurred.
• Parse the extracted text with parseReportMarkdown and persist the normalized report, setting reportCtx.report so the outcome becomes "executed".
• Preserve the existing fallback-template behavior when no tool call occurs and the assistant produced no report text.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.