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

Skip to content

Conversation

blink-so[bot]
Copy link
Contributor

@blink-so blink-so bot commented Sep 5, 2025

Summary

Adds an enable_tasks variable to the agentapi module that allows users to conditionally create the coder_ai_task resource.

Problem

When using the agentapi module with count = 0 at the module level, Terraform still validates all resources inside the module during the planning phase. This causes validation errors because the coder_ai_task resource requires an "AI Prompt" parameter to be defined in the template, even when the module won't be instantiated.

Solution

  • Added enable_tasks boolean variable (defaults to true for backward compatibility)
  • Added count = var.enable_tasks ? 1 : 0 to the coder_ai_task resource
  • Updated README.md with usage example

Usage

module "agentapi" {
  source = "registry.coder.com/coder/agentapi/coder"
  
  # ... other variables ...
  enable_tasks = false  # Disables coder_ai_task creation
}

When enable_tasks = false, the coder_ai_task resource will not be created, avoiding the validation requirement for the "AI Prompt" parameter.

Testing

Tested locally - the changes maintain backward compatibility (defaults to true) and resolve the validation issue when set to false.

blink-so bot added 2 commits September 5, 2025 05:39
…er_ai_task resource

This allows users to disable AI task creation when using the agentapi module
with count = 0, avoiding the validation requirement for 'AI Prompt' parameter.

When enable_tasks = false, the coder_ai_task resource will not be created,
allowing the module to be used without defining the required AI Prompt parameter.
Remove extra space before comment to match prettier formatting rules.
@matifali matifali requested a review from 35C4n0r September 5, 2025 05:43
@matifali matifali added the version:minor Add to PRs requiring a minor version upgrade label Sep 5, 2025
@matifali matifali marked this pull request as ready for review September 5, 2025 05:44
Terraform fmt requires consistent spacing in resource blocks.
Copy link

github-actions bot commented Sep 5, 2025

❌ Version Bump Validation Failed

Bump Type: minor

Module versions need to be updated but haven't been bumped yet.

Required Actions:

  1. Run the version bump script locally: ./.github/scripts/version-bump.sh minor
  2. Commit the changes: git add . && git commit -m "chore: bump module versions (minor)"
  3. Push the changes: git push

Script Output:

🔍 Detecting modified modules...
Found modules:
registry/coder/modules/agentapi

📦 Processing: coder/agentapi
Found git tag: release/coder/agentapi/v1.1.1 (v1.1.1)
Current version: 1.1.1
New version: 1.2.0
Updating version references for coder/agentapi in registry/coder/modules/agentapi/README.md

🔧 Running formatter to ensure consistent formatting...

📋 Summary:
Bump Type: minor

Modules Updated:

- coder/agentapi: v1.1.1 → v1.2.0

READMEs Updated:

- coder/agentapi

✅ Version bump completed successfully!
📝 README files have been updated with new versions.

Next steps:
1. Review the changes: git diff
2. Commit the changes: git add . && git commit -m 'chore: bump module versions (minor)'
3. Push the changes: git push

Please update the module versions and push the changes to continue.

@35C4n0r
Copy link
Collaborator

35C4n0r commented Sep 5, 2025

I'll try to run the claude-code module with these changes, and then merge this.

@matifali matifali marked this pull request as draft September 5, 2025 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version:minor Add to PRs requiring a minor version upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants