feat(FR-1530): Implement window focus check for automatic refresh interval #4358

agatha197 · 2025-09-29T04:31:54Z

Add pauseWhenHidden option to interval hooks

This PR adds a pauseWhenHidden option to the useInterval and useIntervalValue hooks, allowing intervals to be paused when the page is hidden (tab not in focus). The feature is enabled by default but can be disabled by setting pauseWhenHidden={false}.

When enabled:

Intervals pause when the page is hidden
Callbacks execute immediately when the page becomes visible again
Reduces unnecessary background processing

The BAIFetchKeyButton component has been updated to support this new option, with the default behavior maintaining backward compatibility.

Checklist:

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

agatha197 · 2025-09-29T04:32:11Z

feat(FR-1530): Implement window focus check for automatic refresh interval #4358 : 2 dependent PRs (#4365 , #4371 ) 👈 (View in Graphite)
main

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 hot fixes, skip the queue and merge this PR next

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 · 2025-09-29T04:34:04Z

Coverage report for `./react`

St.^❔	Category	Percentage	Covered / Total
🔴	Statements	4.41% (-0.01% 🔻)	501/11351
🔴	Branches	3.56% (-0.01% 🔻)	285/8016
🔴	Functions	2.55% (-0% 🔻)	90/3532
🔴	Lines	4.39% (-0.01% 🔻)	487/11095

Test suite run success

114 tests passing in 13 suites.

Report generated by 🧪jest coverage report action from a737f11

react/src/hooks/useIntervalValue.tsx

nowgnuesLee

LGTM

Copilot

Pull Request Overview

This PR implements a window focus check feature for automatic refresh intervals by adding a pauseOnWindowBlur option to interval hooks. When enabled (default), intervals pause when the window loses focus and resume with an immediate callback execution when focus is regained, reducing unnecessary background processing.

Added pauseOnWindowBlur parameter to useInterval and useIntervalValue hooks with default value of true
Implemented window focus/blur event handling to pause/resume intervals based on window focus state
Updated BAIFetchKeyButton component to support the new pauseOnWindowBlur option

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
react/src/hooks/useIntervalValue.tsx	Added window focus/blur detection and `pauseOnWindowBlur` parameter to both interval hooks
react/src/components/BAIFetchKeyButton.tsx	Added `pauseOnWindowBlur` prop support with default value of `true`
react/src/components/KeypairResourcePolicySettingModal.tsx	Removed commented-out code line

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

react/src/hooks/useIntervalValue.tsx

yomybaby

Please consider document.visibilityState instead of focus.

yomybaby

LGTM

graphite-app · 2025-10-01T08:14:06Z

Merge activity

Oct 1, 8:14 AM UTC: yomybaby added this pull request to the Graphite merge queue.
Oct 1, 8:16 AM UTC: Merged by the Graphite merge queue.

…erval (#4358) Resolves #4357 ([FR-1530](https://lablup.atlassian.net/browse/FR-1530)) # Add pauseWhenHidden option to interval hooks This PR adds a `pauseWhenHidden` option to the `useInterval` and `useIntervalValue` hooks, allowing intervals to be paused when the page is hidden (tab not in focus). The feature is enabled by default but can be disabled by setting `pauseWhenHidden={false}`. When enabled: - Intervals pause when the page is hidden - Callbacks execute immediately when the page becomes visible again - Reduces unnecessary background processing The `BAIFetchKeyButton` component has been updated to support this new option, with the default behavior maintaining backward compatibility. **Checklist:** - [ ] 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 [FR-1530]: https://lablup.atlassian.net/browse/FR-1530?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

github-actions bot assigned agatha197 Sep 29, 2025

github-actions bot added the size:M 30~100 LoC label Sep 29, 2025

agatha197 mentioned this pull request Sep 29, 2025

feat(FR-1240): add agent statistics and resource monitoring capabilities #4344

Merged

4 tasks

agatha197 requested review from nowgnuesLee and yomybaby September 29, 2025 04:32

nowgnuesLee requested changes Sep 29, 2025

View reviewed changes

react/src/hooks/useIntervalValue.tsx Outdated Show resolved Hide resolved

react/src/hooks/useIntervalValue.tsx Outdated Show resolved Hide resolved

react/src/hooks/useIntervalValue.tsx Outdated Show resolved Hide resolved

agatha197 mentioned this pull request Sep 29, 2025

feat(FR-1531): add active agent list to admin dashboard page #4365

Merged

5 tasks

agatha197 changed the base branch from feat_FR-1240_/admin-dashboard to graphite-base/4358 September 29, 2025 13:53

agatha197 force-pushed the graphite-base/4358 branch from 293d794 to c6915f4 Compare September 29, 2025 13:53

agatha197 force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from 09b8426 to 01dee45 Compare September 29, 2025 13:53

agatha197 changed the base branch from graphite-base/4358 to main September 29, 2025 13:53

agatha197 requested a review from nowgnuesLee September 29, 2025 13:54

nowgnuesLee approved these changes Sep 30, 2025

View reviewed changes

yomybaby force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from 01dee45 to 948e77d Compare September 30, 2025 02:07

Copilot AI review requested due to automatic review settings September 30, 2025 02:07

Copilot AI reviewed Sep 30, 2025

View reviewed changes

react/src/hooks/useIntervalValue.tsx Outdated Show resolved Hide resolved

react/src/hooks/useIntervalValue.tsx Outdated Show resolved Hide resolved

yomybaby force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from 948e77d to f108b95 Compare September 30, 2025 02:08

yomybaby requested changes Sep 30, 2025

View reviewed changes

agatha197 force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from f108b95 to 63d1fed Compare September 30, 2025 05:41

agatha197 mentioned this pull request Sep 30, 2025

feat(FR-1530): add useIntervalValue test code and fix broken test code snapshot #4371

Merged

1 task

agatha197 requested a review from yomybaby September 30, 2025 06:17

agatha197 marked this pull request as draft September 30, 2025 06:19

agatha197 force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from 63d1fed to 3ecb178 Compare September 30, 2025 06:27

agatha197 marked this pull request as ready for review September 30, 2025 06:27

agatha197 marked this pull request as draft September 30, 2025 06:28

agatha197 force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch 3 times, most recently from 9c43d1d to 4e1ef17 Compare September 30, 2025 06:57

agatha197 marked this pull request as ready for review September 30, 2025 06:57

agatha197 force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from 4e1ef17 to c99d1cb Compare September 30, 2025 07:36

yomybaby force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from c99d1cb to 874597a Compare October 1, 2025 08:11

yomybaby approved these changes Oct 1, 2025

View reviewed changes

graphite-app bot force-pushed the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch from 874597a to a737f11 Compare October 1, 2025 08:14

graphite-app bot merged commit a737f11 into main Oct 1, 2025
11 checks passed

graphite-app bot deleted the feat_FR-1530_/window-focus-check-for-automatic-refresh-interval branch October 1, 2025 08:16

vercel bot deployed to Production October 1, 2025 08:24 View deployment

This was referenced Oct 13, 2025

refactor(FR-1240): refactor FR-1240 with agentStats query #4399

Closed

feat(FR-873): set dashboard page as a default #3554

Merged

agatha197 mentioned this pull request Oct 21, 2025

feat(FR-1575): merge admin dashboard into user dashboard #4425

Merged

5 tasks

yomybaby mentioned this pull request Oct 21, 2025

style(FR-1577): improve dashboard statistics display design and progress visualization #4427

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

feat(FR-1530): Implement window focus check for automatic refresh interval #4358

feat(FR-1530): Implement window focus check for automatic refresh interval #4358

Uh oh!

agatha197 commented Sep 29, 2025 •

edited by atlassian bot

Loading

Uh oh!

agatha197 commented Sep 29, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Sep 29, 2025 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nowgnuesLee left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

yomybaby left a comment

Uh oh!

yomybaby left a comment

Uh oh!

graphite-app bot commented Oct 1, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

feat(FR-1530): Implement window focus check for automatic refresh interval #4358

feat(FR-1530): Implement window focus check for automatic refresh interval #4358

Uh oh!

Conversation

agatha197 commented Sep 29, 2025 • edited by atlassian bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Add pauseWhenHidden option to interval hooks

Uh oh!

agatha197 commented Sep 29, 2025 • 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 Sep 29, 2025 • 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!

Uh oh!

Uh oh!

Uh oh!

nowgnuesLee left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

yomybaby left a comment

Choose a reason for hiding this comment

Uh oh!

yomybaby left a comment

Choose a reason for hiding this comment

Uh oh!

graphite-app bot commented Oct 1, 2025 • 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!

4 participants

agatha197 commented Sep 29, 2025 •

edited by atlassian bot

Loading

agatha197 commented Sep 29, 2025 •

edited

Loading

github-actions bot commented Sep 29, 2025 •

edited

Loading

Coverage report for `./react`

graphite-app bot commented Oct 1, 2025 •

edited

Loading