Open
Conversation
Allow users to configure agent skills sourced from DIAL prompts via a new `skills` config field (preview feature). DIAL prompts with valid Agent Skills frontmatter are fetched at request time, merged with predefined skills, and surfaced to the agent through the same XML metadata and `read_skill` tool. Key changes: - Make PromptPartProvider and MessagesTransformer interfaces async - Add SkillConfig discriminated union and ApplicationConfig.skills field - Extract parse_frontmatter and XML generation to shared modules - Refactor AgentSkillsProvider to pure data store; add SkillsRegistry - Add DialPromptSkillResolver with parallel fetch and graceful degradation - Wire DialPromptSkillsModule as @preview_module in AppFactory Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Surface DIAL prompt skill resolution warnings to users via stages instead of server logs: parse_frontmatter raises SkillValidationError, resolver returns structured warnings, and SkillsRegistry renders them in a user-facing stage. Add GET /skills listing and POST /skills/validate endpoints to configuration support controller. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…prompts-as-skills # Conflicts: # poetry.lock
- Bump aidial-client to released 0.7.0 (drop temp branch dep) - Raise on compatibility > 500 chars instead of silent truncation - Return 401 on missing/invalid api-key in validate endpoint - Accept SkillConfig union in validate endpoint; dispatch by type - Thread prompt URL through resolver so collision warnings show it - Render catastrophic failure via dedicated stage path - Guard SkillsRegistry._resolve with asyncio.Lock - Move ResolvedDialPromptSkill to dial_prompt_skills package Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Collaborator
Author
|
/deploy-review
|
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
1 participant
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.
Applicable issues
Description of changes
Allow users to configure agent skills sourced from DIAL prompts via a new
skillsconfig field (preview feature). DIAL prompts with valid Agent Skills frontmatter are fetched at request time, merged with predefined skills, and surfaced to the agent through the same XML metadata andread_skilltool.Key changes:
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.