Skip to content

Align agent discovery test with exact-case README.md handling#317703

Draft
Copilot wants to merge 4 commits into
mainfrom
copilot/fix-actions-job-macos-electron
Draft

Align agent discovery test with exact-case README.md handling#317703
Copilot wants to merge 4 commits into
mainfrom
copilot/fix-actions-job-macos-electron

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 21, 2026

macOS / Electron failed because a unit test expected lowercase readme.md to be excluded from .github/agents, while runtime classification excludes only exact-case README.md. This updates the test to match the implemented case-sensitive rule.

  • Failure addressed

    • SessionCustomizationDiscovery test incorrectly treated readme.md as documentation-only.
    • Actual behavior: only README.md is excluded; other .md files in agent folders remain agent candidates unless filtered by more-specific types.
  • Changes

    • Renamed the test to make the rule explicit: exact-case README.md exclusion.
    • Seeded lowercase readme.md as a regular agent file input.
    • Updated expected discovery output to include lowercase readme.md and continue excluding exact-case README.md.
  • Resulting contract

    • Platform-side discovery expectations now match workbench prompt classification for agent-folder markdown files.
await seed('/workspace/.github/agents/README.md', 'docs');      // excluded
const wsLowerReadmeAgent = await seed('/workspace/.github/agents/readme.md', 'docs lower'); // included

DonJayamanne and others added 4 commits May 21, 2026 14:02
Co-authored-by: DonJayamanne <1948812+DonJayamanne@users.noreply.github.com>
Copilot AI changed the title Fix macOS Electron test for exact-case README.md handling Align agent discovery test with exact-case README.md handling May 21, 2026
Copilot AI requested a review from DonJayamanne May 21, 2026 04:48
Base automatically changed from don/find-vscode-agents to main May 21, 2026 05: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