Skip to content

feat:add new api for orion build#1205

Merged
genedna merged 3 commits into
gitmono-dev:mainfrom
benjamin-747:main
Jul 8, 2025
Merged

feat:add new api for orion build#1205
genedna merged 3 commits into
gitmono-dev:mainfrom
benjamin-747:main

Conversation

@benjamin-747

Copy link
Copy Markdown
Collaborator

No description provided.

@vercel

vercel Bot commented Jul 8, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mega ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 8, 2025 9:43am

@benjamin-747 benjamin-747 changed the title feat: add orion-server deploy workflow feat:add new api for orion build Jul 8, 2025
@genedna
genedna requested a review from Copilot July 8, 2025 09:42

Copilot AI 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.

Pull Request Overview

This PR introduces a new Merge Request files-list API, supporting model and service logic, adds a Git tree endpoint, and includes a new GitHub Actions workflow for Orion server deployment.

  • Adds mr_files_list HTTP endpoint and model MrFilesRes for listing MR diff files
  • Implements MrDiffFile enum and service methods (mr_files_list, get_commit_blobs, traverse_tree)
  • Introduces get_tree_file route and an Orion server deployment workflow

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
mono/src/api/mr/mr_router.rs Registered and implemented new mr_files_list route
mono/src/api/mr/mod.rs Added MrFilesRes struct and From<MrDiffFile> conversion
mono/src/api/api_router.rs Added get_tree_file handler for repository tree listing
ceres/src/model/mr.rs Defined MrDiffFile enum representing file diff types
ceres/src/model/mod.rs Exposed the new mr module
ceres/src/api_service/mono_api_service.rs Implemented service methods for computing MR diff and walking trees
.github/workflows/orion-server-deploy.yml New GitHub Actions workflow for deploying the Orion server
Comments suppressed due to low confidence (4)

.github/workflows/orion-server-deploy.yml:58

  • The AWS region is set to ap-southeast-2 here but the credentials were configured with us-east-1. Align both regions to avoid deployment failures.
          AWS_REGION: ap-southeast-2

mono/src/api/mr/mr_router.rs:265

  • [nitpick] The new mr_files_list endpoint lacks tests; consider adding unit or integration tests to verify its behavior.
async fn mr_files_list(

mono/src/api/mr/mod.rs:113

  • The ToSchema derive is used but not imported; add use utoipa::ToSchema; at the top of this file to resolve the missing trait.
#[derive(Serialize, ToSchema)]

ceres/src/api_service/mono_api_service.rs:433

  • The code uses HashSet but there's no use std::collections::HashSet; import; this will fail to compile. Please add the import.
        let unions: HashSet<PathBuf> = old_files.keys().chain(new_files.keys()).cloned().collect();

@genedna
genedna added this pull request to the merge queue Jul 8, 2025
Merged via the queue into gitmono-dev:main with commit 1f4f5d3 Jul 8, 2025
10 checks passed
yyk808 pushed a commit to yyk808/mega that referenced this pull request Jul 9, 2025
* feat: add orion-server deploy workflow

* feat:add new api for orion build

---------

Co-authored-by: Quanyi Ma <eli@patch.sh>
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.

3 participants