Skip to content

chore(deps): bump dotenv 18, workers-types 5, and recharts 3 - #202

Open
jonbeckman wants to merge 3 commits into
masterfrom
cursor/bump-dotenv-workers-types-recharts-80af
Open

jonbeckman wants to merge 3 commits into
masterfrom
cursor/bump-dotenv-workers-types-recharts-80af

Conversation

@jonbeckman

@jonbeckman jonbeckman commented Oct 22, 2026 •

Copy link
Copy Markdown
Owner

Summary

Replaces closed Dependabot pull requests #170, #171, and #172. Those updates changed package specs and left nub.lock stale, so CI failed with ERR_NUB_OUTDATED_LOCKFILE. This pull request applies the same dependency updates, regenerates nub.lock, and keeps the dashboard charts working on Recharts 3.

Dependency bumps

nub install regenerated nub.lock. A frozen install of that lockfile succeeds.

Recharts 3 code changes

  • Removed @ts-nocheck from the uptime chart, both latency charts, and the shared chart wrapper, and included those files in the app TypeScript project.
  • Status legends are HTML next to the uptime and latency charts. Recharts 3 Legend omits the payload prop, and its size reporting looped inside ResponsiveContainer.
  • Tooltips render after the series so they stay above the bars, area, and line.
  • The shared tooltip formatter uses the Recharts 3 signature. The fifth argument is the tooltip payload array. Item keys are strings, because dataKey can be a function. Non-numeric values render as text.
  • The custom uptime bar is typed from the chart row. connectNulls stays on the latency range area; Recharts 3.10.1 still skips null points.
  • The monitor detail page depends on the search string, so the loading overlay clears and tooltips receive pointer events.

A .tegami/ patch entry covers solstatus (dotenv) and @solstatus/app (recharts and the raised floors). workers-types and the infra dotenv pin are devDependencies.

Validation

  • nub run typecheck passed (root, common, api, app, infra).
  • nub run lint passed (exit 0; existing warnings only).
  • nub run format passed (oxfmt --check, 130 files).
  • nub run test passed (7 files, 18 tests).
  • nub run --filter '@solstatus/app' build passed (Vite 8.3.4 client and SSR).
  • Browser check of the mock detail page: uptime and latency charts render, legends stay visible, tooltips show status counts and latency, and the 30m and 1h ranges refresh and then clear the loading overlay.

recharts-detail-charts.mp4

Checklist

  • I added or updated tests for behavior changes, or explained why tests are not needed. Existing unit tests cover the CLI and API. Chart behavior was checked in the browser; no new automated chart test was added.
  • I updated documentation and config examples when behavior or configuration changed. No config or API contract changed. Dependency floors live in the package manifests.
  • I added a .tegami/ release entry, or explained why this pull request uses release:none. Added .tegami/2026-10-22-deps-dotenv-workers-recharts.md as a patch for solstatus and @solstatus/app.
  • I ran nub run typecheck, nub run lint, nub run format, and nub run test.
  • I did not include credentials, private data, internal URLs, or organization-specific defaults.
  • I did not change production deploys, npm publish, or paid spend.
Open in Web Open in Cursor 

Note

Medium Risk
Recharts 2→3 is a major charting upgrade with UI/layout changes; dotenv and workers-types bumps are lower risk but touch CLI env loading and Cloudflare typings repo-wide.

Reviewed by Cursor Bugbot for commit 7b1ee78. Bugbot is set up for automated code reviews on this repo. Configure here.

cursoragent and others added 3 commits October 22, 2026 21:07
Raise the floors that the lockfile already resolved for Vite 8.3,
date-fns 4.4, and the dropdown menu, and regenerate nub.lock.

Co-authored-by: Jonathan Beckman <jonathantbeckman@gmail.com>
Draw status legends outside Recharts, type the tooltip content for
the v3 payload, and stop the monitor page from refetching on every
render. The CLI loads dotenv 18 without an injection line.

Co-authored-by: Jonathan Beckman <jonathantbeckman@gmail.com>
Co-authored-by: Jonathan Beckman <jonathantbeckman@gmail.com>
@cursor

cursor Bot commented Oct 22, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: 9625ed49-c73c-478e-a5a5-312bd6a406ff)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Bugbot was skipped, so that signal was not used, and no applicable approval policy or existing review requires human review.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@github-actions

Copy link
Copy Markdown
Contributor

Tegami

This repository uses Tegami to manage releases. When your changes affect published packages, add a changelog file under .tegami/ before merging.

Create a changelog → · Changelog format

Release preview

Package Bump Version
@solstatus/common major 3.0.0 → 4.0.0
solstatus patch 3.0.0 → 3.0.1
@solstatus/api patch 3.0.0 → 3.0.1
@solstatus/app patch 3.0.0 → 3.0.1
@solstatus/infra patch 3.0.0 → 3.0.1

Changelogs in this PR

Changelog Title
2026-10-22-deps-dotenv-workers-recharts.md Update dotenv and dashboard chart dependencies

Run nub run tegami locally to create a changelog interactively.

Managed by Tegami.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants