feat(FR-2630): add useSToken hook with nuqs and stoken deprecation warning by nowgnuesLee · Pull Request #6852 · lablup/backend.ai-webui

nowgnuesLee · 2026-04-21T14:53:42Z

Resolves FR-2630 (sub-task of Epic FR-2616)

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

Stack

This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the dev plan for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828.

nowgnuesLee · 2026-04-21T14:53:55Z

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.

github-actions · 2026-04-21T14:56:49Z

Coverage report for `./react`

St.^❔	Category	Percentage	Covered / Total
🔴	Statements	8.59% (-0% 🔻)	1758/20472
🔴	Branches	7.86% (-0% 🔻)	1131/14384
🔴	Functions	5.15% (+0.01% 🔼)	286/5552
🔴	Lines	8.31% (-0% 🔻)	1650/19862

Show new covered files 🐣

St.^❔	File	Statements	Branches	Functions	Lines
🔴	hooks/useSToken.ts	0%	0%	0%	0%

Test suite run success

856 tests passing in 39 suites.

Report generated by 🧪jest coverage report action from fa2be96

Copilot

Pull request overview

Adds a dedicated useSToken hook in the React WebUI to read an sToken from URL query params via nuqs, prefer the canonical sToken key, and warn once when the deprecated lowercase stoken key is used. This supports the FR-2616 effort to extract the sToken login flow into a reusable STokenLoginBoundary.

Changes:

Introduces react/src/hooks/useSToken.ts to resolve sToken vs stoken (fallback) and emit a one-time deprecation warning.
Provides a tuple API returning the effective token and a setter intended to clear token-related query params.

yomybaby

LGTM

graphite-app · 2026-04-23T10:31:27Z

Merge activity

Apr 23, 10:31 AM UTC: yomybaby added this pull request to the Graphite merge queue.
Apr 23, 10:41 AM UTC: Merged by the Graphite merge queue.

…rning (#6852) Resolves FR-2630 (sub-task of Epic [FR-2616](https://lablup.atlassian.net/browse/FR-2616)) 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 ## Stack This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the [dev plan](../blob/main/.specs/draft-stoken-login-boundary/dev-plan.md) for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828. [FR-2616]: https://lablup.atlassian.net/browse/FR-2616?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

…int (#6850) Resolves FR-2628 (sub-task of Epic [FR-2616](https://lablup.atlassian.net/browse/FR-2616)) 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 ## Stack This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the [dev plan](../blob/main/.specs/draft-stoken-login-boundary/dev-plan.md) for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828. [FR-2616]: https://lablup.atlassian.net/browse/FR-2616?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

) Resolves FR-2629 (sub-task of Epic [FR-2616](https://lablup.atlassian.net/browse/FR-2616)) 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 ## Stack This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the [dev plan](../blob/main/.specs/draft-stoken-login-boundary/dev-plan.md) for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828. [FR-2616]: https://lablup.atlassian.net/browse/FR-2616?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

Resolves FR-2631 (sub-task of Epic [FR-2616](https://lablup.atlassian.net/browse/FR-2616)) 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 ## Stack This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the [dev plan](../blob/main/.specs/draft-stoken-login-boundary/dev-plan.md) for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828. [FR-2616]: https://lablup.atlassian.net/browse/FR-2616?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

…Boundary (#6855) Resolves FR-2632 (sub-task of Epic [FR-2616](https://lablup.atlassian.net/browse/FR-2616)) 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 ## Stack This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the [dev plan](../blob/main/.specs/draft-stoken-login-boundary/dev-plan.md) for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828. [FR-2616]: https://lablup.atlassian.net/browse/FR-2616?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

Resolves FR-2633 (sub-task of Epic [FR-2616](https://lablup.atlassian.net/browse/FR-2616)) 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 ## Stack This PR is part of the Story 1 stack for Epic FR-2616 (Extract sToken login flow into reusable boundary component). See the [dev plan](../blob/main/.specs/draft-stoken-login-boundary/dev-plan.md) for the full scope. The Story 1 PR stack is #6850 → #6851 → #6852 → #6853 → #6854 → #6855 → #6856 on top of spec PR #6828. [FR-2616]: https://lablup.atlassian.net/browse/FR-2616?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

github-actions Bot assigned nowgnuesLee Apr 21, 2026

This was referenced Apr 21, 2026

feat(FR-2618): add sToken login boundary component spec #6828

Merged

refactor(FR-2628): rename useEduAppApiEndpoint to useResolvedApiEndpoint #6850

Merged

github-actions Bot added the size:M 30~100 LoC label Apr 21, 2026

nowgnuesLee force-pushed the 04-21-feat_fr-2629_extend_tokenlogin_helper_with_optional_extraparams branch from dcb27d3 to ddbf758 Compare April 22, 2026 03:53

nowgnuesLee force-pushed the 04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning branch from 741c222 to 68fc2d7 Compare April 22, 2026 03:53

This was referenced Apr 22, 2026

fix(FR-2627): forward api_version/date/endpoint to token_login #6893

Closed

feat(FR-2616): handle totp and concurrent-session inline in STokenLoginBoundary #6897

Merged

nowgnuesLee marked this pull request as ready for review April 23, 2026 06:09

Copilot AI review requested due to automatic review settings April 23, 2026 06:09

Copilot started reviewing on behalf of nowgnuesLee April 23, 2026 06:10 View session

Copilot AI reviewed Apr 23, 2026

View reviewed changes

Comment thread react/src/hooks/useSToken.ts

Comment thread react/src/hooks/useSToken.ts Outdated

Comment thread react/src/hooks/useSToken.ts

nowgnuesLee force-pushed the 04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning branch from 68fc2d7 to 61a25de Compare April 23, 2026 07:57

nowgnuesLee force-pushed the 04-21-feat_fr-2629_extend_tokenlogin_helper_with_optional_extraparams branch from ddbf758 to 4db323c Compare April 23, 2026 07:57

nowgnuesLee force-pushed the 04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning branch from 61a25de to f6760d3 Compare April 23, 2026 08:02

nowgnuesLee force-pushed the 04-21-feat_fr-2629_extend_tokenlogin_helper_with_optional_extraparams branch from 4db323c to 66e3c56 Compare April 23, 2026 08:02

nowgnuesLee force-pushed the 04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning branch from f6760d3 to b931e8a Compare April 23, 2026 08:35

yomybaby approved these changes Apr 23, 2026

View reviewed changes

graphite-app Bot force-pushed the 04-21-feat_fr-2629_extend_tokenlogin_helper_with_optional_extraparams branch from 66e3c56 to 7d1542e Compare April 23, 2026 10:33

graphite-app Bot force-pushed the 04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning branch from b931e8a to fa2be96 Compare April 23, 2026 10:34

Base automatically changed from 04-21-feat_fr-2629_extend_tokenlogin_helper_with_optional_extraparams to main April 23, 2026 10:40

graphite-app Bot merged commit fa2be96 into main Apr 23, 2026
8 checks passed

graphite-app Bot deleted the 04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning branch April 23, 2026 10:41

github-pages Bot temporarily deployed to github-pages April 23, 2026 10:42 Inactive

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(FR-2630): add useSToken hook with nuqs and stoken deprecation warning#6852

feat(FR-2630): add useSToken hook with nuqs and stoken deprecation warning#6852
graphite-app[bot] merged 1 commit into
mainfrom
04-21-feat_fr-2630_add_usestoken_hook_with_nuqs_and_stoken_deprecation_warning

nowgnuesLee commented Apr 21, 2026 •

edited by atlassian Bot

Loading

Uh oh!

nowgnuesLee commented Apr 21, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Apr 21, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

yomybaby left a comment

Uh oh!

graphite-app Bot commented Apr 23, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

nowgnuesLee commented Apr 21, 2026 • edited by atlassian Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Stack

Uh oh!

nowgnuesLee commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

github-actions Bot commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./react

Test suite run success

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

yomybaby left a comment

Choose a reason for hiding this comment

Uh oh!

graphite-app Bot commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

nowgnuesLee commented Apr 21, 2026 •

edited by atlassian Bot

Loading

nowgnuesLee commented Apr 21, 2026 •

edited

Loading

github-actions Bot commented Apr 21, 2026 •

edited

Loading

Coverage report for `./react`

graphite-app Bot commented Apr 23, 2026 •

edited

Loading