Skip to content

docs(features): секция про кнопку «Обновить» в списках - #1503

Open
ivanarama wants to merge 3 commits into
mainfrom
fix/1449-features-refresh-button
Open

ivanarama wants to merge 3 commits into
mainfrom
fix/1449-features-refresh-button

Conversation

@ivanarama

Copy link
Copy Markdown
Owner

Что не так

PR #1324 (влит 2026-09-07) добавил команду 🔄 «Обновить» в командные панели списков — коммит feat:, пользовательская возможность. Секции в docs/features.md нет ни в самом PR, ни позже. По CLAUDE.md пользовательский feat: обязан её получить: раздел «Тестируем» на сайте читает именно этот файл, и без секции возможность для пользователя не существует.

Что сделано

Добавлена секция «Кнопка «Обновить» в списках» с <!-- status: testing --> и <!-- date: 2026-09-07 --> — дата мержа #1324.

Тело описывает поведение по факту реализации: команда повторно открывает текущий адрес списка целиком, поэтому переживают обновление поиск, отбор, сортировка, номер страницы и подсистема; в режиме вкладок обновляется только фрейм списка, оболочка не перезагружается. Отдельным абзацем названо, что обновление сбрасывает — временное выделение строки и позицию прокрутки: это оговорено в самом #1324, и умолчать об этом значило бы описать не ту возможность.

Ссылку <!-- issue: NN --> не ставлю намеренно: она выключает featureage, а обсуждать здесь нечего — так и указано в разборе заявки.

Проверки

  • go run ./tools/featureage — каталог разбирается, новая секция принята как свежая и в кандидаты на stable не попадает;
  • go test -count=1 ./tools/featureage;
  • go build ./....

Fixes #1449

PR #1324 добавил команду 🔄 «Обновить» в командные панели списков сущностей,
регистров сведений и журналов, но секции в каталоге возможностей не завёл.
По CLAUDE.md пользовательский feat: обязан её получить — раздел «Тестируем»
на сайте читает именно этот файл.

Секция описывает, что команда повторно открывает текущий адрес списка целиком,
поэтому переживают обновление поиск, отбор, сортировка, страница пагинации и
подсистема; в режиме вкладок обновляется только фрейм списка. Отдельно названо,
что сбрасывается: временное выделение строки и позиция прокрутки.

Ссылку на заявку намеренно не ставлю: она выключает featureage, а обсуждать
здесь нечего.

Fixes #1449

Generated-with: Claude Code
@ivanarama

Copy link
Copy Markdown
Owner Author

Ревью. (круг 1)
Reviewed-SHA: 49610d4
Outcome-Label: changes-requested
Что меняется: В docs/features.md добавлена секция о кнопке «Обновить» для списков сущностей, регистров сведений и журналов документов; описаны сохранение параметров URL, обновление фрейма и сценарий проверки.
Проверено: Аудит выполнен в отдельном detached worktree точного HEAD 49610d4; diff содержит только docs/features.md (+23 строки), git diff --check прошёл; Сверены production-обработчики list/infoRegList/journalList, шаблон list-refresh и оболочка tabs.go: ссылка использует RequestURI, обновление остаётся в текущем фрейме; Проверен код до добавления кнопки: e5afb68^:internal/ui/handlers_entity.go уже восстанавливает поиск q, фильтры f., сортировку и page из r.URL.Query(); Раздел единственный, status=testing, date=2026-09-07 совпадает с датой merge PR #1324; issue-маркера нет; go run ./tools/featureage -now 2026-09-17 прошёл, новая секция не попала в созревшие, файл не изменён; go test -count=1 ./tools/featureage — PASS; go vet ./tools/featureage — PASS; go test -count=1 ./internal/ui -run '^(TestListRefreshKeepsCurrentRequestURI|TestListViewSwitchKeepsSearch|TestListSearchFormKeepsView|TestListFilterFormKeepsSearch|TestListSortLinkKeepsSearchAndFolder|TestTabs.)$' — PASS; проверка RequestURI охватывает все три вида списков; Все восемь обязательных CI-проверок прошли: build, lint, postgres-integration, vuln, smoke, e2e, test-windows, launcher-webview-build; run 34788942111 имеет head_sha=49610d43a122223443ee063b3661a66fac91ab99; Полная локальная сборка и полный набор тестов не запускались: изменение касается только документации; локально проверены каталог и описываемое поведение UI.
Блокирующее: 1) [P2] docs/features.md:5926–5928 — исправить утверждение, что перезагрузка списка браузером раньше сбрасывала поиск, отбор и страницу. Для прямой страницы списка F5 повторяет тот же URL с q, f.* и page; parseListParams восстанавливал эти параметры из r.URL.Query() ещё до PR #1324. Новая кнопка делает GET того же RequestURI, поэтому такое отличие от обычной перезагрузки отсутствует. Текст создаёт неверное представление о поведении продукта. Уберите утверждение о потере параметров при перезагрузке браузером либо ограничьте его повторным открытием списка из меню; отдельно можно оставить преимущество обновления одного фрейма без перезагрузки оболочки. По CLAUDE.md фактически неверное утверждение в поставляемой документации является блокирующим.
Хвост:

Вердикт: есть замечания.

@ivanarama

Copy link
Copy Markdown
Owner Author

PromptPilot service marker: REVIEW result publication claimed.

@ivanarama ivanarama added the changes-requested Ревью: есть замечания — фиксер дорабатывает label Sep 17, 2026
@ivanarama

Copy link
Copy Markdown
Owner Author

PromptPilot service marker: REVIEW result committed.

Убрано неверное утверждение, что перезагрузка браузером сбрасывает
поиск, отбор и страницу: эти параметры восстанавливаются из URL.
Описание кнопки объясняет обновление данных и сохранение адреса списка.

Generated-with: Codex
PP-Fix-Transition: from=49610d43a122223443ee063b3661a66fac91ab99 review-comment=5709215616 claim=5709216026 epoch-sha256=964e2d5d555c5581570432f52838fc8363e8c8ae41a6ae9fe1945b0082e21ca5
@ivanarama

Copy link
Copy Markdown
Owner Author

Исправлено блокирующее замечание ревью.

  1. В docs/features.md убрано неверное утверждение, что перезагрузка браузером сбрасывает поиск, отбор и страницу. Вводный абзац теперь объясняет, как увидеть изменения из другого окна или от другого пользователя.
  2. Сохранено описание подтверждённого поведения: повторный запрос текущего URL, сохранение параметров и обновление одного фрейма в режиме вкладок. Код не менялся: замечание касается только документации.

Проверки прошли: go build ./...; go test -count=1 ./tools/featureage; go test -count=1 ./internal/ui -run '^(TestListRefreshKeepsCurrentRequestURI|TestListViewSwitchKeepsSearch|TestListSearchFormKeepsView|TestListFilterFormKeepsSearch|TestListSortLinkKeepsSearchAndFolder|TestTabs.*)$'; go run ./tools/featureage -now 2026-09-17; git diff --check. Проверка каталога файл не изменила, новая секция не требует смены статуса.

TestTabsBehaviorInNode пропущен: Node.js отсутствует в окружении. Остальные выбранные Go-тесты прошли.

@ivanarama ivanarama removed the changes-requested Ревью: есть замечания — фиксер дорабатывает label Sep 17, 2026
@ivanarama

Copy link
Copy Markdown
Owner Author

Ревью. (круг 2)
Reviewed-SHA: f35e0b2
Outcome-Label: changes-requested
Что меняется: PR #1503, HEAD f35e0b2: в docs/features.md добавлен раздел о кнопке «Обновить» в списках. Описаны поддерживаемые списки, сохранение параметров URL, обновление текущего фрейма, сброс временного выделения и прокрутки, а также сценарий проверки..
Проверено: Полный diff относительно base 390617a: только docs/features.md, 21 добавленная строка. Проверен detached worktree точного HEAD; git diff --check проходит.; Утверждения сверены с internal/ui/templates.go:1702,1712,3148,3292, обработчиками handlers_entity.go, handlers_registers.go, handlers_journals.go и internal/ui/static/ui.js. Кнопка использует текущий RequestURI, переход списка остаётся внутри текущего фрейма; замечаний к содержанию документации не найдено.; Метаданные раздела корректны: уникальный заголовок, UTF-8, status: testing, date: 2026-09-07. Дата совпадает с mergedAt PR #1324; отсутствие issue-маркера сохраняет работу featureage.; go build ./... — успешно.; go test -count=1 ./tools/featureage — успешно.; go test -count=1 ./internal/ui -run '^TestListRefreshKeepsCurrentRequestURI$' -v — успешно для обычного списка, регистра сведений и журнала.; go test -count=1 ./internal/ui -run '^(TestTabsBehaviorInNode|TestHead_EmbeddedChromeHidden|TestPageList_EmbeddedOpenUsesShell)$' -v — две Go-проверки прошли; TestTabsBehaviorInNode пропущен самим тестом из-за отсутствия node. Браузерный прогон локально не выполнялся.; go vet ./tools/featureage ./internal/ui — успешно.; go run ./tools/featureage -now 2026-09-18 — успешно, без изменения файлов; новый раздел не попадает в созревшие. Полный go test ./... не запускался: для проверки этой документационной правки выбран целевой набор.; CI точного HEAD проверен через gh pr checks --required, commit check-runs/status и Actions runs: проверок и запусков нет, aggregate status=pending. Перед complete повторно подтверждены ivanarama, OPEN, main, отсутствие draft и тот же HEAD; statusCheckRollup по-прежнему пуст..
Блокирующее: 1) Не пройден обязательный CI для f35e0b2: отсутствуют результаты build, lint, postgres-integration, vuln, smoke, e2e, test-windows и launcher-webview-build. Все восемь проверок требуются живой защитой main и .github/branch-protection.json; workflow .github/workflows/ci.yml не исключает документационные PR. Нужен успешный прогон обязательного CI этого HEAD перед положительным заключением; локальные проверки его не заменяют..
Хвост:

Вердикт: есть замечания.

@ivanarama

Copy link
Copy Markdown
Owner Author

PromptPilot service marker: REVIEW result publication claimed.

@ivanarama ivanarama added the changes-requested Ревью: есть замечания — фиксер дорабатывает label Sep 17, 2026
@ivanarama

Copy link
Copy Markdown
Owner Author

PromptPilot service marker: REVIEW result committed.

Объединён свежий main a8a2744.
В конфликте docs/features.md сохранены оба самостоятельных раздела:
одиночный выбор из main и прежнее описание кнопки «Обновить» из PR.
Относительно main изменение остаётся прежним: только 21 строка каталога.

Доработка устраняет конфликт ветки, из-за которого отсутствовал CI,
указанный единственным блокером во втором ревью PR #1503.

Generated-with: Codex
PP-Fix-Transition: from=f35e0b288cd7170e5de74cae75906e8105b7195d review-comment=5721947333 claim=5721947811 epoch-sha256=2bae355c84bacf0bd7e8c50c01e4576feb321e9d365343d032f8980f757865ff
@ivanarama

Copy link
Copy Markdown
Owner Author

Устранён единственный блокер второго ревью — отсутствие обязательного CI.

  1. GitHub показывал конфликт ветки с main и не создавал CI текущего HEAD. В ветку PR включён main на a8a2744048cea97961d01287a1f0e06520e8411a; единственный конфликт был в конце docs/features.md. Сохранены оба раздела: одиночный выбор из main и описание кнопки «Обновить» из PR.
  2. Описание кнопки сохранено побайтово. Относительно обновлённого main PR по-прежнему меняет только docs/features.md: 21 добавленная строка. Других замечаний к содержанию во втором ревью не было.

Локально прошли go build ./..., go test -count=1 ./tools/featureage ./internal/ui, go run ./tools/featureage -now 2026-09-18 и git diff --check. Каталог после проверки не изменился; новый раздел не требует смены статуса. В локальном окружении нет Node.js, поэтому встроенные Node-проверки UI были пропущены.

Для нового HEAD aac412ae5ecd07a297ef2b9c28c29e2fbaf7c4cb успешно завершились все восемь обязательных проверок: build, lint, postgres-integration, vuln, smoke, e2e, test-windows, launcher-webview-build. Прогон CI.

@ivanarama ivanarama removed the changes-requested Ревью: есть замечания — фиксер дорабатывает label Sep 18, 2026
@ivanarama

Copy link
Copy Markdown
Owner Author

Ревью. (круг 3)
Reviewed-SHA: aac412a
Outcome-Label: reviewed

Что меняется: в docs/features.md добавлен раздел о кнопке «Обновить» в списках сущностей, регистров сведений и журналов документов. Описаны сохранение параметров текущего URL, обновление одного фрейма в режиме вкладок и сброс временного выделения строки и прокрутки; приведён сценарий проверки. Изменение закрывает #1449 по плану триажа.

Проверено:

  • Detached worktree точного HEAD; полный diff относительно a8a2744048cea97961d01287a1f0e06520e8411a содержит только docs/features.md, +21 строку. git diff --check прошёл. При разрешении конфликта сохранён соседний раздел об одиночном выборе.
  • Текст сверен с шаблоном list-refresh, обработчиками list/infoRegList/journalList, static/ui.js и tabs.go: ссылка передаёт текущий RequestURI, навигация списка остаётся в его фрейме, временное выделение хранится в DOM. Неверное утверждение о потере параметров при обычной перезагрузке браузером из первого круга удалено.
  • Заголовок раздела уникален; status: testing, дата 2026-09-07 совпадает с merge feat(ui): добавить обновление списков #1324, issue-маркер отсутствует. go run ./tools/featureage -now 2026-09-18 прошёл, новую секцию не предлагает состарить и файл не изменяет.
  • go test -count=1 ./tools/featureage — PASS.
  • go test -count=1 ./internal/ui -run '^(TestListRefreshKeepsCurrentRequestURI|TestListViewSwitchKeepsSearch|TestListSearchFormKeepsView|TestListFilterFormKeepsSearch|TestListSortLinkKeepsSearchAndFolder|TestTabs.*|TestHead_EmbeddedChromeHidden|TestPageList_EmbeddedOpenUsesShell)$' -v — PASS. Проверка RequestURI прошла для всех трёх видов списков. TestTabsBehaviorInNode пропущен самим тестом из-за отсутствия Node.js; браузерный прогон локально не выполнялся.
  • go vet ./tools/featureage ./internal/ui — PASS. Полную локальную сборку и полный набор тестов не запускал: diff меняет только документацию.
  • Обязательные проверки живой защиты main и .github/branch-protection.json совпадают. Все восемь завершились успешно для точного HEAD: build, lint, postgres-integration, vuln, smoke, e2e, test-windows, launcher-webview-build. CI. Блокер второго круга устранён.

Блокирующее: нет. Оба замечания предыдущих кругов устранены; нерешённого вопроса для эскалации человеку нет.

Хвост:

Вердикт: годится к мержу.

@ivanarama

Copy link
Copy Markdown
Owner Author

No description provided.

@ivanarama ivanarama added the reviewed Ревью: блокирующего нет — ждёт ship label Sep 18, 2026
@ivanarama

Copy link
Copy Markdown
Owner Author

No description provided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewed Ревью: блокирующего нет — ждёт ship

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs/features.md: нет секции у кнопки «Обновить» в списках (#1324)

1 participant