feat: bootstrap command delegates to ConfigService (Task 6B)#14
Closed
raysonmeng wants to merge 3 commits intomasterfrom
Closed
feat: bootstrap command delegates to ConfigService (Task 6B)#14raysonmeng wants to merge 3 commits intomasterfrom
raysonmeng wants to merge 3 commits intomasterfrom
Conversation
/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>
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。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/agentbridge:initdelegates to bundledinit-project.jshelperConfigService.initDefaults()— single source of truthTest plan
/agentbridge:initin Claude Code sessionImplemented by: Codex
Reviewed by: Pending user review
🤖 Generated with Claude Code