[Bug] Передать periodic read lifecycle Query owners и учитывать eligibility #54
Closed
opened 2026-10-11 21:48:56 +03:00 by codex-bot
·
0 comments
No Branch/Tag Specified
Labels
Clear labels
Kind/Dependencies
Kind/Investigation
Kind/Process
Kind/Refactor
audit/2026-10-11
kind/investigation
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Dependency update or dependency-management work
Research or decision required before implementation
Developer workflow, CI/CD, tooling, or process improvement
Internal code or configuration restructuring without new product behavior
Technical Audit Remediation — 2026-10-11
Нужна проверка evidence до решения; runtime-дефект не утверждается
Breaking change that won't be backward compatible
Something is not working
Documentation changes
Improve existing functionality
New functionality
This is security issue
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
Milestone
No items
No Milestone
Technical Audit Remediation — 2026-10-11
Projects
Clear projects
No projects
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: WealthLedger/frontend#54
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Problem
useScreenSync каждые 30 минут manual-refetch всех resources без isEnabled/isFetching. Short search/unresolved UID/session gates обходятся; scope/return facade никто не читает. App focus defaults противоречат portfolio policy; only-test observe/plural APIs добавляют альтернативные paths.
Expected Behavior
Active eligible queries владеют periodic GET; inactive inputs не запрашиваются. Native migration сохраняет осмысленные cadence/cache/identity guarantees.
Evidence
Audit date
2026-10-11; 05, 04, Complexity. IDs FE-003, ARCH-003, CPLX-001, DRY-010, DEAD-004, DEAD-005, FLOW-002, FLOW-008, TA-F07, CA-H12, CA-H14. Revisionc8821200ce699dab1ce65360d1968d5877a2e78b; Confidence High mechanism/Medium equivalence; Verification status static.Affected: screen-sync.ts:7–61; App.tsx:13–20; bond/api/queries.ts:97–104; resource-query.ts:69–149; authenticated-query.ts:15–32,61; initial-availability.ts:10–46; history hook:52–76.
Scope
In scope
Common readiness correction, eight consumers, agreed focus/reconnect/mount/hidden-tab policy, query-native interval; only-test API pruning после caller proof.
Out of scope
Global polling registry, direct fetch bypass, removal session/access/cache fence, backend public observe retirement, uniform staleTime вместо business decision.
Proposed Approach
Immediate shared enabled/not-fetching guard; native refetchInterval у existing owners после multi-observer proof. Scope/facade удаляются вместе с migrated live timer. Prior-only unused fields/plural hook удалить только при current inventory; automatic invalid-cursor recovery оставить.
Acceptance Criteria
Verification
Existing screen-sync/resource/auth/client MSW+fake time, production App options, два observers same key; final frontend verify и relevant keyboard/browser proof при UI change.
Dependencies
Нет; T11 backend compatibility не blocker client-only API pruning.
Traceability
Campaign WealthLedger/main#23 . Findings FE-003, ARCH-003, CPLX-001, DRY-010, DEAD-004, DEAD-005, FLOW-002, FLOW-008, TA-F07, CA-H12, CA-H14. WealthLedger/frontend; T20.