Repository navigation
deps: batch soaked Dependabot updates (2026-09-30) - #1942
Conversation
Go: pgx/v5 5.11.0, klauspost/compress 1.20.1, modernc.org/sqlite 1.59.0 (libc 1.75.7), k8s.io/* 0.37.1 aligned across root and pkg modules. npm: eslint 10.11.0, vite 8.3.1 (rolldown 1.2.11), lucide-react 1.48.0.
PR Summary by QodoBatch soaked Go and frontend dependency updates
AI Description
Diagram
High-Level Assessment
Files changed (6)
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can route each severity your way: inline, summary, both, or drop |
One tested batch replacing this week's soaked, low-risk Dependabot PRs. Dependency-only: no application source changes.
Screening time: 2026-09-30T08:40:02Z (re-checked 09:12Z, base
0490dd87unchanged)72h cutoff: published at or before 2026-09-27T08:40:02Z. Every resolved version below clears it.
Included
streamingv0.37.1 is the same commit as v0.37.0.stdlib. See notes.>=0.400.0is unchanged.Movement beyond the Dependabot PRs (reviewed, all soaked)
k8s.io/kubectlandk8s.io/csi-translation-lib(indirect, via./pkg) and all ofpkg/go.mod's k8s.io requirements move to v0.37.1 as well. Without this, deps(go): bump the k8s group with 7 updates #1932 would leave the graph mixing v0.37.0 and v0.37.1.k8s.io/{code-generator, kms, metrics}v0.37.1 were published 2026-09-23 21:22 / 21:45 / 23:18Z. None of them appears in go.sum.rolldownand all 15@rolldown/binding-*1.2.6 → 1.2.11 (2026-09-24 13:54–14:30Z)@oxc-project/types0.147.0 → 0.151.0 (2026-09-21)picomatch4.0.5 → 4.0.7 (2026-08-24)package-lock.jsonchanged.Held / excluded (these PRs stay open)
^22.12 || ^24 || >=26, butci.ymlpins Node 20 for the Frontend, k8s-ui and Settings jobs. The PR's CI only passed because npm treats the engine mismatch as a warning (EBADENGINE), which Bugbot also flagged. Follow-up: move CI (and CONTRIBUTING's "Node 20+") to Node 22 first. Note thatweb/runs the hoisted vitest without declaring it.packages/k8s-ui/src/components/ui/monacoRuntime.ts(esm/vs/editor/contrib/{find,folding,format,gotoError,hover,suggest}/...) no longer resolve (TS2882), which fails the Frontend, k8s-ui and Settings jobs.TestInvestigationHandlerAnnotatesRealToolCallWithoutChangingPublicContractfails: "private mount handshake did not mark the scope connected". Radar's private-mount handling needs adapting first..tgzrelease assets (#648), so helm-unittest ends up installed twice ("two plugins claim the name "unittest""). It also moved to the node24 runtime (v2.4.0). Needs aci.ymlchange to howhelm-pluginsis specified.Radar usage and risk notes
+in URI query values, bad percent-encoding now errors,#is data, the last repeated parameter wins, bare IPv6 hosts need brackets, and backslashes in keyword/value strings now escape.RADAR_TIMELINE_POSTGRES_DSNvalues with unusual encoding, which is worth a release-note line.Rows.TypeMapinterface method doesn't matter because Radar implements no custompgx.Rows.RADAR_REQUIRE_POSTGRES_TESTS=1against postgres:17.internal/server/compress.go(kgzip.NewWriterLevel); zstd reaches Radar only transitively (helm, go-containerregistry, prometheus). In the smoke test, a live/api/resources/podsresponse came backContent-Encoding: gzipand decoded with systemgunzipto 269 pods.ai-runs.db. libc is exact-pinned to the version sqlite requires.manualChunksdoes its ownnode_modules/matching, so vite 8.3.0's path-segment change doesn't affect it.@vitejs/plugin-react6.1.1 still satisfies its peer range, and vitest 4.1.11'svitepeer covers ^8.trash(Radar usesTrash2) and glyph redraws forTrash2andBuilding2. That's cosmetic; tests that assert icon class names pass.Verification
go mod verifypasses in both root andpkg.go.sum/pkg/go.sumwere regenerated from main with a cleango mod tidy. Each changes 30 / 14 lines each way, only the modules listed above.make tscpasses.cd web && npm run lintgives 0 errors and 441 warnings (the same as main).packages/k8s-uinpm test: 209 files, 4030 passed, 1 skipped.webnpm run test: 151 files, 1764 passed.cd pkg && go test ./...passes.go test ./...passes exceptcmd/desktop. ItsTestGetShellEnvandTestEnrichEnvPrecedenceAndDiagnosticshit their 5s login-shell timeout while the full parallel suite was loading the machine.go test ./cmd/desktop/ -count=3, the tests pass, and they also pass on main.make buildpasses.monacoRuntime,yamlMonacoRuntime, themonacoYaml.workerandeditor.workerchunks, and showed the pod YAML./visual-test. The targeted browser smoke test above covers the rendering-relevant bumps (vite/rolldown chunking, Monaco loading, lucide icons).Supersedes #1932, #1938, #1936, #1934, #1937, #1935, #1931
Note
Low Risk
Patch dependency upgrades with no app code changes; pgx DSN parsing edge cases are the main operational note for unusual Postgres connection strings.
Overview
Dependency-only batch — no application source changes. Bumps Go and npm lockfiles after screening several Dependabot PRs.
Go: Aligns all
k8s.io/*modules to v0.37.1 in the root module,./pkg, and checksums (patch tag bumps only). Also bumps pgx v5.11.0 (Postgres timeline viaRADAR_TIMELINE_POSTGRES_DSN), klauspost/compress v1.20.1 (HTTP gzip ininternal/server/compress.go), and modernc.org/sqlite v1.59.0 with libc v1.75.7 (SQLite timeline /ai-runs.db).Frontend: vite 8.3.1 (pulls rolldown 1.2.11 and related
@rolldown/binding-*/@oxc-project/typestransitives inpackage-lock.json), eslint 10.11.0, and lucide-react 1.48.0 inweb/andpackages/k8s-ui.Reviewed by Cursor Bugbot for commit 7512c17. Bugbot is set up for automated code reviews on this repo. Configure here.