Skip to content

[T-42C5A6] Update implementor default prompt#57

Merged
stilero merged 3 commits into
mainfrom
feature/t-42c5a6-update-implementor-prompt
Mar 20, 2026
Merged

[T-42C5A6] Update implementor default prompt#57
stilero merged 3 commits into
mainfrom
feature/t-42c5a6-update-implementor-prompt

Conversation

@weekly-daniel

Copy link
Copy Markdown
Collaborator

Summary

Replace the minimal implementor default prompt with a comprehensive prompt covering subagent usage, autonomy, core principles, test verification, and definition of done.

Key Changes

  • server/src/config.js (replace the implementation string in DEFAULT_PROMPTS with the expanded prompt)

Review

  • Verdict: PASS
  • Summary: Changed file: server/src/config.js (plus cosmetic peer flag shuffles in three package-lock.json files). The implementation default prompt in DEFAULT_PROMPTS was expanded from 5 bullet points to a well-structured multi-section prompt covering subagent usage, autonomy, core principles, TDD/test verification, and definition of done. All original guidance (install missing deps, commit after logical units, final commit) is preserved. The new text is a string literal change only — no logic, control flow, or API surface is affected. The prompt aligns with the project's own CLAUDE.md principles, which is a nice consistency win. The package-lock.json diffs are harmless peer-dependency metadata reordering with no functional impact.

Risks

  • Users who have not customized their prompts will get the new default on next settings load. Users who have already saved custom prompts are unaffected because normalizeSettingsShape merges saved prompts over defaults.
  • Existing server tests should cover this; no new tests needed since the change is a string literal update. RISKS: - Users who have not customized their prompts will get the new default on next settings load. Users who have already saved custom prompts are unaffected because normalizeSettingsShape merges saved prompts over defaults.

AI Factory and others added 3 commits March 20, 2026 19:13
Add sections for subagent usage, autonomy, core principles,
mandatory test verification, and definition of done.

🤖 Generated by Ban Kan
@stilero
stilero merged commit 65d82ff into main Mar 20, 2026
1 check passed
@stilero
stilero deleted the feature/t-42c5a6-update-implementor-prompt branch March 20, 2026 18:28
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.

2 participants