feat(desktop): add development IPC traffic diagnostics #231

Merged
admin merged 5 commits from feat/electron-ipc-diagnostics into dev 2026-10-14 09:59:06 +00:00
Owner

Closes #230

Summary

  • Log main/preload IPC invokes and their request, response/error, and elapsed time in the Console of the same Electron window's DevTools.
  • Include daemon snapshot and gRPC stream data/end/error events.
  • Redact credential-like fields and bearer/query secrets; bound long strings, arrays, object fields, and binary payloads.
  • Keep logging development-only, with SISYPHUS_IPC_LOG=0 as an opt-out.
  • Keep DevTools closed at launch and add development-only F12, Ctrl+Shift+I, and Cmd+Option+I toggles.

Verification

  • npm test (78 passed)
  • npm run typecheck
  • npm run build
  • Focused sanitizer tests cover redaction and payload bounds.

The build retains existing Vite use client/sourcemap warnings and large-bundle notices.

Closes #230 ## Summary - Log main/preload IPC invokes and their request, response/error, and elapsed time in the Console of the same Electron window's DevTools. - Include daemon snapshot and gRPC stream data/end/error events. - Redact credential-like fields and bearer/query secrets; bound long strings, arrays, object fields, and binary payloads. - Keep logging development-only, with `SISYPHUS_IPC_LOG=0` as an opt-out. - Keep DevTools closed at launch and add development-only F12, Ctrl+Shift+I, and Cmd+Option+I toggles. ## Verification - `npm test` (78 passed) - `npm run typecheck` - `npm run build` - Focused sanitizer tests cover redaction and payload bounds. The build retains existing Vite `use client`/sourcemap warnings and large-bundle notices.
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Sisyphus/Sisyphus#231