Skip to content

Claude/food tracking feature hv6bij#2

Merged
bmingles merged 3 commits into
mainfrom
claude/food-tracking-feature-hv6bij
Jun 27, 2026
Merged

Claude/food tracking feature hv6bij#2
bmingles merged 3 commits into
mainfrom
claude/food-tracking-feature-hv6bij

Conversation

@bmingles

Copy link
Copy Markdown
Member

No description provided.

claude added 3 commits June 27, 2026 18:04
Tapping a food entry in today's list loads it back into the add-form
for editing (name/macros/quantity), with Save Changes/Cancel actions.
Adds FoodStore.update(), backed by IndexedDB get+put preserving id and
createdAt.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K3Q361t53QdxgaV8nHSayT
Switches deploys from the native Pages-artifact flow (which can only
host one atomic deployment) to publishing both builds onto the
gh-pages branch: main still deploys to the site root, while every
other branch overwrites a single shared /tag-notes/preview/ path, so
a feature-branch PWA can be installed once on iOS and just gets
replaced by whichever branch is pushed next.

Makes the base path and PWA manifest/start_url/scope generated rather
than hardcoded (VITE_BASE_PATH/VITE_APP_VARIANT env vars), since the
preview build needs to live under a different subpath and be
distinguishable by name from the production app.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K3Q361t53QdxgaV8nHSayT
The production service worker registers with scope "/tag-notes/", which
also covers "/tag-notes/preview/". Its workbox NavigationRoute had no
denylist, so on devices that had already installed the production PWA,
that active worker intercepted every navigation in scope - including a
first-ever visit to /preview/ - and served back the old production
index.html/JS before the preview build's own worker ever got a chance to
register. This made the preview deploy look like it was missing
features (e.g. the food-entry edit-on-tap) that were actually present in
the deployed files all along.

Add a navigateFallbackDenylist to the production build's workbox config
so its navigation fallback never claims the /preview/ subpath.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K3Q361t53QdxgaV8nHSayT
@bmingles bmingles merged commit 26ebea4 into main Jun 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants