Skip to content

feat: discover external SOPs recursively - #78

Closed
kyletser wants to merge 1 commit into
strands-agents:mainfrom
kyletser:fix/nested-sop-discovery
Closed

kyletser wants to merge 1 commit into
strands-agents:mainfrom
kyletser:fix/nested-sop-discovery

Conversation

@kyletser

Copy link
Copy Markdown

Issue #, if available:

Closes #61

Description of changes:

  • Discover external .sop.md files recursively under every configured SOP path.
  • Sort matches by their relative POSIX path so same-name first-wins precedence is deterministic across platforms.
  • Add regression coverage for discovery order and the MCP, Agent Skills, and Cursor command consumers.
  • Document recursive discovery in the external SOP loading guides.

How to verify:

  • MCP 1.x test axis on Windows: 46 passed, 6 deselected.
  • MCP 2.x test axis on Windows: 46 passed, 6 deselected.
  • The six deselected cases are existing POSIX path-separator expectations that also fail on unchanged main; the affected nested-discovery tests pass in both axes.
  • ruff check .
  • uv build
  • git diff --check
  • Mutation check: replacing recursive discovery with the previous flat glob makes all four nested-discovery regressions fail.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Load nested .sop.md files in deterministic relative-path order across MCP prompts, Agent Skills, and Cursor commands.

Closes strands-agents#61
Copilot AI lite review requested due to automatic review settings August 25, 2026 13:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kyletser

kyletser commented Oct 3, 2026

Copy link
Copy Markdown
Author

Closing this PR: there has been no maintainer response for over a month, so I'm keeping the contribution queue tidy. The change remains on the fork branch — feel free to ask me to reopen it if you'd like to take it up.

@kyletser kyletser closed this Oct 3, 2026
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.

Add support for SOPs in nested directories

2 participants