feature(FR-1471): migrate the GitHub/GitLab importing feature to React with VFolder usage mode option #4600
+795
−954
Conversation
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
github-actions
bot
added
area:ux
5 tasks
Member
Author
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 49.62% | 132/266 |
| 🔴 | Branches | 29.96% | 80/267 |
| 🔴 | Functions | 33.33% | 20/60 |
| 🔴 | Lines | 51.95% | 120/231 |
Test suite run success
55 tests passing in 3 suites.
Report generated by 🧪jest coverage report action from 42aaf48
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 4.74% | 547/11529 |
| 🔴 | Branches | 3.86% | 314/8133 |
| 🔴 | Functions | 2.93% | 104/3548 |
| 🔴 | Lines | 4.69% | 529/11271 |
Test suite run success
125 tests passing in 14 suites.
Report generated by 🧪jest coverage report action from 42aaf48
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
27fa091 to
492e90e
Compare
yomybaby
force-pushed
the
feautre/migrate-to-all-import-feature-to-react
branch
from
7b4c0c3 to
d6ecc7a
Compare
yomybaby
force-pushed
the
feautre/migrate-to-all-import-feature-to-react
branch
from
d6ecc7a to
83b6f17
Compare
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
492e90e to
b076bde
Compare
5 tasks
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
b076bde to
a500584
Compare
graphite-app
bot
changed the base branch from
feautre/migrate-to-all-import-feature-to-react
to
graphite-base/4600
yomybaby
force-pushed
the
graphite-base/4600
branch
from
83b6f17 to
53cfb56
Compare
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
a500584 to
4589c0c
Compare
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
4589c0c to
215abba
Compare
Copilot finished reviewing on behalf of
yomybaby
November 11, 2025 03:39
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the GitHub and GitLab repository import functionality from the legacy web component (backend-ai-import-view.ts) to a modern React-based implementation. The feature enables users to import entire repositories from GitHub and GitLab into Backend.AI virtual folders, expanding beyond the previous notebook-only import capability.
- Removed legacy web component code for GitHub/GitLab importing (~400 lines)
- Created new React components (
ImportRepoForm) with separate UI cards for notebook, GitHub, and GitLab imports - Added
useGetAvailableFolderNamehook to generate unique folder names and prevent collisions - Added translation keys across 18 language files for new UI elements
Reviewed Changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 22 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/backend-ai-import-view.ts | Removed legacy GitHub/GitLab import functionality and unused imports |
| react/src/pages/ImportAndRunPage.tsx | Added separate cards for notebook, GitHub, and GitLab imports with updated layout |
| react/src/components/ImportRepoForm.tsx | New component handling repository imports with URL parsing, validation, and batch session creation |
| react/src/components/ImportNotebook.tsx | Updated form layout from inline to vertical and improved navigation logic |
| react/src/hooks/index.tsx | Added vfolder.create method signature to BackendAIClient type |
| packages/backend.ai-ui/src/hooks/useGetAvailableFolderName.ts | New hook to check folder name availability and generate unique names with hash suffixes |
| packages/backend.ai-ui/src/hooks/index.ts | Exported new useGetAvailableFolderName hook |
| resources/i18n/*.json | Added NewVFolderName and NewVFolderNameHelp translation keys across 18 language files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
215abba to
1d4b0c8
Compare
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
0cb900c to
7ecf347
Compare
yomybaby
requested review from
ironAiken2 and
nowgnuesLee
and removed request for
Copilot
Copilot finished reviewing on behalf of
yomybaby
November 11, 2025 12:39
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
2 times, most recently
from
30c6e69 to
987cc03
Compare
yomybaby
changed the title
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
2 times, most recently
from
2e961aa to
998cd9f
Compare
5 tasks
ironAiken2
requested changes
Nov 12, 2025
nowgnuesLee
requested changes
Nov 12, 2025
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
998cd9f to
adc4754
Compare
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
adc4754 to
16ef141
Compare
yomybaby
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
16ef141 to
95898aa
Compare
5 tasks
Merge activity
|
…t with VFolder usage mode option (#4600) Resolves #4278 ([FR-1471](https://lablup.atlassian.net/browse/FR-1471)) This PR migrates the GitHub and GitLab repository import functionality from the legacy web components to the React UI. The implementation includes: - Added a new `useGetAvailableFolderName` hook to generate unique folder names for imported repositories - Created `ImportRepoForm` component to handle GitHub and GitLab repository imports - Enhanced the import page UI with separate cards for notebook, GitHub, and GitLab imports - Added badge generation functionality that creates HTML/Markdown code for notebooks - Improved the `CopyButton` component to support translations and proper disabled states - Updated translations for all import-related features across multiple languages The implementation ensures repositories are properly downloaded, extracted, and made available as folders that can be mounted in sessions. [FR-1471]: https://lablup.atlassian.net/browse/FR-1471?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
graphite-app
bot
force-pushed
the
feature/migrate-github-gitlab-import
branch
from
95898aa to
42aaf48
Compare
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.
Resolves #4278 (FR-1471)
This PR migrates the GitHub and GitLab repository import functionality from the legacy web components to the React UI. The implementation includes:
useGetAvailableFolderNamehook to generate unique folder names for imported repositoriesImportRepoFormcomponent to handle GitHub and GitLab repository importsCopyButtoncomponent to support translations and proper disabled statesThe implementation ensures repositories are properly downloaded, extracted, and made available as folders that can be mounted in sessions.