chore(autorag+automl): RESTful refactor of S3 BFF endpoints#7358
chore(autorag+automl): RESTful refactor of S3 BFF endpoints#7358GAUNSD wants to merge 3 commits intoopendatahub-io:mainfrom
Conversation
|
Skipping CI for Draft Pull Request. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7358 +/- ##
==========================================
+ Coverage 63.89% 65.03% +1.14%
==========================================
Files 2513 2458 -55
Lines 77939 76402 -1537
Branches 19818 19278 -540
==========================================
- Hits 49797 49691 -106
+ Misses 28142 26711 -1431 see 64 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Propagate S3 endpoint RESTful refactor from OpenAPI specs to BFF Go handlers and frontend TypeScript callers in both automl and autorag. - BFF handlers (s3_handler.go): extract key from URL path param (ps.ByName) instead of query string; update signatures and doc comments. - BFF routing (app.go): update path constants to use :key path segment. - Frontend API callers (api/s3.ts): embed key in URL path via encodeURIComponent instead of query param. - Frontend hooks (queries.ts, mutations.ts): rewrite fetch/XHR URLs to use /s3/files/<key> path. Generated-by: Claude Opus 4.6 noreply@anthropic.com Co-authored-by: Claude Opus 4.6 noreply@anthropic.com
Issues
Description
How Has This Been Tested?
Test Impact
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
main