feat(agentapi,claude-code): expose hidden parameter for web app#730
feat(agentapi,claude-code): expose hidden parameter for web app#730zek wants to merge 1 commit intocoder:mainfrom
Conversation
Add `web_app_hidden` variable to both the `agentapi` and `claude-code` modules, passing it through to `coder_app.agentapi_web.hidden`. This allows users to hide the AgentAPI web chat UI from the Coder dashboard while keeping the app functional for healthchecks and task status reporting (required by `coder_ai_task`). Requires Coder v2.16+ (when `hidden` attribute was added to `coder_app`). Closes coder#729
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 60ef7d1755
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| web_app_group = var.group | ||
| web_app_icon = var.icon | ||
| web_app_display_name = var.web_app_display_name | ||
| web_app_hidden = var.web_app_hidden |
There was a problem hiding this comment.
Update agentapi child version before passing web_app_hidden
This forwards web_app_hidden into the agentapi child module, but the child is still pinned to version = "2.0.0" in the same block. The new input was added in this commit under registry/coder/modules/agentapi/main.tf, so claude-code consumers can hit Unsupported argument during module loading when Terraform resolves agentapi v2.0.0, even if they do not explicitly set web_app_hidden.
Useful? React with 👍 / 👎.
| order = var.web_app_order | ||
| group = var.web_app_group | ||
| subdomain = var.agentapi_subdomain | ||
| hidden = var.web_app_hidden |
There was a problem hiding this comment.
Raise coder provider floor when using coder_app.hidden
The module now sets coder_app.hidden, but required_providers.coder in this file still allows >= 2.13; users with lockfiles or environments pinned below the version that introduced hidden will fail during planning with an unsupported argument/schema mismatch. Bumping the minimum provider version to the first release that supports hidden avoids this compatibility break.
Useful? React with 👍 / 👎.
Summary
web_app_hiddenvariable to theagentapimodule, passing it through tocoder_app.agentapi_web.hiddenweb_app_hiddenvariable to theclaude-codemodule, forwarding it to theagentapisub-moduleThis allows users to hide the AgentAPI web chat UI from the Coder dashboard while keeping the app fully functional for healthchecks and task status reporting (required by
coder_ai_task).Use Case
Users who prefer Claude Code via terminal (SSH / code-server) can set
web_app_hidden = trueto declutter their dashboard without losing the Tasks feature.Changes
registry/coder/modules/agentapi/main.tfweb_app_hiddenvariable + pass tocoder_app.agentapi_web.hiddenregistry/coder/modules/claude-code/main.tfweb_app_hiddenvariable + forward to agentapi moduleRequires Coder v2.16+ (when
hiddenwas added tocoder_app).Closes #729