Thanks to visit codestin.com
Credit goes to github.com

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
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