Skip to content

Tiho/agent skills integration#1270

Draft
tculig wants to merge 7 commits intomainfrom
tiho/agent-skills-integration
Draft

Tiho/agent skills integration#1270
tculig wants to merge 7 commits intomainfrom
tiho/agent-skills-integration

Conversation

@tculig
Copy link
Contributor

@tculig tculig commented Mar 5, 2026

Description

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@@ -0,0 +1,151 @@
# Agent Skills Integration Tracker
Copy link
Contributor

Choose a reason for hiding this comment

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

This should not be committed.

package.json Outdated
]
},
"dependencies": {
"@mongodb-js/agent-skills": "file:../agent-skills",
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not an npm module and should not be taken as a dependency like that - instead, we should take the repo as a submodule.

Comment on lines +1436 to +1440
"chatSkills": [
{
"path": "./node_modules/@mongodb-js/agent-skills/skills/mongodb-client-management/SKILL.md"
},
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it not allowed to contribute an entire folder or do we have to list them 1 by 1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems its not allowed to contribute the entire folder, yes. I will do a double check on that though.

}
}

/** Reported when an agent skill is invoked by Copilot */
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we can track this.

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