Skip to content

Fix RCA reflection verdict parsing#115

Merged
eshulman2 merged 1 commit into
forge-sdlc:mainfrom
eshulman2:fix-rca-reflection-verdict
Jun 30, 2026
Merged

Fix RCA reflection verdict parsing#115
eshulman2 merged 1 commit into
forge-sdlc:mainfrom
eshulman2:fix-rca-reflection-verdict

Conversation

@eshulman2

Copy link
Copy Markdown
Collaborator

Summary

  • Read the RCA reflector verdict from the saved final assistant message when available
  • Fall back to container stdout only if the history file is unavailable
  • Add regression coverage for container stdout logs hiding a VALID final response

Tests

  • uv run pytest tests/unit/workflow/nodes/test_rca_analysis.py -q
  • uv run ruff check src/forge/workflow/nodes/rca_analysis.py tests/unit/workflow/nodes/test_rca_analysis.py

@eshulman2 eshulman2 merged commit 2ab70a3 into forge-sdlc:main Jun 30, 2026
2 checks passed
@eshulman2 eshulman2 deleted the fix-rca-reflection-verdict branch June 30, 2026 15:00
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