Remove RAG reindex functionality from Documents UI and backend by aliasaria · Pull Request #1428 · transformerlab/transformerlab-app

aliasaria · 2026-03-03T12:19:54Z

Summary

Removes all RAG reindex functionality and the "rag" folder concept from the Documents UI and backend.

Frontend

Removed the "Reindex" button from the Documents component
Removed the "Documents for RAG should be uploaded in a folder called rag" message
Removed the additionalMessage prop from the Documents component
Cleaned up unused imports (RotateCcwIcon, TinyButton)
Removed Rag.ReIndex endpoint from the API client

Backend

Removed commented-out rag.reindex() calls from document upload, upload_links, and download_zip handlers
Removed /experiment/{experimentId}/rag/reindex from openapi.json

Summary by CodeRabbit

Changes
- Removed the Reindex button from the Documents interface.
Chores
- Removed obsolete API endpoints and commented code related to document reindexing.

Amp-Thread-ID: https://ampcode.com/threads/T-019cb397-7123-76b4-8bb1-b64fd4919423 Co-authored-by: Amp <[email protected]>

paragon-review · 2026-03-03T12:20:20Z

Paragon Summary

This pull request review analyzed 5 files and found no issues. The review examined code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

Paragon did not detect any problems in the current diff. Proceed with merge after your normal checks.

This PR removes the RAG reindex functionality entirely from both the frontend Documents UI (removing the Reindex button and rag folder messaging) and the backend API (cleaning up reindex handler code and removing the endpoint from the OpenAPI spec).

Key changes:

Removed "Reindex" button and "rag" folder message from Documents UI components
Cleaned up unused imports (RotateCcwIcon, TinyButton) and additionalMessage prop
Removed Rag.ReIndex endpoint from API client
Removed commented-out rag.reindex() calls from document upload handlers
Removed /experiment/{experimentId}/rag/reindex endpoint from OpenAPI spec

Confidence score: 5/5

This PR has low risk with no critical or high-priority issues identified
Score reflects clean code review with only minor suggestions or no issues found
Code quality checks passed - safe to proceed with merge

5 files reviewed, 0 comments

Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

coderabbitai · 2026-03-03T12:20:29Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ef9754 and 47d82c8.

⛔ Files ignored due to path filters (1)

scripts/openapi.json is excluded by !**/*.json

📒 Files selected for processing (4)

api/transformerlab/routers/experiment/documents.py
src/renderer/components/Experiment/Documents/index.tsx
src/renderer/components/Shared/Documents/index.tsx
src/renderer/lib/api-client/endpoints.ts

💤 Files with no reviewable changes (3)

src/renderer/lib/api-client/endpoints.ts
src/renderer/components/Shared/Documents/index.tsx
api/transformerlab/routers/experiment/documents.py

📝 Walkthrough

Walkthrough

This PR removes RAG reindexing functionality across the codebase. Commented-out reindex calls are eliminated from the backend documents router, the Rag.ReIndex endpoint is removed from the API client endpoints, and the Documents component no longer exposes the additionalMessage prop or renders a Reindex button with explanatory messaging.

Changes

Cohort / File(s)	Summary
Backend RAG Reindexing `api/transformerlab/routers/experiment/documents.py`	Removed commented-out `rag.reindex()` calls that previously suggested reindexing the vector store after uploads or ZIP extraction.
API Client Endpoints `src/renderer/lib/api-client/endpoints.ts`	Removed `Rag.ReIndex` public endpoint method from the Endpoints.Rag object.
Frontend UI Components `src/renderer/components/Shared/Documents/index.tsx`, `src/renderer/components/Experiment/Documents/index.tsx`	Removed `additionalMessage` prop from Documents component signature and its parent usage; eliminated Reindex button UI block, related handler, and RotateCcwIcon and TinyButton imports.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Remove a lot of API routes #1391: Both PRs modify RAG reindexing code paths, removing or commenting out rag.reindex() calls in the documents router and the Rag.ReIndex endpoint in the renderer.

Suggested reviewers

deep1401

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Remove RAG reindex functionality from Documents UI and backend' directly and accurately summarizes the main objective of the changeset, clearly communicating the primary purpose of the pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings (stacked PR)
📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch remove-rag-reindex

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

sentry · 2026-03-03T12:22:31Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

aliasaria and others added 2 commits March 3, 2026 07:14

Remove RAG reindex button and rag folder messaging from Documents UI

ec7ff40

Amp-Thread-ID: https://ampcode.com/threads/T-019cb397-7123-76b4-8bb1-b64fd4919423 Co-authored-by: Amp <[email protected]>

Remove commented-out RAG reindex code from backend and openapi spec

47d82c8

Amp-Thread-ID: https://ampcode.com/threads/T-019cb397-7123-76b4-8bb1-b64fd4919423 Co-authored-by: Amp <[email protected]>

paragon-review bot reviewed Mar 3, 2026

View reviewed changes

deep1401 approved these changes Mar 3, 2026

View reviewed changes

Merge branch 'main' into remove-rag-reindex

934bbca

aliasaria merged commit c708a38 into main Mar 3, 2026
6 of 7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Remove RAG reindex functionality from Documents UI and backend#1428

Remove RAG reindex functionality from Documents UI and backend#1428
aliasaria merged 3 commits intomainfrom
remove-rag-reindex

aliasaria commented Mar 3, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

paragon-review bot commented Mar 3, 2026

Uh oh!

coderabbitai bot commented Mar 3, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

❌ Failed checks (1 warning)

Uh oh!

sentry bot commented Mar 3, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

aliasaria commented Mar 3, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Frontend

Backend

Summary by CodeRabbit

Uh oh!

paragon-review bot commented Mar 3, 2026

Paragon Summary

Confidence score: 5/5

Uh oh!

coderabbitai bot commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

❌ Failed checks (1 warning)

Uh oh!

sentry bot commented Mar 3, 2026

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

aliasaria commented Mar 3, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Mar 3, 2026 •

edited

Loading