Skip to content

t258: Exclude skill reference files from subagent stub generation#1035

Merged
marcusquinn merged 1 commit intomainfrom
feature/t258
Feb 11, 2026
Merged

t258: Exclude skill reference files from subagent stub generation#1035
marcusquinn merged 1 commit intomainfrom
feature/t258

Conversation

@marcusquinn
Copy link
Owner

@marcusquinn marcusquinn commented Feb 11, 2026

Summary

  • Add -not -name '*-skill.md' filter to find command in generate-opencode-agents.sh
  • Prevents skill reference files from being processed as OpenCode subagents
  • Skill files are documentation/reference only, not executable agents

Changes

  • Modified .agents/scripts/generate-opencode-agents.sh line 782
  • Added filter alongside existing -not -path '*/loop-state/*' exclusion

Testing

  • Run generate-opencode-agents.sh
  • Verify skill files are excluded from subagent generation (12 skill files excluded, 893 subagents processed)
  • Run shellcheck validation (passes with no violations)

Verification

# Before: Would process all .md files including skills
find ~/.aidevops/agents -mindepth 2 -name '*.md' -type f -not -path '*/loop-state/*' | wc -l
# 905 files

# After: Excludes skill files
find ~/.aidevops/agents -mindepth 2 -name '*.md' -type f -not -path '*/loop-state/*' -not -name '*-skill.md' | wc -l
# 893 files (12 skill files excluded)

Related: t258

Summary by CodeRabbit

  • Chores
    • Updated internal build script to refine file discovery and processing logic.

Add -not -name '*-skill.md' filter to find command in generate-opencode-agents.sh
to prevent skill reference files from being processed as subagents.
@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

A shell script modification that adds an exclusion filter to the subagent markdown generation process, preventing files matching "*-skill.md" from being processed by the find command while preserving all downstream logic and workflows.

Changes

Cohort / File(s) Summary
Agent Generation Script
.agents/scripts/generate-opencode-agents.sh
Added -not -name "*-skill.md" exclusion pattern to the find command, preventing skill markdown files from triggering subagent stub generation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Possibly related PRs

Poem

🎯 Filter flows with measured grace,
Skill files find their rightful place,
One small flag, one clean exclusion,
No more stubs from the right solution! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/t258

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 46 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Wed Feb 11 00:30:54 UTC 2026: Code review monitoring started
Wed Feb 11 00:30:55 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 46

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 46
  • VULNERABILITIES: 0

Generated on: Wed Feb 11 00:30:57 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@marcusquinn marcusquinn marked this pull request as ready for review February 11, 2026 00:31
@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@sonarqubecloud
Copy link

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.

1 participant