Skip to content

merge#341

Open
tmuhlhausen wants to merge 4 commits into
superagent-ai:mainfrom
tmuhlhausen:main
Open

merge#341
tmuhlhausen wants to merge 4 commits into
superagent-ai:mainfrom
tmuhlhausen:main

Conversation

@tmuhlhausen

@tmuhlhausen tmuhlhausen commented Jul 1, 2026

Copy link
Copy Markdown

What does this PR do?

Describe your changes here.

Fixes #

Checklist

  • I tested my changes
  • I reviewed my own code

Note

Low Risk
Documentation-only addition under .agents/ with no runtime, auth, or application code changes.

Overview
Adds a new agent skill at .agents/skills/agent-github/skill.md so agents know when and how to contribute to public GitHub repos via fork + PR instead of ad-hoc shell/git.

The spec covers scope (public repos only), prerequisites (git, gh/PAT), a 10-step preferred flow (delegate to GitHub sub-agent, fork, branch, commit, push, open PR), tool-by-tool guidance for github_* and git_* helpers, reliability rules (atomic PRs, state snapshots, no upstream pushes), and blockers (auth, rate limits, private/archived repos).

It is structured like the existing agent-desktop skill and ends with meta copy noting that alignment; reviewers may want to trim that closing paragraph for consistency with other skills.

Reviewed by Cursor Bugbot for commit dd09067. Configure here.

@open-cla

open-cla Bot commented Jul 1, 2026

Copy link
Copy Markdown

Contributor License Agreement

The following contributors need CLA coverage:

Review and sign the CLA

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit dd09067. Configure here.

@@ -0,0 +1,82 @@
---

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wrong skill manifest filename

Medium Severity

The new skill lives in skill.md, but this repo’s skill loader only reads SKILL.md in each skill directory. Sibling skills such as agent-desktop use that name, so agent-github is never discovered or injected into the agent prompt.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit dd09067. Configure here.

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