Skip to content

Conversation

iamriajul
Copy link
Contributor

Description

Solves: https://discord.com/channels/747933592273027093/1404654368745193492

Type of Change

  • New module
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

Related Issues

Copy link
Contributor

@david-fraley david-fraley left a comment

Choose a reason for hiding this comment

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

Looks good to me. Can you make sure to coordinate with @bcpeinhardt on coder/coder#19718? We're adding the Docker Tasks template to the recommended list and I know that there's a disconnect with changes of one file to the other, so wanting to make sure that we've got in sync copies!

@matifali
Copy link
Member

There is another refactor going on in #423.

@david-fraley can you review that.
Also @DevelopmentCats for making sure we are synchronized.

Comment on lines +52 to +66
# Usage with Claude Code Subscription
# Uncomment options below and comment out the above `anthropic_api_key` variable
# variable "claude_code_oauth_token" {
# type = string
# description = "Generate one using `claude setup-token` command (required if use_claude_code_auth is true)"
# sensitive = true
# }
#
# # Create the OAuth token environment variable
# resource "coder_env" "claude_code_oauth_token" {
# agent_id = coder_agent.main.id
# name = "CLAUDE_CODE_OAUTH_TOKEN"
# value = var.claude_code_oauth_token
# }

Copy link
Member

Choose a reason for hiding this comment

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

Let's also mention this in the readme of the module too

Copy link
Member

Choose a reason for hiding this comment

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

@iamriajul can you update that as per #402

35C4n0r added a commit to 35C4n0r/registry that referenced this pull request Sep 18, 2025
@35C4n0r
Copy link
Collaborator

35C4n0r commented Sep 21, 2025

Closing, as completed in #402.
@iamriajul Thanks for the contribution :)

@35C4n0r 35C4n0r closed this Sep 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants