Skip to content

Conversation

BrunoQuaresma
Copy link

No description provided.

@BrunoQuaresma BrunoQuaresma requested review from bpmct and hugodutka June 3, 2025 20:55
@BrunoQuaresma BrunoQuaresma self-assigned this Jun 3, 2025
bpmct

This comment was marked as duplicate.

bpmct

This comment was marked as duplicate.

@BrunoQuaresma BrunoQuaresma requested a review from bpmct June 3, 2025 23:36
@bpmct
Copy link
Member

bpmct commented Jun 3, 2025

We want to make this configurable in the module. See #129 so we can mark this as a draft once I get it working and perhaps @DevelopmentCats could help take it to the finish line unless you want to add the TF logic

@matifali matifali added the version:minor Add to PRs requiring a minor version upgrade label Jun 4, 2025
@hugodutka hugodutka marked this pull request as draft June 4, 2025 11:10
@hugodutka
Copy link
Contributor

Converted to draft - this needs more work before it's ready to merge.

@code-asher
Copy link
Member

Uh sorry I accidentally pushed to this branch (I am basing the agent status work on this since I need agentapi). I removed the commit.

But also, I think that means the api cannot become optional if we need it for the status reporting?

hugodutka and others added 5 commits June 5, 2025 18:26
Is it OK to add the flag like this or do we need to check the cli
version to determine whether the new flag is available? Or I could just
throw in an `||` to run the command again without the flag if it fails.

Blocked on adding AgentAPI to Claude.

Will need to do the same for Goose.
@hugodutka
Copy link
Contributor

hugodutka commented Jun 19, 2025

this will be finished as part of coder/internal#700

@BrunoQuaresma BrunoQuaresma removed their assignment Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version:minor Add to PRs requiring a minor version upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants