pi-lens 4.4.0 → 4.4.2

package/rules/ast-grep-rules/rule-tests/no-identity-operator-on-literals-test.yml

@@ -4,11 +4,21 @@ valid:
   - "if x is not None:"
   - "if x is y:"  # both are variables
   - "if obj is SingletonClass.instance:"  # pattern
+  - "if getattr(position, \"closed\", None) is True:"
+  - "if getattr(self.profit_take, \"last_close_was_race_drop\", False) is True:"
+  - "if result is not True:"
+  - "if result is not False:"
 invalid:
   - "if 1 is '1':"
   - "if x is 'literal':"
   - "if 'foo' is x:"
   - "if x is not 42:"
-  - "if x is True:"
-  - "if x is not False:"
+  - "if x is 1:"
+  - "if x is not 1.5:"
+  - "if x is -1:"
+  - "if -1 is x:"
+  - "if x is not -2:"
+  - "if x is -1.5:"
+  - "if -1.5 is x:"
+  - "if x is not -2.0:"
   - "if x is []:"  # list literal

package/rules/ast-grep-rules/rule-tests/throw-new-error-js-test.yml

@@ -8,3 +8,5 @@ invalid:
   - 'throw TypeError("x")'
   - 'throw RangeError("x")'
   - 'throw SyntaxError("x")'
+  - 'throw URIError("x")'
+  - 'throw EvalError("x")'

package/rules/ast-grep-rules/rule-tests/throw-new-error-test.yml

@@ -8,3 +8,5 @@ invalid:
   - 'throw TypeError("x")'
   - 'throw RangeError("x")'
   - 'throw SyntaxError("x")'
+  - 'throw URIError("x")'
+  - 'throw EvalError("x")'

package/rules/ast-grep-rules/rules/no-identity-operator-on-literals.yml

@@ -13,11 +13,13 @@ note: |
   literal-kind check. This ensures we ONLY flag literal-vs-X
   comparisons (not `x == "literal"` which is the correct form).
 
-  Two regex variants cover `is True/False` (where `True`/`False`
-  parse as identifiers, not a `true`/`false` AST kind). Two more
-  cover `is not True/False`. The remaining variants use
+  Identity against the bool singletons is deliberately excluded:
+  `is True`, `is False`, `is not True`, and `is not False` are
+  correct singleton checks in Python, including tri-state and mock-
+  robustness checks. The remaining variants use
   `has: kind: <literal>` as direct children of the
-  comparison_operator.
+  comparison_operator. Negative numeric literals are unary_operator
+  children, so their nested integer/float kinds are covered explicitly.
 
   Direct children only — no `stopBy: end` — so string subscripts
   like `summary["verified"] is True` don't fire (the string is
@@ -29,14 +31,6 @@ metadata:
   sonar: S3403
 rule:
   any:
-    - kind: comparison_operator
-      regex: 'is\s+(True|False|true|false)\b'
-    - kind: comparison_operator
-      regex: '\b(True|False|true|false)\s+is\b'
-    - kind: comparison_operator
-      regex: 'is\s+not\s+(True|False|true|false)\b'
-    - kind: comparison_operator
-      regex: '\b(True|False|true|false)\s+is\s+not\b'
     - all:
         - kind: comparison_operator
           regex: '\bis\b'
@@ -47,11 +41,25 @@ rule:
           regex: '\bis\b'
         - has:
             kind: integer
+    - all:
+        - kind: comparison_operator
+          regex: '\bis\b'
+        - has:
+            kind: unary_operator
+            has:
+              kind: float
     - all:
         - kind: comparison_operator
           regex: '\bis\b'
         - has:
             kind: float
+    - all:
+        - kind: comparison_operator
+          regex: '\bis\b'
+        - has:
+            kind: unary_operator
+            has:
+              kind: integer
     - all:
         - kind: comparison_operator
           regex: '\bis\b'

package/rules/ast-grep-rules/rules/throw-new-error-js.yml

@@ -1,9 +1,9 @@
 id: throw-new-error-js
 language: JavaScript
-severity: error
+severity: warning
 message: "Use new Error() not Error() — explicit construction is clearer"
 note: |
-  throw Error() works by accident. throw new Error() is the correct pattern.
+  This is a consistency/style rule: Error() is spec-equivalent to new Error().
 rule:
   any:
     - pattern: "throw Error($$$ARGS)"

package/rules/ast-grep-rules/rules/throw-new-error.yml

@@ -1,9 +1,9 @@
 id: throw-new-error
 language: TypeScript
-severity: error
+severity: warning
 message: "Use new Error() not Error() — explicit construction is clearer"
 note: |
-  throw Error() works by accident. throw new Error() is the correct pattern.
+  This is a consistency/style rule: Error() is spec-equivalent to new Error().
 rule:
   any:
     - pattern: "throw Error($$$ARGS)"

package/dist/chunk-54B42YRX.js

@@ -1,27 +0,0 @@
-import { createRequire as __pilensCreateRequire } from "node:module"; const require = __pilensCreateRequire(import.meta.url);
-import {
-  GovulncheckClient,
-  parseGovulncheckJson
-} from "./chunk-NXPS2L6U.js";
-import "./chunk-7F72PXY2.js";
-import "./chunk-I7SPGOWI.js";
-import "./chunk-NTUJEGDZ.js";
-import "./chunk-EOKRQ3SA.js";
-import "./chunk-WONERJTP.js";
-import "./chunk-VN2AXLEX.js";
-import "./chunk-BW5KFIQC.js";
-import "./chunk-DQ2NZ7C4.js";
-import "./chunk-VNMT7C64.js";
-import "./chunk-2EECBNC5.js";
-import "./chunk-N3YQJI6O.js";
-import "./chunk-O6TQT6RI.js";
-import "./chunk-5THXD6X5.js";
-import "./chunk-UK3CMEAL.js";
-import "./chunk-LJDODBBZ.js";
-import "./chunk-ABBOA7UD.js";
-import "./chunk-Q5U6FMDI.js";
-import "./chunk-NXL4FFQQ.js";
-export {
-  GovulncheckClient,
-  parseGovulncheckJson
-};

package/dist/chunk-CGGAIP7J.js

@@ -1,29 +0,0 @@
-import { createRequire as __pilensCreateRequire } from "node:module"; const require = __pilensCreateRequire(import.meta.url);
-import {
-  importFactProvider,
-  importFactsApplyTo
-} from "./chunk-V5PXFSZK.js";
-import "./chunk-JOLITGCC.js";
-import "./chunk-LW6XKDZQ.js";
-import "./chunk-Z54ZSOBZ.js";
-import "./chunk-TGPXLY2J.js";
-import "./chunk-QOZXJ4NA.js";
-import "./chunk-NTUJEGDZ.js";
-import "./chunk-WONERJTP.js";
-import "./chunk-VN2AXLEX.js";
-import "./chunk-5ZEJHV35.js";
-import "./chunk-DQ2NZ7C4.js";
-import "./chunk-VNMT7C64.js";
-import "./chunk-2EECBNC5.js";
-import "./chunk-N3YQJI6O.js";
-import "./chunk-O6TQT6RI.js";
-import "./chunk-5THXD6X5.js";
-import "./chunk-UK3CMEAL.js";
-import "./chunk-LJDODBBZ.js";
-import "./chunk-ABBOA7UD.js";
-import "./chunk-Q5U6FMDI.js";
-import "./chunk-NXL4FFQQ.js";
-export {
-  importFactProvider,
-  importFactsApplyTo
-};

package/dist/chunk-D6QS3U6M.js

@@ -1,30 +0,0 @@
-import { createRequire as __pilensCreateRequire } from "node:module"; const require = __pilensCreateRequire(import.meta.url);
-import {
-  calcCyclomaticComplexity,
-  functionFactProvider
-} from "./chunk-YLF3PPT5.js";
-import "./chunk-JOLITGCC.js";
-import "./chunk-5W3R6LFT.js";
-import "./chunk-LW6XKDZQ.js";
-import "./chunk-Z54ZSOBZ.js";
-import "./chunk-TGPXLY2J.js";
-import "./chunk-QOZXJ4NA.js";
-import "./chunk-NTUJEGDZ.js";
-import "./chunk-WONERJTP.js";
-import "./chunk-VN2AXLEX.js";
-import "./chunk-5ZEJHV35.js";
-import "./chunk-DQ2NZ7C4.js";
-import "./chunk-VNMT7C64.js";
-import "./chunk-2EECBNC5.js";
-import "./chunk-N3YQJI6O.js";
-import "./chunk-O6TQT6RI.js";
-import "./chunk-5THXD6X5.js";
-import "./chunk-UK3CMEAL.js";
-import "./chunk-LJDODBBZ.js";
-import "./chunk-ABBOA7UD.js";
-import "./chunk-Q5U6FMDI.js";
-import "./chunk-NXL4FFQQ.js";
-export {
-  calcCyclomaticComplexity,
-  functionFactProvider
-};

package/dist/chunk-HMFXILPV.js

@@ -1,83 +0,0 @@
-import { createRequire as __pilensCreateRequire } from "node:module"; const require = __pilensCreateRequire(import.meta.url);
-import {
-  _getWarmWordIndexCacheStateForTests,
-  _resetWarmWordIndexCacheForTests,
-  acquireWarmWordIndex,
-  analyzeFile,
-  summarizeWarningTiers
-} from "./chunk-TLGFBM3F.js";
-import "./chunk-C2AUYXQH.js";
-import "./chunk-I3R7GKDR.js";
-import "./chunk-FC4RP4I2.js";
-import "./chunk-DFVMIKMQ.js";
-import "./chunk-TO6WL3GM.js";
-import "./chunk-7F72PXY2.js";
-import "./chunk-RPD5K5NC.js";
-import "./chunk-ZMOL6RQC.js";
-import "./chunk-I7SPGOWI.js";
-import "./chunk-AS44CFQ4.js";
-import "./chunk-7FBF7NPR.js";
-import "./chunk-HHYAELLJ.js";
-import "./chunk-V5PXFSZK.js";
-import "./chunk-YLF3PPT5.js";
-import "./chunk-JOLITGCC.js";
-import "./chunk-E2WFBRHM.js";
-import "./chunk-6VCD4VKZ.js";
-import "./chunk-F3P7VQ7H.js";
-import "./chunk-LSMJ6KBE.js";
-import "./chunk-3MED75HT.js";
-import "./chunk-3I5QFT3N.js";
-import "./chunk-TVN5N6GY.js";
-import "./chunk-ARFCXG7S.js";
-import "./chunk-OAXNDZJ7.js";
-import "./chunk-XIMYZ3JA.js";
-import "./chunk-B3I2UUT4.js";
-import "./chunk-DIULIWKU.js";
-import "./chunk-VDBPVIKD.js";
-import "./chunk-5W3R6LFT.js";
-import "./chunk-LW6XKDZQ.js";
-import "./chunk-Z54ZSOBZ.js";
-import "./chunk-TGPXLY2J.js";
-import "./chunk-QOZXJ4NA.js";
-import "./chunk-V7R425XG.js";
-import "./chunk-PAK6IDME.js";
-import "./chunk-GUOGVTCQ.js";
-import "./chunk-AJVLWNFN.js";
-import "./chunk-YGVYKNSN.js";
-import "./chunk-LNAKEO63.js";
-import "./chunk-Z3CCVDUB.js";
-import "./chunk-PWFHBTSF.js";
-import "./chunk-2BEMIWEW.js";
-import "./chunk-OQGFCZJ4.js";
-import "./chunk-RUOG5D2C.js";
-import "./chunk-U3VGR4VU.js";
-import "./chunk-NTUJEGDZ.js";
-import "./chunk-EOKRQ3SA.js";
-import "./chunk-WONERJTP.js";
-import "./chunk-22AVIGXS.js";
-import "./chunk-GVXA6CCE.js";
-import "./chunk-6QIUX4EP.js";
-import "./chunk-VHQLPIRJ.js";
-import "./chunk-VN2AXLEX.js";
-import "./chunk-BW5KFIQC.js";
-import "./chunk-QYF5S6P6.js";
-import "./chunk-5ZEJHV35.js";
-import "./chunk-DQ2NZ7C4.js";
-import "./chunk-VNMT7C64.js";
-import "./chunk-2EECBNC5.js";
-import "./chunk-N3YQJI6O.js";
-import "./chunk-O6TQT6RI.js";
-import "./chunk-5THXD6X5.js";
-import "./chunk-UK3CMEAL.js";
-import "./chunk-LJDODBBZ.js";
-import "./chunk-NMABWBZN.js";
-import "./chunk-ABBOA7UD.js";
-import "./chunk-Q5U6FMDI.js";
-import "./chunk-NXL4FFQQ.js";
-export {
-  _getWarmWordIndexCacheStateForTests,
-  _resetWarmWordIndexCacheForTests,
-  acquireWarmWordIndex,
-  analyzeFile,
-  summarizeWarningTiers
-};