coder/terraform-provider-coderPublic

NotificationsYou must be signed in to change notification settings
Fork24
Star52

feat: add coder_task data source#460

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

johnstcn merged 6 commits intomainfromcj/coder_ai_task_prompt_datasource

Nov 4, 2025

Merged

feat: add coder_task data source#460

johnstcn merged 6 commits intomainfromcj/coder_ai_task_prompt_datasource

Nov 4, 2025

Conversation

Copy link

Member

johnstcn commentedNov 3, 2025•
edited
Loading

Terraform will tolerate a certain amount of self-reference, but this changes when usingcount.

For example, the below is accepted

resource"coder_ai_task""task" {app_id=module.agent.task_app_id}module"agent" {source="..."prompt=coder_ai_task.task.prompt}

Whereas this outputs a cycle error:

data"coder_workspace""me" {}resource"coder_ai_task""task" {count=data.coder_workspace.me.start_countapp_id=module.agent[count.index].task_app_id}module"agent" {count=data.coder_workspace.me.start_countsource="..."prompt=coder_ai_task.task[count.index].prompt}

After some cursory searching, I didn't find much more information on this apart fromthis post.

Adding a data source is not too big a deal as the required environment variables are already present.

With this additional data source, the above example becomes:

data"coder_workspace""me" {}data"coder_task""me" {}resource"coder_ai_task""task" {count=data.coder_workspace.me.start_countapp_id=module.agent[count.index].task_app_id}module"agent" {count=data.coder_workspace.me.start_countsource="..."prompt=data.coder_task.me.prompt}

NB: I explicitly named thiscoder_task to sidestepthis check in Coder. I'm open to alternative names, but it can't becoder_ai_task.

Seecoder/coder#20416 for what this looks like in practice.

feat: add coder_ai_task_prompt data source

37908cf

johnstcn self-assigned this

Nov 3, 2025

johnstcn requested review fromDanielleMaywood andmafredri

November 3, 2025 21:58

johnstcn marked this pull request as ready for review

November 3, 2025 21:58

johnstcn mentioned this pull request

Nov 3, 2025

chore: update coder/dogfood template to reference coder_task.promptcoder/coder#20416

Merged

johnstcn added2 commits

November 4, 2025 09:12

add version note

51684f4

rename datasource coder_ai_task_prompt -> coder_task

a27420c

johnstcn changed the title~~feat: add coder_ai_task_prompt data source~~feat: add coder_task data source

Nov 4, 2025

add example

1983ed0

johnstcn commented

Nov 4, 2025

View reviewed changes

provider/script_test.go

Comment on lines +134 to +137

		namestring
		cronExprstring
		expectWarningsbool
		expectErrorsbool

Copy link

MemberAuthor

johnstcnNov 4, 2025

Choose a reason for hiding this comment

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

drive-bygo fmt

mafredri approved these changes

Nov 4, 2025

View reviewed changes

Copy link

Member

mafredri left a comment

Choose a reason for hiding this comment

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

Small suggestions but otherwise 👍🏻

provider/ai_task.go OutdatedShow resolvedHide resolved

provider/ai_task.go

		"id": {
		Type:schema.TypeString,
		Computed:true,
		Description:"The UUID of the task, if executing in a Coder Task context. Empty in a Coder Workspace context.",

Copy link

Member

mafredriNov 4, 2025

Choose a reason for hiding this comment

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

Should we set optional true, is that possible for computed?

Copy link

MemberAuthor

johnstcnNov 4, 2025

Choose a reason for hiding this comment

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

I don't think we should according to the documentation forComputed:

If Required and Optional are both false, the attribute will be considered "read only" for the practitioner, with only the provider able to set its value.