Skip to content

feat(coderd): add experimental tasks send endpoint - #19941

Merged
mafredri merged 14 commits into
mainfrom
mafredri/feat-coderd-tasks-send
Sep 25, 2025
Merged

mafredri merged 14 commits into
mainfrom
mafredri/feat-coderd-tasks-send

Conversation

@mafredri

Copy link
Copy Markdown
Member

@mafredri
mafredri force-pushed the mafredri/feat-coderd-tasks-send branch 2 times, most recently from 8272a1c to 1bd075b Compare September 24, 2025 12:46
@mafredri
mafredri force-pushed the mafredri/feat-coderd-tasks-send branch from 1bd075b to 812b2b2 Compare September 24, 2025 12:55
@mafredri
mafredri marked this pull request as ready for review September 24, 2025 12:55

@DanielleMaywood DanielleMaywood left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The logic looks fine to me, although there are a couple of stylistic nits and some unneeded use of httperror.WriteResponseWriter

Comment thread coderd/aitasks.go Resolved
Comment thread coderd/aitasks.go Resolved
Comment thread coderd/aitasks.go Resolved
Comment thread coderd/aitasks.go Resolved
Comment thread coderd/aitasks_test.go Resolved
Comment thread coderd/aitasks_test.go Resolved

@DanielleMaywood DanielleMaywood left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM :shipit: I'd feel happier with Cian having a second look as we already partly discussed the approach earlier

@johnstcn johnstcn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work @mafredri !

I think we're just missing a pre-flight check on the AgentAPI /status endpoint.

Comment thread coderd/aitasks.go Resolved
Comment thread coderd/aitasks.go Resolved
Comment thread coderd/aitasks.go Resolved
@mafredri
mafredri force-pushed the mafredri/feat-coderd-tasks-send branch from 20d653d to f607b45 Compare September 25, 2025 09:30
@mafredri
mafredri requested a review from johnstcn September 25, 2025 09:40
@mafredri
mafredri enabled auto-merge (squash) September 25, 2025 11:57
@mafredri mafredri changed the title feat(coderd): add tasks send endpoint feat(coderd): add experimental tasks send endpoint Sep 25, 2025
@mafredri
mafredri merged commit 5317d30 into main Sep 25, 2025
@mafredri
mafredri deleted the mafredri/feat-coderd-tasks-send branch September 25, 2025 12:12
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tasks API: Implement POST /tasks/{id}/send

3 participants