Address PR #184 review feedback: fix line lengths and add test coverage

[Copilot]

PR #184 could not merge due to grafted commit history and unaddressed review feedback. Main branch already contains the KeyboardInterrupt handling but has line length violations and missing test coverage.

Changes

Code Quality (main.py)

• Split 190-char Profile ID prompt into 3 lines (<80 chars each) with proper color reset placement
• Added docstring to nested validate_profile_input() function
• Split 131-char error message across 2 lines

Test Coverage (test_main.py)

• Added test_keyboard_interrupt_during_setup - verifies exit code 130, friendly cancellation message, no stack traces
• Updated test_interactive_prompts_show_hints - verifies dashboard URL display
• Fixed test_interactive_input_extracts_id - updated assertion for new prompt text

# Before: 190-char line
print(f"{Colors.CYAN}  You can find this in the URL of your profile in the Control D Dashboard at https://controld.com/dashboard/profiles (or just paste the URL).{Colors.ENDC}")

# After: Clear, scannable 3-line message
print(f"{Colors.CYAN}  Find it in your profile URL at "
      f"https://controld.com/dashboard/profiles"
      f"{Colors.ENDC}")
print(f"{Colors.CYAN}  (You can also paste the full profile URL)."
      f"{Colors.ENDC}")

All 93 tests pass, CodeQL scan clean.

Original prompt

Hi @copilot, please create a new pull request incorporating the feedback and suggested changes from this thread while all resolving any currrent conflicts which are preventing this branch from merging with the main branch - #184. Thank you!

Custom agent used: Development Partner
The Development Partner Protocol

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[trunk-io]

🚫 This pull request was requested to be canceled by Abhi Mehrotra (a GitHub user), so it was removed from the merge queue. See more details here.

• To merge this pull request, check the box to the left or comment /trunk merge below.

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.

[Copilot]

Pull request overview

This PR incorporates feedback from PR #184 to improve CLI UX by handling KeyboardInterrupt gracefully during the setup phase and providing clearer guidance for interactive prompts. The changes address line length issues and add documentation while resolving merge conflicts with the main branch.

Changes:

• Split long print statements in main.py to comply with line length limits
• Added docstring to nested validate_profile_input function for better code documentation
• Enhanced test coverage by adding test_keyboard_interrupt_during_setup to verify graceful exit on Ctrl+C
• Updated test assertions to verify the full dashboard URL is shown in prompts

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
main.py	Split long lines for profile ID prompt and error message; added docstring to validate_profile_input
test_main.py	Added KeyboardInterrupt test; updated assertions to check for full dashboard URL

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.