feat(claude-code): add web_app variable to disable the web app#764
Draft
blinkagent[bot] wants to merge 5 commits intomainfrom
Draft
feat(claude-code): add web_app variable to disable the web app#764blinkagent[bot] wants to merge 5 commits intomainfrom
web_app variable to disable the web app#764blinkagent[bot] wants to merge 5 commits intomainfrom
Conversation
Adds a `web_app` variable (default: true) to both the claude-code and agentapi modules. When set to false, AgentAPI still runs but the web UI app icon is not shown in the Coder dashboard. This mirrors the existing `cli_app` toggle pattern.
The claude-code module needs to reference the local agentapi module so that both sets of changes are tested together. This should be reverted to the registry source before merging once agentapi is published with the web_app variable.
The claude-code module must use the published agentapi registry source. The web_app variable is added to claude-code for future use, but the pass-through to agentapi is deferred until agentapi is published with web_app support.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a
web_appvariable (default:true) to both theclaude-codeandagentapimodules. When set tofalse, AgentAPI still runs but the web UI app icon is not shown in the Coder dashboard.This mirrors the existing
cli_apptoggle pattern.Changes
agentapimoduleweb_appvariable (bool, defaulttrue)coder_app.agentapi_webnow hascount = var.web_app ? 1 : 0task_app_idoutput returns""whenweb_appisfalseclaude-codemoduleweb_appvariable (bool, defaulttrue) — added for future useTODOcomment to wireweb_appthrough to agentapi once the agentapi module is published with this supportUsage (once fully wired)
Merge strategy
This needs to land in two steps:
web_appsupport, and adds theweb_appvariable to claude-code (not yet wired through)TODOwithweb_app = var.web_app