Add skills for webpubsub service#2074
Conversation
93dbf9f to
5c6e375
Compare
|
Consider consolidating this under azure-messaging with separate reference files for pubsub. Please note if there are troubleshooting workflows consider consolidating those reference files under azure-diagnostics. Here's a reference file - #1986 |
|
@kvenkatrajan Thanks for the suggestion. My concern is that the two skills have very different scopes.
I think a cleaner approach is to keep troubleshooting content consolidated where appropriate, but keep Web PubSub best-practice guidance separate. |
jongio
left a comment
There was a problem hiding this comment.
Issues to address:
- SKILL.md:15-16 - Quick Reference lists 3 MCP tools but the MCP Tools section (line 28-30) only documents one. An agent won't know when to use monitor or resourcehealth for Web PubSub.
- triggers.test.ts, unit.test.ts - missing trailing newline
Merge conflict in skills.json needs a rebase (main now includes entra-agent-id).
Co-authored-by: Copilot <copilot@github.com>
jongio
left a comment
There was a problem hiding this comment.
Addresses my previous feedback. Merge conflict resolved, MCP tools table is now consistent, Entra ID roles documented per @JialinXin's suggestion. Looks good.
cc @kvenkatrajan for approval
|
@jongio Please help to merge. |
Description
Add the initial
azure-webpubsubskill with focused guidance for webpubsub best practice.Checklist
cd tests && npm test)npm run test:skills:integration -- <skill>)USE FOR/DO NOT USE FOR/PREFER OVERclauses: confirmed no routing regressions for competing skillsRelated Issues