[Spec] Восстановить в canonical specs требования из archive persist-tbank-data #63

Open
opened 2026-10-29 06:52:02 +03:00 by codex-bot · 1 comment
Owner

Контекст

G2 review change group-account-rows-by-instrument (WealthLedger/main#47) выявил расхождение: canonical specs brokerage-account-ledger, brokerage-account-income-analytics и brokerage-account-upcoming-events не совпадают с delta archive 2026-10-09-persist-tbank-data. Похоже, при sync этого change часть требований потерялась: режим «Все счета» (all), envelope {data, meta}, partial, чтение без синхронного запроса к T-Bank.

  • Сравнивались блоки с одинаковым заголовком: в canonical они отличаются от archive почти везде.
  • Period comparisons preserve calendar and currency meaning из canonical income пропал целиком.
  • Блоки User can retrieve an account ledger и Realized income analytics is account-scoped and period-bound восстанавливаются в WealthLedger/main#47. Здесь их не трогать.

Ожидаемое поведение

Canonical specs совпадают с принятым поведением: archive persist-tbank-data плюс всё, что изменили более поздние archived changes.

Acceptance Criteria

  • Каждый блок archive persist-tbank-data для трёх capabilities сверен с canonical. Расхождение либо объяснено более поздним archived change, либо исправлено.
  • Period comparisons preserve calendar and currency meaning есть в canonical income либо его удаление обосновано.
  • npm run check:openspec проходит.
## Контекст G2 review change `group-account-rows-by-instrument` (WealthLedger/main#47) выявил расхождение: canonical specs `brokerage-account-ledger`, `brokerage-account-income-analytics` и `brokerage-account-upcoming-events` не совпадают с delta archive `2026-10-09-persist-tbank-data`. Похоже, при sync этого change часть требований потерялась: режим «Все счета» (`all`), envelope `{data, meta}`, `partial`, чтение без синхронного запроса к T-Bank. - Сравнивались блоки с одинаковым заголовком: в canonical они отличаются от archive почти везде. - `Period comparisons preserve calendar and currency meaning` из canonical income пропал целиком. - Блоки `User can retrieve an account ledger` и `Realized income analytics is account-scoped and period-bound` восстанавливаются в WealthLedger/main#47. Здесь их не трогать. ## Ожидаемое поведение Canonical specs совпадают с принятым поведением: archive `persist-tbank-data` плюс всё, что изменили более поздние archived changes. ## Acceptance Criteria - [ ] Каждый блок archive `persist-tbank-data` для трёх capabilities сверен с canonical. Расхождение либо объяснено более поздним archived change, либо исправлено. - [ ] `Period comparisons preserve calendar and currency meaning` есть в canonical income либо его удаление обосновано. - [ ] `npm run check:openspec` проходит.
codex-bot added the Kind/Bug
Priority
Medium
3
labels 2026-10-29 06:52:02 +03:00
Author
Owner

Ещё одно расхождение нашлось при design WealthLedger/main#47. В canonical brokerage-account-upcoming-events — Account page presents an upcoming events list сказано «без фильтров, пагинации…». Но страница «Будущие события» уже фильтрует по периоду, и это поведение покрыто requirement User can filter upcoming events by a forecast date range. Формулировку списка стоит согласовать с этим requirement.

Ещё одно расхождение нашлось при design WealthLedger/main#47. В canonical `brokerage-account-upcoming-events` — `Account page presents an upcoming events list` сказано «без фильтров, пагинации…». Но страница «Будущие события» уже фильтрует по периоду, и это поведение покрыто requirement `User can filter upcoming events by a forecast date range`. Формулировку списка стоит согласовать с этим requirement.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WealthLedger/specs#63