Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Aug 31, 2025

Description

Pause and resume rather than interrupt cn


Summary by cubic

Switches CLI chat interruption from a hard stop to a pause/resume flow so users can press Enter to continue the last response without losing context. Also improves /info to show the current session.

  • New Features
    • Pause/resume: interrupt stops streaming, drops partial assistant reply, saves history, and sets wasInterrupted; Enter with empty input resumes; typing a new message cancels the pause.
    • UI: shows a yellow “Interrupted by user — Press enter to resume” message above the input; resume doesn’t add to input history.
    • /info: displays current session Title, ID, and file path.
    • Tests: adds minimal TUI interruption tests; removes a noisy debug log.

@sestinj sestinj requested a review from a team as a code owner August 31, 2025 20:39
@sestinj sestinj requested review from Patrick-Erichsen and removed request for a team August 31, 2025 20:39
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Aug 31, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 8 files

React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.


describe("TUIChat - Interruption UI (Minimal Test)", () => {
beforeEach(() => {
vi.useFakeTimers();
Copy link
Contributor

Choose a reason for hiding this comment

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

Unnecessary use of fake timers in tests; no timer-based behavior is exercised, so this adds complexity without benefit. Consider removing to avoid brittle interactions with async code.

Prompt for AI agents
Address the following comment on extensions/cli/src/ui/__tests__/TUIChat.interruption.minimal.test.tsx at line 9:

<comment>Unnecessary use of fake timers in tests; no timer-based behavior is exercised, so this adds complexity without benefit. Consider removing to avoid brittle interactions with async code.</comment>

<file context>
@@ -0,0 +1,198 @@
+
+describe(&quot;TUIChat - Interruption UI (Minimal Test)&quot;, () =&gt; {
+  beforeEach(() =&gt; {
+    vi.useFakeTimers();
+  });
+
</file context>

@sestinj sestinj merged commit 80c6797 into main Sep 1, 2025
55 checks passed
@sestinj sestinj deleted the pause-cli branch September 1, 2025 00:57
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs Sep 1, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 1, 2025
@github-actions github-actions bot added the tier 2 Important feature that adds new capabilities to the platform or improves critical user journeys label Sep 1, 2025
@sestinj
Copy link
Contributor Author

sestinj commented Sep 1, 2025

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Sep 3, 2025

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

released size:L This PR changes 100-499 lines, ignoring generated files. tier 2 Important feature that adds new capabilities to the platform or improves critical user journeys

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants