Skip to content

fix: restrict local file picker to verified ingestion extensions#1209

Merged
Wallgau merged 3 commits into
release-0.4.0from
type-files
Mar 23, 2026
Merged

fix: restrict local file picker to verified ingestion extensions#1209
Wallgau merged 3 commits into
release-0.4.0from
type-files

Conversation

@Wallgau

@Wallgau Wallgau commented Mar 21, 2026

Copy link
Copy Markdown
Collaborator

I checked manually the file types that we support, here the ones ingested successfully:
documents: [
"txt",
"md",
"html",
"htm",
"adoc",
"asciidoc",
"asc",
"pdf",
"docx",
],
spreadsheets: ["csv"],
}

Scope
Updates are limited to:
--> frontend/components/knowledge-dropdown.tsx
--> docs/docs/core-components/ingestion.mdx

What changed
knowledge-dropdown.tsx

  • SUPPORTED_FILE_TYPES / SUPPORTED_EXTENSIONS (local file picker + folder ingest filter) now only allow extensions that were verified to ingest successfully in your tests:
  • documents — .txt, .md, .html, .htm, .adoc, .asciidoc, .asc, .pdf, .docx
  • spreadsheets — .csv
  • Types you marked as failing (e.g. .mdx, .xml, code, JSON/YAML, etc.) are not in the picker/filter.

ingestion.mdx

  • “Supported file types” section updated to match that verified list.
  • Clarifies that the chat/local picker only offers these until other types are confirmed, and that cloud connectors may still differ.

@github-actions github-actions Bot added documentation 📘 Improvements or additions to documentation frontend 🟨 Issues related to the UI/UX bug 🔴 Something isn't working. and removed documentation 📘 Improvements or additions to documentation labels Mar 21, 2026
@github-actions

This comment has been minimized.

* spreadsheets: csv
*/
export const SUPPORTED_FILE_TYPES = {
"image/*": [".png", ".jpg", ".jpeg", ".gif", ".bmp", ".tiff", ".webp"],

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.

should we add it as comments for Future TODO?

@edwinjosechittilappilly edwinjosechittilappilly left a comment

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.

LGTM

@github-actions github-actions Bot added the lgtm label Mar 23, 2026
@edwinjosechittilappilly

Copy link
Copy Markdown
Collaborator

@Wallgau can you create a same PR for Main branch also?

@github-actions github-actions Bot added documentation 📘 Improvements or additions to documentation bug 🔴 Something isn't working. and removed bug 🔴 Something isn't working. documentation 📘 Improvements or additions to documentation labels Mar 23, 2026
@github-actions

This comment has been minimized.

@github-actions github-actions Bot added documentation 📘 Improvements or additions to documentation bug 🔴 Something isn't working. and removed bug 🔴 Something isn't working. documentation 📘 Improvements or additions to documentation labels Mar 23, 2026
@github-actions

github-actions Bot commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

Build successful! ✅
Deploying docs draft.
Deploy successful! View draft

@Wallgau Wallgau merged commit 9a19e64 into release-0.4.0 Mar 23, 2026
7 checks passed
@github-actions github-actions Bot deleted the type-files branch March 23, 2026 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔴 Something isn't working. frontend 🟨 Issues related to the UI/UX lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants