eagraiclainne/app

Watch

Fork 0

feat: rename a list from its detail view (#115) #141

Merged

nalum merged 2 commits from feat/list-rename into main

2026-08-18 16:43:03 +00:00

nalum commented

2026-08-18 12:38:51 +00:00

Owner

Closes #115. PR 3 of the #137 stack, based on #140.

ItemListService/Update was reachable only from the TUI — a gap on the reference surface itself (parity defect 3). Both daily surfaces now carry the verb:

Web: the list detail page's manager row gains Rename (gated on can("ItemListService","Update")), opening a one-field sheet prefilled with the current name. The write rides the offline queue: the optimistic sketch renames the cached list row and each entry's list ref, so no board shows the old name before sync.
Android: the detail screen's manager row gains the same Rename, mirroring the New list sheet. ListsRepository.renameList goes online-first through the shared OpQueue; the sketch updates item_list.name plus the denormalised item.list_name, and the server's echo overwrites both.

Verified against the live deploy: server rename round-trips (Update returns the new name), the web sheet opens prefilled and gated. Android on-device check pending (no device attached), matching the stack's standing pattern.

🤖 Generated with Claude Code

Closes #115. PR 3 of the #137 stack, based on #140. `ItemListService/Update` was reachable only from the TUI — a gap on the reference surface itself (parity defect 3). Both daily surfaces now carry the verb: - **Web**: the list detail page's manager row gains **Rename** (gated on `can("ItemListService","Update")`), opening a one-field sheet prefilled with the current name. The write rides the offline queue: the optimistic sketch renames the cached list row and each entry's list ref, so no board shows the old name before sync. - **Android**: the detail screen's manager row gains the same **Rename**, mirroring the New list sheet. `ListsRepository.renameList` goes online-first through the shared OpQueue; the sketch updates `item_list.name` plus the denormalised `item.list_name`, and the server's echo overwrites both. Verified against the live deploy: server rename round-trips (`Update` returns the new name), the web sheet opens prefilled and gated. Android on-device check pending (no device attached), matching the stack's standing pattern. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

nalum added 2 commits

2026-08-18 12:38:51 +00:00

feat(web): rename a list from its detail page 3e22bb8aa3

ItemListService/Update was reachable only from the TUI — a gap on the
reference surface itself (parity defect 3). The detail page's manager
row gains Rename, opening a one-field sheet; the write rides the
offline queue like every other list verb, patching the cached list
name and each entry's list ref so boards stay coherent before sync.

feat(android): rename a list from its detail screen

check / commits (pull_request) Successful in 6s

Details

check / web (pull_request) Successful in 1m35s

Details

check / go (pull_request) Successful in 2m42s

Details

check / report (pull_request) Successful in 5s

Details

android / build (pull_request) Successful in 5m24s

Details

android / report (pull_request) Successful in 3s

Details

check / commits (push) Has been cancelled

Details

check / go (push) Has been cancelled

Details

check / report (push) Has been cancelled

Details

check / web (push) Has been cancelled

Details

tag / tag (push) Has been cancelled

Details

android / report (push) Has been cancelled

Details

android / build (push) Has been cancelled

Details

7a83886501

The parity gap on web (defect 3) held on Android too: every list verb
but Update had an affordance. The manager row gains Rename, opening a
sheet mirroring New list; the write goes online-first through the
shared OpQueue, and the sketch renames the cached list row plus each
entry's list_name so offline boards agree with the server's echo.

nalum referenced this pull request

2026-08-18 12:39:16 +00:00

Order of work for the open backlog (#107–#136) #137

forgejo-actions commented

2026-08-18 12:41:39 +00:00

Test report

Suite	Tests	Result	Skipped
Unit	1370	✅ pass	1
Integration	85	✅ pass	—

Coverage: 28.2%

_{Updated by the check workflow · commit 7a83886501}

## Test report | Suite | Tests | Result | Skipped | | --- | --: | --- | --: | | Unit | 1370 | ✅ pass | 1 | | Integration | 85 | ✅ pass | — | **Coverage:** 28.2% <sub>Updated by the check workflow · commit 7a83886501f3eba6bb9aba17b6b1728a22f1c491</sub>

forgejo-actions commented

2026-08-18 12:44:19 +00:00

Android test report

Suite	Tests	Result	Skipped
Unit (debug)	31	✅ pass	0

Coverage: 2.4% of lines

_{Updated by the android workflow · commit 7a83886501}

## Android test report | Suite | Tests | Result | Skipped | | --- | --: | --- | --: | | Unit (debug) | 31 | ✅ pass | 0 | **Coverage:** 2.4% of lines <sub>Updated by the android workflow · commit 7a83886501f3eba6bb9aba17b6b1728a22f1c491</sub>

nalum referenced this pull request

2026-08-18 12:47:36 +00:00

fix: honest MCP verbs — reopen, rota attribution, event clears #142

nalum changed target branch from fix/reward-job-links to main

2026-08-18 16:42:56 +00:00

nalum merged commit 7a83886501 into main

2026-08-18 16:43:03 +00:00

nalum deleted branch feat/list-rename

2026-08-18 16:43:04 +00:00