Skip to content

feat: utilize axios for API calls - #36

Open
AryanKpr wants to merge 3 commits into
mainfrom
feature/issue-15-tanstack-axios
Open

AryanKpr wants to merge 3 commits into
mainfrom
feature/issue-15-tanstack-axios

Conversation

@AryanKpr

Copy link
Copy Markdown
Collaborator

📋 PR Overview

Summary

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • [✅ ] ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🔧 Configuration change (changes to build, CI, or configuration files)
  • 📚 Documentation update (changes to documentation only)
  • 🎨 UI/UX improvement (styling, layout, or user experience changes)
  • ⚡ Performance improvement (code changes that improve performance)
  • [ ✅] 🧹 Code cleanup/refactoring (non-functional changes that improve code quality)

QA Instructions

  1. Verify documents load correctly in the research page
  2. Verify prepare-index works when selecting documents
  3. Note: chat endpoint kept as fetch due to streaming requirement
  4. Run npm install to ensure axios is installed

Related Issues

Resolves #15

  • Closes #

@vercel

vercel Bot commented Sep 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
nexus Ready Ready Preview Sep 17, 2026 5:45pm UTC

Aryan added 2 commits September 16, 2026 20:26
Fixed (16 vulnerabilities):
- critical: tar
- high: brace-expansion, flatted, js-yaml, jws,
  lodash, minimatch, picomatch, ws
- moderate: ajv, mdast-util-to-hast, qs, uuid

Remaining 12 cannot be fixed without breaking changes:
- next/postcss/sharp: locked by @LlamaIndex peer deps
- undici/@ai-sdk: locked by ai package version
- prismjs: locked by react-syntax-highlighter

Closes #4
- Replace fetch with axios in useDocuments.ts
- Replace fetch with axios in prepare-index call
- Chat endpoint kept as fetch due to streaming requirement

Closes #15
- Reset package-lock.json to match main's llamaindex version
- Added axios cleanly on top of working dependency tree
- Fixes WorkflowStream type error in chat/route.ts
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.

Utilize Tanstack/Axios Library for API Calls

1 participant