feat(cli): accept positional <id> on team get (completes PR#50 rollout)

PR#50 added positional IDs to incident/alert curated commands but left
team get requiring --id. Agents using the positional form got syntax errors
(seen 3× in sess_5Cg5UV6WpTY8Yt4VSbHrLw steps 60-61). MaximumNArgs(1) +
PreRunE bypass when arg present is consistent with the PR#50 pattern.

Author: ysyneu

internal/cli/team.go

@@ -92,23 +92,35 @@ func newTeamGetCmd() *cobra.Command {
 	)
 
 	cmd := &cobra.Command{
-		Use:   "get",
+		Use:   "get [<id>]",
 		Short: "Get team detail",
 		Long: curatedLong(`Get detailed information about a specific team.
 
-Specify the team by exactly one of: --id, --name, or --ref-id.
+Specify the team by positional ID, or by exactly one of: --id, --name, or --ref-id.
 The output includes team metadata, member list, and audit information.
 
 Examples:
+  flashduty team get 123
   flashduty team get --id 123
   flashduty team get --name "SRE Team"
   flashduty team get --ref-id "hr-dept-42"
   flashduty team get --id 123 --json`, "Teams", "ReadInfo"),
+		Args: cobra.MaximumNArgs(1),
 		PreRunE: func(cmd *cobra.Command, args []string) error {
+			if len(args) == 1 {
+				return nil
+			}
 			return requireExactlyOneFlag(cmd, "id", "name", "ref-id")
 		},
 		RunE: func(cmd *cobra.Command, args []string) error {
 			return runCommand(cmd, args, func(ctx *RunContext) error {
+				if len(ctx.Args) == 1 {
+					id, err := strconv.ParseInt(ctx.Args[0], 10, 64)
+					if err != nil {
+						return fmt.Errorf("invalid team id %q: must be a number", ctx.Args[0])
+					}
+					teamID = id
+				}
 				team, _, err := ctx.Client.Teams.ReadInfo(cmdContext(ctx.Cmd), &flashduty.TeamInfoRequest{
 					TeamID:   uint64(teamID),
 					TeamName: teamName,
@@ -122,8 +134,6 @@ Examples:
 					return ctx.Printer.Print(team, nil)
 				}
 
-				// TeamItem carries only member person IDs; resolve names/emails
-				// in one batch to replicate the legacy member display.
 				members := resolveTeamMemberInfos(ctx, team.PersonIDs)
 				printTeamDetail(ctx.Writer, team, members)
 				return nil

internal/cli/team_get_test.go

@@ -0,0 +1,29 @@
+package cli
+
+import "testing"
+
+func TestTeamGetAcceptsPositionalID(t *testing.T) {
+	cmd := newTeamGetCmd()
+	// MaximumNArgs(1): two positional args should be rejected
+	if cmd.Args != nil {
+		if err := cmd.Args(cmd, []string{"123456", "789"}); err == nil {
+			t.Fatal("team get should reject two positional args")
+		}
+	}
+}
+
+func TestTeamGetNoArgNoFlagFails(t *testing.T) {
+	cmd := newTeamGetCmd()
+	err := cmd.PreRunE(cmd, []string{})
+	if err == nil {
+		t.Fatal("expected error when no positional arg and no flag provided")
+	}
+}
+
+func TestTeamGetPositionalArgBypassesPreRunE(t *testing.T) {
+	cmd := newTeamGetCmd()
+	err := cmd.PreRunE(cmd, []string{"123456"})
+	if err != nil {
+		t.Fatalf("PreRunE should succeed with positional arg, got: %v", err)
+	}
+}