chore(deps): refresh protobuf and worker types - #978
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
Codex review: needs maintainer review before merge. Reviewed August 10, 2026, 10:40 PM ET / August 11, 2026, 02:40 UTC. ClawSweeper reviewWhat this changesUpdates indirect Google protobuf/genproto modules and the email-tracking worker’s Cloudflare type declarations, with corresponding Go and pnpm lockfile records. Merge readinessKeep open for normal collaborator review: this is a narrow, policy-aware dependency refresh that is not yet on current main, and the remaining hosted checks should complete on the reviewed head before it lands. Priority: P3 Review scores
Verification
How this fits togetherThe gog CLI builds against Google API libraries whose generated protobuf types are resolved through Go modules. Its email-tracking Worker separately type-checks TypeScript against Cloudflare declarations before Wrangler prepares the deployment bundle. flowchart LR
A[Google API libraries] --> B[Go module graph]
B --> C[gog CLI build]
D[Worker TypeScript source] --> E[Cloudflare worker type declarations]
E --> F[Worker typecheck]
F --> G[Wrangler deployment bundle]
Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Merge the narrowly pinned update after the remaining required checks succeed for the reviewed commit. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR updates build-time dependency records rather than addressing a reported runtime failure; its body supplies build, lint, dry-run, and test validation. Is this the best way to solve the issue? Yes. Updating only the resolved package versions and matching lockfile integrity records is the narrow maintainable path, while retaining the Worker workspace’s minimum-release-age policy. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 71c6c1e63787. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
Summary
Wrangler 4.120.1 is visible upstream but intentionally remains excluded until it satisfies the worker's 24-hour
minimumReleaseAgesupply-chain policy.Proof
make cimake build && ./bin/gog versioncorepack pnpm lintcorepack pnpm build— Wrangler dry-run completed; 11.06 KiB upload / 3.56 KiB gzipcorepack pnpm test— 3 files, 15 tests passed