docs: record stock card polish evidence
This commit is contained in:
@@ -7,6 +7,60 @@ delivery commits, точные verification commands с наблюдёнными
|
|||||||
итог independent read-only review. Записи добавляются после archive и являются
|
итог independent read-only review. Записи добавляются после archive и являются
|
||||||
неизменяемой историей поставки.
|
неизменяемой историей поставки.
|
||||||
|
|
||||||
|
## 2026-08-26-polish-stock-position-cards
|
||||||
|
|
||||||
|
**Change:** polish-stock-position-cards (schema `wealth-ledger-sdd` v3)
|
||||||
|
**Archive path:** `specs/openspec/changes/archive/2026-08-26-polish-stock-position-cards`
|
||||||
|
**Specs sync:** `skip_specs: true`; change уточняет presentation существующего
|
||||||
|
Portfolio Position без изменения observable requirements или API contract.
|
||||||
|
|
||||||
|
### Delivery Map
|
||||||
|
|
||||||
|
| Area | Status | Outcome |
|
||||||
|
| --- | --- | --- |
|
||||||
|
| frontend | affected | Карточка акции получила двухзонную иерархию: identity и метаданные слева, стоимость и доходность справа, средняя цена и доля одной приглушённой строкой; мобильный перенос сохраняет количество и ISIN |
|
||||||
|
| backend | N/A | Backend behavior, финансовые расчёты и API contract не менялись |
|
||||||
|
| specs | affected | Утверждённые Proposal, Design и Tasks заархивированы с `skip_specs: true` |
|
||||||
|
| docs | affected | As-built Portfolio documentation описывает поставленную визуальную иерархию карточки акции |
|
||||||
|
|
||||||
|
### Delivery commits
|
||||||
|
|
||||||
|
| Repo | Commit | Subject |
|
||||||
|
| --- | --- | --- |
|
||||||
|
| frontend | `273778c4cfa6ef8bd3ab421e03c57b6e31f7d925` | feat: polish stock position cards |
|
||||||
|
| specs (archive) | `cf6bc1761140f06f44ea29e2eee1a2b24d075e2a` | docs(specs): archive stock card polish |
|
||||||
|
| docs (as-built) | `0c59ce788690ba6d17f33e0b57e122dd67e74904` | docs: describe polished stock cards |
|
||||||
|
| docs (evidence) | current evidence commit | docs: record stock card polish evidence |
|
||||||
|
|
||||||
|
### Verification
|
||||||
|
|
||||||
|
| Command (repo) | Наблюдённый результат |
|
||||||
|
| --- | --- |
|
||||||
|
| `npm run test:run -- src/pages/account/ui/AccountPage.test.tsx` (frontend, RED) | failed as expected: прежняя разметка не объединяла quantity/ISIN и average price/share в спокойные строки |
|
||||||
|
| `npm run test:run -- src/pages/account/ui/AccountPage.test.tsx` (frontend, responsive RED) | failed as expected: metadata имела `data-line-clamp=true`, способный скрыть ISIN на узком экране |
|
||||||
|
| `npm run test:run -- src/pages/account/ui/AccountPage.test.tsx` (frontend, final) | pass; 13/13 tests, metadata больше не line-clamped |
|
||||||
|
| `PATH=/Users/ksv741/.nvm/versions/node/v22.21.0/bin:$PATH bash specs/scripts/verify-workspace-change.sh polish-stock-position-cards` (workspace, pre-archive) | pass; frontend/specs/docs affected, backend/runtime-openapi N/A; 20/20 main specs; frontend 21 files / 68 tests, build and bundle check; docs 37 Markdown files / 0 issues |
|
||||||
|
| `openspec archive polish-stock-position-cards --skip-specs --yes --store wealth-ledger-specs` (specs) | pass; archived as `2026-08-26-polish-stock-position-cards`, spec updates skipped by approved metadata |
|
||||||
|
| `openspec validate --specs --strict --store wealth-ledger-specs` (specs, post-archive) | pass; 20 passed, 0 failed |
|
||||||
|
| `openspec validate --archived --strict --store wealth-ledger-specs` (specs, post-archive) | pass; 33 passed, 0 failed |
|
||||||
|
| `openspec schema validate wealth-ledger-sdd --verbose` (specs, post-archive) | pass; dependency graph and templates valid |
|
||||||
|
| `bash scripts/verify-openspec-authority.sh` (specs, post-archive) | pass; OpenSpec CLI 1.10.0, authority `wealth-ledger-specs` |
|
||||||
|
|
||||||
|
Все четыре Tasks завершены до archive. Semantic verification подтвердила
|
||||||
|
двухзонную компоновку только для секции акций, сохранение состава метрик,
|
||||||
|
nullable hiding, прежнего вида облигаций/прочих позиций и отсутствие изменений
|
||||||
|
API. Account lazy-route baseline зафиксирован на 24,695 bytes; initial
|
||||||
|
JavaScript payload — 510,523 bytes.
|
||||||
|
|
||||||
|
### Independent read-only review
|
||||||
|
|
||||||
|
Verdict: **PASS**. Первый review выявил два Important findings: потенциальное
|
||||||
|
скрытие ISIN из-за `lineClamp` на узком экране и преждевременную отметку
|
||||||
|
workspace task. Task была возвращена в unchecked, metadata получила перенос
|
||||||
|
через `overflow-wrap: anywhere`, добавлен RED/GREEN regression assertion.
|
||||||
|
Повторный review подтвердил: Critical — 0, Important — 0; stock-only routing,
|
||||||
|
nullable hiding и неизменённый branch облигаций/прочих позиций сохранены.
|
||||||
|
|
||||||
## 2026-08-26-harden-openspec-sdd-agent-resilience
|
## 2026-08-26-harden-openspec-sdd-agent-resilience
|
||||||
|
|
||||||
**Change:** harden-openspec-sdd-agent-resilience (schema `wealth-ledger-sdd` v3)
|
**Change:** harden-openspec-sdd-agent-resilience (schema `wealth-ledger-sdd` v3)
|
||||||
|
|||||||
Reference in New Issue
Block a user