pi-lens 3.8.73 → 4.0.0

package/package.json

@@ -1,140 +1,141 @@
 {
-	"name": "pi-lens",
-	"version": "3.8.73",
-	"type": "module",
-	"description": "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo",
-	"repository": {
-		"type": "git",
-		"url": "git+https://github.com/apmantza/pi-lens.git"
-	},
-	"main": "./dist/index.js",
-	"bin": {
-		"pi-lens-mcp": "dist/mcp/server.js",
-		"pi-lens-analyze": "dist/mcp/analyze-cli.js"
-	},
-	"scripts": {
-		"build": "tsc --project tsconfig.build.json",
-		"build:dist": "node -e \"require('fs').rmSync('dist',{recursive:true,force:true})\" && npx --yes -p typescript@7.0.2 tsc --project tsconfig.dist.json --noCheck && npm run bundle:dist",
-		"bundle:dist": "node scripts/bundle-dist.mjs",
-		"prepare": "npm run build:dist && node scripts/download-grammars.js --core --dest grammars",
-		"lint": "tsc --project tsconfig.json",
-		"watch": "tsc --watch",
-		"test": "vitest run",
-		"test:unit": "vitest run --ignore tests/index-integration.test.ts --ignore tests/clients/lsp/integration.test.ts",
-		"test:integration": "vitest run tests/index-integration.test.ts tests/clients/lsp/integration.test.ts",
-		"test:watch": "vitest",
-		"check": "node scripts/check-extensions.mjs",
-		"selftest:install": "node scripts/install-selftest.mjs",
-		"check:lockfile": "node scripts/check-lockfile-sync.mjs",
-		"check:grammars": "node scripts/check-grammar-provenance.mjs",
-		"check:grammar-load": "node scripts/check-grammar-load.mjs",
-		"audit:tree-sitter": "node scripts/audit-tree-sitter-rules.mjs",
-		"audit:rule-catalog": "node scripts/validate-rule-catalog.mjs",
-		"audit:astgrep-rule-pairs": "node scripts/audit-astgrep-rule-pairs.mjs",
-		"audit:playground": "node scripts/playground-verify-rule.mjs",
-		"logs:smells": "node scripts/analyze-pi-lens-logs.mjs",
-		"changelog:check": "node scripts/changelog-release.mjs --check",
-		"changelog:release": "node scripts/changelog-release.mjs",
-		"changelog:extract": "node scripts/changelog-extract.mjs",
-		"release:backfill-notes": "node scripts/backfill-github-releases.mjs",
-		"release:backfill-thanks": "node scripts/backfill-release-thanks.mjs",
-		"docs:rule-catalogs": "node scripts/gen-rule-catalogs.mjs",
-		"banner": "npx -y sharp-cli --input banner.svg --output banner.png resize 2200 620",
-		"download-grammars": "node scripts/download-grammars.js",
-		"harness:poc": "node scripts/run-harness.mjs cases/ts/pipeline-dispatch-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
-		"harness:python-poc": "node scripts/run-harness.mjs cases/python/pipeline-dispatch-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
-		"harness:ts-format-poc": "node scripts/run-harness.mjs cases/ts/format-default-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
-		"harness:python-autofix-poc": "node scripts/run-harness.mjs cases/python/autofix-default-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
-		"harness:go-poc": "node scripts/run-harness.mjs cases/go/pipeline-dispatch-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\""
-	},
-	"keywords": [
-		"pi",
-		"pi-extension",
-		"pi-package",
-		"linter",
-		"biome",
-		"ast-grep",
-		"ruff",
-		"typescript",
-		"code-quality",
-		"feedback",
-		"type-coverage",
-		"jscpd",
-		"knip"
-	],
-	"author": "Apostolos Mantzaris",
-	"license": "MIT",
-	"files": [
-		"dist/",
-		"docs/",
-		"grammars/",
-		"config/",
-		"rules/",
-		"skills/",
-		"scripts/download-grammars.js",
-		"scripts/grammars.lock.json",
-		"scripts/analyze-pi-lens-logs.mjs",
-		"scripts/install-selftest.mjs",
-		"scripts/rpc-load-check.mjs",
-		"README.md",
-		"CHANGELOG.md",
-		"banner.svg",
-		"banner.png"
-	],
-	"dependencies": {
-		"@ast-grep/cli": "^0.45.0",
-		"@ast-grep/napi": "^0.45.0",
-		"@earendil-works/pi-tui": "^0.82.1",
-		"js-yaml": "^5.2.2",
-		"minimatch": "^10.2.5",
-		"pidusage": "^4.0.1",
-		"typebox": "^1.0.0",
-		"vscode-jsonrpc": "^9.0.0"
-	},
-	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": "*"
-	},
-	"peerDependenciesMeta": {
-		"@earendil-works/pi-coding-agent": {
-			"optional": true
-		}
-	},
-	"optionalDependencies": {
-		"web-tree-sitter": "0.25.10"
-	},
-	"devDependencies": {
-		"@biomejs/biome": "^2.4.10",
-		"@earendil-works/pi-coding-agent": "^0.82.1",
-		"@types/node": "^26.0.0",
-		"@types/pidusage": "^2.0.5",
-		"@vitest/coverage-v8": "^4.1.5",
-		"typescript": "^7.0.2",
-		"typescript-language-server": "^5.1.3",
-		"vitest": "^4.1.1"
-	},
-	"pi": {
-		"extensions": [
-			"./dist/index.js"
-		],
-		"skills": [
-			"../../skills"
-		]
-	},
-	"knip": {
-		"entry": [
-			"index.ts"
-		],
-		"project": [
-			"**/*.ts",
-			"!**/*.test.ts"
-		],
-		"ignore": [
-			"**/*.test.ts"
-		]
-	},
-	"allowScripts": {
-		"@ast-grep/cli@0.44.1": true,
-		"@google/genai@1.52.0": true,
-		"protobufjs@7.6.4": true
-	}
+  "name": "pi-lens",
+  "version": "4.0.0",
+  "type": "module",
+  "description": "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/apmantza/pi-lens.git"
+  },
+  "main": "./dist/index.js",
+  "bin": {
+    "pi-lens": "dist/mcp/cli.js",
+    "pi-lens-mcp": "dist/mcp/server.js",
+    "pi-lens-analyze": "dist/mcp/analyze-cli.js"
+  },
+  "scripts": {
+    "build": "tsc --project tsconfig.build.json",
+    "build:dist": "node -e \"require('fs').rmSync('dist',{recursive:true,force:true})\" && npx --yes -p typescript@7.0.2 tsc --project tsconfig.dist.json --noCheck && npm run bundle:dist",
+    "bundle:dist": "node scripts/bundle-dist.mjs",
+    "prepare": "npm run build:dist && node scripts/download-grammars.js --core --dest grammars",
+    "lint": "tsc --project tsconfig.json",
+    "watch": "tsc --watch",
+    "test": "node scripts/with-test-lock.mjs -- vitest run",
+    "test:unit": "node scripts/with-test-lock.mjs -- vitest run",
+    "test:integration": "node scripts/with-test-lock.mjs -- vitest run tests/index-integration.test.ts tests/clients/lsp/integration.test.ts",
+    "test:watch": "vitest",
+    "check": "node scripts/check-extensions.mjs",
+    "selftest:install": "node scripts/install-selftest.mjs",
+    "check:lockfile": "node scripts/check-lockfile-sync.mjs",
+    "check:grammars": "node scripts/check-grammar-provenance.mjs",
+    "check:grammar-load": "node scripts/check-grammar-load.mjs",
+    "audit:tree-sitter": "node scripts/audit-tree-sitter-rules.mjs",
+    "audit:rule-catalog": "node scripts/validate-rule-catalog.mjs",
+    "audit:astgrep-rule-pairs": "node scripts/audit-astgrep-rule-pairs.mjs",
+    "audit:playground": "node scripts/playground-verify-rule.mjs",
+    "logs:smells": "node scripts/analyze-pi-lens-logs.mjs",
+    "changelog:check": "node scripts/rollup-changelog.mjs --check",
+    "changelog:release": "node scripts/changelog-release.mjs",
+    "changelog:extract": "node scripts/changelog-extract.mjs",
+    "release:backfill-notes": "node scripts/backfill-github-releases.mjs",
+    "release:backfill-thanks": "node scripts/backfill-release-thanks.mjs",
+    "docs:rule-catalogs": "node scripts/gen-rule-catalogs.mjs",
+    "banner": "npx -y sharp-cli --input banner.svg --output banner.png resize 2200 620",
+    "download-grammars": "node scripts/download-grammars.js",
+    "harness:poc": "node scripts/run-harness.mjs cases/ts/pipeline-dispatch-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
+    "harness:python-poc": "node scripts/run-harness.mjs cases/python/pipeline-dispatch-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
+    "harness:ts-format-poc": "node scripts/run-harness.mjs cases/ts/format-default-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
+    "harness:python-autofix-poc": "node scripts/run-harness.mjs cases/python/autofix-default-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\"",
+    "harness:go-poc": "node scripts/run-harness.mjs cases/go/pipeline-dispatch-poc --pi-bin \"C:\\Users\\R3LiC\\AppData\\Roaming\\npm\\pi.cmd\""
+  },
+  "keywords": [
+    "pi",
+    "pi-extension",
+    "pi-package",
+    "linter",
+    "biome",
+    "ast-grep",
+    "ruff",
+    "typescript",
+    "code-quality",
+    "feedback",
+    "type-coverage",
+    "jscpd",
+    "knip"
+  ],
+  "author": "Apostolos Mantzaris",
+  "license": "MIT",
+  "files": [
+    "dist/",
+    "docs/",
+    "grammars/",
+    "config/",
+    "rules/",
+    "skills/",
+    "scripts/download-grammars.js",
+    "scripts/grammars.lock.json",
+    "scripts/analyze-pi-lens-logs.mjs",
+    "scripts/install-selftest.mjs",
+    "scripts/rpc-load-check.mjs",
+    "README.md",
+    "CHANGELOG.md",
+    "banner.svg",
+    "banner.png"
+  ],
+  "dependencies": {
+    "@ast-grep/cli": "^0.45.0",
+    "@ast-grep/napi": "^0.45.0",
+    "@earendil-works/pi-tui": "^0.84.1",
+    "js-yaml": "^5.2.2",
+    "minimatch": "^10.2.6",
+    "pidusage": "^4.0.1",
+    "typebox": "^1.0.0",
+    "vscode-jsonrpc": "^9.0.0"
+  },
+  "peerDependencies": {
+    "@earendil-works/pi-coding-agent": "*"
+  },
+  "peerDependenciesMeta": {
+    "@earendil-works/pi-coding-agent": {
+      "optional": true
+    }
+  },
+  "optionalDependencies": {
+    "web-tree-sitter": "0.25.10"
+  },
+  "devDependencies": {
+    "@biomejs/biome": "^2.4.10",
+    "@earendil-works/pi-coding-agent": "^0.84.1",
+    "@types/node": "^26.0.0",
+    "@types/pidusage": "^2.0.5",
+    "@vitest/coverage-v8": "^4.1.5",
+    "typescript": "^7.0.2",
+    "typescript-language-server": "^5.1.3",
+    "vitest": "^4.1.1"
+  },
+  "pi": {
+    "extensions": [
+      "./dist/index.js"
+    ],
+    "skills": [
+      "../../skills"
+    ]
+  },
+  "knip": {
+    "entry": [
+      "index.ts"
+    ],
+    "project": [
+      "**/*.ts",
+      "!**/*.test.ts"
+    ],
+    "ignore": [
+      "**/*.test.ts"
+    ]
+  },
+  "allowScripts": {
+    "@ast-grep/cli@0.45.1": true,
+    "@google/genai@1.52.0": true,
+    "protobufjs@7.6.5": true
+  }
 }

package/rules/ast-grep-rules/rule-tests/hardcoded-url-js-test.yml

@@ -2,6 +2,7 @@ id: hardcoded-url-js
 valid:
   - 'const url = process.env.API_URL'
   - 'const x = "hello"'
+  - 'export const BASE_URL_CLINE = "https://api.cline.bot";'
 invalid:
   - 'const url = "https://api.example.com"'
   - 'const endpoint = "http://localhost:3000"'

package/rules/ast-grep-rules/rule-tests/hardcoded-url-test.yml

@@ -2,6 +2,8 @@ id: hardcoded-url
 valid:
   - 'const url = process.env.API_URL'
   - 'const x = "hello"'
+  - 'export const BASE_URL_CLINE = "https://api.cline.bot";'
+  - 'export const URL_KILO_TOS = "https://kilo.dev/tos";'
 invalid:
   - 'const url = "https://api.example.com"'
   - 'const endpoint = "http://localhost:3000"'

package/rules/ast-grep-rules/rule-tests/no-bare-except-test.yml

@@ -15,6 +15,55 @@ valid:
         doThing()
     except Exception as e:
         handle(e)
+  - |
+    # dotted/qualified name (#1031)
+    try:
+        doThing()
+    except asyncio.TimeoutError:
+        handle()
+  - |
+    # parenthesized single name (mirrors #1026's tree-sitter spec set)
+    try:
+        doThing()
+    except (ValueError):
+        handle()
+  - |
+    # subscripted generic (#1244)
+    try:
+        doThing()
+    except dict[str, int]:
+        handle()
+  - |
+    # subscripted PEP 654 exception group (#1244)
+    try:
+        doThing()
+    except BaseExceptionGroup[TypeError, ValueError]:
+        handle()
+  - |
+    try:
+        doThing()
+    except ValueError | TypeError:
+        handle()
+  - |
+    try:
+        doThing()
+    except cache[key]:
+        handle()
+  - |
+    try:
+        doThing()
+    except (ValueError,):
+        handle()
+  - |
+    try:
+        doThing()
+    except (mod.E1, E2) as error:
+        handle(error)
+  - |
+    try:
+        doThing()
+    except* ValueError:
+        handle()
 invalid:
   - |
     try:

package/rules/ast-grep-rules/rule-tests/no-bare-host-path-in-win32-branch-test.yml

@@ -0,0 +1,87 @@
+id: no-bare-host-path-in-win32-branch
+valid:
+  # DOCUMENTED LIMITATION (#1339 review): the exact pre-fix #1150 shape --
+  # bare dirname in an unguarded helper CALLED FROM the guarded branch.
+  # ast-grep has no cross-function reach, so this is deliberately a
+  # negative fixture; it exists so a future strengthening flips it to a
+  # positive consciously, not accidentally. Review-enforced meanwhile
+  # (AGENTS.md defect-shape 2).
+  - |
+    import * as path from "node:path";
+    import { isWindowsPath } from "./path-utils.js";
+    function resolveNonExisting(filePath: string) {
+      return path.dirname(filePath);
+    }
+    function normalize(filePath: string) {
+      if (isWindowsPath(filePath)) {
+        return resolveNonExisting(filePath);
+      }
+    }
+  - |
+    import * as path from "node:path";
+    import { isWindowsPath } from "./path-utils.js";
+    function f(filePath: string) {
+      if (isWindowsPath(filePath)) {
+        return path.win32.dirname(filePath);
+      }
+      return path.dirname(filePath);
+    }
+  - |
+    import * as path from "node:path";
+    import { isWindowsPath } from "./path-utils.js";
+    function f(filePath: string) {
+      return isWindowsPath(filePath)
+        ? path.win32.basename(filePath)
+        : path.basename(filePath);
+    }
+  - |
+    import { dirname } from "node:path";
+    function f(filePath: string) {
+      if (someOtherPredicate(filePath)) {
+        return dirname(filePath);
+      }
+    }
+invalid:
+  # Fresh probes from the #1339 review, encoded so they stay covered.
+  - |
+    import * as path from "node:path";
+    import { isWindowsPath } from "./path-utils.js";
+    function f(filePath: string) {
+      if (isWindowsPath(filePath)) {
+        for (const part of parts) {
+          const nested = path.join(filePath, part);
+        }
+      }
+    }
+  - |
+    import * as path from "node:path";
+    import { isWindowsPath } from "./path-utils.js";
+    function f(filePath: string) {
+      if (isWindowsPath(filePath)) {
+        return filePath.split(path.sep);
+      }
+    }
+  - |
+    import * as path from "node:path";
+    import { isFullyQualifiedWin32 } from "./path-utils.js";
+    function f(filePath: string, other: boolean) {
+      if (isFullyQualifiedWin32(filePath) && other) {
+        return path.basename(filePath);
+      }
+    }
+  - |
+    import * as path from "node:path";
+    import { isWindowsPath } from "./path-utils.js";
+    function f(filePath: string) {
+      if (isWindowsPath(filePath)) {
+        return path.dirname(filePath);
+      }
+    }
+  - |
+    import { join, sep } from "node:path";
+    import { isFullyQualifiedWin32 } from "./path-utils.js";
+    function f(filePath: string) {
+      if (isFullyQualifiedWin32(filePath)) {
+        return join(filePath, sep);
+      }
+    }

package/rules/ast-grep-rules/rule-tests/no-case-declarations-js-test.yml

@@ -7,6 +7,16 @@ valid:
         break;
       }
     }
+  - |
+    switch (x) {
+      case 1: {
+        if (x) {
+          const y = 1;
+          consume(y);
+        }
+        break;
+      }
+    }
   - |
     var z = 1; switch (x) { case 1: break; }
 invalid:

package/rules/ast-grep-rules/rule-tests/no-case-declarations-test.yml

@@ -7,6 +7,16 @@ valid:
         break;
       }
     }
+  - |
+    switch (x) {
+      case 1: {
+        if (x) {
+          const y = 1;
+          consume(y);
+        }
+        break;
+      }
+    }
   - |
     var z = 1; switch (x) { case 1: break; }
 invalid:

package/rules/ast-grep-rules/rule-tests/no-javascript-url-js-test.yml

@@ -8,7 +8,17 @@ valid:
     it("rejects javascript: urls", () => {
       expect(openBrowser("javascript:alert(1)")).toBe(false);
     });
+  - 'const links = arr.filter((l) => !l.startsWith("javascript:"))'
+  - 'if (url.includes("javascript:")) return null;'
+  - 'const isBlocked = url === "javascript:alert(1)"'
+  - 'const safe = url !== "javascript:alert(1)"'
+  - 'url?.startsWith("javascript:")'
 invalid:
   - 'a.href = "javascript:alert(1)"'
   - 'a.href = "javascript:void(0)"'
   - 'const u = "javascript:alert(1)"'
+  - 'a.href = str.replace("http:", "javascript:alert(1)")'
+  - 'if (candidates.includes(link.href = "javascript:alert(1)")) { doThing(); }'
+  - 'someArray.includes((a.href = "javascript:alert(1)", true));'
+  - 'if ((a.href = "javascript:alert(1)") === expected) { doThing(); }'
+  - 'const ok = (typeof (a.href = "javascript:alert(1)")) === "string";'

package/rules/ast-grep-rules/rule-tests/no-javascript-url-test.yml

@@ -12,7 +12,17 @@ valid:
     test.each(["javascript:alert(1)"])("rejects %s", (url) => {
       expect(openBrowser(url)).toBe(false);
     });
+  - 'const links = arr.filter((l) => !l.startsWith("javascript:"))'
+  - 'if (url.includes("javascript:")) return null;'
+  - 'const isBlocked = url === "javascript:alert(1)"'
+  - 'const safe = url !== "javascript:alert(1)"'
+  - 'url?.startsWith("javascript:")'
 invalid:
   - 'a.href = "javascript:alert(1)"'
   - 'a.href = "javascript:void(0)"'
   - 'const u = "javascript:alert(1)"'
+  - 'a.href = str.replace("http:", "javascript:alert(1)")'
+  - 'if (candidates.includes(link.href = "javascript:alert(1)")) { doThing(); }'
+  - 'someArray.includes((a.href = "javascript:alert(1)", true));'
+  - 'if ((a.href = "javascript:alert(1)") === expected) { doThing(); }'
+  - 'const ok = (typeof (a.href = "javascript:alert(1)")) === "string";'

package/rules/ast-grep-rules/rule-tests/no-nested-links-test.yml

@@ -2,7 +2,17 @@ id: no-nested-links
 valid:
   - '<a href="/x">link</a>'
   - '<a href="/x"><span>nested</span></a>'
+  - '<div><a href="/x">link</a></div>'
+  - '<div><a href="/x">first</a><a href="/y">second</a></div>'
 invalid:
   - '<a href="/x">outer <a href="/y">inner</a></a>'
   - '<a><a>double</a></a>'
-  - '<div><a href="/x">link</a></div>'
+  - '<a href="/x"><span><a href="/y">inner</a></span></a>'
+  - |
+      <a href="/outer">
+        <span>
+          <a href="/middle">
+            <em><a href="/inner">deep</a></em>
+          </a>
+        </span>
+      </a>

package/rules/ast-grep-rules/rule-tests/no-raw-json-store-write-test.yml

@@ -0,0 +1,17 @@
+id: no-raw-json-store-write
+valid:
+  - "writeFileAtomic(file, JSON.stringify(data))"
+  - "writeFileSync(file, text)"
+invalid:
+  # #1349 review: encoding/replacer/indent variants are the same raw write.
+  - "writeFileSync(file, JSON.stringify(data), 'utf8')"
+  - "writeFileSync(file, JSON.stringify(data, null, 2))"
+  - "fs.writeFileSync(file, JSON.stringify(data, replacer), { encoding: 'utf8' })"
+  - "writeFileSync(file, JSON.stringify(data))"
+  - "fs.writeFileSync(file, JSON.stringify(data))"
+  - |
+    const payload = JSON.stringify(data)
+    writeFileSync(file, payload)
+  - |
+    const payload = JSON.stringify(data)
+    fs.writeFileSync(file, payload)

package/rules/ast-grep-rules/rule-tests/no-win32-isabsolute-for-qualification-test.yml

@@ -0,0 +1,16 @@
+id: no-win32-isabsolute-for-qualification
+valid:
+  # DOCUMENTED LIMITATION (#1349 review): the aliased parser -- the shape
+  # the actual pre-#1330 defect used -- is beyond ast-grep's reach and is
+  # deliberately a negative; flipping it to invalid requires an engine
+  # capability decision. Review-enforced meanwhile (AGENTS.md shape 2).
+  - |
+    const w = path.win32;
+    if (w.isAbsolute(filePath)) {
+      return true;
+    }
+  - "isFullyQualified(filePath)"
+  - "path.posix.isAbsolute(filePath)"
+invalid:
+  - "path.win32.isAbsolute(filePath)"
+  - "win32.isAbsolute(filePath)"

package/rules/ast-grep-rules/rule-tests/strict-equality-js-test.yml

@@ -3,6 +3,9 @@ valid:
   - 'a === b'
   - 'a !== b'
   - 'a = b'
+  - 'value == null'
+  - 'null == value'
+  - 'value == undefined'
 invalid:
   - 'a == b'
   - 'a == "x"'

package/rules/ast-grep-rules/rule-tests/strict-equality-test.yml

@@ -3,6 +3,9 @@ valid:
   - 'a === b'
   - 'a !== b'
   - 'a = b'
+  - 'value == null'
+  - 'null == value'
+  - 'value == undefined'
 invalid:
   - 'a == b'
   - 'a == "x"'

package/rules/ast-grep-rules/rule-tests/strict-inequality-js-test.yml

@@ -3,6 +3,9 @@ valid:
   - 'a !== b'
   - 'a === b'
   - 'a < b'
+  - 'value != null'
+  - 'null != value'
+  - 'value != undefined'
 invalid:
   - 'a != b'
   - 'a != "x"'

package/rules/ast-grep-rules/rule-tests/strict-inequality-test.yml

@@ -3,6 +3,10 @@ valid:
   - 'a !== b'
   - 'a === b'
   - 'a < b'
+  - 'value != null'
+  - 'null != value'
+  - 'thinkingMap != null'
+  - 'value != undefined'
 invalid:
   - 'a != b'
   - 'a != "x"'