bug: document.Refresh() panics if page.Reload() returns nil response #11

New Issue

Claude · 2026-02-14T16:06:09Z

Claude commented

2026-02-14 16:06:09 +00:00

Parent: #1

Description

In document.go:58-69:

func (d *document) Refresh() error {
    resp, err := d.page.Reload()
    if err != nil {
        return fmt.Errorf("failed to reload page: %w", err)
    }
    resp.Status() // panics if resp is nil
}

If Playwright's Reload() returns (nil, nil) — which can happen if the page is already navigating or in some edge cases — resp.Status() will cause a nil pointer dereference.

Fix

Add a nil check: if resp == nil { return nil } or treat nil response as an error.

**Parent:** #1 ## Description In `document.go:58-69`: ```go func (d *document) Refresh() error { resp, err := d.page.Reload() if err != nil { return fmt.Errorf("failed to reload page: %w", err) } resp.Status() // panics if resp is nil } ``` If Playwright's `Reload()` returns `(nil, nil)` — which can happen if the page is already navigating or in some edge cases — `resp.Status()` will cause a nil pointer dereference. ## Fix Add a nil check: `if resp == nil { return nil }` or treat nil response as an error.

Claude added the bug priority/medium type/task labels 2026-02-14 16:06:18 +00:00

Claude referenced this issue

2026-02-14 16:09:00 +00:00

Epic: Error Handling & Correctness #1

Claude referenced this issue

2026-02-15 15:59:00 +00:00

Master plan: address all open issues (17 PRs across 5 phases) #31

Claude referenced this issue

2026-02-15 16:11:12 +00:00

Master plan: address all open issues (17 PRs across 5 phases) #31

Claude referenced this issue

2026-02-15 16:13:12 +00:00