Complete K-food deliveries while preserving two-hour return navigation - #477
Merged
Merged
Conversation
Record resolved delivery work immediately and preserve the existing mobile execution contract until a fixed two-hour navigation deadline. Revalidate ownership and stop corrections under the route lock. Constraint: Existing mobile and Shopify releases must remain compatible. Rejected: Delay operational completion for two hours | Completed work would remain active in Shopify. Confidence: high Scope-risk: moderate Directive: Preserve generation and current-child ownership; do not backfill historical outcomes. Tested: Prisma generation/validation, full lint/typecheck/build, 3014 tests including 4 PostgreSQL contracts, additive migration, independent review. Not-tested: Production deployment and authenticated Shopify/mobile deadline behavior.
Existing released apps need a valid command context to submit their return intent. Keep that context inert for GPS inference and guard legacy rollback against stale markers and incompatible deferred completion receipts. Constraint: Preserve mobile 1.3.3 and 1.3.4 without a mobile release. Rejected: Null-policy return context | Released apps never submit its return command. Confidence: high Scope-risk: moderate Directive: Legacy rollback requires zero deferred completion acknowledgements and guarded marker cleanup. Tested: Released app reducer contract, focused PostgreSQL return cases, real rollback guard cases, production-backup restore and additive migration twice with zero drift, independent review. Not-tested: Latest exact-commit full CI and production deployment are pending.
OziinG
force-pushed
the
codex/kfood-completion-navigation-grace
branch
from
October 2, 2026 14:36
0808fd8 to
4a8350d
Compare
OziinG
marked this pull request as ready for review
October 2, 2026 14:38
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
K-food routes stayed active after delivery work finished, and return-to-store navigation on the final Arrived stop did not send a delivery event. This change resolves that final stop on an authenticated return intent, shows delivery work Complete immediately in server/Shopify reads, and preserves return navigation in existing mobile apps for two hours.
Validation: Prisma generation, full lint/typecheck/build and 3,049 passing tests, including 39 PostgreSQL contracts; 189 unrelated opt-in tests skipped. Released-app reducers were executed with the command context. Real PostgreSQL rollback and backup-failure checks passed. A fresh production backup was restored, migrated twice and checked for zero schema drift. Independent review found no remaining blockers. Exact PR CI and merged main CI passed.
Production deployment completed at
2026-10-02T14:56Z, exact revision90827faf60133e7df3b53779d1c2dc8acd286f8f.7a5bf019-f11a-4245-8dcc-7933eb4f1804, Success/0.sha256:b2144fd675f23014646d810f4210eafa6f97a6afb8df7fdbeb84d9c2f2e8a26d; migration digest:sha256:6a6935aaf294e2f6455187156f19b8e166d6b0480ee08f66d58a2947904bdebd. Web static digest remainedsha256:3363abce063d02988137ffe812393ebff73afdd14ab40a4b10c9264b1bdd0b38.20261002140000_kfood_delivery_navigation_graceapplied successfully. Ownership audit found zero invalid associations. Both public health endpoints passed; the exact API and PostgreSQL containers are healthy with zero restarts after startup. The standard migration lane recreated PostgreSQL before API replacement; authenticated postchecks passed afterward.kfood_return_intent_only_v1with all coordinates null and zero candidates. Existing delivery counts remained unchanged, the stale finalization worker remained enabled, and location inference remained disabled. Post-start logs contained only informational JSON levels and no fatal/unhandled error lines in the observed window.No mobile or Shopify source release. The private backup and restore proof are retained outside Git; no production delivery outcomes were fabricated for verification.