Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Sep 27, 2025

Description

experimental status tool for cn to report back the state of the agent


Summary by cubic

Adds a beta Status tool so agents can report task state to users. Introduces a shared authenticated API client and refactors remote/ls to use it for cleaner requests and errors.

  • New Features

    • Status tool (beta) to set agent status: PLANNING, WORKING, DONE, BLOCKED.
    • Enable with --beta-status-tool. Uses --id to find the agent and POSTs to agents/{id}/status.
  • Refactors

    • Added shared API client (get/post/put/del) with typed responses and custom errors (AuthenticationRequiredError, ApiRequestError).
    • remote and ls now use the client instead of manual fetch/auth; improved error messages. Added unit tests for the API client.

@sestinj sestinj requested a review from a team as a code owner September 27, 2025 06:55
@sestinj sestinj requested review from RomneyDa and removed request for a team September 27, 2025 06:55
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Sep 27, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 6 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="extensions/cli/src/tools/status.ts">

<violation number="1" location="extensions/cli/src/tools/status.ts:44">
This tool issues a POST to update agent status but is marked readonly, so plan-mode restrictions would allow a state-changing call. Mark it non-readonly so plan mode stays side-effect free.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

RomneyDa
RomneyDa previously approved these changes Sep 30, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Sep 30, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 30, 2025
@sestinj sestinj merged commit 2d50cfd into main Oct 1, 2025
119 of 129 checks passed
@sestinj sestinj deleted the nate/status-tool branch October 1, 2025 15:44
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Oct 1, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 1, 2025
@github-actions github-actions bot added the tier 2 Important feature that adds new capabilities to the platform or improves critical user journeys label Oct 1, 2025
@sestinj
Copy link
Contributor Author

sestinj commented Oct 1, 2025

🎉 This PR is included in version 1.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Oct 1, 2025

🎉 This PR is included in version 1.21.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Oct 15, 2025

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:XL This PR changes 500-999 lines, ignoring generated files. tier 2 Important feature that adds new capabilities to the platform or improves critical user journeys

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants