Conversation
Resolves 5 advisories reported by `pnpm audit` (3 high, 1 moderate, 1 low),
all transitive: ws (DoS + uninitialized memory disclosure) via viem,
brace-expansion (DoS x2) via eslint/typescript-eslint, and esbuild
(arbitrary file read in dev server) via tsx. Audit is now clean.
- viem 2.48.11 -> 2.55.8
- @grafana/grafana-foundation-sdk 0.0.12 -> 0.0.18
- @typescript-eslint/{eslint-plugin,parser} 8.59.3 -> 8.65.0
- eslint 10.3.0 -> 10.7.0
- tsx 4.21.0 -> 4.23.1 (esbuild 0.27.7 -> 0.28.1)
Supply chain: raise minimumReleaseAge from 3 to 7 days (10080 minutes) and
copy pnpm-workspace.yaml into the Docker build context so the policy also
applies to image builds.
Drop the `pnpm.onlyBuiltDependencies` field from package.json: pnpm 11 no
longer reads it (it warned on every command) and the setting already lives
in `allowBuilds` in pnpm-workspace.yaml.
Grafana SDK 0.0.18 started emitting its own timeseries `options` defaults
with showLegend: false, which would have hidden the legend on all six
timeseries panels. Set the legend explicitly in timeseriesPanel() to keep
the previous rendering.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The previous run aborted with "Your provider API key hit its rate limit." without posting any review. Empty commit to re-run the check on PR #5. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (6 files)
Reviewed by gpt-5.6-terra · Input: 144.7K · Output: 6.5K · Cached: 965.4K |
iicc1
approved these changes
Jul 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release v1.1.1 — security patch
Patch bump per the versioning rules in
.cursor/rules/project-overview.mdc(dependency bumps, no behaviour change).Security
pnpm auditreported 5 advisories (3 high, 1 moderate, 1 low), all transitive. All are resolved — audit is now clean. This also clears the 4 Dependabot alerts currently open onmain.wsviemwsviembrace-expansion×2eslint,typescript-eslintesbuildtsxDependency updates
viem2.48.11 → 2.55.8@grafana/grafana-foundation-sdk0.0.12 → 0.0.18@typescript-eslint/{eslint-plugin,parser}8.59.3 → 8.65.0eslint10.3.0 → 10.7.0tsx4.21.0 → 4.23.1 (esbuild 0.27.7 → 0.28.1)viem 2.55.10andeslint 10.8.0were deliberately left out: the release-age policy below blocks them for being too recent. They will land on the next update pass.Supply chain
minimumReleaseAgeraised from 3 to 7 days (10080 minutes) inpnpm-workspace.yaml.Dockerfilenow copiespnpm-workspace.yamlinto the build context, so the policy applies to image builds too — it previously did not.pnpm.onlyBuiltDependenciesfield frompackage.json: pnpm 11 no longer reads it (it warned on every command) and the setting already lives inallowBuilds.Grafana dashboard
SDK 0.0.18 started emitting its own timeseries
optionsdefaults withshowLegend: false, which would have silently hidden the legend on all six timeseries panels.timeseriesPanel()now sets the legend explicitly. Verified with a key-order-normalized diff against the previousdashboard.json: the only change is the addedoptionsblock, matching Grafana's own defaults.Verification
pnpm run test:all— 49 + 8 tests passpnpm run lint— cleanpnpm run grafana:build— committed dashboard matches generated outputdocker build— succeedspnpm audit— no known vulnerabilitiesAfter merge
Tag
v1.1.1onmainto trigger the Docker Hub publish (stakely/stvaults-watcher:v1.1.1+:latest).🤖 Generated with Claude Code