Skip to content

Conversation

@ryanmac
Copy link
Owner

@ryanmac ryanmac commented Jul 23, 2025

Summary

  • Fixed confusing UX where uncommitted changes appeared immediately after initial commit
  • Reorganized installer flow to perform all file modifications before the commit prompt
  • Users now have a clean working directory when starting their first agent

Problem

After running the installer and committing the initial setup, users were immediately faced with uncommitted changes when trying to start an agent:

Would you like to start a dev agent now? [Y/n]: y
🤖 Starting dev agent...
⚠️ Uncommitted changes detected.
Stash them automatically before starting agent? [Y/n]: 

The uncommitted changes were from the installer itself:

  • Role configuration modifications to .conductor/config.yaml
  • Demo task creation modifications to .conductor/workflow-state.json

Solution

Reorganized the installer steps from:

Setup → Commit → Role Config → Demo Tasks → Launch Agent

To:

Setup → Role Config → Demo Tasks → Commit → Launch Agent

This ensures all file modifications happen before the commit, resulting in a clean working directory when the user starts their first agent.

Test plan

  • Run the installer in a fresh repo
  • Verify role configuration happens before commit
  • Verify demo tasks are created before commit
  • Verify no uncommitted changes after commit
  • Verify agent starts without stash prompt

🤖 Generated with Claude Code

Reorganized the installer flow to prevent uncommitted changes after
the initial commit. Previously, role configuration and demo task
creation happened after the commit, leaving the user with immediate
uncommitted changes. This was confusing UX.

New flow:
1. Initial setup (Steps 1-4)
2. Role configuration (Step 5) - modifies config.yaml
3. Demo task creation (Step 6) - modifies workflow-state.json
4. Commit everything together (Step 7)
5. Launch agent (Step 8) - clean working directory

This ensures users have a clean git state when starting their first
agent, eliminating the confusing "uncommitted changes" prompt.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ryanmac
Copy link
Owner Author

ryanmac commented Jul 23, 2025

Closing due to conflicts with merged PR #9. The installer UX improvements from this PR have been superseded by the environment selection feature in PR #9, which includes similar flow reorganization.

@ryanmac ryanmac closed this Jul 23, 2025
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.

2 participants