From b4cb48bef80743aec375e49b72e5b173223a78ef Mon Sep 17 00:00:00 2001 From: Kayos Date: Thu, 30 Apr 2026 21:34:51 -0700 Subject: [PATCH] =?UTF-8?q?list:=20drop=20confusing=20'clear'=20button=20+?= =?UTF-8?q?=20bump=20enrich=20timeout=2090=E2=86=92180s?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit (1) /list had a 'clear' button that just unchecked all the localStorage shopping checkboxes. Cobb correctly flagged it as confusing — looks like a destructive action ('clear what?'). It was just a UX nicety for re-using the same shopping list, not a plan-touching operation. The plan-level reset already lives on /plan via a properly-scoped button. Removing the /list 'clear' so there's exactly one reset concept in the app: /plan → ⌫ reset week. (2) forge.enrich_recipe was hitting clawdforge with timeout_secs=90 which proved too short for one recipe (peppery-barbecue-glazed-shrimp type long ingredient list). Bumped to 180s to match the sterilizer's post-fix timeout. Walks now have more slack before timing out a single recipe. --- cauldron/forge.py | 2 +- cauldron/templates/list.html | 8 -------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/cauldron/forge.py b/cauldron/forge.py index 88394de..9447328 100644 --- a/cauldron/forge.py +++ b/cauldron/forge.py @@ -650,7 +650,7 @@ class Forge: "'chicken stir fry with rice'.\n" "- When uncertain on a categorical, use 'unknown' or 'other' rather than guessing." ) - result = self.run(prompt, model=model or "sonnet", timeout_secs=90) + result = self.run(prompt, model=model or "sonnet", timeout_secs=180) return _extract_recipe_meta(result) def fetch_food_info(self, name: str, *, model: str | None = None) -> dict: diff --git a/cauldron/templates/list.html b/cauldron/templates/list.html index 1cfb392..7c1c87d 100644 --- a/cauldron/templates/list.html +++ b/cauldron/templates/list.html @@ -172,7 +172,6 @@
-
@@ -214,13 +213,6 @@ } save(checked); }); - document.getElementById('clear-all').addEventListener('click', () => { - for (const li of list.querySelectorAll('li.row')) { - li.classList.remove('checked'); - } - checked.clear(); - save(checked); - }); })();