mirror of
https://github.com/coder/registry.git
synced 2026-06-02 20:48:14 +00:00
d21f55a322
## Description Updates AGENTS.md to be more concise (~36 lines) while adding missing commands and a PR review checklist based on recent PR feedback. ## Changes - **Commands section**: Added `bun run tftest`, `bun run tstest`, single test commands, and version-bump script - **Structure section**: Added note that URLs must be relative (from #639) - **Code Style section**: Added `tf` vs `hcl` guidance and relative icon paths - **New PR Review Checklist**: Based on patterns from recent PRs including: - Version bumping requirements (#661, #617) - Breaking changes documentation (#636) - Graceful error handling in scripts (#658) - Diagnostic logging for tests (#643) - **AI attribution requirement**: PRs should note model/tool used ## Type of Change - [ ] New module - [ ] New template - [ ] Bug fix - [ ] Feature/enhancement - [x] Documentation - [ ] Other --- Generated with [Amp](https://ampcode.com/threads/T-019bcb7e-2e92-76f2-a1aa-2023ecdb0763) using Claude Sonnet 4
2.4 KiB
2.4 KiB
AGENTS.md
Coder Registry: Terraform modules/templates for Coder workspaces under registry/[namespace]/modules/ and registry/[namespace]/templates/.
Commands
bun run fmt # Format code (Prettier + Terraform) - run before commits
bun run tftest # Run all Terraform tests
bun run tstest # Run all TypeScript tests
terraform init -upgrade && terraform test -verbose # Test single module (run from module dir)
bun test main.test.ts # Run single TS test (from module dir)
./scripts/terraform_validate.sh # Validate Terraform syntax
./scripts/new_module.sh ns/name # Create new module scaffold
.github/scripts/version-bump.sh patch | minor | major # Bump module version after changes
Structure
- Modules:
registry/[ns]/modules/[name]/withmain.tf,README.md(YAML frontmatter),.tftest.hcl(required) - Templates:
registry/[ns]/templates/[name]/withmain.tf,README.md - Validation:
cmd/readmevalidation/(Go) validates structure/frontmatter; URLs must be relative, not absolute
Code Style
- Every module MUST have
.tftest.hcltests; optionalmain.test.tsfor container/script tests - README frontmatter:
display_name,description,icon,verified: false,tags - Use semantic versioning; bump version via script when modifying modules
- Docker tests require Linux or Colima/OrbStack (not Docker Desktop)
- Use
tf(nothcl) for code blocks in README; use relative icon paths (e.g.,../../../../.icons/)
PR Review Checklist
- Version bumped via
.github/scripts/version-bump.shif module changed (patch=bugfix, minor=feature, major=breaking) - Breaking changes documented: removed inputs, changed defaults, new required variables
- New variables have sensible defaults to maintain backward compatibility
- Tests pass (
bun run tftest,bun run tstest); add diagnostic logging for test failures - README examples updated with new version number; tooltip/behavior changes noted
- Shell scripts handle errors gracefully (use
|| echo "Warning..."for non-fatal failures) - No hardcoded values that should be configurable; no absolute URLs (use relative paths)
- If AI-assisted: include model and tool/agent name at footer of PR body (e.g., "Generated with Amp using Claude")