Uh oh!
There was an error while loading. Please reload this page.
refactor(registry/coder/modules/coder-utils)!: derive names from module_directory - #874
Conversation
…t test - README: set the usage example version to 2.0.0. The previous value was a placeholder. - main.test.ts: prettier-format the imports and expect line.
de3d8a4 to
cfa7a24Compare@matifali If we merge this:
|
matifali
commented
Apr 24, 2026
This is not required. terraform will fail even if the nested validation fails
That is fine. Being restrictive is better than permissive. We can relax it later if needed.
I think it is ok. We are the only consumer of this module today. We can add support for multiple instances when needed. |
Uh oh!
There was an error while loading. Please reload this page.
Summary
Derives
coder-utilsscript names frommodule_directoryinstead of a separateagent_nameinput. Themodule_directoryalready encodes both the namespace and the module name, so carrying both is redundant and error-prone. Callers likeclaude-codeno longer need to passagent_name.Scripts this module materializes lose the
${agent_name}-utils-prefix becausemodule_directoryalready namespaces them per-caller.We will address multiple instances of coder-utils per caller in a future iteration if needed.
Versioning Note
Previous tags (
v1.0.0throughv1.3.0) have been deleted because no published module ever consumed them — the module was effectively unreleased. This PR ships the first real public version asv0.0.1, treating it as a fresh start rather than a breaking bump from a version that was never in production use.Changes
agent_namevariable.caller_name = "${namespace}-${module_name}"frommodule_directory.module_directorymatches$HOME/.coder-modules/<namespace>/<module-name>.${agent_name}-utils-<phase>.shto plain<phase>.sh.module_directoryvalidation.v0.0.1(first published version; all prior tags removed).Breaking Changes
agent_name = "myagent"module_directory)module_directory = ".my-module"module_directory = "$HOME/.coder-modules/<ns>/<name>"(validated)${agent_name}-utils-install.shinstall.sh${agent_name}-install_script${namespace}-${module_name}-install_scriptNo callers were depending on the old format (prior tags were unpublished).
Validation
terraform fmt -recursivecleanterraform validatecleanterraform test→ 17/17 passbun test registry/coder/modules/coder-utils→ 5/5 passprettier --checkcleanConsumer
#861 (
claude-code) consumes this and is currently pinned to the commit SHA until this merges and ships asv0.0.1.