chore(deps): refresh compatible workspace dependencies - #87
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: needs maintainer review before merge. Reviewed September 5, 2026, 7:59 AM ET / 11:59 UTC. ClawSweeper reviewWhat this changesUpdates compatible workspace dependencies, synchronizes Biome’s configuration schema, and regenerates the pnpm lockfile. Merge readiness✅ Ready for maintainer review This remains a useful landing candidate: the refresh is absent from pinned main, and review found no blocking correctness or security defect. Priority: P3 Review scores
Verification
How this fits togetherWorkspace dependencies support Krillswitch’s operator dashboard, Cloudflare Worker, SDKs, and development tools. The Worker validates evaluation requests and reads D1 configuration to return flag values to applications. flowchart LR
A[Workspace manifests] --> B[pnpm lockfile]
B --> C[Dashboard and build tools]
B --> D[Worker runtime]
E[Evaluation requests] --> D
F[D1 flag configuration] --> D
D --> G[Flag values for applications]
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Adopt the bounded refresh while keeping authentication migrations and runtime-support changes in separately reviewed work. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR refreshes dependencies rather than reporting a specific broken behavior. Is this the best way to solve the issue? Yes. Synchronized manifests and a generated lockfile provide a focused update without mixing in schema migrations or a higher runtime support floor. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 59f724c8d5aa. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (14 earlier review cycles; latest 8 shown)
|
What Problem This Solves
Refreshes the workspace's compatible npm dependencies while preserving the current authentication schema, SDK contracts, Node support declaration, and two-day dependency release cooldown.
Why This Change Was Made
The refresh uses pnpm 11.24.0 and its generated lockfile. No direct dependencies were added or removed, and no application source or CI assertions changed.
@biomejs/biomeand config schema@tanstack/react-queryreact-router@vitejs/plugin-reactzod@cloudflare/vitest-plugin@cloudflare/workers-typeswrangler@types/node@testing-library/reactThe lockfile also refreshes compatible transitive packages, including workerd, Miniflare, Rolldown, and sourcemap-codec. All twelve GitHub Actions dependencies were checked against upstream release tags and peeled commit SHAs; their existing pins are current. pnpm and Carapace are current too.
Deferred updates:
Account.issuerand an identity backfill. The current D1 schema has no issuer column. Keep runtime and schema-generator CLI at 1.6.30 and handle the migration with a separately reviewed data-preservation and rollout plan. This also affects the Better Auth update proposed by chore(deps): bump the production group with 2 updates #86; this PR overlaps its React Query update only.^22.22.2 || ^24.15.0 || >=26.0.0, which narrows the repository's declared>=22.22.0support. Keep the direct React test dependency at 29.1.1 pending a deliberate support-floor change. Existing jsdom 30 optional peer snapshots elsewhere in the lockfile predate this PR.User Impact
No intentional user-visible behavior change, public API change, or database migration. This is dependency maintenance, so no new changelog entry is added.
Evidence
Local validation used Node 26.8.1 and pnpm 11.24.0:
All passed. Lint reported six warnings and no errors. Full suite output excerpts:
Live local proof
The existing migrations and synthetic seed were applied only to local D1. A built dashboard and local Worker were started with these commands (the auth secret below is a disposable local fixture):
Actual response:
A local Node integration script exercised conditional HTTP requests, rejected unauthenticated management access, invoked the compiled SDK evaluator against the running Worker, and ran the built CLI using an ephemeral local viewer token. The token was kept in process memory, passed through the child environment, and revoked afterward. The CLI command was:
node packages/cli/dist/index.js eval -p clawhub -e development --key dependency-refresh --attr role=admin --jsonActual integration output excerpts:
Actual CLI output excerpt:
{ "flags": { "souls": { "value": true, "variationId": "var_souls_on", "reason": { "kind": "default" } }, "theme": { "value": "dark", "variationId": "var_theme_dark", "reason": { "kind": "rule", "attribute": "role" } } } }The excerpt omits two additional fixture flags. The integration finished with
Live proof passed.The dashboard also returned HTTP 200.CI reasoning
Default-branch build/test CI was already green at
59f724c: CI run. Its jobs cover workflow lint, lint, typecheck, tests, and builds. CodeQL is a separate security analysis workflow, also green: scheduled CodeQL run. Stale and ClawSweeper Dispatch are operational automation, not build/test proof. Docs publishes on main; Cloudflare deployment is manual and SDK publication is tag-triggered. This PR does not deploy, publish, or alter production.The API suite was also repeated using the installed Node 24.20.0 runtime, matching CI's Node major:
Both successful Worker runs printed two non-failing
Error: internal errordiagnostics; neither run failed an assertion or exited unsuccessfully. No tests, assertions, or jobs were weakened.Codex autoreview completed with no actionable findings:
.agents/skills/autoreview/scripts/autoreview --mode local --engine codex --model gpt-5.6-sol --thinking highThe invocation additionally supplied scope context with
--promptcovering the deferred updates and unchanged contracts. Result:Final checks for commit
0a0649f4ed4e6ad3e9f5bef9561a4eea8bbc9e2c: CI passed with all five jobs green (workflow lint, lint, typecheck, test, build), and CodeQL passed. No reruns were required.pnpm install --frozen-lockfilealso passed locally withAlready up to date.