Skip to content

Add DB Semantic Search template to gallery - #1024

Open
Marina He (hemarina) with Copilot wants to merge 6 commits into
mainfrom
copilot/text-semantic-search-embedding-models
Open

Marina He (hemarina) with Copilot wants to merge 6 commits into
mainfrom
copilot/text-semantic-search-embedding-models

Conversation

Copilot AI commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Add the community .NET template for semantic text search using Microsoft Foundry embeddings and Azure PostgreSQL with pgvector.

  • Gallery metadata: Includes the Blazor framework, Azure services, Bicep IaC, and a unique ID.
  • Preview: Adds the template’s architecture image to the gallery assets.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: hemarina <104857065+hemarina@users.noreply.github.com>
Copilot AI changed the title [WIP] Implement text semantic search using embedding models Add DB Semantic Search template to gallery Oct 2, 2026
@hemarina
Marina He (hemarina) marked this pull request as ready for review October 2, 2026 22:14
Copilot AI balanced review requested due to automatic review settings October 2, 2026 22:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

Review effort: Lite
Findings: 1 High severity · 1 Medium severity

Open (2)
What changed in this PR

Adds a new community gallery entry for a .NET semantic search template using Microsoft Foundry embeddings and Azure Database for PostgreSQL (pgvector), including preview metadata.

Changes:

  • Added a new template record (title/description/source/author) to the gallery index.
  • Defined metadata for languages/frameworks/Azure services and Bicep IaC.
  • Linked a preview architecture image asset path for the gallery card.
File Description
website/​static/​templates.json Adds the new “DB Semantic Search” template entry and its gallery metadata (tags/services/preview/id).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread website/static/templates.json
Comment thread website/static/templates.json Outdated
"author": "Massimo Bonanni",
"source": "https://github.com/massimobonanni/DBSemanticSearch",
"tags": [
"community",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot New community template contribution should ask for testing. Update that into skill and ping v-xuto.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated the submission skill to request manual testing for community templates, except for submissions from trusted authors. Tong Xu (MSFT) (@v-xuto) Please test this community template to ensure it works as expected. The guidance and requested author URL are committed as e768e9f.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated the testing contact in the skill and committed as COMMIT_HASH. Minghao Chen (@HarveyLink) Please test this community template before merge.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: hemarina <104857065+hemarina@users.noreply.github.com>
@hemarina

Copy link
Copy Markdown
Contributor

Copilot Fix failed pipeline.

Co-authored-by: hemarina <104857065+hemarina@users.noreply.github.com>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I found two items to address before merge.

"author": "Massimo Bonanni",
"source": "https://github.com/massimobonanni/DBSemanticSearch",
"tags": [
"community",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is a community template, and the submission issue author is not on the trusted-author list. Could we add the required manual testing request before merge?

Tong Xu (MSFT) (@v-xuto) Please test this community template to ensure it works as expected.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi Kristen Womack (@kristenwomack), we have tested this community template, and its core functionality works as expected.

created by a trusted author. Do not request manual testing or deployment proof
for trusted authors.

For community templates (those without the `msft` tag), comment on the PR and

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This changes repository-wide submission policy in a PR whose stated purpose is adding one gallery template. The skill already existed, but these new attribution and testing rules affect every future submission and need their own review. The testing step also cannot run reliably in this workflow because the PR is created asynchronously after the skill files the issue.

Could we remove the SKILL.md changes from this PR and propose them separately, with a clear follow-up mechanism and the correct person to contact for Microsoft templates?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please keep this PR focused on the gallery template. Move the repository-wide submission skill policy changes into a separate PR, and complete the required community-template testing before merge.

Co-authored-by: hemarina <104857065+hemarina@users.noreply.github.com>

This branch has not been deployed

No deployments
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.

Text Semantic Search using embedding models in Microsoft Foundry and Azure PostgreSQL

6 participants