feat(FR-1679): add auto refresh to container logs #4640

agatha197 · 2025-11-14T05:58:40Z

Add auto-refresh feature to container logs with configurable interval

This PR enhances the container log modal with an auto-refresh feature that allows users to automatically refresh logs at configurable intervals. Users can toggle auto-refresh on/off and set their preferred refresh interval using a time picker.

The implementation includes:

Enhanced AutoRefreshSwitch component to accept additional props and use a configurable interval
Added auto-refresh controls to the container log modal with a time picker for interval selection
Stored user preferences for auto-refresh settings in user settings
Added internationalization support for "Auto refresh" text in all language files

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-11-14T05:58:59Z

feat(FR-1679): add auto refresh to container logs #4640 👈 (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-11-14T06:00:53Z

Coverage report for `./react`

St.^❔	Category	Percentage	Covered / Total
🔴	Statements	4.7%	548/11648
🔴	Branches	3.81%	314/8250
🔴	Functions	2.91%	104/3571
🔴	Lines	4.65%	530/11391

Test suite run success

125 tests passing in 14 suites.

Report generated by 🧪jest coverage report action from 879081b

Copilot

Pull Request Overview

This PR adds an auto-refresh feature to the container log modal with a configurable refresh interval. Users can toggle auto-refresh on/off and set their preferred interval using a TimePicker. The implementation refactors the existing AutoRefreshSwitch component to be more flexible and stores user preferences persistently.

Key changes:

Enhanced AutoRefreshSwitch to accept interval and other props as configurable parameters
Added auto-refresh controls with TimePicker to the container log modal
Persisted auto-refresh preferences (enabled state and interval) in user settings
Added i18n support for "Auto refresh" across all supported languages

Reviewed Changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated 4 comments.

File	Description
react/src/components/AutoRefreshSwitch.tsx	Refactored to accept interval and switch props, making the component more flexible and reusable
react/src/components/ComputeSessionNodeItems/ContainerLogModal.tsx	Added auto-refresh UI controls with TimePicker and integrated with user settings
react/src/hooks/useBAISetting.tsx	Added TypeScript interface properties for storing auto-refresh preferences
resources/i18n/*.json	Added "AutoRefresh" translation key in all 20 supported language files

react/src/components/ComputeSessionNodeItems/ContainerLogModal.tsx

yomybaby

It doesn't work correctly on the first activation. You can reproduce this by clearing the relevant localStorage keys and trying again.
Instead of using a TimePicker, what do you think about accepting just a single number representing seconds?

agatha197 · 2025-11-18T13:25:24Z

Oops, I didn't use autoRefreshIntervalValue in below. Thanks!

Copilot

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

yomybaby

LGTM

graphite-app · 2025-11-19T02:58:59Z

Merge activity

Nov 19, 2:58 AM UTC: agatha197 added this pull request to the Graphite merge queue.
Nov 19, 3:00 AM UTC: Merged by the Graphite merge queue.

resolves FR-1679 # Add auto-refresh feature to container logs with configurable interval This PR enhances the container log modal with an auto-refresh feature that allows users to automatically refresh logs at configurable intervals. Users can toggle auto-refresh on/off and set their preferred refresh interval using a time picker. The implementation includes: - Enhanced `AutoRefreshSwitch` component to accept additional props and use a configurable interval - Added auto-refresh controls to the container log modal with a time picker for interval selection - Stored user preferences for auto-refresh settings in user settings - Added internationalization support for "Auto refresh" text in all language files <img width="312" height="53" alt="image" src="https://github.com/user-attachments/assets/eb5cd40f-d030-44dc-990f-84bc55003443" /> **Checklist:** - [x] 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

Copilot AI review requested due to automatic review settings November 14, 2025 05:58

github-actions bot assigned agatha197 Nov 14, 2025

github-actions bot added area:ux UI / UX issue. area:i18n Localization size:M 30~100 LoC labels Nov 14, 2025

Copilot started reviewing on behalf of agatha197 November 14, 2025 05:59 View session

Copilot finished reviewing on behalf of agatha197 November 14, 2025 06:00

agatha197 requested review from ironAiken2, kyujin-cho, nowgnuesLee and yomybaby and removed request for Copilot November 14, 2025 06:00

agatha197 requested a review from Beomi November 14, 2025 06:01

agatha197 marked this pull request as draft November 14, 2025 06:43

agatha197 removed request for Beomi, kyujin-cho and nowgnuesLee November 14, 2025 06:43

agatha197 force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch 2 times, most recently from 1f39760 to 0355b24 Compare November 17, 2025 02:58

github-actions bot added size:L 100~500 LoC and removed size:M 30~100 LoC labels Nov 17, 2025

agatha197 force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from 0355b24 to 579480c Compare November 17, 2025 02:59

agatha197 marked this pull request as ready for review November 17, 2025 03:00

Copilot AI review requested due to automatic review settings November 17, 2025 03:00

Copilot started reviewing on behalf of agatha197 November 17, 2025 03:01 View session

Copilot finished reviewing on behalf of agatha197 November 17, 2025 03:02

Copilot AI reviewed Nov 17, 2025

View reviewed changes

yomybaby requested changes Nov 18, 2025

View reviewed changes

agatha197 force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from 579480c to dab66f2 Compare November 18, 2025 13:24

agatha197 mentioned this pull request Nov 18, 2025

feat(FR-1634): Configure Copilot to accept 'use memo' React directive #4667

Merged

1 task

agatha197 changed the base branch from main to graphite-base/4640 November 18, 2025 13:37

agatha197 force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from dab66f2 to 2528469 Compare November 18, 2025 13:37

agatha197 changed the base branch from graphite-base/4640 to fix/FR-1634 November 18, 2025 13:37

agatha197 force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from 2528469 to 6a783b4 Compare November 18, 2025 13:41

agatha197 requested review from Copilot and yomybaby November 18, 2025 13:41

Copilot AI reviewed Nov 18, 2025

View reviewed changes

Copilot started reviewing on behalf of agatha197 November 18, 2025 13:42 View session

Copilot finished reviewing on behalf of agatha197 November 18, 2025 13:47

yomybaby changed the base branch from fix/FR-1634 to graphite-base/4640 November 19, 2025 02:51

yomybaby force-pushed the graphite-base/4640 branch from 00ce91c to 38f1cb8 Compare November 19, 2025 02:51

yomybaby force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from 6a783b4 to 1822856 Compare November 19, 2025 02:51

yomybaby changed the base branch from graphite-base/4640 to main November 19, 2025 02:51

yomybaby approved these changes Nov 19, 2025

View reviewed changes

yomybaby force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from 1822856 to 5dca640 Compare November 19, 2025 02:53

graphite-app bot force-pushed the feat_FR-1679_/add-auto-refresh-to-container-logs branch from 5dca640 to 879081b Compare November 19, 2025 02:59

graphite-app bot merged commit 879081b into main Nov 19, 2025
11 checks passed

graphite-app bot deleted the feat_FR-1679_/add-auto-refresh-to-container-logs branch November 19, 2025 03:00

github-pages bot temporarily deployed to github-pages November 19, 2025 03:00 Inactive

vercel bot had a problem deploying to Production November 19, 2025 03:44 Failure

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(FR-1679): add auto refresh to container logs #4640

feat(FR-1679): add auto refresh to container logs #4640

Uh oh!

agatha197 commented Nov 14, 2025 •

edited

Loading

Uh oh!

agatha197 commented Nov 14, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Nov 14, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

yomybaby left a comment

Uh oh!

agatha197 commented Nov 18, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

yomybaby left a comment

Uh oh!

graphite-app bot commented Nov 19, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

feat(FR-1679): add auto refresh to container logs #4640

feat(FR-1679): add auto refresh to container logs #4640

Uh oh!

Conversation

agatha197 commented Nov 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Add auto-refresh feature to container logs with configurable interval

Uh oh!

agatha197 commented Nov 14, 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 Nov 14, 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!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

yomybaby left a comment

Choose a reason for hiding this comment

Uh oh!

agatha197 commented Nov 18, 2025

Uh oh!

Copilot AI 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 Nov 19, 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!

3 participants

agatha197 commented Nov 14, 2025 •

edited

Loading

agatha197 commented Nov 14, 2025 •

edited

Loading

github-actions bot commented Nov 14, 2025 •

edited

Loading

Coverage report for `./react`

graphite-app bot commented Nov 19, 2025 •

edited

Loading