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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant