WealthLedger/frontend
5
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

perf(tbank-connection): poll screen sync every five seconds #47

Merged
ksv741 merged 1 commits from codex/reduce-screen-sync-polling into develop 2026-10-07 14:56:40 +03:00
Conversation 0 Commits 1 Files Changed 2
+162 -31
codex-bot commented 2026-10-07 14:05:20 +03:00
Owner
Copy Link
Copy Source

Implements OpenSpec change reduce-screen-sync-polling (WealthLedger/specs#21):

  • STATUS_INTERVAL_MS 1s → 5s: status checks and observe reads during active screen sync run at most once per five seconds.
  • Terminal sync state observes screen resources immediately (unchanged path, now covered by test).
  • Waiting states, partial results, read recovery, unmount behavior unchanged.

Tests: 18/18 in screen-sync.test.tsx (cadence/gap/freeze, immediate terminal observe). npm run verify green.

Implements OpenSpec change `reduce-screen-sync-polling` ([WealthLedger/specs#21](https://git.personal.ksv741.ru/WealthLedger/specs/pulls/21)): - `STATUS_INTERVAL_MS` 1s → 5s: status checks and observe reads during active screen sync run at most once per five seconds. - Terminal sync state observes screen resources immediately (unchanged path, now covered by test). - Waiting states, partial results, read recovery, unmount behavior unchanged. Tests: 18/18 in `screen-sync.test.tsx` (cadence/gap/freeze, immediate terminal observe). `npm run verify` green.
codex-bot added 1 commit 2026-10-07 14:05:20 +03:00
perf(tbank-connection): poll screen sync every five seconds
Frontend verify / commits (pull_request) Successful in 48s
Details
Frontend verify / security (pull_request) Successful in 10s
Details
Frontend verify / verify (pull_request) Successful in 6m57s
Details
Frontend verify / commits (push) Skipped
Details
Frontend verify / security (push) Successful in 11s
Details
Frontend verify / verify (push) Successful in 8m21s
Details
74f8a72ed5 
During active screen sync, status checks and observe reads run every 5s
instead of 1s; terminal state observes resources immediately.
OpenSpec change: WealthLedger/specs#21
ksv741 merged commit 74f8a72ed5 into develop 2026-10-07 14:56:40 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Kind/Dependencies
Dependency update or dependency-management work
 Kind/Investigation
Research or decision required before implementation
 Kind/Process
Developer workflow, CI/CD, tooling, or process improvement
 Kind/Refactor
Internal code or configuration restructuring without new product behavior
 audit/2026-10-11
Technical Audit Remediation — 2026-10-11
 kind/investigation
Нужна проверка evidence до решения; runtime-дефект не утверждается
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Medium
3
The priority is medium
Priority
Low
4
The priority is low
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status
Abandoned
3
Somebody has started to work on this but abandoned work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
codex-bot hermes-agent ksv741 publisher-bot renovate-bot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WealthLedger/frontend#47