ci: clear v0.1.9 release PR's audit + prettier failures#21
Conversation
The v0.1.9 release PR had two non-gating CI failures. Security Audit (`npm audit --omit=dev --audit-level=high`): posthog-js → @opentelemetry/exporter-logs-otlp-http → @opentelemetry/otlp-transformer pulled in protobufjs ^7.3.0, which resolved to 7.5.5 — vulnerable to a high severity advisory (GHSA-66ff-xgx4-vchm and others, all fixed in 7.5.6). No OpenTelemetry release advances the floor yet, so we use an npm override to force the whole tree onto protobufjs ^7.5.8 (latest 7.x). Prettier (`npm run format:check`): SetupWizard.tsx had two lines that exceeded the print width after the notification-permission step landed. Auto-formatted. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Two more findings while looking at why the v0.1.9 draft release was missing Root cause of the missing updater pair
impl Default for Updater {
fn default() -> Self {
Self::Bool(false)
}
}Our The Same bug hit v0.1.6, v0.1.7, v0.1.8, v0.1.9 — every release since v0.1.5. v0.1.7/v0.1.8 shipped because someone manually uploaded the missing files (e.g., v0.1.8's x86_64 leg removedThe Intel matrix leg has not landed a release artifact since v0.1.6:
GitHub's Verification plan after merge
|
Summary
The v0.1.9 release PR (#20) shipped with two non-gating CI failures. This PR clears both so the next release is green.
posthog-jspulls in@opentelemetry/exporter-logs-otlp-http→@opentelemetry/otlp-transformer, which depends onprotobufjs ^7.3.0. That resolved to 7.5.5, which is affected by the GHSA-66ff-xgx4-vchm family of high-severity advisories (all fixed in7.5.6). No OpenTelemetry release advances the floor yet, so this adds an npmoverridesentry forcing the whole tree ontoprotobufjs ^7.5.8(latest 7.x).SetupWizard.tsxhad two lines past print-width after the notification-permission step landed in feat(onboarding): add notification permission step to setup wizard #19. Re-ranprettier --writeon the file.After the change locally:
The lockfile diff also reflects the
version: "0.1.8" → "0.1.9"bump that #20 missed, plus a couple of@protobufjs/*sub-packages that float along withprotobufjsitself.Test plan
Security Auditjob goes greenLint & Formatjob goes greenType Checkstays green🤖 Generated with Claude Code