## Summary
- Make `staticCookieJar` thread-safe with `sync.RWMutex` (reads use RLock, writes use Lock)
- Add `SameSite` field to `Cookie` struct with `SameSiteStrict`, `SameSiteLax`, `SameSiteNone` constants
- Update Playwright cookie conversion functions to preserve SameSite attribute
- Replace hardcoded 4-country switch in Google search with dynamic `"country" + strings.ToUpper(c.Country)` supporting all countries
## Test plan
- [x] `go build ./...` passes
- [x] `go test ./...` passes (existing tests updated for new struct layout)
- [x] `go vet ./...` passes
Closes #20, #22, #23
**Parent:** #31 (Phase 3, PR 11)
- Wrap staticCookieJar in struct with sync.RWMutex for thread safety
- Add SameSite field to Cookie struct with Strict/Lax/None constants
- Update Playwright cookie conversion functions for SameSite
- Replace hardcoded 4-country switch with dynamic country code generation
Closes#20, #22, #23
Co-Authored-By: Claude Opus 4.6 <[email protected]>
Claude
merged commit ddb701fca0 into main2026-02-15 16:35:11 +00:00
Claude
deleted branch enhance/cookies-and-google2026-02-15 16:35:11 +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
staticCookieJarthread-safe withsync.RWMutex(reads use RLock, writes use Lock)SameSitefield toCookiestruct withSameSiteStrict,SameSiteLax,SameSiteNoneconstants"country" + strings.ToUpper(c.Country)supporting all countriesTest plan
go build ./...passesgo test ./...passes (existing tests updated for new struct layout)go vet ./...passesCloses #20, #22, #23
Parent: #31 (Phase 3, PR 11)