Skip to content

feat: add project summary sidebar - #5386

Open
harshitsinghbhandari wants to merge 9 commits into
mainfrom
ao/agent-orchestrator-320/project-summary
Open

feat: add project summary sidebar#5386
harshitsinghbhandari wants to merge 9 commits into
mainfrom
ao/agent-orchestrator-320/project-summary

Conversation

@harshitsinghbhandari

@harshitsinghbhandari harshitsinghbhandari commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a durable daemon-owned Project Summary projection in migration 0142 with a separate source watermark
  • generate a running narrative through the project orchestrator provider using verified Codex or Claude noninteractive CLI invocation
  • serialize refreshes, preserve unchanged prose through the update prompt, and advance the watermark only after valid output
  • retain the last good narrative and surface provider, authentication, or generation failures
  • consume delivery-independent report facts through a narrow Summary-owned read interface, including needs-input reports and opaque outputs
  • add a responsive right sidebar with compact counts, decisions, outputs, freshness, navigation, and composer seed without sending

Tests

  • go test ./internal/service/projectsummary
  • go vet ./internal/service/projectsummary ./internal/daemon
  • renderer ProjectSummaryPanel, localization coverage, and catalog tests
  • npm run api
  • focused SQLite migration tests

Invocation verification

  • Codex 0.154.0 completed the noninteractive output probe successfully
  • Claude accepted the print-mode invocation and returned the expected local organization authentication error, which the summary service surfaces while retaining the last good narrative

Integration boundary

Report delivery PR #5064 owns report schema, persistence, delivery, and report.Service.ListProject. This PR does not copy those symbols or migrations and never claims, acknowledges, wakes, interrupts, or dereferences report outputs. Its report consumer logic is independently tested behind a Summary-owned ReportReader. After #5064 lands, the remaining integration is a narrow adapter and daemon wiring from report.Service.ListProject into that reader.

@harshitsinghbhandari
harshitsinghbhandari marked this pull request as ready for review September 14, 2026 08:45
@harshitsinghbhandari
harshitsinghbhandari force-pushed the ao/agent-orchestrator-320/project-summary branch from c9291cf to 3b2ef2c Compare September 14, 2026 09:30
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