fix OOM

Author: Frotty

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstio/WurstCompilerJassImpl.java

@@ -645,12 +645,9 @@ private void beginPhase(int phase, String description) {
 
     private void printDebugImProg(String debugFile) {
         if (!errorHandler.isUnitTestMode() || !errorHandler.isOutputTestSource()) {
-            // output only in unit test mode
             return;
         }
-
         try {
-            // TODO remove test output
             File file = new File(debugFile);
             file.getParentFile().mkdirs();
             try (Writer w = Files.newWriter(file, Charsets.UTF_8)) {

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/attributes/ErrorHandler.java

@@ -19,7 +19,9 @@ public class ErrorHandler {
 
     private final WurstGui gui;
     private boolean unitTestMode = false;
-    public static boolean outputTestSource = true;
+    /** Write intermediate IM debug files during tests. Off by default — only tests that
+     *  explicitly assert on IM output (e.g. DeterministicChecks) should set this to true. */
+    public static boolean outputTestSource = false;
 
     public ErrorHandler(WurstGui gui) {
         this.gui = gui;

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/attributes/HasAnnotation.java

@@ -7,9 +7,10 @@
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.WeakHashMap;
 
 public class HasAnnotation {
-    // OPTIMIZATION 1: Cache normalized annotations
+    // OPTIMIZATION 1: Cache normalized annotations (String→String: safe as static, strings are interned/small)
     private static final Map<String, String> normalizationCache = new HashMap<>();
 
     @NotNull
@@ -74,11 +75,19 @@ public static Annotation getAnnotation(NameDef e, String annotation) {
         return null;
     }
 
-    // OPTIMIZATION 8: Cache normalized annotation types per Annotation object
-    private static final Map<Annotation, String> annotationTypeCache = new HashMap<>();
+    // OPTIMIZATION 8: Cache normalized annotation types per Annotation object.
+    // WeakHashMap: entries are collected automatically when the AST node (Annotation) is GC'd,
+    // preventing this static cache from pinning entire compilation trees across tests.
+    private static final Map<Annotation, String> annotationTypeCache = new WeakHashMap<>();
 
     private static String getNormalizedType(Annotation a) {
         return annotationTypeCache.computeIfAbsent(a,
             ann -> normalizeAnnotation(ann.getAnnotationType()));
     }
+
+    /** Explicitly clear both caches. Called from GlobalCaches.clearAll() between tests. */
+    public static void clearCaches() {
+        annotationTypeCache.clear();
+        normalizationCache.clear();
+    }
 }

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/translation/imtranslation/LuaNativeLowering.java

@@ -298,7 +298,7 @@ private static boolean hasHandleParam(ImFunction f) {
     }
 
     /** Returns true for WC3 handle types (ImSimpleType that is not int/real/boolean/string). */
-    static boolean isHandleType(ImType type) {
+    public static boolean isHandleType(ImType type) {
         if (!(type instanceof ImSimpleType)) {
             return false;
         }

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/validation/GlobalCaches.java

@@ -1,6 +1,7 @@
 package de.peeeq.wurstscript.validation;
 
 import de.peeeq.wurstscript.ast.Element;
+import de.peeeq.wurstscript.attributes.HasAnnotation;
 import de.peeeq.wurstscript.intermediatelang.ILconst;
 import de.peeeq.wurstscript.intermediatelang.interpreter.LocalState;
 import it.unimi.dsi.fastutil.objects.Object2ObjectOpenHashMap;
@@ -142,6 +143,7 @@ public static void clearAll() {
         LOCAL_STATE_CACHE.clear();
         LOCAL_STATE_NOARG_CACHE.clear();
         lookupCache.clear();
+        HasAnnotation.clearCaches();
     }
 
     public enum LookupType {

de.peeeq.wurstscript/src/test/java/tests/wurstscript/tests/WurstScriptTest.java

@@ -30,6 +30,7 @@
 import de.peeeq.wurstscript.utils.Utils;
 import de.peeeq.wurstscript.validation.GlobalCaches;
 import org.testng.Assert;
+import org.testng.annotations.AfterMethod;
 import org.testng.annotations.BeforeMethod;
 
 import java.io.*;
@@ -66,6 +67,13 @@ public void _clearBefore() {
         GlobalCaches.clearAll();
     }
 
+    @AfterMethod(alwaysRun = true)
+    public void _clearAfter() {
+        // Release all AST strong-refs held by caches so the GC can reclaim the
+        // stdlib copy from the previous test before the next test allocates its own.
+        GlobalCaches.clearAll();
+    }
+
     class TestConfig {
         private final String name;
         private boolean withStdLib;