recipe: the depth switch says what it does about every other token

golangci-lint reads a switch on token.Type without a default as incomplete, and
it is right about the shape while being wrong about this function: everything
that is not a bracket or a brace leaves the nesting where it was, and listing
the other thirty members would say less than one line saying so.

Measured rather than assumed, because the natural conclusion was that flattening
the function had introduced this. It had not. The version before the flattening,
restored from 3a32c9d and run through the same pinned linter, reports the same
thing at line 78 - the branch had simply never been through preflight, which is
also how a shape cap sat one over its measurement for three commits.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: donislawdev

internal/recipe/limits.go

@@ -91,12 +91,18 @@ func flowDepth(src []byte) int {
 // the shape guard counts how many functions sit that deep as well as how deep
 // the deepest one is. Splitting is what that guard asks for and it costs
 // nothing here.
+// The default is not decoration. token.Type has thirty four members and a
+// switch on it without one is reported as incomplete, which is correct of the
+// linter and wrong about this function: everything that is not a bracket or a
+// brace leaves the nesting where it was, and listing thirty of them would say
+// less than one line saying so.
 func depthChange(t token.Type) int {
 	switch t {
 	case token.SequenceStartType, token.MappingStartType:
 		return 1
 	case token.SequenceEndType, token.MappingEndType:
 		return -1
+	default:
+		return 0
 	}
-	return 0
 }