preprocessor: remove Step as a mode of matching

Sanitisers and comparators are configured at the site or page level to
work against commands. This configuration allows for them to match based
on one of three modes: the full command, the command prefix, or the step
name.

The step name matching mode is not implemented and by definition is not
used and does not work.

In this CL we remove it, in favour of per statement-tag directives that
will allow for more precise setting of sanitisers/comparators.
(Step-level setting applies, in the general case, to the list of
statements contain therein... which doesn't really make much sense).

Preprocessor-No-Write-Cache: true
Signed-off-by: Paul Jolly <[email protected]>
Change-Id: I86a0629a9a9344923ce0672e6c5a7a232be4e079
Dispatch-Trailer: {"type":"trybot","CL":1176699,"patchset":3,"ref":"refs/changes/99/1176699/3","targetBranch":"alpha"}

internal/cmd/preprocessor/cmd/sanitisers.go

@@ -54,10 +54,6 @@ type kindAndCommand struct {
 	// equal to the value set.
 	CommandPrefix string `json:"commandPrefix"`
 
-	// Step defines that X should apply for the specific multi-step script
-	// command step (steps names are unique).
-	Step string `json:"step"`
-
 	// stmt is a parsed version of Command or CommandPrefix.
 	stmt *syntax.Stmt
 }

internal/cmd/preprocessor/cmd/schema.cue

@@ -12,15 +12,9 @@ package preprocessor
 		{
 			command!:       string
 			commandPrefix?: _|_
-			step?:          _|_
 		} | {
 			command?:       _|_
 			commandPrefix!: string
-			step?:          _|_
-		} | {
-			command?:       _|_
-			commandPrefix?: _|_
-			step!:          string
 		}
 	}