Skip to content

fix(web): delay thread shortcut hints by 200 ms - #8172

Merged
t3dotgg merged 4 commits into
mainfrom
t3code/fix-sticky-command-thread-hint
Aug 25, 2026
Merged

fix(web): delay thread shortcut hints by 200 ms#8172
t3dotgg merged 4 commits into
mainfrom
t3code/fix-sticky-command-thread-hint

Conversation

@t3dotgg

@t3dotgg t3dotgg commented Aug 25, 2026

Copy link
Copy Markdown
Member

Thread shortcut hints appeared immediately during brief Command presses from dictation and other shortcuts.

Wait 200 ms before showing the hints. Both sidebars now use the existing shared delay.

Made with GPT-5.6 Sol using the Codex harness.


Note

Low Risk
Sidebar UI timing only; behavior is covered by existing createThreadJumpHintVisibilityController tests.

Overview
Thread jump shortcut labels in the sidebar no longer flash during brief Command-key presses (e.g. dictation or other chords that momentarily look like “show hints”).

THREAD_JUMP_HINT_SHOW_DELAY_MS is raised from 100ms to 200ms, and the main Sidebar stops toggling hints synchronously with the modifier. It now uses useThreadJumpHintVisibility, which waits for the delay before showing hints, hides immediately on release, and cancels a pending show if the key is released early—the same shared behavior LegacySidebar already relied on.

Reviewed by Cursor Bugbot for commit 97d9afd. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Increase THREAD_JUMP_HINT_SHOW_DELAY_MS from 100 to 200 in sidebar

  • Raises the thread jump hint display delay from 100 ms to 200 ms to reduce premature hint flashing
  • Extracts hint visibility state out of Sidebar into a new useThreadJumpHintVisibility hook, replacing the local useState(showJumpHints)
  • Risk: Sidebar now depends on useThreadJumpHintVisibility for showThreadJumpHints and updateThreadJumpHintsVisibility — verify the hook is exported and the useEffect call site in Sidebar.tsx passes the correct boolean

Macroscope summarized 97d9afd.

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 95653926-c08c-4d76-a915-3d8b944da4f4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Aug 25, 2026

@macroscopeapp macroscopeapp Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One finding: the jump-hint delay test no longer derives its boundary from THREAD_JUMP_HINT_SHOW_DELAY_MS. The rest of the change (moving Sidebar onto the shared useThreadJumpHintVisibility hook, so both sidebars share one debounce contract) looks consistent.

Posted via Macroscope — UI Consistency

Comment thread apps/web/src/components/Sidebar.logic.test.ts Outdated
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.3 KiB 13.3 KiB +30 B (+0.2%) 15.1 KiB
Codex Thread snapshot wire 6.9 KiB 6.9 KiB +9 B (+0.1%) 7.3 KiB
Codex Live turn WebSocket wire 6.4 KiB 6.4 KiB +21 B (+0.3%) 7.8 KiB
Codex Live turn WebSocket decoded 55.6 KiB 55.6 KiB +44 B (+0.1%) 66.4 KiB
Codex Live turn messages 10 11 +1 (+10.0%) 21
Claude Total thread wire 13.3 KiB 13.3 KiB +1 B (+0.0%) 15.1 KiB
Claude Thread snapshot wire 6.9 KiB 6.9 KiB +6 B (+0.1%) 7.3 KiB
Claude Live turn WebSocket wire 6.5 KiB 6.4 KiB −5 B (−0.1%) 7.8 KiB
Claude Live turn WebSocket decoded 56.4 KiB 56.4 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 11 11 0 (0.0%) 21

Baseline: afc8342 · PR result: 97d9afd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 109.4 KiB
  • Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

macroscopeapp Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 97d9afd

Macroscope's review found this PR approvable — The PR makes a narrowly scoped sidebar UI timing adjustment, using an existing visibility controller to delay shortcut hints without changing navigation or other workflows. Its runtime impact is limited and self-contained.

Notes:

  • No code objects were reviewed. Approvability was decided on eligibility alone.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit e6d487e into main Aug 25, 2026
24 checks passed
@t3dotgg
t3dotgg deleted the t3code/fix-sticky-command-thread-hint branch August 25, 2026 07:01
BarretoDiego added a commit to BarretoDiego/t3code that referenced this pull request Aug 25, 2026
- Providers no server: compatibilidade preservada após upgrades (pingdotgg#8176), CLIs disponíveis no serviço do macOS (pingdotgg#8173) e fim do probing do Cursor enquanto não estiver habilitado (pingdotgg#8175)
- Client-runtime: retry de queries após interrupção de conexão (pingdotgg#8117), com ajustes em authorization/remote e state/runtime
- Claude: compactação de threads antigas antes de estourar o uso (pingdotgg#8144), incluindo novo módulo claudeCompaction em packages/shared e medidor de janela de contexto na web
- Web: dicas de atalho de thread agora atrasadas em 200 ms (pingdotgg#8172)
- Grok: limite nas atualizações cumulativas de saída de ferramentas (pingdotgg#7279)
- Mobile: coleta de modelo de dispositivo e versão de SO (pingdotgg#8169)
- Docs: release passa a verificar atualizações remotas com migrações de banco (pingdotgg#8177) e doc de providers do Claude atualizada
- Conflitos: 2 arquivos conflitaram e foram resolvidos manualmente, mantendo as modificações próprias do fork e incorporando as mudanças do upstream por cima

Sync automatico t3code-sync em 2026-08-25T06:05:15-03:00
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 25, 2026
## What's Changed
* feat(mobile): track device models and OS versions by @t3dotgg in pingdotgg/t3code#8169
* fix(grok): bound cumulative tool output updates by @lnieuwenhuis in pingdotgg/t3code#7279
* fix(web): delay thread shortcut hints by 200 ms by @t3dotgg in pingdotgg/t3code#8172
* fix(server): stop probing Cursor until enabled by @t3dotgg in pingdotgg/t3code#8175
* docs(release): verify remote updates with database migrations by @t3dotgg in pingdotgg/t3code#8177
* fix(server): keep provider CLIs available in the macOS service by @t3dotgg in pingdotgg/t3code#8173
* feat(claude): compact old threads before they burn through usage by @t3dotgg in pingdotgg/t3code#8144
* fix(client-runtime): retry queries after connection interruption by @tris203 in pingdotgg/t3code#8117
* fix(server): keep previously used providers working after upgrades by @t3dotgg in pingdotgg/t3code#8176


**Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260825.1182...v0.0.34-nightly.20260825.1183

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260825.1183
frankdavidcorona pushed a commit to frankdavidcorona/t3code that referenced this pull request Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant