Skip to content

Add sub-issue discovery to github-issues skill#817

Merged
drbenvincent merged 1 commit into
mainfrom
fix-skill-sub-issue-discovery
Apr 14, 2026
Merged

Add sub-issue discovery to github-issues skill#817
drbenvincent merged 1 commit into
mainfrom
fix-skill-sub-issue-discovery

Conversation

@drbenvincent

Copy link
Copy Markdown
Collaborator

Summary

Add sub-issue discovery guidance to the github-issues skill so agents check for native GitHub sub-issues when evaluating issues.

  • issue-evaluation.md: Add a "Check for sub-issues" step using the subIssues / parent GraphQL fields, and note to review sub-issues during context analysis.
  • parent-child-issues.md: Add a "Discover existing sub-issues" section (step 6) with the correct GraphQL query and a warning not to use the older trackedIssues / trackedInIssues fields.

Motivation

GitHub's native sub-issues use the subIssues GraphQL field, not trackedIssues. Without this guidance, agents default to the wrong API and silently miss existing sub-issues on a parent issue.

Test plan

  • Skill files are valid markdown
  • No code changes; documentation-only

Agents were using the older trackedIssues/trackedInIssues GraphQL fields
which miss native GitHub sub-issues. Add the correct subIssues/parent
query to both issue-evaluation and parent-child-issues workflows.

Made-with: Cursor
@drbenvincent drbenvincent added the agents Agent related issues specifically for use by developers label Apr 2, 2026
@drbenvincent drbenvincent requested a review from maresb April 2, 2026 15:14
@drbenvincent drbenvincent added the OSS_PRODUCT OSS_PRODUCT project priorities. Labs members should get approval before logging hours. label Apr 2, 2026
@codecov

codecov Bot commented Apr 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.77%. Comparing base (7db0672) to head (842616c).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #817   +/-   ##
=======================================
  Coverage   93.77%   93.77%           
=======================================
  Files          77       77           
  Lines       11881    11881           
  Branches      696      696           
=======================================
  Hits        11142    11142           
  Misses        546      546           
  Partials      193      193           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 causalpy | 🛠️ Build #32095324 | 📁 Comparing 842616c against latest (7db0672)

  🔍 Preview build  

Show files changed (1 files in total): 📝 1 modified | ➕ 0 added | ➖ 0 deleted
File Status
404.html 📝 modified

@drbenvincent

Copy link
Copy Markdown
Collaborator Author

@cetagostini could we review/merge this one (and #808) before I review your #837 ? Logic is that my 2 are open agent PR's and yours looks to try to unify things, so makes sense to do that after?

@maresb maresb left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sorry, just now seeing this.

Also just now learning about subissues! I think I should be using them.

Regarding this PR, this looks very benign and I think you should just merge it.

@drbenvincent drbenvincent merged commit cd89079 into main Apr 14, 2026
14 checks passed
@drbenvincent drbenvincent deleted the fix-skill-sub-issue-discovery branch April 14, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent related issues specifically for use by developers OSS_PRODUCT OSS_PRODUCT project priorities. Labs members should get approval before logging hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants