Skip to content

feat: bootstrap command delegates to ConfigService (Task 6B)#14

Closed
raysonmeng wants to merge 3 commits intomasterfrom
feat/bootstrap-command
Closed

feat: bootstrap command delegates to ConfigService (Task 6B)#14
raysonmeng wants to merge 3 commits intomasterfrom
feat/bootstrap-command

Conversation

@raysonmeng
Copy link
Copy Markdown
Owner

Summary

  • /agentbridge:init delegates to bundled init-project.js helper
  • Reuses ConfigService.initDefaults() — single source of truth
  • 3 new tests: bootstrap, idempotent re-run, missing codex error

Test plan

  • Unit tests pass
  • Smoke test: run init-project.js in temp directory
  • End-to-end: /agentbridge:init in Claude Code session

Implemented by: Codex
Reviewed by: Pending user review

🤖 Generated with Claude Code

rayson951005@gmail.com and others added 3 commits March 25, 2026 09:14
/agentbridge:init slash command now calls a bundled init-project.js
helper that reuses ConfigService for project config initialization,
eliminating template drift between CLI and plugin paths.

Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1. Replace execSync with execFileSync + argument array to eliminate
   shell interpretation surface in defaultRunCommand.

2. Add missing test for bun prerequisite check failure.

Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The bundled helper was still using the old execSync implementation
while the source was already fixed. Regenerate to close the gap.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@raysonmeng
Copy link
Copy Markdown
Owner Author

Closing: This PR is 5 days old and master has changed significantly (PR #31, #35, #36, #38 merged since). The bootstrap command work (Task 6B) needs to be re-evaluated against the current codebase.

If this is still needed, please open a fresh PR rebased on current master.

关闭:此 PR 已过期 5 天,master 变化很大(PR #31#35#36#38 已合并)。Bootstrap 命令功能(Task 6B)需要在当前代码基础上重新评估。

如果仍需要此功能,请基于最新 master 重新开 PR。

@raysonmeng raysonmeng closed this Mar 30, 2026
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.

1 participant