Add to this week and favorite (Feature 15) #147

2026-10-08T20:59:21-04:00

2026-10-08 20:59:21 -04:00

Add to this week and favorite (Feature 15, task 3)

Maestro task 07423ba7-9f91-4ec5-8ab8-1fb92e3c7d2e. Wires the two controls the mobile detail screen rendered in their resting state.

"Add to this week" opens a Sheet listing the current week's seven days with each day's current dinner and a toggle to the next week; choosing a day swaps via the Plan board's own endpoint (PUT /api/v1/plans/{weekStart}/days/{date}), reusing its query keys and mutation pattern rather than adding a parallel swap path. A week with no plan offers "Draft this week first" linking to /plan — the AI never invents a plan. Success shows a confirmation and invalidates the plan query.

The heart toggles is_favorite through POST /dinners/{id}/favorite, optimistically with rollback on failure and an aria-pressed state. The detail cache holds the detail shape while that endpoint answers with the list-style dinner, so the heart patches the cached dinner rather than overwriting it. Weeks are Monday-start in the household timezone, using the existing week utilities.

The developer reports typecheck, tests and build passing; no Go file changed. Recorded for the next task (tablet kitchen reading mode): the detail screen still has one column switched by breakpoint, ActionBar is now just AddToWeek plus FavoriteHeart (reuse both), and any change to a dinner should update dinnerDetailKey(id) directly rather than replacing it with the list-style shape.

## Add to this week and favorite (Feature 15, task 3) Maestro task `07423ba7-9f91-4ec5-8ab8-1fb92e3c7d2e`. Wires the two controls the mobile detail screen rendered in their resting state. "Add to this week" opens a Sheet listing the current week's seven days with each day's current dinner and a toggle to the next week; choosing a day swaps via the Plan board's own endpoint (`PUT /api/v1/plans/{weekStart}/days/{date}`), reusing its query keys and mutation pattern rather than adding a parallel swap path. A week with no plan offers "Draft this week first" linking to /plan — the AI never invents a plan. Success shows a confirmation and invalidates the plan query. The heart toggles `is_favorite` through `POST /dinners/{id}/favorite`, optimistically with rollback on failure and an `aria-pressed` state. The detail cache holds the detail shape while that endpoint answers with the list-style dinner, so the heart patches the cached dinner rather than overwriting it. Weeks are Monday-start in the household timezone, using the existing week utilities. The developer reports typecheck, tests and build passing; no Go file changed. Recorded for the next task (tablet kitchen reading mode): the detail screen still has one column switched by breakpoint, `ActionBar` is now just `AddToWeek` plus `FavoriteHeart` (reuse both), and any change to a dinner should update `dinnerDetailKey(id)` directly rather than replacing it with the list-style shape.

orchestrator-agent added 1 commit 2026-10-08 20:59:22 -04:00

Wire Add to this week and the favorite heart on the dinner detail

CI / ci (push) Successful in 5m11s

Details

CI / ci (pull_request) Successful in 5m6s

Details

5267e8a2a7

Add to this week opens a sheet of the household's current week (or next
week by the toggle), Monday to Sunday in the household timezone, each day
with its current dinner. Choosing a day swaps the dinner in through the
Plan's own useSwapDay (PUT /api/v1/plans/{weekStart}/days/{date}); the
screen then says it is being added, added, or why not with Retry, and the
week's plan query is invalidated once added. A week with no plan offers
"Draft this week first", linking to /plan. A dinner already on the week
cannot be added again.

The heart toggles is_favorite through the existing useToggleFavorite,
which now also updates the detail cache optimistically, rolls it back on
failure, and takes only is_favorite from the list-shaped answer so the
detail keeps its recipe. It exposes aria-pressed; a failure shows the
error with Retry.

useSwapDay now also reports its latest swap when it succeeded.

Maestro task 07423ba7-9f91-4ec5-8ab8-1fb92e3c7d2e.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

review_agent requested changes

2026-10-08 21:00:48 -04:00

Dismissed

review_agent left a comment

Independent Codex gpt-6-sol review: CHANGES_REQUIRED for this exact commit.

overseer added 1 commit 2026-10-08 21:04:19 -04:00

Share the Draft board's swap step with Add to this week

CI / ci (push) Successful in 4m57s

Details

CI / ci (pull_request) Successful in 5m0s

Details

bbd66e49f2

useWeekSwap is now the one way a day is swapped: it refuses while a swap
or push of the week is pending, and leaves the week's settled swaps and
pushes behind before sending the swap. The Draft board (sheet, drop and
retry) and the dinner's Add to this week sheet and its Retry all use it,
so a swap from the dinner no longer leaves the Plan saying the earlier
plan's Supply run was made.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

review_agent approved these changes

2026-10-08 21:12:31 -04:00

review_agent left a comment

Independent Codex gpt-6-sol review: APPROVED for this exact commit.

approver_agent merged commit c454c76816 into feature/293a89c5-bd98-4de4-9afd-5ca371e5e679-recipe-detail-and-kitchen-reading-mode

2026-10-08 21:31:21 -04:00

Sign in to join this conversation.

3 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Maniacs_Mansion/MealQuest#147