Skip to content

Conversation

@Angelmmiguel
Copy link
Contributor

Improves the rover iterate command experience by using the common display utilities and simplifying the command flow. The changes make the output cleaner, more consistent with other CLI commands, and easier to follow.

Changes

  • Refactored the iterate command to use showTitle() and showProperties() for consistent output formatting
  • Replaced verbose chat messages with clean task property displays
  • Integrated ProcessManager for step-by-step progress feedback during iteration creation
  • Simplified iteration context retrieval using the TaskDescriptionManager.getLastIteration() method
  • Moved task loading and workspace validation earlier in the command flow for better error handling
  • Removed getLatestIterationContext() helper in favor of using task manager methods directly
  • Updated success tips to include iteration number in logs and inspect commands

Notes

Closes #310

The --interactive option has been commented out in preparation for future implementation.

@Angelmmiguel Angelmmiguel added this to the v1.7 milestone Nov 27, 2025
@Angelmmiguel Angelmmiguel self-assigned this Nov 27, 2025
@Angelmmiguel
Copy link
Contributor Author

@ereslibre I noticed the iterate command still used the old CLI format and updated it. This PR doesn't include any code for the interactive feature.

@ereslibre
Copy link
Collaborator

@ereslibre I noticed the iterate command still used the old CLI format and updated it. This PR doesn't include any code for the interactive feature.

Thanks!

Copy link
Collaborator

@ereslibre ereslibre left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines +315 to +318
// .option(
// '-i, --interactive',
// 'Open an interactive command session to iterate on the task'
// )
Copy link
Collaborator

Choose a reason for hiding this comment

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

😀

@Angelmmiguel Angelmmiguel merged commit d2b8a5f into main Nov 27, 2025
5 checks passed
@Angelmmiguel Angelmmiguel deleted the 310/interactive branch December 5, 2025 10:56
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.

Add interactive mode to iterate command

3 participants