Conversation
Contributor
Wanawrin
commented
Feb 7, 2026
Wanawrin
commented
- Added a v3 prompt, removing atomicity, changing skill manager prompt formatting, changing the external agent wrapping prompt
- Improved agentic system prompting experience by adding cli support and automatic injection generation
Introduces SKILL_MANAGER_V3_PROMPT using XML tags for better LLM parsing, consolidated rejection criteria, and 4 structured examples. ~35% more concise than v2.1 while improving clarity. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…anager Double-brace template variables were not being substituted by .format(). Changed to single braces and registered SKILL_MANAGER_V3_PROMPT as version "3.0". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…section field - Fix KeyError by escaping JSON braces in TAG semantics examples - Remove section field from external agent wrapper (redundant with ID prefix) - Update docstrings to reflect id+content only format 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…PDATE - Remove "under 15 words" constraint from atomicity scoring - Change UPDATE rule to "preserve ALL enumerated items (lists, criteria)" - Update ADD rule to remove word limit - Change key rule from word limit to "preserve enumerated items on UPDATE" - Update reminder to "never narrow enumerated items" This prevents UPDATE operations from condensing skills that contain multiple criteria (e.g., cancellation eligibility with 4 conditions) into shorter, less useful versions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
… prompting 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add argparse CLI for agentic_system_prompting.py with options for traces_dir, model, epochs, threshold, input-skillbook, output-dir - Update README with CLI usage documentation - Update prompts import to v3 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Lanzelot1
force-pushed
the
core_improvements
branch
from
8579b12 to
8654705
Compare
- Create ace/prompt_manager.py with unified PromptManager supporting v1.0-v3.0 - Remove duplicate PromptManager classes from prompts_v2.py, prompts_v2_1.py, prompts_v3.py - Add backward-compatible re-exports via __getattr__ in version modules - Remove unused MIGRATION_GUIDE constants and compare_prompt_versions() - Update all integrations to import from ace.prompt_manager Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
davidfarah2003
pushed a commit
that referenced
this pull request
Mar 6, 2026
* feat(prompts): add v3 SkillManager prompt with XML structure Introduces SKILL_MANAGER_V3_PROMPT using XML tags for better LLM parsing, consolidated rejection criteria, and 4 structured examples. ~35% more concise than v2.1 while improving clarity. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(prompts): fix v3 SkillManager template vars and wire into PromptManager Double-brace template variables were not being substituted by .format(). Changed to single braces and registered SKILL_MANAGER_V3_PROMPT as version "3.0". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(prompts_v3): escape braces in TAG semantics and remove redundant section field - Fix KeyError by escaping JSON braces in TAG semantics examples - Remove section field from external agent wrapper (redundant with ID prefix) - Update docstrings to reflect id+content only format 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix(prompts_v3): remove 15-word limit, preserve enumerated items on UPDATE - Remove "under 15 words" constraint from atomicity scoring - Change UPDATE rule to "preserve ALL enumerated items (lists, criteria)" - Update ADD rule to remove word limit - Change key rule from word limit to "preserve enumerated items on UPDATE" - Update reminder to "never narrow enumerated items" This prevents UPDATE operations from condensing skills that contain multiple criteria (e.g., cancellation eligibility with 4 conditions) into shorter, less useful versions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * feat(examples): add external agent injection output to agentic system prompting 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * feat(examples): add CLI support to agentic system prompting - Add argparse CLI for agentic_system_prompting.py with options for traces_dir, model, epochs, threshold, input-skillbook, output-dir - Update README with CLI usage documentation - Update prompts import to v3 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * feat(examples): add convert_skillbook_to_injection.py script 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: revert personal path and format code Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor(prompts): consolidate PromptManager into single module - Create ace/prompt_manager.py with unified PromptManager supporting v1.0-v3.0 - Remove duplicate PromptManager classes from prompts_v2.py, prompts_v2_1.py, prompts_v3.py - Add backward-compatible re-exports via __getattr__ in version modules - Remove unused MIGRATION_GUIDE constants and compare_prompt_versions() - Update all integrations to import from ace.prompt_manager Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: halluton <bush.mitoses-75@icloud.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Lanzelot1 <35410605+Lanzelot1@users.noreply.github.com>
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
3 participants
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.