Skip to content

content review#995

Merged
aaronpowell merged 6 commits intostagedfrom
content-review
Mar 13, 2026
Merged

content review#995
aaronpowell merged 6 commits intostagedfrom
content-review

Conversation

@aaronpowell
Copy link
Contributor

Removing/reworking some of the items in the repo to better align with way the current frontier models work, and be more in line with our policies on content inclusion.

  • Cleaned up some tool names
  • Removing some instructionsThese instructions are no longer useful as the knowledge the add is already well handled by frontier models, so the instructions potentially provide conflicting or incorrect information to the agent while it undertakes a task
  • Improved the skill to be more explicit on how to use playwright
  • Removing a skill that is of low value

…the knowledge the add is already well handled by frontier models, so the instructions potentially provide conflicting or incorrect information to the agent while it undertakes a task
The information captured in this skill is mostly just what is found in the links that are at the top of the references, and thus the model will already have that knowledge available to it, meaning that the skill will potentially provide conflicting guidance to the agent as it works
Copilot AI review requested due to automatic review settings March 13, 2026 01:08
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ This PR targets main, but PRs should target staged.

The main branch is auto-published from staged and should not receive direct PRs.
Please close this PR and re-open it against the staged branch.

You can change the base branch using the Edit button at the top of this PR,
or run: gh pr edit 995 --base staged

@aaronpowell aaronpowell changed the base branch from main to staged March 13, 2026 01:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions
Copy link
Contributor

🔍 Potential Duplicate Resources Detected

This PR modifies resources that may be similar to existing ones in the repository. Please review this potential overlap before merging to avoid redundancy.

Possible Duplicates

Group 1: C#/.NET janitorial/cleanup agents with identical descriptions

Resource Type Description
agents/csharp-dotnet-janitor.agent.md (modified in this PR) Agent 'Perform janitorial tasks on C#/.NET code including cleanup, modernization, and tech debt remediation.'
agents/dotnet-upgrade.agent.md (existing) Agent 'Perform janitorial tasks on C#/.NET code including cleanup, modernization, and tech debt remediation.'

Why flagged: These two agents share identical description fields and both target C#/.NET code. While their body content differs (csharp-dotnet-janitor focuses on code cleanup/tech debt, and dotnet-upgrade focuses on framework version migration), the front matter description is word-for-word the same, which may confuse users discovering these resources.

Suggestion: Consider whether dotnet-upgrade.agent.md's description should be updated to better reflect its actual migration focus (e.g., "Upgrade and migrate C#/.NET projects to newer framework versions"), or whether the two agents should be consolidated. If they are intentionally distinct, updating the description of dotnet-upgrade.agent.md would make both resources easier to differentiate.


💡 This is an advisory check only. If these are intentionally different, no action is needed — feel free to proceed with your PR.

Generated by PR Duplicate Check for issue #995 ·

@aaronpowell aaronpowell merged commit b1f3346 into staged Mar 13, 2026
9 checks passed
@aaronpowell aaronpowell deleted the content-review branch March 13, 2026 02:45
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