docs: plan runner subscription status path - #5985
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR adds design documentation for runner subscription-status reporting. It defines authenticated runner and agent-service APIs, sanitized per-harness states, frontend query and display behavior, implementation stages, research findings, unresolved decisions, and QA scenarios. ChangesRunner subscription-status design
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…tus plan The review pass approved the plan without changes. status.md now records the five decisions taken to unblock implementation: public route on the Python agent service, invoke-level authentication, one state per harness for Pi in the first version, the frontend text as drafted, and deployment runner scope only.
|
Implementation is starting from this plan. I added one commit (3407663) recording the five decisions taken to unblock it, each reversible before the implementation PR lands: the public route lives on the Python agent service, it uses the same authentication as agent invocation, Pi reports one state per harness in the first version, the frontend text ships as drafted in api-design.md, and the scope is the deployment runner only (the resolver seam is kept for later per-user cloud runners). |
Users can select a model subscription before Agenta knows whether the runner is active or can read the mounted login. They see setup errors only after a run fails.
This draft plan defines a safe status path:
The runner returns status values only. It does not return tokens, file paths, account details, or file contents.
The plan also records a routing limit. The current code selects one deployment runner through
AGENTA_RUNNER_INTERNAL_URL. A different runner for each Agenta Cloud user requires a server-owned runner connection and a shared resolver for status checks and model runs.Included documents
Checks
git diff --checkHow to review
README.mdfor the proposed path and document order.research.mdfor the current code behavior and routing limit.api-design.mdfor the response shapes and security boundary.plan.mdfor the implementation order.status.md.