feat: template builder backend fixes (DEVEX-287)#26432
Open
jeremyruppel wants to merge 4 commits into
Open
Conversation
Docs preview📖 View docs preview for |
This was referenced Jun 16, 2026
Contributor
Author
This was referenced Jun 16, 2026
39f07dd to
bc5ca0b
Compare
8868d03 to
9ec0473
Compare
9ec0473 to
df0f468
Compare
bc5ca0b to
6723495
Compare
df0f468 to
e41a23d
Compare
6723495 to
4c13d89
Compare
Implement the module settings wizard step: - Add ModuleSettingsStep component that renders a ModuleConfiguration card per selected module with configurable variable fields - Map non-sensitive variables to ConfigurationField definitions (switch for bool, text input for string/number) - Show info notice for sensitive variables indicating they will be collected from developers at workspace creation - Export moduleSettingsComplete helper to validate required fields - Extend canContinue in PageView to gate module-settings step - Wire the step into TemplateBuilderPageView with SET_MODULE_VARIABLES dispatch
…URL default Terraform module registry source addresses must not include a URL scheme. The default value 'https://registry.coder.com' produced invalid source paths like 'https://registry.coder.com/coder/code-server/coder', causing 'terraform init' to fail with 'failed to parse module registry address'. Change the default to 'registry.coder.com'.
… declarations Sensitive variable blocks in module .tf.tmpl files and the modulegen script were missing default = "", causing terraform plan to prompt for values during template import.
The backend now accepts raw string values from callers (e.g. "anthropic") and wraps them in HCL quotes automatically via hclQuote(). Previously callers were required to send pre-quoted HCL literals, which is not a reasonable API contract for frontend consumers. - validateStringValue now validates raw strings (rejects interpolation and overlong values) - toHCLLiteral wraps string values in quotes with proper escaping - hclQuote handles backslash, quote, newline, and carriage return escaping - null is passed through unquoted for all types
4c13d89 to
9d3600f
Compare
e41a23d to
6589566
Compare
This was referenced Jun 18, 2026
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
1 participant
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.
Part of the Template Builder wizard PR stack.
Backend fixes
Registry URL scheme fix: Default
CODER_TEMPLATE_BUILDER_REGISTRY_URLwashttps://registry.coder.combut Terraform module registry addresses must be scheme-less. Changed toregistry.coder.com.Sensitive variable defaults: Module
.tf.tmplfiles for claude-code, aider, amazon-q had sensitivevariableblocks withoutdefault, causingterraform planto fail during template import. Also fixed thetemplatebuildermodulegenscript.Auto-quote string variables: The backend now accepts raw string values from callers and wraps them in HCL quotes automatically. Previously callers were required to send pre-quoted HCL literals, which is not a reasonable API contract.
Note
Generated by Coder Agents on behalf of @jeremyruppel