Shared: Only make unlifted models in case the API itself is relevant.

Author: michaelnebel

csharp/ql/src/utils/modelgenerator/internal/CaptureModels.qll

@@ -131,7 +131,10 @@ module ModelGeneratorInput implements ModelGeneratorInputSig<Location, CsharpDat
 
     Callable lift() { result = lift }
 
-    predicate isRelevant() { relevant(this) }
+    predicate isRelevant() {
+      relevant(this) and
+      not hasManualSummaryModel(this)
+    }
   }
 
   /**

java/ql/src/utils/modelgenerator/internal/CaptureModels.qll

@@ -110,7 +110,10 @@ module ModelGeneratorInput implements ModelGeneratorInputSig<Location, JavaDataF
 
     Callable lift() { result = lift }
 
-    predicate isRelevant() { relevant(this) }
+    predicate isRelevant() {
+      relevant(this) and
+      not hasManualSummaryModel(this)
+    }
   }
 
   private string isExtensible(Callable c) {

shared/mad/codeql/mad/modelgenerator/internal/ModelGeneratorImpl.qll

@@ -848,7 +848,11 @@ module MakeModelGenerator<
         input = parameterNodeAsContentInput(p) + printReadAccessPath(reads) and
         output = getContentOutput(returnNodeExt) + printStoreAccessPath(stores) and
         input != output and
-        (if mentionsField(reads) or mentionsField(stores) then lift = false else lift = true)
+        (
+          if mentionsField(reads) or mentionsField(stores)
+          then lift = false and api.isRelevant()
+          else lift = true
+        )
       )
     }