From 80f429faf10a4e31ba3ab8849c439e96e221c7d0 Mon Sep 17 00:00:00 2001 From: DevCats Date: Tue, 30 Sep 2025 07:44:41 -0500 Subject: [PATCH 1/3] chore: remove it wrappers from required variables tests (#442) ## Description Remove it wrappers from required variables tf test in jfrog-oauth and jfrog-token modules. This solves the failing tf tests that we were encountering in all PR's across the board. ## Type of Change - [ ] New module - [X] Bug fix - [ ] Feature/enhancement - [ ] Documentation - [ ] Other ## Testing & Validation - [X] Tests pass (`bun test`) - [X] Code formatted (`bun run fmt`) - [X] Changes tested locally ## Related Issues --- registry/coder/modules/jfrog-oauth/main.test.ts | 10 ++++------ registry/coder/modules/jfrog-token/main.test.ts | 12 +++++------- 2 files changed, 9 insertions(+), 13 deletions(-) diff --git a/registry/coder/modules/jfrog-oauth/main.test.ts b/registry/coder/modules/jfrog-oauth/main.test.ts index 0a8e62c0..940d166b 100644 --- a/registry/coder/modules/jfrog-oauth/main.test.ts +++ b/registry/coder/modules/jfrog-oauth/main.test.ts @@ -24,12 +24,10 @@ describe("jfrog-oauth", async () => { const fakeFrogUrl = "http://localhost:8081"; const user = "default"; - it("can run apply with required variables", async () => { - testRequiredVariables(import.meta.dir, { - agent_id: "some-agent-id", - jfrog_url: fakeFrogUrl, - package_managers: "{}", - }); + testRequiredVariables(import.meta.dir, { + agent_id: "some-agent-id", + jfrog_url: fakeFrogUrl, + package_managers: "{}", }); it("generates an npmrc with scoped repos", async () => { diff --git a/registry/coder/modules/jfrog-token/main.test.ts b/registry/coder/modules/jfrog-token/main.test.ts index 9e3097b0..419b5f28 100644 --- a/registry/coder/modules/jfrog-token/main.test.ts +++ b/registry/coder/modules/jfrog-token/main.test.ts @@ -55,13 +55,11 @@ describe("jfrog-token", async () => { const user = "default"; const token = "xxx"; - it("can run apply with required variables", async () => { - testRequiredVariables(import.meta.dir, { - agent_id: "some-agent-id", - jfrog_url: fakeFrogUrl, - artifactory_access_token: "XXXX", - package_managers: "{}", - }); + testRequiredVariables(import.meta.dir, { + agent_id: "some-agent-id", + jfrog_url: fakeFrogUrl, + artifactory_access_token: "XXXX", + package_managers: "{}", }); it("generates an npmrc with scoped repos", async () => { From 80acbd7e3a95290fe1ebb562b6675d1f08a074c9 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 30 Sep 2025 12:53:03 +0000 Subject: [PATCH 2/3] chore(deps): bump crate-ci/typos from 1.36.2 to 1.36.3 in the github-actions group (#438) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- .github/workflows/ci.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 1bdc5f68..39eb0197 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -48,7 +48,7 @@ jobs: - name: Validate formatting run: bun fmt:ci - name: Check for typos - uses: crate-ci/typos@v1.36.2 + uses: crate-ci/typos@v1.36.3 with: config: .github/typos.toml validate-readme-files: From 44354b202dc76988462bb508cad4433d432171df Mon Sep 17 00:00:00 2001 From: Atif Ali Date: Tue, 30 Sep 2025 18:02:35 +0500 Subject: [PATCH 3/3] Fix claude-code module not passing workdir to agentapi (#439) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary Fixes #436 - The claude-code 3.0.0 module was not passing the custom `workdir` variable to the agentapi module, causing it to default to `/home/coder` instead of using the specified working directory. ## Changes - Added missing `folder = local.workdir` parameter to the agentapi module call in `main.tf:247` - This ensures that custom working directories are properly propagated to the agentapi module ## Test Plan - [x] Terraform validation passes - [x] Code formatting applied with `bun run fmt` - [x] Basic terraform test passes (one pre-existing test failure unrelated to this change) ## Verification The fix adds the missing parameter that was identified in the issue: ```terraform module "agentapi" { # ... other parameters folder = local.workdir # <- Added this line # ... rest of configuration } ``` 🤖 Generated with [Claude Code](https://claude.ai/code) --------- Co-authored-by: DevCats --- registry/coder/modules/claude-code/README.md | 8 ++++---- registry/coder/modules/claude-code/main.tf | 1 + 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/registry/coder/modules/claude-code/README.md b/registry/coder/modules/claude-code/README.md index 04d8f8c8..3c334e74 100644 --- a/registry/coder/modules/claude-code/README.md +++ b/registry/coder/modules/claude-code/README.md @@ -13,7 +13,7 @@ Run the [Claude Code](https://docs.anthropic.com/en/docs/agents-and-tools/claude ```tf module "claude-code" { source = "registry.coder.com/coder/claude-code/coder" - version = "3.0.0" + version = "3.0.1" agent_id = coder_agent.example.id workdir = "/home/coder/project" claude_api_key = "xxxx-xxxxx-xxxx" @@ -49,7 +49,7 @@ data "coder_parameter" "ai_prompt" { module "claude-code" { source = "registry.coder.com/coder/claude-code/coder" - version = "3.0.0" + version = "3.0.1" agent_id = coder_agent.example.id workdir = "/home/coder/project" @@ -85,7 +85,7 @@ Run and configure Claude Code as a standalone CLI in your workspace. ```tf module "claude-code" { source = "registry.coder.com/coder/claude-code/coder" - version = "3.0.0" + version = "3.0.1" agent_id = coder_agent.example.id workdir = "/home/coder" install_claude_code = true @@ -108,7 +108,7 @@ variable "claude_code_oauth_token" { module "claude-code" { source = "registry.coder.com/coder/claude-code/coder" - version = "3.0.0" + version = "3.0.1" agent_id = coder_agent.example.id workdir = "/home/coder/project" claude_code_oauth_token = var.claude_code_oauth_token diff --git a/registry/coder/modules/claude-code/main.tf b/registry/coder/modules/claude-code/main.tf index d391e479..4836347b 100644 --- a/registry/coder/modules/claude-code/main.tf +++ b/registry/coder/modules/claude-code/main.tf @@ -244,6 +244,7 @@ module "agentapi" { web_app_group = var.group web_app_icon = var.icon web_app_display_name = var.web_app_display_name + folder = local.workdir cli_app = var.cli_app cli_app_slug = var.cli_app ? "${local.app_slug}-cli" : null cli_app_display_name = var.cli_app ? var.cli_app_display_name : null