Skip to content

feat: rc10 UX improvements + stall warning fixes (#186, #187, #188)#189

Merged
Nathan Schram (nathanschram) merged 5 commits intodevfrom
feature/rc10-ux-stall-fixes
Mar 22, 2026
Merged

feat: rc10 UX improvements + stall warning fixes (#186, #187, #188)#189
Nathan Schram (nathanschram) merged 5 commits intodevfrom
feature/rc10-ux-stall-fixes

Conversation

@nathanschram
Copy link
Member

Summary

  • Emoji buttons + edit-in-place for outline approval — ExitPlanMode buttons now show ✅/❌/📋 emoji prefixes; post-outline approval edits feedback message in-place instead of creating a second message #186
  • Redesign startup message layout — italic subheadings, split engine lines, renamed "projects" to "directories", added bug report link #187
  • 🧹 emoji on /new session clear messages
  • Succinct stall warnings — truncate Last: to 80 chars, recognise command: prefix for Bash, reassuring "still running" when CPU active, drop PID diagnostics from Telegram, "may be stuck" only when genuinely stuck #188
  • Frozen ring escalation fix — when a known tool is running (main sleeping, CPU active on children), show "Bash command still running" instead of alarming "No progress" message #188

Test plan

  • 125 exec_bridge tests pass (124 existing + 1 new frozen ring test)
  • 244 related tests pass (exec_bridge + exec_runner + claude_control)
  • Integration tested via @untether_dev_bot: emoji buttons, outline flow, /new emoji, all 6 engines
  • CI pipeline: format, ruff, ty, pytest (3.12/3.13/3.14), build, lockfile, pip-audit, bandit

🤖 Generated with Claude Code

Add emoji prefixes to ExitPlanMode and post-outline buttons (✅/❌/📋).
Post-outline approve/deny now edits the "Asked Claude Code to outline
the plan" message in-place instead of creating a second message.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Split engine info into separate lines, add italic subheadings,
rename "projects" to "directories", add bug report link.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Part of startup message UX improvements (#187).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…#188)

Truncate Last: to 80 chars, recognise command: prefix for Bash tools,
use reassuring "still running" when CPU active, drop PID diagnostics
from Telegram messages, only say "may be stuck" when genuinely stuck.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…mmands (#188)

When a known tool is running (main sleeping, CPU active on children),
frozen ring escalation now shows "Bash command still running" instead
of alarming "No progress" message. Found via wpnav staging session
where benchmark scripts ran for 60+ min with false warnings.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: defaults

Review profile: CHILL

Plan: Pro

Run ID: 941863a2-b135-484e-a8f4-95c4cf322763

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/rc10-ux-stall-fixes

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

@nathanschram Nathan Schram (nathanschram) merged commit f050916 into dev Mar 22, 2026
21 checks passed
@nathanschram Nathan Schram (nathanschram) deleted the feature/rc10-ux-stall-fixes branch March 22, 2026 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant