Fixes for failing PR checks

- Syncs "server/dist/**"
- Applies "codeql query format" rules for all server/ql/**/*.ql files
- Fixes formatting of markdown text in query markdown docs
- Addresses other PR review feedback for #168

Author: data-douser

server/dist/codeql-development-mcp-server.js

@@ -18,6 +18,4 @@ query predicate nodes(Node node, string property, string value) {
   value = node.toString()
 }
 
-query predicate edges(Node pred, Node succ) {
-  pred.getASuccessor() = succ
-}
+query predicate edges(Node pred, Node succ) { pred.getASuccessor() = succ }

server/dist/codeql-development-mcp-server.js.map

@@ -18,9 +18,7 @@ external string sourceFunction();
 /**
  * Gets a single source function name from the comma-separated list.
  */
-string getSourceFunctionName() {
-  result = sourceFunction().splitAt(",").trim()
-}
+string getSourceFunctionName() { result = sourceFunction().splitAt(",").trim() }
 
 /**
  * Gets a function by matching against the selected source function names.
@@ -30,7 +28,8 @@ Function getSourceFunction() {
     selectedFunc = getSourceFunctionName() and
     (
       // Match by exact function name
-      result.getName() = selectedFunc or
+      result.getName() = selectedFunc
+      or
       // Match by qualified name
       result.getQualifiedName() = selectedFunc
     )
@@ -46,10 +45,7 @@ where
     source = getSourceFunction()
     or
     // Fallback for unit tests: include test files
-    (
-      not exists(getSourceFunction()) and
-      source.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
-    )
+    not exists(getSourceFunction()) and
+    source.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
   )
-select call,
-  "Call from `" + source.getQualifiedName() + "` to `" + callee.getQualifiedName() + "`"
+select call, "Call from `" + source.getQualifiedName() + "` to `" + callee.getQualifiedName() + "`"

server/ql/actions/tools/src/PrintCFG/PrintCFG.ql

@@ -33,7 +33,7 @@ void source() {
 }
 ```
 
-Running with `sourceFunction = "source"` and `targetFunction = "target"` produces results showing each call site on the path with the message pattern ``Reachable call from `source`to`intermediate```.
+Running with `sourceFunction = "source"` and `targetFunction = "target"` produces results showing each call site on the path with a message like: "Reachable call from `source` to `intermediate`".
 
 ## Output Format
 

server/ql/cpp/tools/src/CallGraphFrom/CallGraphFrom.ql

@@ -39,7 +39,8 @@ Function getSourceFunction() {
     selectedFunc = getSourceFunctionName() and
     (
       // Match by exact function name
-      result.getName() = selectedFunc or
+      result.getName() = selectedFunc
+      or
       // Match by qualified name
       result.getQualifiedName() = selectedFunc
     )
@@ -54,7 +55,8 @@ Function getTargetFunction() {
     selectedFunc = getTargetFunctionName() and
     (
       // Match by exact function name
-      result.getName() = selectedFunc or
+      result.getName() = selectedFunc
+      or
       // Match by qualified name
       result.getQualifiedName() = selectedFunc
     )
@@ -82,11 +84,9 @@ where
     )
     or
     // Fallback for unit tests: include test files
-    (
-      not exists(getSourceFunctionName()) and
-      not exists(getTargetFunctionName()) and
-      caller.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
-    )
+    not exists(getSourceFunctionName()) and
+    not exists(getTargetFunctionName()) and
+    caller.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
   )
 select call,
   "Reachable call from `" + caller.getQualifiedName() + "` to `" + callee.getQualifiedName() + "`"

server/ql/cpp/tools/src/CallGraphFromTo/CallGraphFromTo.md

@@ -18,9 +18,7 @@ external string targetFunction();
 /**
  * Gets a single target function name from the comma-separated list.
  */
-string getTargetFunctionName() {
-  result = targetFunction().splitAt(",").trim()
-}
+string getTargetFunctionName() { result = targetFunction().splitAt(",").trim() }
 
 /**
  * Gets a function by matching against the selected target function names.
@@ -30,7 +28,8 @@ Function getTargetFunction() {
     selectedFunc = getTargetFunctionName() and
     (
       // Match by exact function name
-      result.getName() = selectedFunc or
+      result.getName() = selectedFunc
+      or
       // Match by qualified name
       result.getQualifiedName() = selectedFunc
     )
@@ -46,10 +45,7 @@ where
     target = getTargetFunction()
     or
     // Fallback for unit tests: include test files
-    (
-      not exists(getTargetFunction()) and
-      target.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
-    )
+    not exists(getTargetFunction()) and
+    target.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
   )
-select call,
-  "Call to `" + target.getQualifiedName() + "` from `" + caller.getQualifiedName() + "`"
+select call, "Call to `" + target.getQualifiedName() + "` from `" + caller.getQualifiedName() + "`"

server/ql/cpp/tools/src/CallGraphFromTo/CallGraphFromTo.ql

@@ -18,9 +18,7 @@ external string selectedSourceFiles();
 /**
  * Gets a single source file from the comma-separated list.
  */
-string getSelectedSourceFile() {
-  result = selectedSourceFiles().splitAt(",").trim()
-}
+string getSelectedSourceFile() { result = selectedSourceFiles().splitAt(",").trim() }
 
 /**
  * Gets a file by matching against the selected source file paths.
@@ -30,11 +28,14 @@ File getSelectedFile() {
     selectedFile = getSelectedSourceFile() and
     (
       // Match by exact relative path from source root
-      result.getRelativePath() = selectedFile or
+      result.getRelativePath() = selectedFile
+      or
       // Match by file name if no path separators
-      (not selectedFile.matches("%/%") and result.getBaseName() = selectedFile) or
+      not selectedFile.matches("%/%") and result.getBaseName() = selectedFile
+      or
       // Match by ending path component
-      result.getAbsolutePath().suffix(result.getAbsolutePath().length() - selectedFile.length()) = selectedFile
+      result.getAbsolutePath().suffix(result.getAbsolutePath().length() - selectedFile.length()) =
+        selectedFile
     )
   )
 }
@@ -49,9 +50,7 @@ class Cfg extends PrintAstConfiguration {
     decl.getFile() = getSelectedFile()
     or
     // Fallback for unit tests: include test files
-    (
-      not exists(getSelectedFile()) and
-      decl.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
-    )
+    not exists(getSelectedFile()) and
+    decl.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
   }
 }

server/ql/cpp/tools/src/CallGraphTo/CallGraphTo.ql

@@ -17,6 +17,4 @@ query predicate nodes(ControlFlowNode node, string property, string value) {
   value = node.toString()
 }
 
-query predicate edges(ControlFlowNode pred, ControlFlowNode succ) {
-  pred.getASuccessor() = succ
-}
+query predicate edges(ControlFlowNode pred, ControlFlowNode succ) { pred.getASuccessor() = succ }

server/ql/cpp/tools/src/PrintAST/PrintAST.ql

@@ -18,9 +18,7 @@ external string sourceFunction();
 /**
  * Gets a single source method name from the comma-separated list.
  */
-string getSourceFunctionName() {
-  result = sourceFunction().splitAt(",").trim()
-}
+string getSourceFunctionName() { result = sourceFunction().splitAt(",").trim() }
 
 /**
  * Gets a method by matching against the selected source method names.
@@ -41,10 +39,7 @@ where
     source = getSourceFunction()
     or
     // Fallback for unit tests: include test files
-    (
-      not exists(getSourceFunction()) and
-      source.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
-    )
+    not exists(getSourceFunction()) and
+    source.getFile().getParentContainer().getParentContainer().getBaseName() = "test"
   )
-select call,
-  "Call from `" + source.getName() + "` to `" + callee.getName() + "`"
+select call, "Call from `" + source.getName() + "` to `" + callee.getName() + "`"