Check error from page.Context().Cookies() in playwright.go:updateCookies() before iterating, preventing silent failures when cookie retrieval fails
Replace time.Sleep(5s) with context-aware select using time.After in archive.go:Archive(), allowing the wait to be cancelled promptly when the context is done
Test plan
go build ./... passes
go test ./... passes
go vet ./... passes
Verify archive cancellation is responsive during the initial 5s wait
## Summary
- Check error from `page.Context().Cookies()` in `playwright.go:updateCookies()` before iterating, preventing silent failures when cookie retrieval fails
- Replace `time.Sleep(5s)` with context-aware `select` using `time.After` in `archive.go:Archive()`, allowing the wait to be cancelled promptly when the context is done
## Test plan
- [x] `go build ./...` passes
- [x] `go test ./...` passes
- [x] `go vet ./...` passes
- [ ] Verify archive cancellation is responsive during the initial 5s wait
Closes #7, #18
**Parent:** #31 (Phase 2, PR 6)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
- playwright.go: check error from page.Context().Cookies() before
iterating over results, preventing silent failures
- archive.go: replace time.Sleep(5s) with context-aware select using
time.After, allowing the operation to be cancelled promptly
Closes#7, #18
Co-Authored-By: Claude Opus 4.6 <[email protected]>
Claude
merged commit 85e4632ea9 into main2026-02-15 16:20:06 +00:00
Claude
deleted branch fix/cookies-error-and-context-sleep2026-02-15 16:20:07 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
page.Context().Cookies()inplaywright.go:updateCookies()before iterating, preventing silent failures when cookie retrieval failstime.Sleep(5s)with context-awareselectusingtime.Afterinarchive.go:Archive(), allowing the wait to be cancelled promptly when the context is doneTest plan
go build ./...passesgo test ./...passesgo vet ./...passesCloses #7, #18
Parent: #31 (Phase 2, PR 6)
🤖 Generated with Claude Code