Skip to content

Improve initial flow and enabling/disabling#678

Merged
Soph merged 7 commits intomainfrom
soph/improve-enable
Mar 16, 2026
Merged

Improve initial flow and enabling/disabling#678
Soph merged 7 commits intomainfrom
soph/improve-enable

Conversation

@Soph
Copy link
Copy Markdown
Collaborator

@Soph Soph commented Mar 10, 2026

This changes a few things with entire:

  • running entire, entire enable or entire configure in a repo that has no .entire folder yet, will bring you to the setup wizard
  • running entire enable on a repo that is setup earlier already it only tells you that entire is enabled
  • running entire disable then entire enable will disable -> enable entire now
  • running entire configure let's you add more agents later but not remove them, for that you need to use entire configure --remove <agent> to actually remove it

Happy to hear thoughts.

Soph added 4 commits March 10, 2026 14:54
Entire-Checkpoint: 05683eec3f48
Entire-Checkpoint: 4e46c2515716
Entire-Checkpoint: 68f77b8a4adb
Copilot AI review requested due to automatic review settings March 10, 2026 17:55
@cursor
Copy link
Copy Markdown

cursor bot commented Mar 10, 2026

PR Summary

Medium Risk
Alters core CLI control flow and settings read/write behavior (including local-vs-project settings), which could change enable/disable semantics across existing repos if edge cases aren’t covered.

Overview
Improves first-run UX by adding entire configure and making entire (no args) automatically launch the setup wizard when run inside a git repo that isn’t set up yet.

Refactors setup/enable behavior: enable now re-enables an existing repo without re-running full setup (and prints installed agents), while fresh repos funnel through a shared runSetupFlow; configure can add additional agents later (without allowing removal).

Changes setup detection to treat either .entire/settings.json or .entire/settings.local.json as “set up”, and adjusts runEnable to write enabled=true to local settings by default so disableenable reliably re-enables via the local override. Minor prompt/output text and tests are updated accordingly.

Written by Cursor Bugbot for commit 354cb84. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refines the CLI’s first-run and enable/disable/configure flows so repositories without an .entire setup are guided into an initial configuration wizard, and subsequent commands behave more predictably once setup exists.

Changes:

  • Add a shared “first-time setup flow” and invoke it from entire, entire enable, and entire configure when the repo isn’t set up yet.
  • Make entire enable a no-op (with status output) when already enabled; otherwise re-enable by updating settings.
  • Introduce settings.IsSetUpAny() to treat either project or local settings as “already set up”.

Reviewed changes

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

Show a summary per file
File Description
cmd/entire/cli/strategy/common_test.go Minor test formatting alignment.
cmd/entire/cli/setup_test.go Update tests for new runEnable signature and adjusted prompt output.
cmd/entire/cli/setup.go Adds configure/setup flows, changes enable behavior, and adds agent-add flow.
cmd/entire/cli/settings/settings.go Adds IsSetUpAny() helper to detect setup via either settings file.
cmd/entire/cli/root.go Runs setup wizard on bare entire in fresh repos; updates getting-started text; registers configure command.

Entire-Checkpoint: 948380b80fc1
@Soph Soph marked this pull request as ready for review March 11, 2026 10:45
@Soph Soph requested a review from a team as a code owner March 11, 2026 10:45
pjbgf
pjbgf previously approved these changes Mar 12, 2026
pjbgf
pjbgf previously approved these changes Mar 16, 2026
Co-authored-by: Paulo Gomes <paulo@entire.io>
@Soph Soph merged commit 1778f6a into main Mar 16, 2026
3 checks passed
@Soph Soph deleted the soph/improve-enable branch March 16, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants