From 23303a759b1bfb549d8099c6c7dfaabe1a22b115 Mon Sep 17 00:00:00 2001
From: Karl Seguin <k@openmymind.io>
Date: Fri, 23 Jan 2026 19:09:36 +0800
Subject: [PATCH] Prevents fast double navigate

This puppeteer script [likely will] crash the brower:

page.goto(baseURL + '/campfire-commerce/');
await page.goto(baseURL + '/campfire-commerce/');

The quick double-navigation means parse_state remains .pre and thus the page
isn't reset. We introduce a new load_state, .waiting, which can be used to
detect this state.
---
 src/browser/Page.zig | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/browser/Page.zig b/src/browser/Page.zig
index b5bc29af..1862258b 100644
--- a/src/browser/Page.zig
+++ b/src/browser/Page.zig
@@ -256,7 +256,7 @@ fn reset(self: *Page, comptime initializing: bool) !void {
     self.window._location = &default_location;
 
     self._parse_state = .pre;
-    self._load_state = .parsing;
+    self._load_state = .waiting;
     self._queued_navigation = null;
     self._parse_mode = .document;
     self._attribute_lookup = .empty;
@@ -335,11 +335,12 @@ pub fn isSameOrigin(self: *const Page, url: [:0]const u8) !bool {
 
 pub fn navigate(self: *Page, request_url: [:0]const u8, opts: NavigateOpts) !void {
     const session = self._session;
-    if (self._parse_state != .pre) {
+    if (self._parse_state != .pre or self._load_state != .waiting) {
         // it's possible for navigate to be called multiple times on the
         // same page (via CDP). We want to reset the page between each call.
         try self.reset(false);
     }
+    self._load_state = .parsing;
 
     const req_id = self._session.browser.http_client.nextReqId();
     log.info(.page, "navigate", .{
@@ -2708,6 +2709,9 @@ const ParseState = union(enum) {
 };
 
 const LoadState = enum {
+    // waiting for the main HTML
+    waiting,
+
     // the main HTML is being parsed (or downloaded)
     parsing,