Skip to content

fix(web): resource monitor no longer claims to sample every 0 ms - #1

Open
M0xoo wants to merge 1998 commits into
mainfrom
fix/diagnostics-on-demand-sampling
Open

M0xoo wants to merge 1998 commits into
mainfrom
fix/diagnostics-on-demand-sampling

Conversation

@M0xoo

@M0xoo M0xoo commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Problem

When the native monitor is idle or unavailable, Settings → Diagnostics → Resource monitor says "Sampling every 0 ms" next to a green live dot. The server reports sampleIntervalMs: 0 in that state (IDLE_SAMPLE_INTERVAL_MS = 0 in apps/server/src/resourceTelemetry/NativeTelemetryClient.ts), which means it samples on request, not every 0 ms. The Diagnostics page's own Interval stat already reads 0 as "On demand" (DiagnosticsSettings.tsx). Only the resource monitor line took it literally.

Change

  • The formatter moves to ResourceTelemetryDiagnostics.logic.ts as formatSamplingStatus. It returns "Sampling on demand" for 0; every other interval reads as before.
  • The status dot is green only while the server samples on an interval. Otherwise it's muted.

Before / after

Settings → Diagnostics at 1440px. Before on the left, after on the right; light on top, dark below.

Sampling status before and after

Verification

New case in the existing ResourceTelemetryDiagnostics.logic.test.ts for 0, 500 and 2000 ms.

On main's logic:

vp test run src/components/settings/ResourceTelemetryDiagnostics.logic.test.ts
AssertionError: expected [ 'Sampling every 0 ms', …(2) ] to deeply equal [ 'Sampling on demand', …(2) ]
      Tests  1 failed | 1 passed (2)

With the fix:

      Tests  2 passed (2)

vp run check:changed: format, lint, knip, typecheck (2 packages), tests and the react compiler check all pass. I took the screenshots with vp run dev (isolated worktree state, native monitor unavailable) in headless Chrome, swapping main's files in and out on the same server.

Surfaces: only the web/desktop Diagnostics page shows this line. Mobile has no diagnostics view. Not checked: the desktop (Electron) shell, which renders the same component.

Model and harness: Claude Opus 5.5 in Claude Code.

🤖 Generated with Claude Code

Tabonx and others added 30 commits November 13, 2026 18:50
…ingdotgg#31146)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…ged revalidation (pingdotgg#23775)

Signed-off-by: Shaswat Raj <shaswat340@gmail.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…ell (pingdotgg#33064)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: lyn1dev <lyn1dev@users.noreply.github.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
… local port (pingdotgg#24247)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…pingdotgg#23743)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: only21mil <only21mil@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…omposer (pingdotgg#26692)

Co-authored-by: nibarAhmed <nibarahmed0@gmail.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…hite (pingdotgg#34569)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…dotgg#36290)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…hes (pingdotgg#36477)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…gg#36504)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Grok 4.7 <noreply@x.ai>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…n launch (pingdotgg#36042)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…g#36250)

Co-authored-by: Roger <roger@Rogerflix.local>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…dotgg#35907)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: SAPHID <saphid@users.noreply.github.com>
Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…tgg#35617)

Co-authored-by: Simone <185146821+Lucenx9@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
M0xoo and others added 22 commits November 14, 2026 07:00
…every chunk (pingdotgg#29422)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…ngdotgg#27458)

Co-authored-by: maximkes <22573701+maximkes@users.noreply.github.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…es the refusal (pingdotgg#28360)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…28518)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
… status (pingdotgg#29855)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…otgg#29628)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…ead of running forever (pingdotgg#29208)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…itions (pingdotgg#29612)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
… diff (pingdotgg#29629)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…der (pingdotgg#29846)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…oved (pingdotgg#29699)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…es (pingdotgg#29867)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…36785)

Co-authored-by: Alex Southwell <4596216+saphid@users.noreply.github.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…changes (pingdotgg#36786)

Co-authored-by: Alex Southwell <4596216+saphid@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
… 09:00 (pingdotgg#36789)

Co-authored-by: Alex Southwell <4596216+saphid@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ingdotgg#29802)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…tgg#29395)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Keep the fix to the on-demand sampling label and leave the status dot
as it is on main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted Author is trusted by repo permissions or the VOUCHED list. size:XXL labels Nov 14, 2026
saphid and others added 6 commits November 14, 2026 07:36
…6793)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…s load or after a network blip (pingdotgg#28708)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
…ngdotgg#28945)

Co-authored-by: maria-rcks <maria@kuuro.net>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
…mand-sampling

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The dot stayed green while the Native badge beside it reported the
monitor unavailable. Use the badge's neutral dot unless the server
samples on an interval.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted Author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.