Skip to content

Make workspace data source uploads configurable#25012

Merged
EngincanV merged 1 commit intorel-10.2from
10.2-ai-management-docs
Mar 4, 2026
Merged

Make workspace data source uploads configurable#25012
EngincanV merged 1 commit intorel-10.2from
10.2-ai-management-docs

Conversation

@enisn
Copy link
Member

@enisn enisn commented Mar 4, 2026

Update AI Management docs to describe configurable workspace data source upload options. Adds notes that allowed file extensions and max file size are configurable (default 10 MB), and that deleting a data source removes its embeddings, chunks, and blob. Introduces a new "Configuring Data Source Upload Options" section with a WorkspaceDataSourceOptions example (AllowedFileExtensions, MaxFileSize, ContentTypeMap), lists available properties and helper methods (GetMaxFileSizeDisplay, GetAllowedExtensionsDisplay, GetAcceptAttribute), and a note about registering content extractors for new file types. Also documents the WorkspaceDataSourceManager repository responsibility for full cleanup.

Update AI Management docs to describe configurable workspace data source upload options. Adds notes that allowed file extensions and max file size are configurable (default 10 MB), and that deleting a data source removes its embeddings, chunks, and blob. Introduces a new "Configuring Data Source Upload Options" section with a WorkspaceDataSourceOptions example (AllowedFileExtensions, MaxFileSize, ContentTypeMap), lists available properties and helper methods (GetMaxFileSizeDisplay, GetAllowedExtensionsDisplay, GetAcceptAttribute), and a note about registering content extractors for new file types. Also documents the WorkspaceDataSourceManager repository responsibility for full cleanup.
@enisn enisn added this to the 10.2-final milestone Mar 4, 2026
@enisn enisn requested a review from EngincanV March 4, 2026 09:47
@enisn enisn requested review from Copilot and removed request for Copilot March 4, 2026 09:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the AI Management module documentation to describe the newly configurable workspace data source upload options. It clarifies that file extensions and max file size are no longer hardcoded, documents the cleanup behavior when deleting a data source, and introduces a full configuration reference for WorkspaceDataSourceOptions.

Changes:

  • Adds (configurable) annotations and a deletion cleanup note to the Workspace Data Sources section.
  • Introduces a new "Configuring Data Source Upload Options" sub-section with a code example, property table, helper method table, and an important note about registering content extractors.
  • Extends the DELETE endpoint description and Domain Services list to reflect full cleanup behavior and the new WorkspaceDataSourceManager.

{
Configure<WorkspaceDataSourceOptions>(options =>
{
// Add support for additional file types
Copy link

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

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

The comment on line 403 says "Add support for additional file types," but the code on line 404 replaces the AllowedFileExtensions array entirely (overwriting the defaults .txt, .md, .pdf) rather than appending to them. This comment is misleading because readers might expect the code to append new extensions to the existing ones rather than replacing all of them. The comment should be updated to clarify that this line sets (replaces) the full list of allowed extensions.

Suggested change
// Add support for additional file types
// Set the full list of allowed file extensions (including additional types)

Copilot uses AI. Check for mistakes.
@EngincanV EngincanV merged commit 3bd7284 into rel-10.2 Mar 4, 2026
10 checks passed
@EngincanV EngincanV deleted the 10.2-ai-management-docs branch March 4, 2026 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants