Merge pull request 'plan --diff says which modules a push would leave out, and why' (#181) from fix/plan-diff-says-what-it-leaves-out into main

2026-10-09 12:41:15 +00:00
parent af025562c7 54b04a7604
commit c58516f819
1 changed files with 4 additions and 0 deletions
@@ -1239,6 +1239,10 @@ func planCommand(ctx context.Context, args []string) error {
 		if err != nil {
 			return err
 		}
+		// A module left out is not in the body, so the diff alone would say "nothing would change" for
+		// a module just assigned whose settings cannot compose — success-shaped silence. Said first, with
+		// why, as push and the plain plan say it (novox/hq ADR 0163, rule 6).
+		reportLeftOut(args[0], declared)
 		return writePlanDiff(ctx, open.inventory, args[0], body)
 	}
 	if *asJSON {