Add --in as global alias for --project

[robzolkos]

Summary

• Promotes --in from per-command local flag to a root-level persistent flag, aliasing --project
• Fixes basecamp show todo <id> --in <project> returning "Unknown option: --in"
• Commands that define their own local --in continue to shadow the global one, no behavior change

Fixes the inconsistency described in the Figuring it out card: most commands accepted --in but show only accepted --project.

---

Summary by cubic

Promotes --in to a global alias for --project across the CLI and makes people subcommands accept it. Fixes unknown/ignored --in cases and aligns behavior with docs.

• Bug Fixes
  • Register project name completion for --in; use projectFlagChanged() so config project honors --project or --in.
  • Make people list/add/remove accept --in by adding a local alias, validating either flag, and registering completion for both.
  • Fall back to root --project/--in in people list/add/remove when local flags are unset, so basecamp --in 55555 people ... works.

^{Written for commit 85de871. Summary will update on new commits.}

[cubic-dev-ai]

1 issue found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="internal/cli/root.go">

<violation number="1" location="internal/cli/root.go:187">
P2: Missing tab-completion registration for `--in`. The `--project` flag has `RegisterFlagCompletionFunc` wired up, but the new `--in` alias does not, so `--in <TAB>` won't offer project-name completions.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[Copilot]

Pull request overview

This PR promotes --in to a root-level persistent flag so it works consistently across commands (notably basecamp show ...), aligning it as an alias for --project.

Changes:

• Add a new root persistent --in flag wired to the same underlying project flag value.
• Update the CLI surface snapshot (.surface) to include --in across commands.

Reviewed changes

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

File	Description
internal/cli/root.go	Adds root-level --in flag intended to alias --project.
.surface	Updates the surface snapshot to reflect the new --in flag availability.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[Copilot]

Pull request overview

This PR promotes --in to a root-level persistent flag as a global alias for --project, aiming to make project scoping consistent across commands (notably fixing basecamp show todo <id> --in <project>). It also updates completion and ensures config project detects either flag.

Changes:

• Adds --in as a persistent root flag bound to the same value as --project, plus shell completion.
• Updates config project to treat --project and --in as equivalent for “explicit flag provided” detection.
• Extends people command tests and flags to accept --in as an alias in list/add/remove.

Reviewed changes

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

Show a summary per file

File	Description
internal/cli/root.go	Adds global persistent --in and registers project-name completion for it.
internal/commands/helpers.go	Introduces projectFlagChanged() helper for alias-aware .Changed detection.
internal/commands/config.go	Uses projectFlagChanged() so config project recognizes --in as explicit input.
internal/commands/config_test.go	Adds test coverage ensuring config project --in ... works and persists config.
internal/commands/people.go	Adds per-subcommand --in flags (aliasing local --project) and completion; adjusts required-flag behavior for add/remove.
internal/commands/people_test.go	Adds mock-server-based tests verifying people list/add/remove behavior with --in.
.surface	Updates CLI surface snapshot to include the new global flag everywhere.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.