Document ingestion support for PDF and docx and WAV chunk decoding - #63
Merged
Conversation
…etrics (ATA parity)
…ATA React UI compatibility
sachinkaushik
approved these changes
Jul 24, 2026
sainijit
previously approved these changes
Jul 24, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Adds richer ingestion and observability features to the smart kiosk stack by expanding RAG context upload to support document formats (PDF/DOCX), enhancing query-time metrics streaming, and improving browser audio streaming compatibility with WAV-container chunks. Also introduces endpoints to serve synthesized response audio segments and to proxy platform hardware info.
Changes:
- Add PDF/DOCX parsing for
/api/v1/context/fileuploads and batch ingestion response types. - Extend RAG query request flags and SSE output to optionally include performance/LLM metrics.
- Decode browser-uploaded WAV chunks into PCM frames and add endpoints for response-audio playback and platform-info proxying.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| smart-kiosk-assistant/rag-service/requirements.txt | Adds dependencies for DOCX/PDF ingestion (python-docx, pypdf). |
| smart-kiosk-assistant/rag-service/dto/query_dto.py | Adds request flags for metrics and new DTOs for per-file + batch ingest responses. |
| smart-kiosk-assistant/rag-service/api/custom_endpoints.py | Implements DOCX/PDF text extraction, batch file ingestion, and optional SSE metrics emission. |
| smart-kiosk-assistant/main.py | Adds endpoint to serve synthesized response-audio WAV segments. |
| smart-kiosk-assistant/kiosk_core/service.py | Adds lookup helper for synthesized response-audio segment file paths. |
| smart-kiosk-assistant/kiosk_core/audio_session.py | Decodes WAV-container chunks to PCM to stabilize downstream RMS/VAD/ASR processing. |
| smart-kiosk-assistant/kiosk_core/api/endpoints.py | Adds /api/v1/platform-info proxy endpoint to metrics-collector. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
avinash-palleti
previously approved these changes
Jul 24, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
Author
|
I have included the changes suggested by copilot. |
avinash-palleti
approved these changes
Jul 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.