Repository navigation
Add community project catalogs - #9107
Eduard van Valkenburg (eavanvalkenburg) merged 2 commits into
Conversation
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The catalogs, external links, disclaimer, schema, and review guidance are consistent and complete.
Review effort: Balanced
Findings: None
What changed in this PR
Adds SDK-specific catalogs for discovering eligible community integrations and defines their review policy.
Changes:
- Adds Python and .NET community-project catalogs and README links.
- Lists verified Engram and MemorySync Python integrations.
- Adds and wires catalog eligibility guidance.
| File | Description |
|---|---|
python/samples/README.md |
Links the Python catalog. |
python/samples/community-projects.md |
Adds Python entries and disclaimer. |
dotnet/samples/README.md |
Links the .NET catalog. |
dotnet/samples/community-projects.md |
Adds the empty .NET catalog. |
.github/skills/pull-requests/SKILL.md |
Triggers catalog review guidance. |
.github/skills/community-projects/SKILL.md |
Defines eligibility and review rules. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
MAF Automated Review — Iteration 1
Result: Findings reported
Scope: full PR (1 commit(s)): b6361c2e660a
Model: gpt-5.6-sol
Overview
The PR establishes SDK-specific community catalogs with strong eligibility, ownership, support-link, taxonomy, and cross-SDK guardrails, and the initial entries satisfy those catalog rules. The review workflow also correctly requires direct inspection rather than accepting maintainer assertions. However, that inspection crosses into submitter-controlled content without defining an untrusted-content boundary, leaving automated reviewers exposed to prompt-injection attempts.
Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
1 verified finding remained after source verification (1 medium) across 1 file. Details are attached to the affected lines below.
Affected areas: .github/skills/community-projects/SKILL.md
Jose Alvarez (jpalvarezl)
left a comment
There was a problem hiding this comment.
| Rule | Result | Evidence |
|---|---|---|
| Complete submission | Pass | Both entries provide all catalog fields, identify Python integrations, and are presented as published reader-facing projects. |
| Project URL shape | Pass | Engram links to its project repository; MemorySync links to its project-specific package page. |
| Direct external issue-creation URL shape | Pass | Both issue links use external GitHub /issues/new/choose paths. |
| Markdown link check | Pass | The markdown-link-check workflow passed on the current head. |
| Allowed primary component and table shape | Pass | Engram is listed once under Tools and MemorySync once under Context providers; both tables use the required five columns. |
| Neutral wording | Pass | Names, descriptions, and scenarios are factual and make no Microsoft ownership, support, endorsement, testing, or compatibility claims. |
| Exclusions | Pass | Neither entry is an official Microsoft integration, generic endpoint/OTLP listing, planned project, or Microsoft-routed support link. |
| Ordering and cross-SDK consistency | Pass | Each populated component contains one correctly placed row; no unsupported .NET copy was added. |
Eligible
Motivation & Context
External Agent Framework integrations do not have an authoritative, SDK-specific discovery surface or a consistent eligibility policy. This change adds source catalogs in the code repository for the Agent Framework team to sync to Microsoft Learn while keeping project ownership, support, compatibility, and security guidance with each external maintainer.
Description & Review Guide
Related Issue
Closes #6341
Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.