refactor: port status page migrate commands to SDK + RunContext

Replaces the CLI-local raw-HTTP transport introduced by #1 with the
flashduty-sdk v0.7.0 methods and routes the 4 migrate subcommands
through the RunContext / runCommand pattern used by every other
command.

Changes:
- Bump flashduty-sdk from v0.6.0 to v0.7.0 (adds status page migration
  methods).
- Extend flashdutyClient interface with StartStatusPageMigration,
  StartStatusPageEmailSubscriberMigration, GetStatusPageMigrationStatus,
  CancelStatusPageMigration.
- Add default mockClient stubs for the four new interface methods.
- Rewrite internal/cli/status_page_migrate.go (~590 lines to ~280):
  delete statusPageMigrationAPI, statusPageMigrationService, the
  CLI-local migration transport, manual sanitization, and duplicate
  local types. Each cobra command now uses runCommand(cmd, args,
  func(ctx *RunContext) error {...}) and delegates to ctx.Client.*.
  Print helpers rewired from flagJSON / newPrinter() globals to
  ctx.JSON / ctx.Printer / ctx.Writer.
- Rewrite internal/cli/status_page_migrate_test.go (387 lines to ~340):
  replace httptest-backed transport tests with newClientFn mock-swap
  tests using execCommand, matching the pattern established in
  command_test.go. One httptest-backed integration test per SDK method
  is not kept here because the flashduty-sdk repo owns wire-format
  tests now.

Post-review fixes (independent reviews from go-reviewer + Codex):
- Nil-guard mock methods so tests overriding a subset do not panic.
- Assert "atlassian" substring in the unsupported-source error so the
  supported list cannot silently drift.
- Emit both "command" (PR #1 schema) and "next_command" in cancel
  --json output to preserve the shipped JSON contract.
- Move validateMigrationSource before runCommand so an invalid --from
  fails with the source error before any client/auth work, matching
  PR #1 ordering. Add two tests locking the ordering.

Command output and JSON schemas remain compatible with PR #1. The
file internal/cli/status_page_migrate.go is no longer an architectural
outlier: 100% of the CLI now flows through ctx.Client.

Verification:
- go test -race -count=1 ./... green
- go test -tags=e2e ./e2e/... green (24.4s)
- gofmt, goimports, go vet clean
- Coverage on ported commands: 80-100%.

Author: debidong

go.mod

@@ -3,7 +3,7 @@ module github.com/flashcatcloud/flashduty-cli
 go 1.25.1
 
 require (
-	github.com/flashcatcloud/flashduty-sdk v0.6.0
+	github.com/flashcatcloud/flashduty-sdk v0.7.0
 	github.com/spf13/cobra v1.10.2
 	golang.org/x/term v0.42.0
 	gopkg.in/yaml.v3 v3.0.1

go.sum

@@ -1,6 +1,6 @@
 github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
-github.com/flashcatcloud/flashduty-sdk v0.6.0 h1:2dzJJ5s7Wgq7KWbsnbGMSo0+yN8yuNMVc50M+dtF8rU=
-github.com/flashcatcloud/flashduty-sdk v0.6.0/go.mod h1:dG4eJfdZaj4jNBMwEexbfK/3PmcIMhNeJ88L/DcZzUY=
+github.com/flashcatcloud/flashduty-sdk v0.7.0 h1:yPW8ghyHB60/34fz5sBITXhMWtbsm2mxYVFORgs+jpE=
+github.com/flashcatcloud/flashduty-sdk v0.7.0/go.mod h1:dG4eJfdZaj4jNBMwEexbfK/3PmcIMhNeJ88L/DcZzUY=
 github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
 github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
 github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=

internal/cli/command_test.go

@@ -191,6 +191,22 @@ func (m *mockClient) SearchAuditLogs(context.Context, *flashduty.SearchAuditLogs
 	return nil, fmt.Errorf("mockClient: SearchAuditLogs not implemented")
 }
 
+func (m *mockClient) StartStatusPageMigration(context.Context, *flashduty.StartStatusPageMigrationInput) (*flashduty.StartStatusPageMigrationOutput, error) {
+	return nil, fmt.Errorf("mockClient: StartStatusPageMigration not implemented")
+}
+
+func (m *mockClient) StartStatusPageEmailSubscriberMigration(context.Context, *flashduty.StartStatusPageEmailSubscriberMigrationInput) (*flashduty.StartStatusPageMigrationOutput, error) {
+	return nil, fmt.Errorf("mockClient: StartStatusPageEmailSubscriberMigration not implemented")
+}
+
+func (m *mockClient) GetStatusPageMigrationStatus(context.Context, string) (*flashduty.StatusPageMigrationJob, error) {
+	return nil, fmt.Errorf("mockClient: GetStatusPageMigrationStatus not implemented")
+}
+
+func (m *mockClient) CancelStatusPageMigration(context.Context, string) error {
+	return fmt.Errorf("mockClient: CancelStatusPageMigration not implemented")
+}
+
 // saveAndResetGlobals saves the current state of all global vars that commands
 // mutate, resets them to safe defaults, and returns a restore function for
 // t.Cleanup.

internal/cli/root.go

@@ -68,6 +68,12 @@ type flashdutyClient interface {
 	QueryInsightIncidentList(ctx context.Context, input *flashduty.QueryInsightIncidentListInput) (*flashduty.QueryInsightIncidentListOutput, error)
 	QueryNotificationTrend(ctx context.Context, input *flashduty.QueryNotificationTrendInput) (*flashduty.QueryNotificationTrendOutput, error)
 	SearchAuditLogs(ctx context.Context, input *flashduty.SearchAuditLogsInput) (*flashduty.SearchAuditLogsOutput, error)
+
+	// === PHASE 4: Status Page Migration ===
+	StartStatusPageMigration(ctx context.Context, input *flashduty.StartStatusPageMigrationInput) (*flashduty.StartStatusPageMigrationOutput, error)
+	StartStatusPageEmailSubscriberMigration(ctx context.Context, input *flashduty.StartStatusPageEmailSubscriberMigrationInput) (*flashduty.StartStatusPageMigrationOutput, error)
+	GetStatusPageMigrationStatus(ctx context.Context, jobID string) (*flashduty.StatusPageMigrationJob, error)
+	CancelStatusPageMigration(ctx context.Context, jobID string) error
 }
 
 // newClientFn creates a flashdutyClient. Override in tests to inject a mock.