Skip to content

fix: expose agentapi_port in codex/auggie/gemini/opencode modules #838

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.

Sign up for GitHub

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

Open
qinkangdeid wants to merge 4 commits into
base: main
Choose a base branch
from
Open

fix: expose agentapi_port in codex/auggie/gemini/opencode modules #838

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2e8d99b
fix: expose agentapi_port in codex/auggie/gemini/opencode modules
Apr 8, 2026
a5aba0e
fix: propagate agentapi_port to install scripts and MCP configs
Apr 9, 2026
68a536e
Merge branch 'main' into fix/expose-agentapi-port
DevelopmentCats Apr 13, 2026
29ba14d
chore: bump versions and add tests for agentapi_port
Apr 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions registry/coder-labs/modules/auggie/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,12 @@ variable "agentapi_version" {
}
}

variable "agentapi_port" {
type = number
description = "The port used by AgentAPI."
default = 3284
}

variable "pre_install_script" {
type = string
description = "Custom script to run before installing Auggie."
Expand Down Expand Up @@ -194,6 +200,7 @@ module "agentapi" {
module_dir_name = local.module_dir_name
install_agentapi = var.install_agentapi
agentapi_version = var.agentapi_version
agentapi_port = var.agentapi_port
pre_install_script = var.pre_install_script
post_install_script = var.post_install_script
start_script = <<-EOT
Expand Down
7 changes: 7 additions & 0 deletions registry/coder-labs/modules/codex/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,12 @@ variable "agentapi_version" {
default = "v0.12.1"
}

variable "agentapi_port" {
type = number
description = "The port used by AgentAPI."
default = 3284
}

variable "codex_model" {
type = string
description = "The model for Codex to use. Defaults to gpt-5.3-codex."
Expand Down Expand Up @@ -254,6 +260,7 @@ module "agentapi" {
install_agentapi = var.install_agentapi
agentapi_subdomain = var.subdomain
agentapi_version = var.agentapi_version
agentapi_port = var.agentapi_port
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 8, 2026

Choose a reason for hiding this comment

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

P1 Badge Propagate configurable AgentAPI port into module configs

Passing agentapi_port into the agentapi submodule here does not actually reconfigure the client-side MCP wiring, because the module install paths still hardcode http://localhost:3284 (for example: registry/coder-labs/modules/codex/scripts/install.sh:132, auggie/scripts/install.sh:88, gemini/main.tf:167 and gemini/scripts/install.sh:145, opencode/scripts/install.sh:97). With any non-default agentapi_port, these modules will still point tools at 3284, so task reporting/MCP calls hit the wrong endpoint or fail entirely, which breaks the multi-tool port-conflict fix this commit is trying to add.

Useful? React with 👍 / 👎.

enable_state_persistence = var.enable_state_persistence
pre_install_script = var.pre_install_script
post_install_script = var.post_install_script
Expand Down
7 changes: 7 additions & 0 deletions registry/coder-labs/modules/gemini/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,12 @@ variable "agentapi_version" {
default = "v0.10.0"
}

variable "agentapi_port" {
type = number
description = "The port used by AgentAPI."
default = 3284
}

variable "gemini_model" {
type = string
description = "The model to use for Gemini (e.g., gemini-2.5-pro)."
Expand Down Expand Up @@ -191,6 +197,7 @@ module "agentapi" {
module_dir_name = local.module_dir_name
install_agentapi = var.install_agentapi
agentapi_version = var.agentapi_version
agentapi_port = var.agentapi_port
pre_install_script = var.pre_install_script
post_install_script = var.post_install_script
install_script = <<-EOT
Expand Down
7 changes: 7 additions & 0 deletions registry/coder-labs/modules/opencode/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,12 @@ variable "agentapi_version" {
default = "v0.11.2"
}

variable "agentapi_port" {
type = number
description = "The port used by AgentAPI."
default = 3284
}

variable "ai_prompt" {
type = string
description = "Initial task prompt for OpenCode."
Expand Down Expand Up @@ -163,6 +169,7 @@ module "agentapi" {
module_dir_name = local.module_dir_name
install_agentapi = var.install_agentapi
agentapi_version = var.agentapi_version
agentapi_port = var.agentapi_port
pre_install_script = var.pre_install_script
post_install_script = var.post_install_script
start_script = <<-EOT
Expand Down
Loading