Skip to content

feat(web): add shortcut to unsettle the last thread - #58

Closed
maria-rcks wants to merge 1 commit into
TrogonStack:mainfrom
maria-rcks:t3code/06c37b52
Closed

maria-rcks wants to merge 1 commit into
TrogonStack:mainfrom
maria-rcks:t3code/06c37b52

Conversation

@maria-rcks

@maria-rcks maria-rcks commented Sep 21, 2026

Copy link
Copy Markdown

Adds a default mod+shift+u shortcut that finds the most recently settled thread and un-settles it. The action uses the persisted settlement timestamp, works across environments, and reports command failures through the existing toast.

Verification: vp test run apps/web/src/keybindings.test.ts; vp run --filter @t3tools/web typecheck; targeted lint (all passed, with existing warnings only).

Note

Written by gpt-6 on behalf of Maria


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository: TrogonStack/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 1ea8e54d-1293-4219-bb66-2a441e891634


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.

@cursor

cursor Bot commented Sep 21, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
User-triggered shortcut reuses the existing unsettle mutation; scope is limited to picking one settled thread by timestamp with no auth or data-model changes.

Overview
Adds mod+shift+u (thread.unsettleLast) to undo settlement on the most recently settled thread, using settledAt and filtering out archived threads and environments that do not support settlement.

The handler is wired in ChatView (in-thread shortcuts) and ChatRouteGlobalShortcuts in _chat.tsx so it works on chat routes even when the thread view is not focused. Both paths call the existing unsettle command with reason: "user" and show the same error toast on failure.

Contracts and shared default keybindings register the new command; keybindings.test.ts asserts mod+shift+u resolves outside the terminal.

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

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Sep 21, 2026
@maria-rcks
maria-rcks force-pushed the t3code/06c37b52 branch 2 times, most recently from 0b5a757 to 020f0f4 Compare September 21, 2026 22:09
@github-actions github-actions Bot added size:M and removed size:XXL labels Sep 21, 2026
@maria-rcks
maria-rcks force-pushed the t3code/06c37b52 branch 2 times, most recently from ff9f335 to 9596f30 Compare September 21, 2026 22:21
Signed-off-by: maria-rcks <maria@kuuro.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 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