Use `Microsoft.Extensions.DataIngestion` in AI Chat Web template #7023

MackinnonBuck · 2025-11-06T23:44:54Z

This PR makes the following changes to the chat template:

Adds support for Markdown documents
- Replaces Example_GPS_Watch.pdf with its markdown equivalent
- Adds a Markdown viewer to view citations
Removes the dependency on PdfPig
- ...but now has an implicit dependency on markitdown

This reverts commit e1d066034962c9686bf8150984b6adf0e25846c8.

This reverts commit a369be9.

src/Libraries/Microsoft.Extensions.DataIngestion/Writers/VectorStoreWriter.cs

...rc/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Components/Pages/Chat/ChatCitation.razor

...es/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs

....AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/IngestedChunk.cs

...AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/SemanticSearch.cs

src/ProjectTemplates/Microsoft.Extensions.AI.Templates/THIRD-PARTY-NOTICES.TXT

MackinnonBuck · 2025-11-07T00:40:34Z

Marking as ready for review to get some eyes on this. Note that there are still pending improvements:

Better UX for document ingestion
Possibly re-introducing PdfPig dependency to replace the markitdown dependency (see Use Microsoft.Extensions.DataIngestion in AI Chat Web template #7023 (comment))

Copilot

Pull Request Overview

This PR modernizes the AI chat web templates by replacing the custom PDF ingestion pipeline (using PdfPig) with the new Microsoft.Extensions.DataIngestion library suite. The changes enable support for both Markdown and PDF document formats while simplifying the ingestion architecture.

Key Changes

Replaced custom PDFDirectorySource and IIngestionSource with the standardized Microsoft.Extensions.DataIngestion APIs
Removed IngestedDocument tracking class as document versioning is now handled by the ingestion pipeline
Added Markdown viewer support (viewer.html and viewer.mjs) for rendering .md files
Updated citation format to remove page numbers, now supporting document-level citations
Changed ingestion trigger from startup to lazy initialization on first search request

Reviewed Changes

Copilot reviewed 94 out of 100 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
VectorStoreWriter.cs	Added workaround for QdrantVectorStore key type incompatibility using string name check
SemanticSearch.cs (all variants)	Added lazy ingestion on first search with `_initialized` flag
DocumentReader.cs	New custom reader supporting both Markdown and PDF via MarkdownReader and MarkItDownReader
DataIngestor.cs (all variants)	Simplified to use IngestionPipeline with SemanticSimilarityChunker
IngestedChunk.cs (all variants)	Changed Key type to Guid, made constants public, added JSON serialization attributes
ChatCitation.razor	Added Markdown viewer support alongside existing PDF viewer
ChatMessageItem.razor	Removed page number from citation regex and data structure
Program.cs variants	Removed startup ingestion, added vector store registrations, changed DataIngestor to singleton
*.csproj.in	Replaced PdfPig with DataIngestion packages and ML.Tokenizers
THIRD-PARTY-NOTICES.TXT	Removed PdfPig license notice
GeneratedContent.targets	Updated package version variables

src/Libraries/Microsoft.Extensions.DataIngestion/Writers/VectorStoreWriter.cs

...AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/SemanticSearch.cs

...tensions.AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Program.Aspire.cs

jeffhandley

Looks great, @MackinnonBuck!

src/ProjectTemplates/Microsoft.Extensions.AI.Templates/THIRD-PARTY-NOTICES.TXT

...enAI_Qdrant_Aspire.verified/aichatweb/aichatweb.Web/Components/Pages/Chat/ChatCitation.razor

…uids

adamsitnik

Big thanks for a great contribution and detailed testing @MackinnonBuck !

Please enable the tracing, this could be done by modyfing:

extensions/test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/aichatweb.AzureOpenAI_Qdrant_Aspire.verified/aichatweb/aichatweb.ServiceDefaults/Extensions.cs

Line 79 in c3e0c73

.AddSource("Experimental.Microsoft.Extensions.AI");

with:

.AddSource("Experimental.Microsoft.Extensions.DataIngestion");

src/Libraries/Microsoft.Extensions.DataIngestion/Writers/VectorStoreWriter.cs

....AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/IngestedChunk.cs

src/ProjectTemplates/Microsoft.Extensions.AI.Templates/THIRD-PARTY-NOTICES.TXT

...ts/aichatweb.AzureOpenAI_Qdrant_Aspire.verified/aichatweb/aichatweb.Web/aichatweb.Web.csproj

...ates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DataIngestor.cs

...es/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs

…ata-ingestion

jeffhandley

Just one comment that I think needs resolved.

...es/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs

…tWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com>

jeffhandley · 2025-11-10T08:18:07Z

@MackinnonBuck FYI - I added some commits, including one that shows a message about documents being loaded. All tests are passing and I did a lot of end-to-end functional validation too. I've marked it to auto-merge when CI is green after my latest push.

adamsitnik

We are almost there, we just need to disable the incremental ingestion and remove the SK dependency from the PDF reader.

...ates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DataIngestor.cs

...ates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/PdfPigReader.cs

...src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/ChatWithCustomData-CSharp.Web.csproj.in

...zureOpenAI_Qdrant_Aspire.verified/aichatweb/aichatweb.Web/Services/Ingestion/DataIngestor.cs

.../aichatweb.Ollama_Qdrant.verified/aichatweb/aichatweb.Web/Services/Ingestion/DataIngestor.cs

...ests/Snapshots/aichatweb.Ollama_Qdrant.verified/aichatweb/aichatweb.Web/aichatweb.Web.csproj

...apshots/aichatweb.OpenAI_AzureAISearch.verified/aichatweb/Services/Ingestion/DataIngestor.cs

...apshots/aichatweb.OpenAI_AzureAISearch.verified/aichatweb/Services/Ingestion/PdfPigReader.cs

...ntegrationTests/Snapshots/aichatweb.OpenAI_AzureAISearch.verified/aichatweb/aichatweb.csproj

adamsitnik

@MackinnonBuck To save your time, I've addressed my feedback by pushing to your branch. Please perform the manual verification before merging (I don't know how to do it).

...plates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Components/Pages/Chat/Chat.razor

…x code formatting.

…net#7023) * Add Markdown support * Remove PDF support * Revert "Remove PDF support" This reverts commit e1d066034962c9686bf8150984b6adf0e25846c8. * Add 'Example_GPS_Watch.md' * Add MEDI dependencies * Revert "[MEDI] Remove collection key type workaround (dotnet#7010)" This reverts commit a369be9. * MEDI integration into chat template * Remove PdfPig dependency * Fix citation + normalize identifier path * Undo changes to `M.E.DI.csproj` * Update snapshots * Update DataIngestion unit tests to handle keys as either strings or guids * Update SK and fix MEDI version * Remove SK workaround * Fix sandbox paths to allow running tests multiple times * Reliable data ingestion * Enable MEDI tracing * Simplify log message * Add `PdfPigReader` for non-Aspire template * Invert PdfPigReader exclusion condition * Use Markitdown MCP * Update snapshots * Undo changes to `IngestionPipelineTests.cs` * Update src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com> * Update snapshots * Improve template execution test failure output * Support .NET 10 in aichatweb, using it by default * Show a message when loading documents by loading docs as a separate tool * disable the incremental ingestion * map every PDF page to a single section * drop SK dependency * Add system prompt instructions for calling the LoadDocuments tool. Fix code formatting. --------- Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com> Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com>

- Use `Microsoft.Extensions.DataIngestion` in AI Chat Web template (dotnet#7023) - Add a new Microsoft.Agents.AI.Templates package with an aiagents-webapi project template (dotnet#7014) - Add Agent Framework DevUI into the aiagent-webapi template (dotnet#7026)

MackinnonBuck added 10 commits November 5, 2025 14:39

Add Markdown support

f8d2fe5

Remove PDF support

3631508

Revert "Remove PDF support"

b311c28

This reverts commit e1d066034962c9686bf8150984b6adf0e25846c8.

Add 'Example_GPS_Watch.md'

fd6b2fc

Add MEDI dependencies

6da6a21

Revert "[MEDI] Remove collection key type workaround (#7010)"

3a41be8

This reverts commit a369be9.

MEDI integration into chat template

3c3a27c

Remove PdfPig dependency

8c41874

Fix citation + normalize identifier path

da477f5

Undo changes to M.E.DI.csproj

eb7b2cd

github-actions bot added the area-ai-templates Microsoft.Extensions.AI.Templates label Nov 6, 2025

dotnet-policy-service bot assigned MackinnonBuck Nov 6, 2025

MackinnonBuck commented Nov 7, 2025

View reviewed changes

Update snapshots

7f758af

MackinnonBuck marked this pull request as ready for review November 7, 2025 00:40

MackinnonBuck requested a review from a team as a code owner November 7, 2025 00:40

Copilot AI review requested due to automatic review settings November 7, 2025 00:40

Copilot AI reviewed Nov 7, 2025

View reviewed changes

src/Libraries/Microsoft.Extensions.DataIngestion/Writers/VectorStoreWriter.cs Outdated Show resolved Hide resolved

...AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/SemanticSearch.cs Outdated Show resolved Hide resolved

jeffhandley reviewed Nov 7, 2025

View reviewed changes

...tensions.AI.Templates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Program.Aspire.cs Show resolved Hide resolved

jeffhandley approved these changes Nov 7, 2025

View reviewed changes

src/ProjectTemplates/Microsoft.Extensions.AI.Templates/THIRD-PARTY-NOTICES.TXT Show resolved Hide resolved

...enAI_Qdrant_Aspire.verified/aichatweb/aichatweb.Web/Components/Pages/Chat/ChatCitation.razor Show resolved Hide resolved

jeffhandley added 2 commits November 6, 2025 22:46

Merge branch 'main' into mbuck/chat-template-data-ingestion

3955026

Update DataIngestion unit tests to handle keys as either strings or g…

28ec490

…uids

adamsitnik reviewed Nov 7, 2025

View reviewed changes

MackinnonBuck added 3 commits November 7, 2025 10:53

Update SK and fix MEDI version

8108e80

Remove SK workaround

5296eea

Fix sandbox paths to allow running tests multiple times

0f56e53

MackinnonBuck requested a review from a team as a code owner November 7, 2025 19:45

MackinnonBuck added 2 commits November 7, 2025 12:51

Reliable data ingestion

1a65d68

Enable MEDI tracing

2bffe17

MackinnonBuck added 7 commits November 7, 2025 12:58

Simplify log message

012d08e

Add PdfPigReader for non-Aspire template

baa407d

Invert PdfPigReader exclusion condition

9f9921c

Merge remote-tracking branch 'origin/main' into mbuck/chat-template-d…

9c9c596

…ata-ingestion

Use Markitdown MCP

aa9d026

Update snapshots

a799f0b

Undo changes to IngestionPipelineTests.cs

960bbc5

jeffhandley approved these changes Nov 7, 2025

View reviewed changes

...es/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs Show resolved Hide resolved

...es/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs Show resolved Hide resolved

MackinnonBuck and others added 6 commits November 7, 2025 15:50

Update src/ProjectTemplates/Microsoft.Extensions.AI.Templates/src/Cha…

537474d

…tWithCustomData/ChatWithCustomData-CSharp.Web/Services/Ingestion/DocumentReader.cs Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com>

Update snapshots

45c07c9

Improve template execution test failure output

b51f32b

Support .NET 10 in aichatweb, using it by default

537b616

Merge branch 'main' into mbuck/chat-template-data-ingestion

491e808

Show a message when loading documents by loading docs as a separate tool

84cfea6

jeffhandley enabled auto-merge (squash) November 10, 2025 08:16

adamsitnik reviewed Nov 10, 2025

View reviewed changes

adamsitnik added 3 commits November 10, 2025 15:08

disable the incremental ingestion

0467a99

map every PDF page to a single section

cfc845f

drop SK dependency

855cfd2

adamsitnik approved these changes Nov 10, 2025

View reviewed changes

jeffhandley disabled auto-merge November 10, 2025 16:58

MackinnonBuck commented Nov 10, 2025

View reviewed changes

...plates/src/ChatWithCustomData/ChatWithCustomData-CSharp.Web/Components/Pages/Chat/Chat.razor Show resolved Hide resolved

Add system prompt instructions for calling the LoadDocuments tool. Fi…

17ba491

…x code formatting.

MackinnonBuck merged commit f38b9c0 into main Nov 10, 2025
6 checks passed

MackinnonBuck deleted the mbuck/chat-template-data-ingestion branch November 10, 2025 23:54

jeffhandley mentioned this pull request Nov 11, 2025

Update .NET AI Chat Web App template to support .NET 10 #7033

Closed

Use Microsoft.Extensions.DataIngestion in AI Chat Web template #7023

Use Microsoft.Extensions.DataIngestion in AI Chat Web template #7023

Conversation

MackinnonBuck commented Nov 6, 2025 • edited by jeffhandley Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

MackinnonBuck commented Nov 7, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Key Changes

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jeffhandley left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

adamsitnik left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jeffhandley left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

jeffhandley commented Nov 10, 2025

Uh oh!

adamsitnik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

adamsitnik left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Use `Microsoft.Extensions.DataIngestion` in AI Chat Web template #7023

Use `Microsoft.Extensions.DataIngestion` in AI Chat Web template #7023

MackinnonBuck commented Nov 6, 2025 •

edited by jeffhandley

Loading

adamsitnik left a comment •

edited

Loading