ci: fold parity into QA release validation

clawsweeper[bot] · clawsweeper[bot] · commit 34826540587c · 2026-05-02T19:51:04.000Z
diff --git a/scripts/check-docker-e2e-boundaries.mjs b/scripts/check-docker-e2e-boundaries.mjs
@@ -58,6 +58,7 @@ function validateUniqueLanes(label, lanes) {
 }
 
 function validateLane(label, lane) {
+  const resources = laneResources(lane);
   if (!lane.name || typeof lane.name !== "string") {
     errors.push(`${label}: Docker E2E lane is missing a string name`);
   }
@@ -70,7 +71,7 @@ function validateLane(label, lane) {
       `${label}: Docker E2E lane '${lane.name}' has invalid image kind '${lane.e2eImageKind}'`,
     );
   }
-  if (lane.live && lane.e2eImageKind) {
+  if (lane.live && lane.e2eImageKind && !resources.includes("npm")) {
     errors.push(`${label}: live Docker E2E lane '${lane.name}' must not require a package image`);
   }
   if (!lane.live && !lane.e2eImageKind) {
@@ -79,7 +80,7 @@ function validateLane(label, lane) {
   if (laneWeight(lane) < 1) {
     errors.push(`${label}: Docker E2E lane '${lane.name}' must have positive weight`);
   }
-  if (!laneResources(lane).includes("docker")) {
+  if (!resources.includes("docker")) {
     errors.push(`${label}: Docker E2E lane '${lane.name}' must include the docker resource`);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ function validateUniqueLanes(label, lanes) {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`function validateLane(label, lane) {`
	`61`	`+ const resources = laneResources(lane);`
`61`	`62`	`if (!lane.name \|\| typeof lane.name !== "string") {`
`62`	`63`	errors.push(`${label}: Docker E2E lane is missing a string name`);
`63`	`64`	`}`
`@@ -70,7 +71,7 @@ function validateLane(label, lane) {`
`70`	`71`	`${label}: Docker E2E lane '${lane.name}' has invalid image kind '${lane.e2eImageKind}'`,
`71`	`72`	`);`
`72`	`73`	`}`
`73`		`- if (lane.live && lane.e2eImageKind) {`
	`74`	`+ if (lane.live && lane.e2eImageKind && !resources.includes("npm")) {`
`74`	`75`	errors.push(`${label}: live Docker E2E lane '${lane.name}' must not require a package image`);
`75`	`76`	`}`
`76`	`77`	`if (!lane.live && !lane.e2eImageKind) {`
`@@ -79,7 +80,7 @@ function validateLane(label, lane) {`
`79`	`80`	`if (laneWeight(lane) < 1) {`
`80`	`81`	errors.push(`${label}: Docker E2E lane '${lane.name}' must have positive weight`);
`81`	`82`	`}`
`82`		`- if (!laneResources(lane).includes("docker")) {`
	`83`	`+ if (!resources.includes("docker")) {`
`83`	`84`	errors.push(`${label}: Docker E2E lane '${lane.name}' must include the docker resource`);
`84`	`85`	`}`
`85`	`86`