Skip to content

feat: precise inbox action feedback and Recently handled outcomes (#309) - #318

Merged
douglasjarquin merged 13 commits into
mainfrom
sum/t-411463d7e28b
Oct 18, 2026
Merged

douglasjarquin merged 13 commits into
mainfrom
sum/t-411463d7e28b

Conversation

@douglasjarquin

@douglasjarquin douglasjarquin commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Summary

Closes #309.

After an answer or handoff, the inbox now says exactly what sum knows instead of implying that the worker received or applied it.

The two inbox render golden files were regenerated. No test source was added or changed. The feature map says the golden render covers the Recently handled view and default feedback text, with no focused coverage of the projection or delivery-state variants.

Verification

  • Coordinator mise run verify on aebf94e: pass (internal/cli 466s); 7fa6252 has the same code with a one-line feature-map wording fix.
  • Worker verify runner on 7fa6252: pass (run 20261018T052134Z-0d644ce9).
  • Independent review (Codex GPT-6 Luna): approve on 7fa6252, including the conflict resolutions.

https://claude.ai/code/session_01LmjWPRZjxcGH3kQmkpRToB

Pipeline

Generated by sum

Stage Status Result
Intent ✅ Approved brief recorded
Rebase ✅ Up to date with main (main at 920df23, observed 2026-10-18T05:28Z)
Review ✅ Passed after 3 remediation passes
Test ✅ Passed; the run certifies no candidate
Document ✅ Passed
Lint ✅ Passed (mise run lint)
Push ✅ Pushed 7fa6252 to origin/sum/t-411463d7e28b
PR ✅ Open: #318
CI ➖ No checks reported for this PR (as of 2026-10-18T05:28Z)

Review summary

Review remediation log

Pass 1

  • Limitation: This pass recorded no structured notes.

Pass 2

  • Limitation: This pass recorded no structured notes.

@douglasjarquin
douglasjarquin marked this pull request as ready for review October 18, 2026 04:29
@douglasjarquin douglasjarquin changed the title Factory issue douglasjarquin/sum#309 feat: precise inbox action feedback and Recently handled outcomes (#309) Oct 18, 2026
…315)

* Fix reviewer launch identity and MCP startup

* Align reviewer launch coverage with behavior

* Update expected reviewer MCP argv
…313) (#316)

* fix: direct task workers to installed helper

* fix: include installed init command in worker brief
…307) (#317)

* feat: track incidental finding dispositions

* feat: expose findings through inbox actions

* Track scoped repair links for findings

* Mark finding verification as manual

* fix finding repair link record handling
@douglasjarquin
douglasjarquin merged commit 7f3b46d into main Oct 18, 2026
3 checks passed
@douglasjarquin
douglasjarquin deleted the sum/t-411463d7e28b branch October 18, 2026 16:16
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.

[P2] Show precise inbox action feedback and recently handled outcomes

1 participant