@kernlang/review-python 3.4.6-canary.42.1.df0d73a1 → 3.4.6-canary.45.1.130ca3d2

package/dist/mapper.js

@@ -594,7 +594,10 @@ function extractFastApiSuccessStatusCodes(decText, fnDef, source) {
     //    rather than a name whitelist; the API_SUCCESS_STATUS_CODES filter
     //    keeps the noise tax low.
     const mutationCodes = new Set();
-    const mutateRe = /\b[A-Za-z_]\w*\.status_code\s*=\s*([^\n;]+)/g;
+    // `=(?!=)` distinguishes assignment from `==` comparison so
+    // `if response.status_code == 200:` doesn't masquerade as a dynamic
+    // mutation (forge round, Claude engine).
+    const mutateRe = /\b[A-Za-z_]\w*\.status_code\s*=(?!=)\s*([^\n;]+)/g;
     for (const match of bodyText.matchAll(mutateRe)) {
         const code = parseFastApiStatusValue(match[1].trim());
         if (code === undefined)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kernlang/review-python",
-  "version": "3.4.6-canary.42.1.df0d73a1",
+  "version": "3.4.6-canary.45.1.130ca3d2",
   "description": "Python concept mapper for kern review — tree-sitter based",
   "type": "module",
   "main": "dist/index.js",
@@ -8,8 +8,8 @@
   "dependencies": {
     "tree-sitter": "^0.25.0",
     "tree-sitter-python": "^0.25.0",
-    "@kernlang/core": "3.4.6-canary.42.1.df0d73a1",
-    "@kernlang/review": "3.4.6-canary.42.1.df0d73a1"
+    "@kernlang/core": "3.4.6-canary.45.1.130ca3d2",
+    "@kernlang/review": "3.4.6-canary.45.1.130ca3d2"
   },
   "devDependencies": {
     "ts-morph": "^28.0.0",

package/src/mapper.ts

@@ -686,7 +686,10 @@ function extractFastApiSuccessStatusCodes(
   //    rather than a name whitelist; the API_SUCCESS_STATUS_CODES filter
   //    keeps the noise tax low.
   const mutationCodes = new Set<number>();
-  const mutateRe = /\b[A-Za-z_]\w*\.status_code\s*=\s*([^\n;]+)/g;
+  // `=(?!=)` distinguishes assignment from `==` comparison so
+  // `if response.status_code == 200:` doesn't masquerade as a dynamic
+  // mutation (forge round, Claude engine).
+  const mutateRe = /\b[A-Za-z_]\w*\.status_code\s*=(?!=)\s*([^\n;]+)/g;
   for (const match of bodyText.matchAll(mutateRe)) {
     const code = parseFastApiStatusValue(match[1].trim());
     if (code === undefined) sawDynamic = true;

package/tests/route-entrypoints.test.ts

@@ -339,6 +339,25 @@ def create():
     expect(routes[0].payload.successStatusCodes).toEqual([202]);
   });
 
+  it('does NOT treat `== 200` comparison as a dynamic mutation (forge round, Claude engine)', () => {
+    // `if response.status_code == 200:` is a comparison, not an assignment.
+    // Without the `=(?!=)` lookahead, the mutation regex captured `200) ...`
+    // as a dynamic RHS and incorrectly marked the route resolved=false.
+    const routes = routePayloads(`
+from fastapi import Response
+
+@router.post("/items", status_code=201)
+def create(response: Response):
+    if response.status_code == 200:
+        return {"unreachable": True}
+    return {"id": 1}
+`);
+    expect(routes[0].payload.successStatusCodesResolved).toBe(true);
+    // Decorator says 201, plain returns inherit it. The `==` comparison
+    // contributes nothing.
+    expect(routes[0].payload.successStatusCodes).toEqual([201]);
+  });
+
   it('detects status_code mutation under non-conventional Response param name (Codex impl-review #2)', () => {
     // Param injected as `out: Response` instead of the conventional
     // `response`. Without broadened receiver matching, mutation is missed and