chore(deps): update dependencies - #139
Conversation
|
Codex review: needs maintainer review before merge. Reviewed August 2, 2026, 3:30 AM ET / 07:30 UTC. ClawSweeper reviewWhat this changesRefreshes Go, web, desktop, Cloudflare, package-manager, container, and GitHub Actions dependencies, then regenerates ClickClack’s embedded web bundle. Merge readinessThis PR remains necessary: current main still uses the older pnpm, container, action, web, desktop, and Go dependency pins. The prior Docker-pnpm mismatch is resolved on the latest head, but Socket’s new warning for the SQLite dependency chain needs a maintainer-aware supply-chain decision before merge. Priority: P2 Review scores
Verification
How this fits togetherClickClack compiles its Svelte web client into tracked files embedded in the Go API binary; Docker and GitHub Actions build that same client using pinned Node and pnpm versions. This PR updates those dependency inputs and the resulting browser assets delivered by the API server. flowchart LR
A[Dependency manifests] --> B[Package and Go resolvers]
B --> C[Web and desktop builds]
E[Docker and CI toolchain pins] --> C
C --> D[Embedded web bundle]
D --> F[Go API binary]
F --> G[Browser and API clients]
Decision needed
Why: The alert is concrete but does not prove a defect that an automated repair can safely fix; accepting, investigating, or declining the upstream dependency is a maintainer supply-chain decision. Before merge
Findings
Agent review detailsSecurityNeeds attention: The patch has no demonstrated code-level security defect, but Socket flags a newly introduced transitive SQLite dependency for likely obfuscated code and an owner should explicitly assess that warning before merge. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Keep the aligned dependency and generated-bundle refresh, but merge only after an owner reviews the modernc.org/libc warning and accepts it as expected upstream generated code or replaces the SQLite update with an acceptable version. Do we have a high-confidence way to reproduce the issue? Not applicable: this is a dependency-refresh PR, so the relevant proof is real build and runtime validation rather than reproduction of a reported defect. The PR body provides a SQLite-backed executable run with successful root and session HTTP responses. Is this the best way to solve the issue? Yes, subject to the supply-chain decision: aligning the declared package-manager, Docker, and CI pins avoids split build toolchains, and regenerating the embedded assets is the repository’s established delivery path. AGENTS.md: found, but no applicable review policy affected this item. Codex review notes: model internal, reasoning high; reviewed against 3205c07468ab. LabelsLabel changes:
Label justifications:
EvidenceSecurity concerns:
What 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
HistoryReview history (4 earlier review cycles)
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
Summary
mainThe three Fontsource packages remain on their current releases because their 5.3.0 artifacts replace tracked WOFF2 binaries, which the repository's required pre-commit review deliberately refuses to accept as unreviewable binary content.
Proof
npx -y pnpm@11.19.0 run checknpx -y pnpm@11.19.0 run buildclickclackexecutable, started it against SQLite with dev bootstrap, and fetched both/and/api/v1/session: HTTP 200 (6,273 and 5,989 bytes)start.CfWZRA5o.js,app.B4wbC3r8.js, and0.DA0bPGL_.js, matching the regenerated production bundleThe first exact-head CI attempt had a single spreadsheet-viewer timeout while 133 tests passed. The focused viewer path passed locally, the full local suite passed, and the unchanged-head rerun passed all jobs.
No release or tag is included.