fix: make published research visible in Goal results and faithful review cards - #5575
loopx-agent wants to merge 6 commits into
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
|
This pull request has merge conflicts with Choose the remote for the base repository, not an out-of-date fork. git fetch upstream
git rebase upstream/main
# Resolve each conflict, git add the resolved files, then git rebase --continue.
git push --force-with-lease origin HEADFor a same-repository clone whose Keep the DCO |
Problem and outcome
A published research projection could be read through
serve-status, but the ordinary Chat-backed Goal could not discover it. Its existing research renderer therefore remained detached from Files & Outputs. Review cards also reduced a research view to period metrics, losing adjudication, uncertainty and counterevidence when those metrics were absent.This change connects the existing Goal results tab to the native projection readback and renders the same validated research view in a complete review card. Loading is lazy and Goal-scoped; revision/hash/Origin/public-safe checks remain enforced. Read failures, disabled extensions and Goal changes clear earlier content. Cards preserve exact metric strings, original clocks and review conditions; oversized cards fail rather than truncate the review basis.
Ownership and delivery boundary
Core shares its existing HTTP presentation handler between Chat and status; Finance retains the research semantics and card rendering. No new protocol, model call, account access, scheduler or external message send is introduced. This is a bounded S5/S8/S9 results-consumption slice; deployed App upgrades, actual notification delivery and financial/method utility remain separate acceptance.
Validation
f35978e3a, 202 targeted Python tests passed on headf66004a44, covering native HTTP/presentation, Finance cards and Todo settlement/phase-identity recovery. The merge preserves the existing PR commits and DCO sign-offs.review_due. Goal directory and corruption cases are scripted; the published projection and HTTP reads are real isolated Core operations. No operational Goal, agent or Todo is created.linkify-itaudit advisory remains outside this change; lockfiles are unchanged.Desktop synthetic results · Mobile synthetic results