Skip to content

Commit ca1fcda

Browse files
committed
Address CodeRabbit review feedback
- Split changelog entries into proper Added/Changed sections - Remove architect from "entirely new" agent list (it's a Plan replacement) - Fix Node.js version requirement: 16.13+ or 18+, not just 18+ - Replace sudo npm install recommendation with nvm/npm-prefix advice
1 parent a08e964 commit ca1fcda

File tree

4 files changed

+15
-11
lines changed

4 files changed

+15
-11
lines changed

.devcontainer/CHANGELOG.md

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,18 @@
22

33
## [Unreleased]
44

5+
### Added
6+
7+
#### Documentation
8+
- **DevContainer CLI guide** — dedicated Getting Started page for terminal-only workflows without VS Code
9+
- **v2 Migration Guide** — path changes, automatic migration, manual steps, breaking changes, and troubleshooting
10+
- Documented 4 previously undocumented agents in agents.md: implementer, investigator, tester, documenter
11+
- Added missing git-workflow and prompt-snippets to configuration.md enabledPlugins example
12+
- Added CONFIG_SOURCE_DIR deprecation note in environment variables reference
13+
- Added cc-orc orchestrator command to first-session launch commands table
14+
- Tabbed client-specific instructions on the installation page
15+
- Dedicated port forwarding reference page covering VS Code auto-detect, devcontainer-bridge, and SSH tunneling
16+
517
### Changed
618

719
#### Performance
@@ -14,22 +26,14 @@
1426
- Dynamic port forwarding for all ports in VS Code — previously only port 7847 was statically forwarded; now all ports auto-forward with notification
1527

1628
#### Documentation
17-
- Added **DevContainer CLI guide** — dedicated Getting Started page for terminal-only workflows without VS Code
18-
- Added **v2 Migration Guide** — path changes, automatic migration, manual steps, breaking changes, and troubleshooting
1929
- Updated **Port Forwarding reference** — VS Code dependency warning, devcontainer-bridge platform support matrix, CLI guide cross-link
2030
- Slimmed **Installation page** — moved troubleshooting to dedicated reference page, CLI details to new CLI guide
2131
- Full documentation review — accuracy, consistency, and completeness fixes across all 30+ pages
22-
- Documented 4 previously undocumented agents in agents.md: implementer, investigator, tester, documenter
2332
- Trimmed disabled ccms usage section from commands reference
2433
- Clarified codeforge-lsp plugin description (declarative config, not "no configuration")
2534
- Improved magic-docs agent explanation in agent-system plugin docs
2635
- Clarified plugin count as "13 local + 1 official" in reference index
27-
- Added missing git-workflow and prompt-snippets to configuration.md enabledPlugins example
28-
- Added CONFIG_SOURCE_DIR deprecation note in environment variables reference
29-
- Added cc-orc orchestrator command to first-session launch commands table
3036
- Updated prerequisites and installation docs to support all DevContainer clients (VS Code, CLI, JetBrains Gateway, DevPod, Codespaces)
31-
- Added tabbed client-specific instructions on the installation page
32-
- Added dedicated port forwarding reference page covering VS Code auto-detect, devcontainer-bridge, and SSH tunneling
3337
- **Ported `.devcontainer/docs/` to docs site** — migrated content from 5 legacy reference docs into the Starlight documentation site:
3438
- New **Keybindings** page (Customization) — VS Code/Claude Code shortcut conflicts and resolution options
3539
- New **Troubleshooting** page (Reference) — 12+ problem/solution entries for build, auth, plugins, and performance issues

docs/src/content/docs/features/agents.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ CodeForge **entirely replaces** all six built-in agents with enhanced custom spe
3030

3131
The redirect is fully transparent — you can use either the built-in name or the custom name interchangeably. Asking Claude to "explore the codebase" triggers the same enhanced explorer agent whether the system selects the `Explore` type or the `explorer` type.
3232

33-
Beyond the six replacements, CodeForge adds **15 entirely new specialists** that have no built-in equivalent: architect, debug-logs, dependency-analyst, doc-writer, documenter, git-archaeologist, implementer, investigator, migrator, perf-profiler, refactorer, researcher, security-auditor, spec-writer, tester, and test-writer. These are available only in CodeForge.
33+
Beyond the six replacements, CodeForge adds **15 entirely new specialists** that have no built-in equivalent: debug-logs, dependency-analyst, doc-writer, documenter, git-archaeologist, implementer, investigator, migrator, perf-profiler, refactorer, researcher, security-auditor, spec-writer, tester, and test-writer. These are available only in CodeForge.
3434

3535
:::tip[Why This Matters]
3636
The redirect happens at the hook level, not the prompt level. This means the upgrade is enforced — not suggested. Even if Claude's internal routing tries to use a stock Explore agent, the hook intercepts the call and swaps in the enhanced explorer before any code executes. The result is a strictly better agent every time, with zero user effort.

docs/src/content/docs/getting-started/devcontainer-cli.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ The DevContainer CLI lets you build and run CodeForge containers without VS Code
1313
npm install -g @devcontainers/cli
1414
```
1515

16-
Requires Node.js 18+. Verify the installation with `devcontainer --version`.
16+
Requires Node.js 16.13+ or 18+ (when installing via npm). Verify the installation with `devcontainer --version`.
1717

1818
## Build and Start
1919

docs/src/content/docs/reference/troubleshooting.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ Solutions for common issues with the CodeForge devcontainer. If your problem isn
1313

1414
- **Node.js not installed** — the installer requires Node.js 18+ and npm. Run `node --version` to check; install from [nodejs.org](https://nodejs.org/) if missing.
1515
- **Network issues** — npm needs to reach the registry. Check your connection or try `npm config set registry https://registry.npmjs.org/`.
16-
- **Permission errors**on some systems, global npm installs need `sudo`. Try `npx --yes codeforge-dev` or install globally with `sudo npm install -g codeforge-dev`.
16+
- **Permission errors**prefer `npx --yes codeforge-dev` to avoid global install permission issues. If you need global installs, configure your npm prefix or use a Node version manager (e.g., nvm) instead of `sudo`.
1717

1818
**Problem: VS Code doesn't show "Reopen in Container".**
1919

0 commit comments

Comments
 (0)