Fix for Superfluous trailing arguments (#6)

To fix this, update the call to `isDocumentWrite` so it matches the
function’s expected arity by removing the extra trailing `checker`
argument.

Best single fix without changing functionality:
- In `internal/codeguard/checks/support/typescript_semantic_runner.js`,
inside `analyzeCallExpression` around line 923, change:
  - `isDocumentWrite(node.expression, checker)`
  - to `isDocumentWrite(node.expression)`
  
This preserves runtime behavior (since extra args were ignored) while
resolving the CodeQL issue and improving call clarity. No imports, new
methods, or definitions are needed.


_Suggested fixes powered by Copilot Autofix. Review carefully before
merging._

Author: alxxjohn

internal/codeguard/checks/support/typescript_semantic_runner.js

@@ -920,7 +920,7 @@ function analyzeCallExpression(node, sourceFile, relPath, flavor, bindings, chec
     pushFinding("security", sourceFile, relPath, flavor, rule(flavor, "security.typescript.vm-dynamic-code", "security.javascript.vm-dynamic-code"), "warn", "Node vm dynamic code execution should be reviewed", node.expression.getStart(sourceFile));
   }
 
-  if (isInsertAdjacentHTML(node.expression) || isDocumentWrite(node.expression, checker)) {
+  if (isInsertAdjacentHTML(node.expression) || isDocumentWrite(node.expression)) {
     pushFinding("security", sourceFile, relPath, flavor, rule(flavor, "security.typescript.unsafe-html-sink", "security.javascript.unsafe-html-sink"), "warn", "unsafe HTML injection sink should be reviewed", node.expression.getStart(sourceFile));
   }