feat: refactor claude code to use agentapi module #402

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

35C4n0r wants to merge 18 commits into coder:main

from 35C4n0r:feat-claude-agentapi

Open

feat: refactor claude code to use agentapi module #402

35C4n0r wants to merge 18 commits into coder:main from 35C4n0r:feat-claude-agentapi

+866 −668

Conversation

35C4n0r

Copy link

Collaborator

@35C4n0r 35C4n0r commented Aug 30, 2025 •

edited

Loading

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 added 2 commits

August 31, 2025 00:14

@35C4n0r


 feat: refactor claude code to use agentapi module

36cebbd

@35C4n0r


 ref: bun fmt

6c5fbee

@35C4n0r 35C4n0r requested review from matifali and DevelopmentCats

August 30, 2025 18:46

@35C4n0r 35C4n0r self-assigned this

Aug 30, 2025

35C4n0r added 5 commits

August 31, 2025 00:18

@35C4n0r


 Merge branch 'main' into feat-claude-agentapi

02c8c4a

# 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


 feat: additional documentation

@35C4n0r


 ref: bun fmt

7fb8b86

@35C4n0r


 feat: need to test if this works

2b23801

@35C4n0r


 feat: option to disable agentapi

4d53c76

matifali

matifali requested changes

Sep 4, 2025

View reviewed changes

registry/coder/modules/claude-code/README.md Outdated Show resolved Hide resolved

registry/coder/modules/claude-code/README.md

The `codercom/oss-dogfood:latest` container image can be used for testing on container-based workspaces.

- You must add the [Coder Login](https://registry.coder.com/modules/coder-login) module to your template for task reporting features.

- An **Anthropic API key** is required for tasks. You can get one from the [Anthropic Console](https://console.anthropic.com/dashboard).

Copy link

Member

@matifali matifali Sep 4, 2025

Choose a reason for hiding this comment

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

Lets make Tasks also work with CLAUDE_SESSION_TOKEN. For Claude.ai Pro and Max plan users.
Here is community member talking about it.
https://discord.com/channels/747933592273027093/1404654368745193492/1404692501268529206

Copy link

Collaborator Author

@35C4n0r 35C4n0r Sep 5, 2025

Choose a reason for hiding this comment

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

@matifali thanks for linking this, added it to the module.

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

@35C4n0r Can we also mention the CLAUDE_SESSION_TOKEN here?

registry/coder/modules/claude-code/README.md Show resolved Hide resolved

registry/coder/modules/claude-code/README.md Outdated Show resolved Hide resolved

registry/coder/modules/claude-code/README.md Show resolved Hide resolved

35C4n0r added 6 commits

September 5, 2025 21:59

@35C4n0r


 revert changes

fb2578c

@35C4n0r


 fix: fix tests

f5970dd

@35C4n0r


 fix: fix tests

e51b46f

@35C4n0r


 feat: add default

76fbed1

@35C4n0r


 feat: remove bedrock flag

da5a5ee

@35C4n0r


 feat: remove bedrock flag

edbf22d

@35C4n0r

Copy link

Collaborator Author

35C4n0r commented Sep 10, 2025

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

@35C4n0r


 feat: add subdomain

70dbf5f

@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


 chore: format test

06bb857

@35C4n0r 35C4n0r requested a review from matifali

September 11, 2025 02:48

35C4n0r added 2 commits

September 12, 2025 18:08

@35C4n0r


 chore: resolve claude comments

90d54c8

@35C4n0r


 feat: set subdomain default to false

a8eb4a7

- ref: coder#419

DevelopmentCats

DevelopmentCats approved these changes

Sep 12, 2025

View reviewed changes

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


 Merge branch 'main' into feat-claude-agentapi

b821ddd

matifali

matifali requested changes

Sep 13, 2025

View reviewed changes

registry/coder/modules/claude-code/README.md

The `codercom/oss-dogfood:latest` container image can be used for testing on container-based workspaces.

- You must add the [Coder Login](https://registry.coder.com/modules/coder-login) module to your template for task reporting features.

- An **Anthropic API key** is required for tasks. You can get one from the [Anthropic Console](https://console.anthropic.com/dashboard).

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

@35C4n0r Can we also mention the CLAUDE_SESSION_TOKEN here?

registry/coder/modules/claude-code/README.md Show resolved Hide resolved

registry/coder/modules/claude-code/README.md

agent_id = coder_agent.example.id

workdir = "/home/coder/project"

# --- Authentication --- required for tasks

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

Suggested change

# --- Authentication --- required for tasks

registry/coder/modules/claude-code/README.md

# OR

claude_code_oauth_token = "xxxxx-xxxx-xxxx"

# --- Versioning ---

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

Suggested change

# --- Versioning ---

registry/coder/modules/claude-code/README.md

claude_code_version = "1.0.82" # Pin to a specific version

agentapi_version = "v0.6.1"

# --- Task Configuration ---

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

Suggested change

# --- Task Configuration ---

registry/coder/modules/claude-code/README.md

continue = true # will fail in a new workspace with no conversation/session to continue

model = "sonnet"

# --- Permissions & Tools ---

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

Suggested change

# --- Permissions & Tools ---

registry/coder/modules/claude-code/README.md

# --- Permissions & Tools ---

permission_mode = "plan"

# --- MCP Configuration ---

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

Suggested change

# --- MCP Configuration ---

registry/coder/modules/claude-code/README.md

Run Claude Code as a standalone app in your workspace. This will install Claude Code and run it without any task reporting to the Coder UI.

Run Claude Code as a standalone CLI in your workspacewithout task reporting to the Coder UI.

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

Suggested change

Run Claude Code as a standalone CLI in your workspace without task reporting to the Coder UI.

Run and configure Claude Code as a standalone CLI in your workspace.

registry/coder/modules/claude-code/README.md

| -------------------------------- | ------------------------------------------------------------- | ------------------------------ |

| `CLAUDE_API_KEY` | Your Anthropic API key. | `""` |

| `CODER_MCP_CLAUDE_SYSTEM_PROMPT` | A custom system prompt for Claude. | "Send a task status update..." |

| `CODER_MCP_CLAUDE_CODER_PROMPT` | A custom coder prompt for Claude. | `""` |

Copy link

Member

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

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

What is it for?

Copy link

Collaborator Author

@35C4n0r 35C4n0r Sep 13, 2025

Choose a reason for hiding this comment

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

I'll remove the environment variable section, it's causing confusion.
ref: https://coder-uae3647.slack.com/archives/C08PHACTZRB/p1756400068946909?thread_ts=1756398053.136979&cid=C08PHACTZRB

registry/coder/modules/claude-code/README.md Show resolved Hide resolved

Labels

None yet

3 participants

@35C4n0r @matifali @DevelopmentCats

feat: refactor claude code to use agentapi module #402

Are you sure you want to change the base?

feat: refactor claude code to use agentapi module #402

Conversation

@35C4n0r 35C4n0r commented Aug 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of Change

Module Information

Testing & Validation

Related Issues

Uh oh!

Uh oh!

Uh oh!

@matifali matifali Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

@35C4n0r 35C4n0r Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

35C4n0r commented Sep 10, 2025

Uh oh!

35C4n0r commented Sep 10, 2025

Uh oh!

@DevelopmentCats DevelopmentCats left a comment

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@matifali matifali Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

@35C4n0r 35C4n0r Sep 13, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

@35C4n0r 35C4n0r commented Aug 30, 2025 •

edited

Loading