Maniacs_Mansion/MealQuest
11
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Discover deck screen (mock 1a) #105

Merged
approver_agent merged 3 commits from task/discover-deck-screen_1ecc121c-86eb-4784-a9fb-1314709b4a59 into feature/e3a8e4ea-8dcb-49e8-878f-552a6af109cb-discover-side-quest-suggestions 2026-10-07 01:28:14 -04:00
Conversation 3 Commits 3 Files Changed 13
+1378 -43
3 Commits
Author SHA1 Message Date
dev_agentandClaude Opus 5.5 7e2bf897a2 Validate the job envelope and make Retry poll the batch's job
CI / ci (push) Successful in 2m54s
Details
CI / ci (pull_request) Successful in 2m37s
Details 
useJob now takes the job kind and checks each GET /api/v1/jobs/{id}
answer: the requested id, the expected kind, a known status, a result
and a text or null error. An answer that is not the job is exposed as the
error and stops polling, so an unknown status no longer leaves the
Discover skeleton cards running with no error or Retry. useJob also
returns poll, which asks for the job at once and restarts the backoff;
the Discover page's "Couldn't check on new side quests" Retry uses it
instead of posting another batch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
 2026-10-07 01:22:17 -04:00
dev_agentandClaude Opus 5.5 fe87968ac9 Fix the Discover deck review findings
CI / ci (pull_request) Successful in 2m39s
Details
CI / ci (push) Successful in 3m10s
Details 
The Quest log pill is now a button that shows and hides the names of
the dinners in the Quest log, read from GET
/api/v1/dinners?status=quest_log; the designed Quest log sheet replaces
this list in its own task.

A batch whose job cannot be polled, or whose result cannot be read,
now shows an error with a Retry that asks for the batch again, instead
of leaving skeleton cards on screen with no way out.

Decisions go through one submit guarded by a ref set before the request
is sent, so presses before the next render cannot send a second,
conflicting decision. A decision counts as saved only when the answer
is that suggestion, accepted or declined as chosen; anything else shows
in the "Couldn't save that choice" banner.

Pass and Try it are IconButtons: IconButton gains a neutral tone and
showLabel, which shows the label as an 11 px caption under the circle.

Maestro task 1ecc121c-86eb-4784-a9fb-1314709b4a59.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
 2026-10-07 01:13:00 -04:00
dev_agentandClaude Opus 5.5 38fec723f6 Add the Discover deck screen (mock 1a)
CI / ci (push) Successful in 2m53s
Details
CI / ci (pull_request) Successful in 2m37s
Details 
/discover now shows the Side quests deck: the "Discover" kicker and
"Side quests" title, the teal "Quest log · N" pill, and the three
stacked cards in a 440 px deck with perspective 1000px. The top card
shows the washed photo, the name, the difficulty dots and the
"Why this?" box, with hidden "Try it!" and "Pass" stamps. The 62 px
Pass and Try it buttons and the ←/→ keys all go through one decide
handler that ignores input while a decision is in flight.

An empty deck shows "All caught up" with "Get more now", which starts a
suggestion batch and shows skeleton cards while its job runs, reading
the deck again as the batch stores Side quests.

Maestro task 1ecc121c-86eb-4784-a9fb-1314709b4a59.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
 2026-10-07 01:03:48 -04:00