feat: add cmux module #523
Conversation
|
|
||
| - **Parallel Agent Execution**: Run multiple AI agents simultaneously on different tasks | ||
| - **Workspace Isolation**: Each agent works in its own isolated environment | ||
| - **Git Divergence Visualization**: Track changes across different agent workspaces |
There was a problem hiding this comment.
Calling out workspaces here seems potentially confusing for a module that is being installed inside a single workspace. Arguably it can be used from within the workspace to connect to other remote workspaces too but that seems like a power-user feature. I propose adjusting the wording here.
There was a problem hiding this comment.
It's a naming difference between cmux and Coder. For cmux a "workspace" is a single instace of a git worktree on a project (repo) you're working on. It's verbatim from: https://github.com/coder/cmux/?tab=readme-ov-file#features
There was a problem hiding this comment.
Oh, that's fine then. I still think it's worth calling out the difference in say, a parenthesis, (this refers to cmux workspaces, not Coder workspaces). Mainly because as a Coder module this may confuse others as it has me 😄.
| default = null | ||
| } | ||
|
|
||
| variable "offline" { |
There was a problem hiding this comment.
Suggestion: This could also just be install, default true. I know it's probably used in other modules, but I think offline is not as descriptive and could be confusing.
| lifecycle { | ||
| precondition { | ||
| condition = !var.offline || !var.use_cached | ||
| error_message = "Offline and Use Cached can not be used together" |
There was a problem hiding this comment.
| error_message = "Offline and Use Cached can not be used together" | |
| error_message = "The \"offline\" and \"use_cached\" parameters can not be used together." |
3 participants
Description
Type of Change
Module Information
Path:
registry/[namespace]/modules/[module-name]New version:
v1.0.0Breaking change: [ ] Yes [ ] No
Template Information
Path:
registry/[namespace]/templates/[template-name]Testing & Validation
bun test)bun fmt)Related Issues