Skip to content

fix(FR-2529): include backend.ai-client src in craco TS loader and tsconfig paths#6615

Closed
yomybaby wants to merge 1 commit intographite-base/6615from
04-11-fix_fr-2529_add_backend.ai-client_to_babel-loader_include_and_tsconfig_paths
Closed

fix(FR-2529): include backend.ai-client src in craco TS loader and tsconfig paths#6615
yomybaby wants to merge 1 commit intographite-base/6615from
04-11-fix_fr-2529_add_backend.ai-client_to_babel-loader_include_and_tsconfig_paths

Conversation

@yomybaby
Copy link
Copy Markdown
Member

@yomybaby yomybaby commented Apr 11, 2026

resolves #NNN (FR-MMM)

Checklist: (if applicable)

  • Documentation
  • Minium required manager version
  • Specific setting for review (eg., KB link, endpoint or how to setup)
  • Minimum requirements to check during review
  • Test case(s) to demonstrate the difference of before/after

@yomybaby
fix(FR-2529): add backend.ai-client to babel-loader include and tscon…
e8ffb5a 
…fig paths

Fix dev mode compilation error by adding backend.ai-client/src to
webpack babel-loader include paths (matching backend.ai-ui pattern).
Also restore tsconfig path mapping pointing to built .d.ts
for TypeScript module resolution.
@github-actions github-actions Bot added the size:S 10~30 LoC label Apr 11, 2026
@yomybaby Graphite App
Copy link
Copy Markdown
Member Author

yomybaby commented Apr 11, 2026 •
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of 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.

@yomybaby yomybaby mentioned this pull request Apr 11, 2026
Merged
4 tasks
@yomybaby yomybaby changed the base branch from refactor/FR-2529-extract-backend-ai-client to graphite-base/6615 April 11, 2026 08:41
@yomybaby yomybaby requested a review from ironAiken2 April 30, 2026 04:40
@yomybaby yomybaby mentioned this pull request Apr 30, 2026
Merged
@nowgnuesLee nowgnuesLee mentioned this pull request Apr 30, 2026
Merged
2 tasks
This was referenced Apr 30, 2026
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Apr 30, 2026
Merged
Merged
@yomybaby yomybaby changed the title fix(FR-2529): add backend.ai-client to babel-loader include and tsconfig paths fix(FR-2529): include backend.ai-client src in craco TS loader and tsconfig paths Apr 30, 2026
@yomybaby
Copy link
Copy Markdown
Member Author

yomybaby commented Apr 30, 2026

Obsolete after rebase — changes already covered by parent commit (ffe57b6) and Vite migration (Craco retired in FR-2611).

@yomybaby yomybaby closed this Apr 30, 2026
@yomybaby yomybaby deleted the 04-11-fix_fr-2529_add_backend.ai-client_to_babel-loader_include_and_tsconfig_paths branch April 30, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ironAiken2 ironAiken2 Awaiting requested review from ironAiken2

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

@yomybaby yomybaby

Labels

size:S 10~30 LoC

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yomybaby