Skip to content

fix: skip mandatory .agent.md suffix for custom agent detection (#307)#310

Draft
wherka-ama wants to merge 1 commit into
AmadeusITGroup:mainfrom
wherka-ama:fix/issue-307-skip-agent-md-suffix
Draft

fix: skip mandatory .agent.md suffix for custom agent detection (#307)#310
wherka-ama wants to merge 1 commit into
AmadeusITGroup:mainfrom
wherka-ama:fix/issue-307-skip-agent-md-suffix

Conversation

@wherka-ama

Copy link
Copy Markdown
Member

Description

VS Code no longer requires the .agent.md suffix for custom agents. Any .md file in an agents/ directory is now recognized as an agent.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 📝 Documentation update

Related Issues

Fixes #307

Changes Made

  • determineFileType: add directory-based detection for agents/ dir
  • APM adapters (apm, local-apm): findPromptFiles picks up .md from agents/, detectFileType accepts full path and checks agents/ dir
  • Awesome Copilot adapters: fix ID extraction for plain .md files
  • bundle-installer: use promptDef.type from manifest when available
  • validate-apm-command: scan .md files in agents/ directories
  • Tests: add directory-based agent detection tests, fixture with plain .md agent file, adapter test verifying agent type detection
  • Docs: update file type tables and naming conventions

Testing

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing completed
  • All existing tests pass

Tested On

  • Linux

  • VS Code Insiders

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Documentation

  • README.md updated
  • JSDoc comments added/updated

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache License 2.0.

…eusITGroup#307)

VS Code no longer requires the .agent.md suffix for custom agents.
Any .md file in an agents/ directory is now recognized as an agent.

Changes:
- determineFileType: add directory-based detection for agents/ dir
- APM adapters (apm, local-apm): findPromptFiles picks up .md from
  agents/, detectFileType accepts full path and checks agents/ dir
- Awesome Copilot adapters: fix ID extraction for plain .md files
- bundle-installer: use promptDef.type from manifest when available
- validate-apm-command: scan .md files in agents/ directories
- Tests: add directory-based agent detection tests, fixture with
  plain .md agent file, adapter test verifying agent type detection
- Docs: update file type tables and naming conventions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

[Bug]: Agents are not copied from the bundle to copilotconfiguration

1 participant