docs: mark insight notifications as do-not-migrate (report API retiring)

The notifications command's backing report API (/report/* QueryNotificationTrend)
is being retired, so it is intentionally excluded from go-flashduty's spec — there
is no SDK method and there won't be one. Reword the TODO from a pending-migration
note to an explicit do-not-migrate so it isn't re-pointed at the SDK later.

Author: ysyneu

internal/cli/insight.go

@@ -342,10 +342,11 @@ func newInsightNotificationsCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "notifications",
 		Short: "Query notification volume trends",
-		// TODO(go-flashduty migration): not migrated. go-flashduty v0.4.0's
-		// Analytics service has no notification-trend endpoint (only by-dimension
-		// insight, top-k alerts, incident list/export). There is no matching
-		// method, so this stays on the legacy SDK's QueryNotificationTrend.
+		// DO NOT MIGRATE to go-flashduty. This command's backing report API
+		// (QueryNotificationTrend, a /report/* endpoint) is being retired, so it
+		// is intentionally excluded from go-flashduty's spec — there is no
+		// go-flashduty method and there won't be one. It stays on the legacy SDK
+		// until the feature itself is removed; do not re-point it at the SDK.
 		RunE: func(cmd *cobra.Command, args []string) error {
 			return runCommand(cmd, args, func(ctx *RunContext) error {
 				startTime, err := timeutil.Parse(since)