Further improve bad shape link check

inexorabletash · inexorabletash · commit ccabdd67fcc9 · 2025-04-24T09:15:24.000-07:00
diff --git a/tools/lint.mjs b/tools/lint.mjs
@@ -333,11 +333,11 @@ for (const match of text.matchAll(/\bnot greater\b/g)) {
   error(`Prefer "less or equal" to "not greater" (etc):  ${format(match)}`);
 }
 
-// [WebNN] Look for incorrect use of shape for an MLOperandDescriptor
+// [WebNN] Ensure MLOperandDescriptor's shape is linked, not MLOperand's.
 // This looks for variables containing 'desc' (descriptor, desc2, etc).
 // FUTURE: Implement a "type checker" for specs.
 for (const match of source.matchAll(/(\|\w*desc\w*\|)'s \[=MLOperand\/shape=\]/ig)) {
-  error(`Use ${match[1]}.{{MLOperandDescriptor/shape}} not shape: ${format(match)}`);
+  error(`Use ${match[1]}.{{MLOperandDescriptor/shape}} not MLOperand's shape: ${format(match)}`);
 }
 
 // [Generic] Look for missing dict-member dfns

Original file line number	Diff line number	Diff line change
`@@ -333,11 +333,11 @@ for (const match of text.matchAll(/\bnot greater\b/g)) {`
`333`	`333`	error(`Prefer "less or equal" to "not greater" (etc): ${format(match)}`);
`334`	`334`	`}`
`335`	`335`
`336`		`-// [WebNN] Look for incorrect use of shape for an MLOperandDescriptor`
	`336`	`+// [WebNN] Ensure MLOperandDescriptor's shape is linked, not MLOperand's.`
`337`	`337`	`// This looks for variables containing 'desc' (descriptor, desc2, etc).`
`338`	`338`	`// FUTURE: Implement a "type checker" for specs.`
`339`	`339`	`for (const match of source.matchAll(/(\\|\wdesc\w\\|)'s \[=MLOperand\/shape=\]/ig)) {`
`340`		- error(`Use ${match[1]}.{{MLOperandDescriptor/shape}} not shape: ${format(match)}`);
	`340`	+ error(`Use ${match[1]}.{{MLOperandDescriptor/shape}} not MLOperand's shape: ${format(match)}`);
`341`	`341`	`}`
`342`	`342`
`343`	`343`	`// [Generic] Look for missing dict-member dfns`