Skip to content

Add Bucket And Provider Routes - #65

Merged
llam36 merged 14 commits into
mainfrom
long/bucket-provider-routes
Feb 9, 2026
Merged

llam36 merged 14 commits into
mainfrom
long/bucket-provider-routes

Conversation

@llam36

@llam36 llam36 commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

Add Bucket And Provider Routes

@greptile-apps

greptile-apps Bot commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

Disclaimer: This comment was AI-generated and is not necessarily completely accurate. Please take code comments with a grain of salt.

Greptile Summary

  • Added complete CRUD operations for file buckets and providers (delete, get all, get by config)
  • Refactored settings functionality by removing SettingsAPI and moving getEmailConfig to EmailAPI
  • Updated auto-generated API code with new models and reorganized imports

Confidence Score: 5/5

  • This PR is safe to merge with no concerns.
  • All changes are well-implemented with proper validation, error handling, and consistent patterns. The auto-generated code follows established conventions, and the refactoring improves code organization by consolidating related functionality.
  • No files require special attention

Important Files Changed

Filename Overview
src/lib/file.ts Added bucket and provider management methods with proper validation.
src/lib/juno.ts Removed deprecated SettingsAPI and reorganized imports.
src/lib/email.ts Moved getEmailConfig from SettingsAPI to EmailAPI with proper implementation.

Sequence Diagram

sequenceDiagram
    participant User
    participant FileAPI
    participant BucketAPI
    participant ProviderAPI
    participant ConfigAPI
    
    User->>FileAPI: "registerProvider(options)"
    FileAPI->>ProviderAPI: "fileProviderControllerRegisterFileProvider()"
    ProviderAPI-->>FileAPI: "FileProviderPartial"
    FileAPI-->>User: "FileProviderPartial"
    
    User->>FileAPI: "registerBucket(options)"
    FileAPI->>BucketAPI: "fileBucketControllerRegisterFileBucket()"
    BucketAPI-->>FileAPI: "FileBucket"
    FileAPI-->>User: "FileBucket"
    
    User->>FileAPI: "getAllFileProviders()"
    FileAPI->>ProviderAPI: "fileProviderControllerGetAllFileProviders()"
    ProviderAPI-->>FileAPI: "Array<FileProvider>"
    FileAPI-->>User: "Array<FileProvider>"
    
    User->>FileAPI: "getBucketsByConfigIdAndEnv(configId)"
    FileAPI->>BucketAPI: "fileBucketControllerGetBucketsByConfigIdAndEnv()"
    BucketAPI-->>FileAPI: "Array<FileBucket>"
    FileAPI-->>User: "Array<FileBucket>"
    
    User->>FileAPI: "deleteBucket(options)"
    FileAPI->>BucketAPI: "fileBucketControllerDeleteFileBucket()"
    BucketAPI-->>FileAPI: "FileBucket"
    FileAPI-->>User: "FileBucket"
    
    User->>FileAPI: "deleteProvider(name)"
    FileAPI->>ProviderAPI: "fileProviderControllerDeleteFileProvider()"
    ProviderAPI-->>FileAPI: "FileProviderPartial"
    FileAPI-->>User: "FileProviderPartial"
Loading

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

14 files reviewed, no comments

Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format

@llam36
llam36 requested a review from UZ9 November 18, 2025 05:38
@llam36
llam36 requested a review from shaply January 28, 2026 20:07
@llam36
llam36 merged commit 1766e21 into main Feb 9, 2026
2 checks passed
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.

1 participant