Skip to content

Conversation

@rushilpatel0
Copy link
Contributor

Motivation

Content

Testing

Please check the following before marking your PR as ready for review

  • I have added tests for my changes
  • I have updated the documentation or added new documentation as needed
@rushilpatel0 rushilpatel0 requested review from a team and codegen-team as code owners September 17, 2025 20:39
@codecov
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

❌ Patch coverage is 0% with 32 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/codegen/cli/tui/app.py 0.00% 20 Missing ⚠️
src/codegen/cli/commands/claude/main.py 0.00% 12 Missing ⚠️
cursor[bot]

This comment was marked as outdated.

print(" \033[37m→ Run Claude Code\033[0m")
print()
print("Press Enter to launch Claude Code with session tracking.")

Copy link

Choose a reason for hiding this comment

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

Bug: Claude Tab Layout Inconsistent Across Installations

The _display_claude_tab function calls _pad_to_lines(7) with a hardcoded value. This doesn't account for the varying number of lines printed before padding (3 lines when Claude Code is installed, 9 lines when not). As a result, the Claude tab's content area doesn't consistently maintain its intended 10-line fixed layout, displaying 12 lines when Claude is not installed and 6 lines when it is.

Fix in Cursor Fix in Web

@rushilpatel0 rushilpatel0 merged commit e9f8e63 into develop Sep 17, 2025
15 of 19 checks passed
@rushilpatel0 rushilpatel0 deleted the rpatel/claude-installation-check branch September 17, 2025 20:49
@github-actions
Copy link
Contributor

🎉 This PR is included in version 0.56.24 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

2 participants