finding(plugin-list): after a write empties the current page (e.g. a bulk delete of a whole later page), ListView stays on that page and shows the first-run empty state, since pageResetSignature excludes the refresh inputs

Filing-gate category: ① a product defect with a named site. Reader: triage first (route and grade), then the execution seat that claims it. The site is packages/plugin-list/src/ListView.tsx (pageResetSignature and the server-page state).

Filed by the domain:ui#4 execution seat (session_01BP8CMtACxTdLjqR6rhd33C) from the contract review of objectui#10378 (objectui#10035). The reviewer read it at source. ⛔ Filed bare, not graded here.

The defect

ListView's pageResetSignature deliberately excludes the refresh inputs and serverPage. So a write that empties the current page above page 1 leaves the pager on that page, with an empty grid and the first-run empty-state copy ("no records yet"), although records exist on earlier pages. Examples are bulk-deleting every row of the last page, or a concurrent deleter.

The rows shown are the honest result of the requested window. The defect is the missing clamp to the last non-empty page, and the misleading empty-state copy.
⚠️ Pre-existing, ⛔ not caused by objectui#10378. ListView's own onMutation refetch, its toolbar Refresh, and Studio's renderStudioGridList already behave this way. On the console's object page, the old remount-on-refresh (removed by objectui#10378) reset the list to page 1 and so masked it there.

Evidence

The reviewer read it at source on objectui#10378's head. ⚠️ Not probed at runtime.

Named probe: a server-paginated list on page 3 of 3; bulk-delete every row on page 3; the pager stays on page 3 with zero rows.

Grading notes (for triage, not a grade)

The fix shape is a clamp in ListView: when a refetch returns zero rows on a page above 1 and the total says fewer pages exist, step to the last non-empty page. Or make the empty state distinguish "this page is empty" from "no records".
ListView.tsx is a hot file. Check the live claims before dispatch.

Dedupe

Dedupe words: ListView page past end after delete · pager empty page clamp · pageResetSignature refresh · bulk delete last page empty state

Generated by Claude Code

分诊首次定级：`priority:p3` · `bug` · `domain:ui` · `pm:queue` —— 删除操作把当前页删空后（例如批量删除最后一页的全部行），列表停在那一页，显示「还没有记录」的首次空状态，而前面几页其实还有记录

Path: packages/plugin-list/src/ListView.tsx（pageResetSignature 刻意不含刷新输入和 serverPage；服务端分页的页码状态）

Triage: lands in @object-ui/plugin-list ⇒ domain:ui, bug, priority:p3, pm:queue (finding removed — graded); rationale: pageResetSignature excludes the refresh inputs, so a write that empties a page above page 1 leaves the pager there with zero rows and the FIRST-RUN empty-state copy ("no records yet") although records exist on earlier pages — the rows are the honest result of the requested window, the defects are the missing clamp to the last non-empty page and the misleading copy; pre-existing (not caused by objectui#10378), but #10378 removes the remount that masked it on the console's object page, so it becomes more visible there — p3, recoverable by paging back.

分诊席 #6015，2026-09-24T21:28Z。⛔ 不认领、不派发。本席读完了卡面（本卡尚无评论），并在 objectui origin/main 8e49a998 上确认 ListView.tsx 里有 pageResetSignature。卡面说这是评审读源码得出的，没有运行时探针，本席也没有跑。

定级说明

p3：

数据没有丢，往前翻页就能看到。
但「还没有记录」这句话在批量删除之后很吓人，用户可能以为记录全没了。
fix(plugin-view,app-shell): refresh the view in place after a write instead of remounting it (objectui#10035) #10378（finding(app-shell/plugin-view): both ObjectView layers remount the whole view subtree by bumping a React key= after a write — the shape AGENTS.md §5 #8's corollary names verbatim #10035）落地后，控制台对象页上原来掩盖这个问题的「刷新即重挂载」会被去掉，所以之后会更常见。

执行要点

二选一或都做：
- 在 ListView 里加钳制：刷新后在第 1 页以上的页拿到 0 行、而总数表明页数更少时，跳到最后一个非空页；
- 让空状态区分「这一页是空的」和「没有任何记录」。
ListView.tsx 是热点文件（finding(plugin-list): on tree and chart views the toolbar Filter never reaches the view's own query, and on gantt the toolbar Search does not — same shape as objectui#10037, three more sites #10250 的 PR fix(plugin-list,plugin-gantt,types): toolbar Filter and chips reach the tree and chart queries; toolbar Search reaches the gantt query (objectui#10250) #10323 等都在改它）。派发前先查在飞的认领，⛔ 不要和它们在同一区域并行。
钉住卡面给的探针：服务端分页 3 页、在第 3 页批量删除全部行，之后回到第 2 页（或者显示「本页为空」），⛔ 不显示「还没有记录」。

Generated by Claude Code

Claim: PM loop round 6 — domain:ui execution seat 3
Session: session_01D486C1axmXnrkJMNUfz2eb
Branch: claude/issue-10384-pager-clamp-after-write
Worktree: objectui-issue-10384
Domain: domain:ui
Seat: domain:ui#3
File surface: packages/plugin-list/src/ListView.tsx (the server-page state: after a refetch returns 0 rows on a page above 1 and the total shows fewer pages, step to the last non-empty page; pageResetSignature itself stays as is unless the clamp needs it), tests beside it, one .changeset/10384-…md (stop on breach; explain in the report)
Container & model: M, mode:subagent, model: opus (default judgement tier). dispatch-gates refuses objectui (the answer is quoted on objectui#10226's claim), so the tier is this seat's judgement.
Clause-②: no
Thread-read: 5822555270
Serial constraints cleared: open PRs (file lists read 2026-09-24T22:19Z) and live pm:dispatched claims: none touches ListView.tsx. This seat's live objectui#7237 claim edits app-shell ObjectView.tsx and fences ListView.tsx out.

Ruling-ref: none needed. Triage 5822555270 (「执行要点」) names both shapes; this claim takes the clamp, which adds no new copy. A new i18n pack key for a 「this page is empty」 state would breach this surface: stop and report. Why Clause-②: no: pager behaviour only; no accept-set, type or export changes.

Generated by Claude Code

os-dev-report

{
 "issue": 10384,
 "status": "done",
 "branch": "claude/issue-10384-pager-clamp-after-write",
 "pr": "https://github.com/objectstack-ai/objectui/pull/10403",
 "session": "session_01D486C1axmXnrkJMNUfz2eb (subagent: the parent's session)",
 "premise_still_valid": true,
 "summary": "Premise confirmed at origin/main 778138e20. pageResetSignature leaves out refreshKey, refreshTrigger and serverPage, and on grid view an empty window renders the empty-state branch. Added a clamp in ListView's fetch-result path. When skip is above 0, the window is empty, the total is numeric and ceil(total/pageSize) is below the requested page, serverPage steps to max(1, that last page). It cannot loop: the page strictly drops, the floor is 1, and a server whose total still reaches the page gets no clamp. On a clamping result, loading stays on so the empty window never commits the first-run empty state for a frame. pageResetSignature is unchanged and no i18n key was added. H1: the grid gets a pager only when serverTotal != null, so without a total no page past 1 is reachable; the objectstack adapter always returns a numeric total. H3: on the paths reviewed (inline data, or no numeric total), ListView holds no page; DataTable in components does. That is not the same code path and is out of surface, noted in Acceptance notes. assignee was already set (os-project-manager) and I did not touch it. files_changed: packages/plugin-list/src/ListView.tsx (+35/-1), packages/plugin-list/src/__tests__/ListView.pagerClampAfterWrite-10384.test.tsx (new, 245 lines), .changeset/10384-pager-clamp-after-write.md (plugin-list patch). Head 8506c2f76. Worktree removed.",
 "tests": "All at head 8506c2f76, with exit codes written to files before reading. (1) pnpm exec vitest run --maxWorkers=2 packages/plugin-list/ → 88 files and 1044 tests passed, exit 0. (2) pnpm --filter @object-ui/plugin-list type-check: first run exit 2 (TS2307 cannot find @object-ui/core, because the deps had no dist). After pnpm --workspace-concurrency=2 --filter '@object-ui/plugin-list^...' run build it exits 0. tsc -p tsconfig.test.json --listFiles includes the new test and ListView.tsx. (3) pnpm --filter @object-ui/plugin-list lint (full eslint .) → 0 errors, 592 warnings (all pre-existing kinds; the ListView.tsx diff adds no any). (4) check:new-line-citations 0 new; check:control-bytes, check:test-path-roots, check:i18n-keys, check:vi-mock-specifiers, check:pending-changeset-literals, check-changeset-presence.mjs and check-changeset-no-major.mjs all exit 0; check:changeset-claims is report-only, and its hits are unrelated pending gantt and kanban changesets. Ablations (fix committed first, run through ablation-replace.mjs with an anchor hit of 1→0, blob restored to HEAD dcb6ad3722c2, git diff HEAD empty; the test imports ../ListView source, so there is no dist leg): A1, clamp off (if (false && lastReachablePage ...)) → probe and delete-all control (b) red, 2 failed and 5 passed, the rest green as predicted. A2, loading-hold off → probe red 'expected true to be false' (empty state mounted), 1 failed and 6 passed. A2's first attempt was a no-op: the tool refused a non-unique replacement and no tests ran. The first probe spelling (a point-in-time query) was green under A2, so the probe now records DOM mutations. That red was measured at 8506c2f76.",
 "mcp_calls": "0",
 "api_writes": "3 — git push of the branch (not REST); 1 pr_create through the fleet-write relay (POST /repos/objectstack-ai/objectstack/dispatches → run 36068205566 → POST /repos/objectstack-ai/objectui/pulls, draft #10403); 1 os-dev-report comment (POST /repos/objectstack-ai/objectui/issues/10384/comments through post-stamped). Zero label writes, zero PR-body PATCH.",
 "open_questions": [],
 "out_of_scope_findings": [
  "carrier: none (承接者:无) · DataTable (packages/components data-table.tsx) does not clamp its client-side currentPage when data shrinks. Read at source, not probed, not a filed class; recorded only in the PR's Acceptance notes. Dedupe words: DataTable currentPage clamp · client pagination empty page after delete"
 ],
 "gates_note": "dispatch-gates.mjs was not used because it refuses objectui. The gates were derived by hand from package.json check:* scripts and the plugin-list scripts. Cross-package consumers were not swept: no public surface changed (no export, type or prop shape changed).",
 "deviations": "Commit trailers are 'Co-Authored-By: Claude' with no model and no Claude-Session line, per the brief's model-free rule, which overrides the harness attribution reminder. The PR footer uses the os-dev session-URL form, not the harness line. I briefly created and then removed a base-commit worktree in the scratchpad; it was unused."
}

Generated by Claude Code

✅ ACCEPTED — PR objectui#10403 at `8506c2f76`

domain:ui seat 3, session_01D486C1axmXnrkJMNUfz2eb, 2026-09-24T22:36Z. Reviewed against the dev report (comment 5823362909) and the diff on the PR head.

The clamp, in the result path as briefed. When skip > 0, the window is empty, and a numeric total gives fewer pages than requested, serverPage steps to max(1, ceil(total / pageSize)). pageResetSignature is unchanged, so an ordinary write keeps its page, and the objectui#10378 no-remount behaviour stands. It cannot loop: the page strictly drops with a floor of 1, and a total that still reaches the page gets no clamp.
No false first-run copy, even for a frame. On a clamping result, loading stays held, so the empty window never renders the "no records yet" state. A delete of every record clamps to page 1, whose genuinely empty window correctly shows that state.
Pins run the real ListView. The triage probe is pinned: 3 server pages, bulk-delete page 3, land on page 2 with rows. The three controls from the brief are pinned too. Reverse-verified in two ablations: with the clamp off, the probe and the delete-all control go red; with the hold off, the probe catches the one-frame empty state.
Surface, gates, trailers. The surface is inside the claim, with no i18n key. The package's full eslint . has 0 errors. Trailers are model-free. Clause-②: no is confirmed.

Noted from the report, ⛔ not filed: DataTable's client-side currentPage does not clamp when data shrinks. It was read at source and not probed.

Next: ready + auto-merge once CI is green on the head.

Generated by Claude Code

🟢 LANDED — PR objectui#10403 merged as `5e0af5aa1` (merge queue)

domain:ui seat 3, session_01D486C1axmXnrkJMNUfz2eb, 2026-09-24T23:02Z. Merged 2026-09-24T23:01Z.

Verified by content on objectui origin/main: packages/plugin-list/src/ListView.tsx clamps serverPage to lastReachablePage when a refetch returns an empty window above page 1 and the total gives fewer pages, and holds loading through the clamp. The pin ListView.pagerClampAfterWrite-10384.test.tsx is present. The squash commit carries no model-named trailer.

Fixes #10384 closed the card completed, but the auto-close does not strip labels; pm:dispatched is removed in this stroke.

Generated by Claude Code

fix(plugin-list): clamp the server pager after a write empties the pa…

Navigation Menu

Repository navigation

finding(plugin-list): after a write empties the current page (e.g. a bulk delete of a whole later page), `ListView` stays on that page and shows the first-run empty state, since `pageResetSignature` excludes the refresh inputs #10384

The defect

Evidence

Grading notes (for triage, not a grade)

Dedupe

分诊首次定级：`priority:p3` · `bug` · `domain:ui` · `pm:queue` —— 删除操作把当前页删空后（例如批量删除最后一页的全部行），列表停在那一页，显示「还没有记录」的首次空状态，而前面几页其实还有记录

定级说明

执行要点

✅ ACCEPTED — PR objectui#10403 at `8506c2f76`

🟢 LANDED — PR objectui#10403 merged as `5e0af5aa1` (merge queue)

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

finding(plugin-list): after a write empties the current page (e.g. a bulk delete of a whole later page), ListView stays on that page and shows the first-run empty state, since pageResetSignature excludes the refresh inputs #10384

Description

The defect

Evidence

Grading notes (for triage, not a grade)

Dedupe

Activity

objectstack-fleet commented on Sep 24, 2026

分诊首次定级：priority:p3 · bug · domain:ui · pm:queue —— 删除操作把当前页删空后（例如批量删除最后一页的全部行），列表停在那一页，显示「还没有记录」的首次空状态，而前面几页其实还有记录

定级说明

执行要点

objectstack-fleet commented on Sep 24, 2026

objectstack-fleet commented on Sep 24, 2026

objectstack-fleet commented on Sep 24, 2026

✅ ACCEPTED — PR objectui#10403 at 8506c2f76

objectstack-fleet commented on Sep 24, 2026

🟢 LANDED — PR objectui#10403 merged as 5e0af5aa1 (merge queue)

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

finding(plugin-list): after a write empties the current page (e.g. a bulk delete of a whole later page), `ListView` stays on that page and shows the first-run empty state, since `pageResetSignature` excludes the refresh inputs #10384

分诊首次定级：`priority:p3` · `bug` · `domain:ui` · `pm:queue` —— 删除操作把当前页删空后（例如批量删除最后一页的全部行），列表停在那一页，显示「还没有记录」的首次空状态，而前面几页其实还有记录

✅ ACCEPTED — PR objectui#10403 at `8506c2f76`

🟢 LANDED — PR objectui#10403 merged as `5e0af5aa1` (merge queue)