Skip to content

Conversation

35C4n0r
Copy link
Collaborator

@35C4n0r 35C4n0r commented Aug 30, 2025

Closes #302

Description

Type of Change

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

Module Information

Path: registry/[namespace]/modules/[module-name]
New version: v3.0.0
Breaking change: [ ] Yes [ ] No

Testing & Validation

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

Related Issues

@35C4n0r 35C4n0r self-assigned this Aug 30, 2025
# Conflicts:
#	registry/coder/modules/claude-code/README.md
#	registry/coder/modules/claude-code/main.tf
#	registry/coder/modules/claude-code/scripts/agentapi-start.sh
#	registry/coder/modules/claude-code/scripts/agentapi-wait-for-start.sh
#	registry/coder/modules/claude-code/testdata/agentapi-mock.js
@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Sep 10, 2025

Linked the issue, the issue also asks for subdomain, will add that today.

@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Sep 10, 2025

Linked the issue, the issue also asks for subdomain, will add that today.

added subdomain 👍🏻

@35C4n0r 35C4n0r requested a review from matifali September 11, 2025 02:48
Copy link
Contributor

@DevelopmentCats DevelopmentCats left a comment

Choose a reason for hiding this comment

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

LGTM

Provided we are cool with the shift of standardization in variables for AI Modules this looks good to me. I have tested and it works fine.

@DevelopmentCats
Copy link
Contributor

@35C4n0r can you resolve the conflicts here?

# Conflicts:
#	registry/coder/modules/claude-code/README.md
#	registry/coder/modules/claude-code/main.test.ts
#	registry/coder/modules/claude-code/main.tf
#	registry/coder/modules/claude-code/testdata/agentapi-mock.js
@DevelopmentCats
Copy link
Contributor

I updated the tasks template for this update.

We should be good here to merge but Ill let you guys take a look at the PR for the the template and I can push these all at once

@DevelopmentCats
Copy link
Contributor

@matifali

We are good here but I cant do final approval since I was the one who made the last commit.

@matifali
Copy link
Member

@DevelopmentCats let's also add an explicit example to use the Claude Max subscription as suggested by @bpmct here: #358 (comment)

@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Sep 18, 2025

@matifali @DevelopmentCats added the example

@matifali
Copy link
Member

@35C4n0r, let's merge and release this and then test the template and release that too. Thanks for your work on this.

@DevelopmentCats DevelopmentCats merged commit d212de4 into coder:main Sep 18, 2025
4 checks passed
DevelopmentCats added a commit that referenced this pull request Sep 19, 2025
## Description

Refactor template for claude-code module update for tasks

## Type of Change

- [ ] New module
- [ ] Bug fix
- [ ] Feature/enhancement
- [ ] Documentation
- [X] Other

## Testing & Validation

- [X] Tests pass (`bun test`)
- [X] Code formatted (`bun run fmt`)
- [X] Changes tested locally

## Related Issues

#402

---------

Co-authored-by: Atif Ali <[email protected]>
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.

claude-code module: refactor to use agentapi

3 participants