Skip to content

Remove session socket debug traces - #183

Merged
jonbeckman merged 3 commits into
masterfrom
cursor/remove-session-socket-debug-logs-ed62
Nov 9, 2026
Merged

jonbeckman merged 3 commits into
masterfrom
cursor/remove-session-socket-debug-logs-ed62

Conversation

@jonbeckman

Copy link
Copy Markdown
Contributor

Summary

Removes console.log traces from the session WebSocket hook. They recorded subscribe, connect, close, and reconnect chatter, and they printed the prompt text. Connection state, reconnect timing, and the rendered session do not use those logs.

console.error stays on the failure paths: token fetch, parse errors, a socket that is not open, and reconnect exhaustion.

release:none: no user-visible session behavior changes.

Validation

  • nub run typecheck
  • nub run lint
  • nub run format
  • nub run test was not run. The hook's state updates are unchanged, and no end-to-end step asserts on browser console output.

Checklist

  • I added or updated tests for behavior changes, or explained why tests are not needed.
  • I updated documentation and config examples when behavior or configuration changed. (No documented behavior changed.)
  • I added a .tegami/ release entry, or explained why this pull request uses release:none.
  • I ran nub run config:check, nub run typecheck, nub run lint, nub run format, and nub run test. (config:check and test were not run; see Validation.)
  • I did not include credentials, private data, internal URLs, or organization-specific defaults.
  • I updated THIRD_PARTY_NOTICES.md for new third-party code or separately licensed assets. (No new third-party code.)
Open in Web Open in Cursor 

These console.log calls record connection chatter and the prompt text. They do not change socket state, reconnects, or what the page renders. console.error stays on the failure paths.

Co-authored-by: Jonathan Beckman <jonathantbeckman@gmail.com>
@jonbeckman jonbeckman added the release:none label Nov 8, 2026 — with Cursor
@jonbeckman
jonbeckman marked this pull request as ready for review November 8, 2026 09:28
@github-actions

github-actions Bot commented Nov 8, 2026 •

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

No changelogs yet

This PR has no pending changelog files. If your changes require a release, add a changelog before merging.

Run nub run tegami locally to create a changelog interactively.

Managed by Tegami.

@github-actions

github-actions Bot commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

🟢 e2e solzero-core: 4 passed, 27 left out by filters

1 agent step · 1 replayed from cache

Change 1 file changed since 487a836
Filter decision kept 3 of 30 tests, left out 27 (1 call, 16.6k tokens)

27 tests left out by filters
Test Filter Reason
tests/accounts.e2e.ts › create, use and revoke an isolated user API key decision 5% likely affected
tests/accounts.e2e.ts › Better Auth 1.7 account APIs select the local account row id decision 5% likely affected
tests/accounts.e2e.ts › Accounts settings shows GitHub as unlinked without OAuth providers decision 5% likely affected
tests/admin.e2e.ts › administrator pages show live control-plane configuration decision 5% likely affected
tests/advanced-sessions.e2e.ts › incognito sessions stay outside normal lists but open directly decision 17% likely affected
tests/api-contract.e2e.ts › exported API key and session schemas decode actual Worker responses decision 7% likely affected
tests/auth.e2e.ts › sign-in gate protects every application route decision 5% likely affected
tests/auth.e2e.ts › credential sign-in opens the requested settings page decision 8% likely affected
tests/auth.e2e.ts › anonymous callers cannot access any protected API group decision 1% likely affected
tests/auth.e2e.ts › sign-out removes authenticated UI access decision 6% likely affected
tests/bots.e2e.ts › create a bot and manage a temporary routine through the UI decision 5% likely affected
tests/branding.e2e.ts › the selected deployment brand reaches the browser decision 2% likely affected
tests/branding.e2e.ts › health identifies the configured application without exposing configuration decision 2% likely affected
tests/branding.e2e.ts › OpenAPI uses the selected deployment brand decision 1% likely affected
tests/session-transfer.e2e.ts › admin authentication transfer redeems once and retains its deep link decision 5% likely affected
tests/sessions.e2e.ts › a second account cannot read or mutate another account's session decision 10% likely affected
tests/sessions.e2e.ts › opencode executes a real prompt and persists its output decision 11% likely affected
tests/sessions.e2e.ts › codex executes a real prompt and persists its output decision 7% likely affected
tests/settings.e2e.ts › all settings categories load real data decision 5% likely affected
tests/settings.e2e.ts › runtime default persists after reload decision 9% likely affected
tests/settings.e2e.ts › MCP deep link and legacy settings URL open the MCP tab decision 5% likely affected
tests/settings.e2e.ts › secret editor creates, finds and deletes a disposable secret decision 5% likely affected
tests/settings.e2e.ts › theme preference survives reload decision 5% likely affected
tests/skills.e2e.ts › global skill defaults and per-user preferences persist independently decision 5% likely affected
tests/workflows.e2e.ts › workflow authoring, rename, export, disable and archive persist decision 5% likely affected
tests/workflows.e2e.ts › manual workflow executes JavaScript and stores a real R2 artifact decision 5% likely affected
tests/workflows.e2e.ts › invalid workflow definitions are rejected before storage decision 5% likely affected
All 4 tests in 3 files
Test Agent Time
🟢 tests/advanced-sessions.e2e.ts · 1 passed 4.1s
🟢 session MCP definitions and subagent preferences persist 4.1s
🟢 tests/auth.setup.e2e.ts · 1 passed 5.9s
🟢 authenticate isolated admin 5.9s
🟢 tests/sessions.e2e.ts · 2 passed 1 step 14.1s
🟢 session tools, archive and restoration persist through the real Worker 1 step 5.8s
🟢 isolate executes a real prompt and persists its output 8.4s

e2e 0.21.3 · 1m 17s · local-cloudflare · run artifacts

cursoragent and others added 2 commits November 8, 2026 09:31
Co-authored-by: Jonathan Beckman <jonathantbeckman@gmail.com>
Co-authored-by: Jonathan Beckman <jonathantbeckman@gmail.com>
@jonbeckman
jonbeckman merged commit 8790609 into master Nov 9, 2026
14 of 15 checks passed
@jonbeckman
jonbeckman deleted the cursor/remove-session-socket-debug-logs-ed62 branch November 9, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants