Продолжение Issue #43: отдельная аналитика брокерского счёта — пополнения, выводы, купоны/дивиденды/проценты, комиссии, реализованный результат продаж по FIFO и нереализованный результат по последнему срезу. Сделки не попадают в бытовые доходы/расходы.
Проверки: npm run build (shared, backend, frontend), npm run test:portfolio:performance, npm run test:broker:xlsx.
Продолжение Issue #43: отдельная аналитика брокерского счёта — пополнения, выводы, купоны/дивиденды/проценты, комиссии, реализованный результат продаж по FIFO и нереализованный результат по последнему срезу. Сделки не попадают в бытовые доходы/расходы.
Проверки: `npm run build` (shared, backend, frontend), `npm run test:portfolio:performance`, `npm run test:broker:xlsx`.
Независимое ревью PR #49 завершено. Проверены полный diff и контекст: расчёт пополнений/выводов и доходов, FIFO по сделкам с десятичными количествами, перевод копеек/рублей, SQL latest snapshot, авторизация через общий requireAuth, UI и тесты. На реальном temp/broker.xlsx стороны сделок и суммы комиссий согласуются (комиссии в движении ДС — дублирующее представление, не добавляются повторно). Сборки shared/backend/frontend и test:portfolio, test:portfolio:performance проходят. Блокирующих замечаний не обнаружено.
Независимое ревью PR #49 завершено. Проверены полный diff и контекст: расчёт пополнений/выводов и доходов, FIFO по сделкам с десятичными количествами, перевод копеек/рублей, SQL latest snapshot, авторизация через общий requireAuth, UI и тесты. На реальном temp/broker.xlsx стороны сделок и суммы комиссий согласуются (комиссии в движении ДС — дублирующее представление, не добавляются повторно). Сборки shared/backend/frontend и test:portfolio, test:portfolio:performance проходят. Блокирующих замечаний не обнаружено.
Финальное независимое ревью полного текущего PR #49 (включая c82b26a fix cost basis и 5adbcbd changelog) завершено. Предыдущий риск неполного FIFO-cost basis устранён: при отсутствии полного соответствия текущей позиции историческим лотам unrealizedResult корректно становится null, без ложного результата. Проверены SQL latest snapshot, классификация broker cash без попадания в бытовые доходы/расходы, FIFO и единицы рублей/копеек, auth middleware, UI/доступность, тесты и changelog. shared/backend/frontend build, portfolio performance и broker XLSX tests проходят. Блокирующих замечаний нет; PR готов к слиянию.
Финальное независимое ревью полного текущего PR #49 (включая c82b26a fix cost basis и 5adbcbd changelog) завершено. Предыдущий риск неполного FIFO-cost basis устранён: при отсутствии полного соответствия текущей позиции историческим лотам unrealizedResult корректно становится null, без ложного результата. Проверены SQL latest snapshot, классификация broker cash без попадания в бытовые доходы/расходы, FIFO и единицы рублей/копеек, auth middleware, UI/доступность, тесты и changelog. shared/backend/frontend build, portfolio performance и broker XLSX tests проходят. Блокирующих замечаний нет; PR готов к слиянию.
Финальное независимое ревью PR #49 повторено после 491a51c. Проверены полный diff, предыдущий cost-basis fix и новый integration test overlapping portfolio reports: повторный отчёт остаётся идемпотентным, новый период и новая сделка сохраняются, cleanup теста присутствует. Backend patch 0.14.1 корректен; changelog и предыдущие проверки без регрессий. shared/backend/frontend build, portfolio performance и broker XLSX tests проходят. Локальный запуск test:portfolio:db в этой среде завершился connection timeout к PostgreSQL (инфраструктурное ограничение, не ошибка кода); по предоставленным результатам PR тест заявлен пройденным. Блокирующих замечаний нет, PR готов к слиянию.
Финальное независимое ревью PR #49 повторено после 491a51c. Проверены полный diff, предыдущий cost-basis fix и новый integration test overlapping portfolio reports: повторный отчёт остаётся идемпотентным, новый период и новая сделка сохраняются, cleanup теста присутствует. Backend patch 0.14.1 корректен; changelog и предыдущие проверки без регрессий. shared/backend/frontend build, portfolio performance и broker XLSX tests проходят. Локальный запуск test:portfolio:db в этой среде завершился connection timeout к PostgreSQL (инфраструктурное ограничение, не ошибка кода); по предоставленным результатам PR тест заявлен пройденным. Блокирующих замечаний нет, PR готов к слиянию.
Финальное независимое ревью полного PR #49 повторено после 94506e3. Новый frontend API contract test проверен и проходит: вызываются ровно /api/portfolio, /api/portfolio/history и /api/portfolio/performance; версия frontend 0.15.1 согласована. Предыдущие cost-basis, overlap-import test и changelog повторно проверены. Frontend/shared/backend build и frontend portfolio test проходят; блокирующих замечаний и регрессий не обнаружено. SQL integration по-прежнему не запускается в текущей среде из-за timeout PostgreSQL, что не является дефектом PR. PR готов к слиянию.
Финальное независимое ревью полного PR #49 повторено после 94506e3. Новый frontend API contract test проверен и проходит: вызываются ровно /api/portfolio, /api/portfolio/history и /api/portfolio/performance; версия frontend 0.15.1 согласована. Предыдущие cost-basis, overlap-import test и changelog повторно проверены. Frontend/shared/backend build и frontend portfolio test проходят; блокирующих замечаний и регрессий не обнаружено. SQL integration по-прежнему не запускается в текущей среде из-за timeout PostgreSQL, что не является дефектом PR. PR готов к слиянию.
Финальное независимое ревью полного PR #49 повторено после c69e55e. Changelog для frontend API contract test проверен, версия frontend 0.15.1 согласована. Повторно проверены performance/FIFO, защита incomplete cost basis, overlap-import test, SQL latest snapshot, auth, UI и отсутствие смешивания брокерских сделок с бытовыми операциями. Frontend API contract test проходит; по предоставленным результатам полный набор сборок/unit/API checks зелёный. SQL integration по-прежнему недоступен только из-за PostgreSQL timeout в среде. Блокирующих замечаний нет, PR готов к слиянию.
Финальное независимое ревью полного PR #49 повторено после c69e55e. Changelog для frontend API contract test проверен, версия frontend 0.15.1 согласована. Повторно проверены performance/FIFO, защита incomplete cost basis, overlap-import test, SQL latest snapshot, auth, UI и отсутствие смешивания брокерских сделок с бытовыми операциями. Frontend API contract test проходит; по предоставленным результатам полный набор сборок/unit/API checks зелёный. SQL integration по-прежнему недоступен только из-за PostgreSQL timeout в среде. Блокирующих замечаний нет, PR готов к слиянию.
Повторное финальное независимое ревью полного PR #49 после b943408 завершено. Новый текст карточки «Купоны, дивиденды и проценты» точен и соответствует backend-классификации; frontend 0.15.2 корректно повышен. Предыдущие performance/FIFO, incomplete cost basis, overlap-import, API contract test и changelog повторно проверены. Блокирующих замечаний и регрессий нет; PR готов к слиянию. SQL integration по-прежнему ограничен timeout PostgreSQL в среде.
Повторное финальное независимое ревью полного PR #49 после b943408 завершено. Новый текст карточки «Купоны, дивиденды и проценты» точен и соответствует backend-классификации; frontend 0.15.2 корректно повышен. Предыдущие performance/FIFO, incomplete cost basis, overlap-import, API contract test и changelog повторно проверены. Блокирующих замечаний и регрессий нет; PR готов к слиянию. SQL integration по-прежнему ограничен timeout PostgreSQL в среде.
Финальное независимое ревью полного PR #49 после be1c2cc завершено. Changelog для frontend 0.15.2 проверен и корректно описывает уточнение подписи «Купоны, дивиденды и проценты». Предыдущие performance/FIFO, incomplete cost basis, overlap-import и API contract test повторно подтверждены; версия и UI согласованы. Блокирующих замечаний и регрессий нет, PR готов к слиянию. SQL integration всё ещё недоступен из-за PostgreSQL timeout в среде.
Финальное независимое ревью полного PR #49 после be1c2cc завершено. Changelog для frontend 0.15.2 проверен и корректно описывает уточнение подписи «Купоны, дивиденды и проценты». Предыдущие performance/FIFO, incomplete cost basis, overlap-import и API contract test повторно подтверждены; версия и UI согласованы. Блокирующих замечаний и регрессий нет, PR готов к слиянию. SQL integration всё ещё недоступен из-за PostgreSQL timeout в среде.
admin
merged commit 71a18e8656 into main2026-08-27 07:02:27 +00:00
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.
Продолжение Issue #43: отдельная аналитика брокерского счёта — пополнения, выводы, купоны/дивиденды/проценты, комиссии, реализованный результат продаж по FIFO и нереализованный результат по последнему срезу. Сделки не попадают в бытовые доходы/расходы.
Проверки:
npm run build(shared, backend, frontend),npm run test:portfolio:performance,npm run test:broker:xlsx.Независимое ревью PR #49 завершено. Проверены полный diff и контекст: расчёт пополнений/выводов и доходов, FIFO по сделкам с десятичными количествами, перевод копеек/рублей, SQL latest snapshot, авторизация через общий requireAuth, UI и тесты. На реальном temp/broker.xlsx стороны сделок и суммы комиссий согласуются (комиссии в движении ДС — дублирующее представление, не добавляются повторно). Сборки shared/backend/frontend и test:portfolio, test:portfolio:performance проходят. Блокирующих замечаний не обнаружено.
Финальное независимое ревью полного текущего PR #49 (включая
c82b26afix cost basis и5adbcbdchangelog) завершено. Предыдущий риск неполного FIFO-cost basis устранён: при отсутствии полного соответствия текущей позиции историческим лотам unrealizedResult корректно становится null, без ложного результата. Проверены SQL latest snapshot, классификация broker cash без попадания в бытовые доходы/расходы, FIFO и единицы рублей/копеек, auth middleware, UI/доступность, тесты и changelog. shared/backend/frontend build, portfolio performance и broker XLSX tests проходят. Блокирующих замечаний нет; PR готов к слиянию.Финальное независимое ревью PR #49 повторено после
491a51c. Проверены полный diff, предыдущий cost-basis fix и новый integration test overlapping portfolio reports: повторный отчёт остаётся идемпотентным, новый период и новая сделка сохраняются, cleanup теста присутствует. Backend patch 0.14.1 корректен; changelog и предыдущие проверки без регрессий. shared/backend/frontend build, portfolio performance и broker XLSX tests проходят. Локальный запуск test:portfolio:db в этой среде завершился connection timeout к PostgreSQL (инфраструктурное ограничение, не ошибка кода); по предоставленным результатам PR тест заявлен пройденным. Блокирующих замечаний нет, PR готов к слиянию.Итоговое независимое ревью полного PR #49 повторено после
19d3c36(changelog). Проверены changelog, overlap-тест повторных/пересекающихся отчётов, предыдущий cost-basis fix, SQL latest snapshot, FIFO, единицы рублей/копеек, классификация cash, auth, UI и регрессии. Backend 0.14.1 согласован с changelog. Финальные backend build, performance/FIFO и broker XLSX tests проходят; SQL integration остаётся недоступным только из-за timeout PostgreSQL в среде. Блокирующих замечаний нет, PR готов к слиянию.Финальное независимое ревью полного PR #49 повторено после
94506e3. Новый frontend API contract test проверен и проходит: вызываются ровно /api/portfolio, /api/portfolio/history и /api/portfolio/performance; версия frontend 0.15.1 согласована. Предыдущие cost-basis, overlap-import test и changelog повторно проверены. Frontend/shared/backend build и frontend portfolio test проходят; блокирующих замечаний и регрессий не обнаружено. SQL integration по-прежнему не запускается в текущей среде из-за timeout PostgreSQL, что не является дефектом PR. PR готов к слиянию.Финальное независимое ревью полного PR #49 повторено после
c69e55e. Changelog для frontend API contract test проверен, версия frontend 0.15.1 согласована. Повторно проверены performance/FIFO, защита incomplete cost basis, overlap-import test, SQL latest snapshot, auth, UI и отсутствие смешивания брокерских сделок с бытовыми операциями. Frontend API contract test проходит; по предоставленным результатам полный набор сборок/unit/API checks зелёный. SQL integration по-прежнему недоступен только из-за PostgreSQL timeout в среде. Блокирующих замечаний нет, PR готов к слиянию.Повторное финальное независимое ревью полного PR #49 после
b943408завершено. Новый текст карточки «Купоны, дивиденды и проценты» точен и соответствует backend-классификации; frontend 0.15.2 корректно повышен. Предыдущие performance/FIFO, incomplete cost basis, overlap-import, API contract test и changelog повторно проверены. Блокирующих замечаний и регрессий нет; PR готов к слиянию. SQL integration по-прежнему ограничен timeout PostgreSQL в среде.Финальное независимое ревью полного PR #49 после
be1c2ccзавершено. Changelog для frontend 0.15.2 проверен и корректно описывает уточнение подписи «Купоны, дивиденды и проценты». Предыдущие performance/FIFO, incomplete cost basis, overlap-import и API contract test повторно подтверждены; версия и UI согласованы. Блокирующих замечаний и регрессий нет, PR готов к слиянию. SQL integration всё ещё недоступен из-за PostgreSQL timeout в среде.