feat(cli): render a storage plan the way every other plan is rendered

pkg/apitypes/apitypes.go

@@ -840,8 +840,18 @@ func (r *PlanResponse) HasErrors() bool {
 
 // UnsafeChange represents a table change that is potentially destructive.
 type UnsafeChange struct {
-	Table      string
-	Reason     string
+	Table  string
+	Reason string
+	// Reasons is the change's findings, already separated. Renderers list and
+	// count these when they are set.
+	//
+	// It exists because Reason carries the engine-reported form, which joins a
+	// table's findings with "; ", so a renderer given only that has to split it
+	// back apart to list them. A reason written for one change is one sentence,
+	// and splitting it renders half of it as a finding of its own. Setting this
+	// is how a producer that knows its own findings says what they are; leaving
+	// it unset keeps the splitting, which is what an engine's plan needs.
+	Reasons    []string `json:",omitempty"`
 	DDL        string
 	ChangeType string
 }

pkg/cmd/commands/apply.go

@@ -182,7 +182,7 @@ func (cmd *ApplyCmd) Run(g *Globals) error {
 
 	// Show unsafe warning if --allow-unsafe was used
 	if cmd.AllowUnsafe {
-		templates.WriteUnsafeWarningAllowed(planResult.UnsafeChanges())
+		templates.WriteUnsafeWarningAllowed(planResult.UnsafeChanges(), templates.UnsafeConsentAllowFlag)
 	}
 
 	// Show options if any flags are set
@@ -396,7 +396,7 @@ func blockUnsafeApply(planResult *apitypes.PlanResponse, database, environment,
 
 	// Then show the unsafe changes warning
 	unsafeChanges := planResult.UnsafeChanges()
-	templates.WriteUnsafeChangesBlocked(unsafeChanges, database, environment, schemaDir)
+	templates.WriteUnsafeChangesBlocked(unsafeChanges, fmt.Sprintf("apply -s %s -e %s --allow-unsafe", schemaDir, environment))
 	return ErrSilent
 }