AURORA-1887: скрыть сателлитные строки на листе АХР, заблокировать добавление для аренды/коммуналки/охраны #273
Open
aep
wants to merge 6 commits from
feat/272-aurora-1887-ahr-hide-satellite-rows into test
pull from: feat/272-aurora-1887-ahr-hide-satellite-rows
merge into: RSHB:test
RSHB:main
RSHB:feat/270-aurora-1907-source-sheet
RSHB:feat/266-aurora-1897-ahr-limit-period-vat-units
RSHB:feat/265-aurora-1909-ahr-manual-row-capture
RSHB:feat/264-aurora-1895-formula-localstorage
RSHB:feat/254-aurora-1906-spod-column
RSHB:feat/255-aurora-1938-transfer-econ-calc
RSHB:feat/250-aurora-1888-stages-role-filter
RSHB:feat/253-aurora-1905-form1-form4-export-layout
RSHB:feat/252-aurora-1902-form2-export-layout
RSHB:feat/251-aurora-1901-smeta-rf-column-order
RSHB:feat/249-aurora-1936-row-delete-confirm
RSHB:feat/248-aurora-1877-token-refresh
RSHB:feat/246-aurora-1944-wheel-scroll-step
RSHB:feat/236-aurora-1843-svod2-server-paging-perf
RSHB:feat/236-aurora-1843-svod2-server-paging-web
RSHB:feat/236-aurora-1843-svod2-server-paging
RSHB:feat/214-aurora-1841-dict-vsp-virtual
RSHB:feat/220-aurora-1842-svod2-column-virtual
RSHB:feat/216-aurora-1846-form4-sheet-sum
RSHB:feat/235-aurora-1850-upd-form-cells-subtree
RSHB:feat/233-aurora-1848-sheet-json-no-pydantic-b
RSHB:feat/234-aurora-1839-cell-memo
RSHB:feat/233-aurora-1848-sheet-json-no-pydantic
RSHB:feat/237-aurora-1845-ws-connection-cache
RSHB:feat/215-aurora-1844-upd-form-cells-full-rows
RSHB:feat/226-aurora-1837-cell-styles-css
RSHB:feat/207-aurora-1831-project-tn-race
RSHB:feat/208-aurora-1832-phase-dates-tz
RSHB:feat/209-aurora-1833-greenlet-dev
RSHB:feat/206-aurora-1830-users-me-local-email
RSHB:feat/212-aurora-1838-ws-immutable-update
RSHB:feat/211-aurora-1836-row-measure
RSHB:feat/205-aurora-1922-pg15-migration-0012
RSHB:feat/213-aurora-1840-sheet-modals
RSHB:feat/210-aurora-1835-svod1-formatter
RSHB:test
RSHB:svod2-filter-fix2
RSHB:indent-export
RSHB:export-hierarchy
RSHB:svod2-filter-fix
RSHB:fix-id-column
RSHB:export-year-fix
RSHB:form3-source
RSHB:columns-svod-2
RSHB:export-optimize-refresh
RSHB:last-column
RSHB:svod-fix
RSHB:fix-columns
RSHB:form3-expense
RSHB:sattelite-fix
RSHB:delete-sattelite-rows
RSHB:form3-pages
RSHB:form1-sub_sub-fix
RSHB:sort-fix
RSHB:fixes-form
RSHB:new-expenses
RSHB:new-rows
RSHB:fix-style
RSHB:phase-template-edit
RSHB:fixes
RSHB:add-btn-reset-filter
RSHB:vsp-choice-fix
RSHB:project-filters-fix
RSHB:width-column-header-name
RSHB:task-filter-navigation
RSHB:auth
RSHB:icons-program-project
RSHB:export-optimize
RSHB:org-unit-limits
RSHB:auth-v2
RSHB:tasks-filters
RSHB:insert-row-fix
RSHB:svod-roles
RSHB:new-role-fix
RSHB:phase-edit-fix
RSHB:new-role
RSHB:delete-from-cell
RSHB:svod-cahe-and-table-direction
RSHB:direction-stage-fix
RSHB:backslash-fix
RSHB:svod
RSHB:front-svod-form-1
RSHB:projects-filters
RSHB:remove-row-fix
RSHB:computed-fix
RSHB:modal-add-year-project
RSHB:name-fix
RSHB:audit-logs-projects
RSHB:meta-form-3
RSHB:project-back-fix
RSHB:fix-sticky-columns
RSHB:fixies-3-form
RSHB:tech-num-fix
RSHB:fix-logic-limitation
RSHB:projects-fix
RSHB:3-form-project
RSHB:project-add-year-handler
RSHB:projects-summary-handler
RSHB:AURORA-1247
RSHB:projects-handlers
RSHB:vsp-choose-form2-2
RSHB:directions-fix
RSHB:add-validation
RSHB:add-checkbox-export-vsp
RSHB:fix-update-column-config
RSHB:fix-editable-columns
RSHB:add-biome-front
RSHB:vsp-choose-form2
RSHB:export-form-4
RSHB:booking-fix
RSHB:fix-front-tables
RSHB:phase-directions
RSHB:AURORA-1225
RSHB:fix-mapping-form-2-oper
RSHB:AURORA-1076-fix
RSHB:fix-add-row-program-4-form
RSHB:form2-limit-migrations
RSHB:delete-projects
RSHB:AURORA-1076
RSHB:form-4-add-row
RSHB:fix-logs
RSHB:AURORA-1206
RSHB:AURORA-894
RSHB:add-line-log-hotfix
RSHB:form-export-fix
RSHB:form-4-sql
RSHB:table-optimization-fix
RSHB:header_forms
RSHB:form-4-expense-item-filters
RSHB:entity-type-hotfix
RSHB:form-3-phases-timezones
RSHB:form-4-handlers
RSHB:export-fix
RSHB:close-date-vsp-fix
RSHB:deleted-user-auth-fix
RSHB:smeta-fix
RSHB:front-fixes
RSHB:line_id_code_full
RSHB:line_id_code
RSHB:timezones
RSHB:filenames-fix
RSHB:AURORA-1174-73
RSHB:project-sheets
RSHB:AURORA-1166
RSHB:websocket-projects
RSHB:project-front
RSHB:form3-permissions
RSHB:location_from_to_placement_type
RSHB:fix-vsp-filters
RSHB:fix-tests-and-functions
RSHB:mig2-fix
RSHB:vsp_fixes
RSHB:fix-ssp-title
RSHB:AURORA-1148
RSHB:fix-vsp-form-edit
RSHB:sql-funcs2-fix
RSHB:AURORA-1132
RSHB:logs
RSHB:AURORA-1147
RSHB:export-optimization
RSHB:AURORA-1131
RSHB:sql-funcs-fix
RSHB:vsp-dropdown-form
RSHB:form_2-back
RSHB:migrations
RSHB:websocket-self-flag
RSHB:reload-fix
RSHB:test-stand-fix
RSHB:excel-empty-fix
RSHB:requirements-fix
RSHB:AURORA-1096-back
RSHB:AURORA-1098
RSHB:AURORA-1096
RSHB:ws-user-id-fix
RSHB:expense-item-handler
RSHB:fix-date-phase
RSHB:excel-fix
RSHB:vsp-edit-field-fix
RSHB:vsp-fix-back
RSHB:vsp-back
RSHB:create-form-back
RSHB:export-direction-fix
RSHB:change-role-fix
RSHB:org-unit-users-fix
RSHB:audit-fix
RSHB:org-archive
RSHB:del_row_fix
RSHB:websocket_locks
RSHB:new_line_id
RSHB:admin-back
RSHB:form-fix
RSHB:ws
RSHB:ssp-fix
RSHB:stages-back
RSHB:form_124_write
RSHB:form_124
RSHB:new-model
RSHB:test-tsygankov
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.
No description provided.
Delete Branch "feat/272-aurora-1887-ahr-hide-satellite-rows"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes #272.
Контекст
Упрощённый дизайн (согласован в чате с человеком 2026-10-31) — расходится с исходным архитектурным текстом тикета AURORA-1887 (который предлагал слить budget_line по статье + лист-источник, снять UNIQUE(target_line_id) в form2_ahr_satellite_line_source, миграцию данных и правку протокола WS). Согласованная версия ограничивается визуальным скрытием и UI-гейтом, без изменения модели данных:
Что сделано
Скрытие сателлитных INPUT-строк на листе AHR.
v3.v_form2_sheet_jsonb(новая миграция0054_form2_ahr_hide_satellite_rows.sql, CREATE OR REPLACE полным актуальным текстом + 3 точечных изменения) теперь приp_sheet = 'AHR'не возвращаетINPUT-строки, у которыхbudget_line.source_sheet IN ('AHR_LIMIT','AHR_RENT','AHR_UTILITY','AHR_SECURITY'). Ручные строки (source_sheet IS NULL, включая «сверх лимита» из AURORA-1907) остаются видимыми без изменений. Листы-источники (AHR_LIMIT/AHR_RENT/AHR_UTILITY/AHR_SECURITY) не затронуты — полная детализация там сохраняется.Суммы на заголовках статей (
plan.year/plan.limit_year/plan.over_limit_year) считаются вv3.v_form2_sheet_sections(CTEtw_agg) и вlimit_treeCTE независимо от видимости строк в финальной выдаче — фильтр применяется только к финальномуSELECT, логика агрегации не тронута.Блокировка «Добавить строку» для аренды/коммуналки/охраны. Новое поле
header.can_add_row(только приp_sheet = 'AHR') —falseдля статей аренды/коммуналки/охраны,trueдля всех прочих (включая лимитные — AURORA-1907 «сверх лимита» не трогаем). Новый небольшой helperv3._form2_satellite_expense_item(p_sheet)возвращает единственныйexpense_item_idдля'AHR_RENT'/'AHR_UTILITY'/'AHR_SECURITY'(314/24/779 соответственно — для'AHR_LIMIT'единого id нет, там множество LEAF-статей).Фронт (
RealtimeTable.jsxhandleAddRow, через чистую функциюcanAddRowForHeaderвutils/rowUtils.js) читает флаг и блокирует клик с тостом «Добавление строк недоступно для этой статьи» — до вызова API.Рулинги
header.can_add_row, читаемый фронтом, без серверной валидации вv3.add_budget_line. Обоснование: это консистентно с существующими паттернами в этом же компоненте —FormsNoCanAddRow,isVspAdditionRow,isAdditionExpenseItem— ни один из них тоже не продублирован проверкой на backend.v3.add_budget_lineи так принимает произвольныйexpense_item_idна любом сочетании sheet/статья без серверной валидации специфичных для UI правил (убедился чтением кода — тот же паттерн уже используется дляAHR_LIMITвFormsNoCanAddRow, которая тоже чисто фронтовая).psqlна разовой тестовой Postgres 16 (SELECT id, item_id, name, sheet, direction, depth FROM v3.expense_item WHERE id IN (314,24,779)): все три —sheet='AHR',direction='Support'. Совпадают с актуальным кодомv3.add_budget_line(0051) иv3.sync_form2_ahr_satellite_lines(0052), где эти же id уже захардкожены как текущие (не как "старые" из 0034) — т.е. это не было новым открытием, но было дважды подтверждено независимо.notify_ahr_about_satellite_change/get_form2_ahr_satellite_delta(websocket.py, 0053) иcellUtils.js/RealtimeContext.jsна фронте.get_form2_ahr_satellite_deltaсама вызываетv3.v_form_view(..., 'AHR', ...), то есть автоматически наследует новый фильтр — скрытая INPUT-строка просто не попадёт в WS-дельту, аwebsocket.pyпри отсутствии INPUT-строки в дельте дляrow_addedуже (без изменений с моей стороны) откатывается наcell_updated-событие с заголовками. На фронтеcellUtils.deleteRow/updateCells— no-op при несуществующемline_id/row_type+depth+sort_order(прочитано, не изменено). Протокол WS НЕ менялся.test_form2_ahr_limit.py,test_form2_ahr_satellite_spod_integration.py, частьtest_form2_ahr_satellite_integration.py) проверяли суммы/источник напрямую на теперь скрытых INPUT-строках листа AHR. Переведены на сравнение через заголовок статьи (ITEM/SUB_ITEM/SUB_SUB_ITEM — глубина статьи в дереве расходов разная для разных категорий, проверено эмпирически) по схеме «до/после патча» (устойчиво к существующим сидовым данным), либо через лист-источник, где детализация не пряталась.Тесты
Интеграционные (
api/tests/integration/test_form2_ahr_satellite_integration.py, общий файл серии задач 1907/1909/1887) — 10 новых тестов на все пункты приёмки (скрытие лимитных LEAF с верной суммой на заголовке, скрытие записи аренды/коммуналки/охраны с видимостью на листе-источнике, регрессия на ручную «сверх лимита» строку,can_add_row=false/true, регрессия на обычную статью). Плюс адаптация 3 ранее существовавших тестов, сломанных скрытием строк (см. R-4).Фронт:
canAddRowForHeader(чистая функция вutils/rowUtils.js) покрытаnode --test(rowUtils.test.js).Одноразовая Postgres 16 в Docker (не
perf-pg, снесена после работы).cd api && pytest: 247 passed, 3 failed (прежний известный набор, не связанный с этой веткой:test_backend_calls_form3_add_project_and_add_line_functions,test_export_full_form_ignores_non_applicable_params_for_other_sheets,test_export_form_payload_forwards_ignore_flag), 4 skipped — итого 254 (было 244 до +10 новых).cd web && node --test src/components/RealtimeTable/utils/*.test.js: 15 passed.Test plan
api/pytest: разница — ровно +10 новых тестов, прежние 3 известных падения без измененийnode --test: 15/15 зелёныхperf-pgне затронут (использовалась отдельная одноразовая Postgres 16, снесена)🤖 Generated with Claude Code
Миграция 0053: v3.budget_line.source_sheet (NULL | AHR_LIMIT | AHR_RENT | AHR_UTILITY | AHR_SECURITY) — явный признак источника строки вместо сопоставления по совпадению имени с LEAF limit_template. Backfill существующих строк: сателлиты — из v3.form2_ahr_satellite_line_source, лимитные — минимальный id в группе (форма, статья, имя LEAF), остальные совпавшие по имени становятся ручными (RAISE NOTICE со списком). _ensure_form2_ahr_limit_sync_line теперь ищет строку по source_sheet='AHR_LIMIT' (а не по имени) — тот же класс бага, что чинил #265/AURORA-1909 для сателлитов аренды/коммуналки/охраны, теперь для лимитов. is_form2_ahr_satellite_line, del_budget_line (guard на удаление) и get_form2_ahr_satellite_delta переведены на признак источника. Новая v3._form2_expense_item_has_limit выносит правило «у статьи есть лимит» в отдельную функцию. v_form2_sheet_jsonb: header.source ('AHR_LIMIT'/'AHR_RENT'/'AHR_UTILITY'/ 'AHR_SECURITY'/'MANUAL'), header.is_over_limit для ручных строк на статьях с лимитом; plan.limit_year/over_limit_year (только на листе AHR, свёрнуты по дереву статей через mv_expense_item_tree.desc_ids) — plan.year не меняется, остаётся суммой всех строк. Фронт: AHR.js — колонки «В пределах лимита, год»/«Сверх лимита, год» после «План, год» (нередактируемые — нет случая в v3._apply_form_cell); Cell.jsx — чип «сверх лимита» в ячейке наименования по header.is_over_limit, без заливки строки. export_mappings.py — те же два поля в экспорт АХР. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>Упрощённый дизайн (согласован в чате 2026-10-31, расходится с архитектурным предложением из тикета): без слияния budget_line, без миграции данных, без изменения протокола WS, без изменения v_svod2_rf. - v3.v_form2_sheet_jsonb (0054): при p_sheet='AHR' не возвращает INPUT-строки с budget_line.source_sheet IN ('AHR_LIMIT','AHR_RENT','AHR_UTILITY', 'AHR_SECURITY'); ручные строки (включая «сверх лимита», AURORA-1907) остаются видимыми. Листы-источники не затронуты. Суммы на заголовках статей считаются независимо от видимости строк — не меняются. - header.can_add_row (новое поле, только p_sheet='AHR'): false для статей аренды/коммуналки/охраны (v3._form2_satellite_expense_item, новый helper), true для лимитных и обычных статей. Фронт (RealtimeTable.jsx handleAddRow, utils/rowUtils.js canAddRowForHeader) блокирует «Добавить строку» по этому флагу — тот же паттерн, что у FormsNoCanAddRow/isVspAdditionRow (без серверной валидации в add_budget_line). - Обновлены тесты, которые проверяли суммы/источник через теперь скрытые INPUT-строки на AHR (test_form2_ahr_limit.py, test_form2_ahr_satellite_spod_integration.py, test_form2_ahr_satellite_integration.py) — сравнение переведено на заголовок статьи (до/после патча) и на лист-источник. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>Checkout
From your project repository, check out a new branch and test the changes.