Skip to content

feat(coder/modules/boundary): add boundary module#840

Open
35C4n0r wants to merge 41 commits intomainfrom
35C4n0r/feat-boundary-module
Open

feat(coder/modules/boundary): add boundary module#840
35C4n0r wants to merge 41 commits intomainfrom
35C4n0r/feat-boundary-module

Conversation

@35C4n0r
Copy link
Copy Markdown
Collaborator

@35C4n0r 35C4n0r commented Apr 13, 2026

Description

Extracts boundary installation and wrapper logic into a standalone coder/boundary module, decoupling it from agentapi.

Why

Boundary is currently embedded inside agentapi (scripts/boundary.sh) and duplicated in claude-code. This couples network isolation to the AI/Tasks stack, but boundary is a general-purpose primitive — users running a plain agent with no agentapi or tasks should be able to use it too.

What this adds

registry/coder/modules/boundary/ — a new first-class module that:

  • Installs boundary via one of three strategies:
    1. coder boundary subcommand (default, zero-install)
    2. Direct binary from release (use_boundary_directly = true)
    3. Compiled from source (compile_boundary_from_source = true)
  • Creates a wrapper script at $HOME/.coder-modules/coder/boundary/boundary-wrapper.sh
  • Exports BOUNDARY_WRAPPER_PATH as a coder_env so any workspace process can use it
  • Strips CAP_NET_ADMIN from the coder binary (copies to coder-no-caps) to allow execution inside network namespaces without sys_admin
  • Supports pre_install_script / post_install_script hooks
  • Exposes boundary_wrapper_path output and sync_script_names for script coordination

Usage

module "boundary" {
  count    = data.coder_workspace.me.start_count
  source   = "registry.coder.com/coder/boundary/coder"
  version  = "0.0.1"
  agent_id = coder_agent.main.id
}

Works standalone with any agent — no agentapi dependency required.

Testing

  • Terraform variable validation (boundary.tftest.hcl)
  • TypeScript integration tests (main.test.ts): state verification, coder subcommand happy path, custom hooks, env var correctness, wrapper execution, idempotent installation

Type of Change

  • New module

Module Information

Path: registry/coder/modules/boundary
New version: v0.0.1
Breaking change: No

Related Issues

Closes #844

🤖 Generated by Coder Agents

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new coder/boundary registry module intended to set up Boundary-related tooling for Coder workspaces.

Changes:

  • Introduces a Boundary install/setup shell script that can compile from source, install from release, or rely on coder boundary.
  • Adds a Terraform module (main.tf) that deploys and runs the install script on an agent.
  • Adds module README and Terraform native tests (.tftest.hcl).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File Description
registry/coder/modules/boundary/scripts/install.sh Installs Boundary (or validates coder boundary) and generates a wrapper script.
registry/coder/modules/boundary/main.tf Defines module variables and a coder_script to deliver/execute install.sh.
registry/coder/modules/boundary/README.md Documents module usage and examples.
registry/coder/modules/boundary/boundary.tftest.hcl Adds Terraform plan-time assertions for basic module wiring.

Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh
Comment thread registry/coder/modules/boundary/main.tf Outdated
Comment thread registry/coder/modules/boundary/README.md
@35C4n0r 35C4n0r marked this pull request as draft April 13, 2026 03:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Comment thread registry/coder/modules/boundary/main.tf Outdated
Comment thread registry/coder/modules/boundary/main.tf Outdated
Comment thread registry/coder/modules/boundary/main.test.ts Outdated
Comment thread registry/coder/modules/boundary/testdata/boundary-mock.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Comment thread registry/coder/modules/boundary/scripts/install.sh Outdated
Jay Kumar and others added 8 commits April 14, 2026 18:19
…missing mock support

Three root causes:

1. boundary_script_destination used 'install.sh' - same filename that coder-utils
   writes to. This caused the running script to overwrite itself, corrupting
   bash's incremental read and producing empty install.log / no wrapper.
   Fix: rename to 'boundary-install.sh'.

2. coder-mock.sh didn't handle 'coder exp sync' commands used by coder-utils
   for script ordering. With set -o errexit, scripts failed immediately.
   Fix: add exp sync as no-op (exit 0).

3. Test setup used setupUtil which only extracts ONE coder_script, but
   coder-utils creates multiple (pre_install, install, post_install).
   Fix: extract all coder_scripts from terraform state and run them
   sequentially in lifecycle order.

4. wrapper-script-execution test called 'wrapper.sh --help' which the mock
   couldn't handle after the '--' separator (tried to exec '--help').
   Fix: test with 'echo boundary-test' instead.
The boundary command (both 'coder boundary' and standalone 'boundary')
expects a '--' separator before the command to execute. The wrapper
scripts were passing arguments directly without this separator, causing
the wrapper-script-execution test to fail.

🤖 Generated by Coder Agents
The boundary wrapper scripts pass arguments directly without a '--'
separator. Updated the coder mock to match this behavior and adjusted
the test comment accordingly.

🤖 Generated by Coder Agents
@35C4n0r 35C4n0r marked this pull request as ready for review April 15, 2026 17:27
@matifali matifali requested review from SasSwart, dannykopping and jcjiang and removed request for matifali April 16, 2026 04:50
35C4n0r and others added 5 commits April 23, 2026 10:59
Update README and tests to match the env var and resource rename from
the latest code changes.

🤖 Generated by Coder Agents
…_WRAPPER_PATH

Update install.sh comments/echo, and boundary.tftest.hcl resource
references to match the env var rename.

🤖 Generated by Coder Agents
Comment thread registry/coder/modules/boundary/README.md Outdated
Jay Kumar added 2 commits April 23, 2026 05:53
The sync_script_names output changed from an object with
.script_names.install to a flat list of script name strings.
Updated boundary.tftest.hcl to use contains() and README to
document the actual script name values.

🤖 Generated by Coder Agents
Comment thread registry/coder/modules/boundary/main.tf Outdated
Comment thread registry/coder/modules/boundary/README.md Outdated
Comment thread registry/coder/modules/boundary/README.md Outdated
Comment thread registry/coder/modules/boundary/README.md Outdated
Comment thread registry/coder/modules/boundary/README.md Outdated
Comment thread registry/coder/modules/boundary/main.tf
Comment thread registry/coder/modules/boundary/scripts/install.sh.tftpl
35C4n0r and others added 14 commits April 23, 2026 12:13
…utput to scripts

- Wrapper path: boundary-wrapper.sh → scripts/boundary-wrapper.sh
- Log paths: *.log → logs/*.log
- Output renamed: sync_script_names → scripts
- Updated boundary.tftest.hcl, main.test.ts, and README.md

🤖 Generated by Coder Agents
The wrapper path moved to scripts/boundary-wrapper.sh but install.sh
never created the scripts/ subdirectory, causing a 'No such file or
directory' error at runtime.

🤖 Generated by Coder Agents
The wrapper script moved to scripts/ but coder-no-caps remains in the
module root. Updated the wrapper to use ${SCRIPT_DIR}/../coder-no-caps
so it correctly resolves the binary path.

🤖 Generated by Coder Agents
- boundary-install.sh → scripts/boundary-install.sh
- coder-no-caps → scripts/coder-no-caps
- Wrapper script now references coder-no-caps in the same directory
- Updated test path for coder-no-caps

🤖 Generated by Coder Agents
- Destination script renamed to scripts/install.sh for consistency
- Fixed coder-utils module source (stale branch ref → main)

🤖 Generated by Coder Agents
@35C4n0r 35C4n0r force-pushed the 35C4n0r/feat-boundary-module branch from eed787d to 4dd0176 Compare April 23, 2026 18:06
35C4n0r and others added 6 commits April 24, 2026 21:35
- Renamed scripts/install.sh → scripts/install.sh.tftpl
- Use templatefile() to inject variables at plan time instead of
  base64-encoding the script and passing env vars at runtime
- Removed boundary_script/boundary_script_destination locals
- Fixed /home/coder expansion in MODULE_DIR and BOUNDARY_WRAPPER_PATH
  (double quotes for shell expansion)
- Updated custom module_directory test paths to match coder-utils
  validation pattern ($HOME/.coder-modules/<ns>/<name>)
- Updated script name references (coder_boundary → coder-boundary)
- Fixed coder-utils source to use branch ref

🤖 Generated by Coder Agents
🤖 Generated by Coder Agents
🤖 Generated by Coder Agents
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Decouple boundary into its own standalone module

4 participants