ReviewOS

also looking at this

craft-native/craft

test(mobile-e2e): prove iOS location settles, and that Zig served it

#213
Merged glennmichael123 wants to merge fix/ios-location-settles into main
6 files +157 -28
packages/zig/src/ios_dispatch.zigmodified+9-1
Changes to packages/zig/src/ios_dispatch.zig
@@ -588,7 +588,15 @@ export fn craft_ios_handle_action(
588588 // is a served action with an unhappy result, not an unserved one.
589589 return true;
590590 };
591 return offer == .answered;
591 if (offer == .answered) return true;
592
593 // Said out loud for the same reason the dispatch line is. The page gets
594 // the same answer from Swift and cannot tell, so without this an action
595 // Zig has quietly stopped serving looks exactly like one it serves - the
596 // dispatch line above is logged either way. The mobile E2E runtime leg
597 // fails when an action it expects Zig to serve shows up here.
598 std.log.info("ios: {s} is not served here; handing it back to the host", .{action});
599 return false;
592600}
593601
594602/// Deliver a result the host shim produced.