PCGen
diff --git a/‎.github/workflows/gradle-release-manual.yml‎
Lines changed: 4 additions & 2 deletions b/‎.github/workflows/gradle-release-manual.yml‎
Lines changed: 4 additions & 2 deletions
diff --git a/‎.github/workflows/gradle-release.yml‎
Lines changed: 4 additions & 2 deletions b/‎.github/workflows/gradle-release.yml‎
Lines changed: 4 additions & 2 deletions
diff --git a/‎.github/workflows/gradle-test.yml‎
Lines changed: 6 additions & 0 deletions b/‎.github/workflows/gradle-test.yml‎
Lines changed: 6 additions & 0 deletions
diff --git a/‎AGENTS.md‎
Lines changed: 1 addition & 2 deletions b/‎AGENTS.md‎
Lines changed: 1 addition & 2 deletions
diff --git a/‎PCGen-Formula/build.gradle‎
Lines changed: 63 additions & 156 deletions b/‎PCGen-Formula/build.gradle‎
Lines changed: 63 additions & 156 deletions
diff --git a/‎PCGen-Formula/code/src/java/module-info.java‎
Lines changed: 20 additions & 0 deletions b/‎PCGen-Formula/code/src/java/module-info.java‎
Lines changed: 20 additions & 0 deletions
diff --git a/‎…va/pcgen/base/formula/AddingFormula.java‎ ‎…va/pcgen/base/formula/AddingFormula.java‎code/src/java/pcgen/base/formula/AddingFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/AddingFormula.java b/‎…va/pcgen/base/formula/AddingFormula.java‎ ‎…va/pcgen/base/formula/AddingFormula.java‎code/src/java/pcgen/base/formula/AddingFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/AddingFormula.java
diff --git a/‎…/pcgen/base/formula/DividingFormula.java‎ ‎…/pcgen/base/formula/DividingFormula.java‎code/src/java/pcgen/base/formula/DividingFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/DividingFormula.java b/‎…/pcgen/base/formula/DividingFormula.java‎ ‎…/pcgen/base/formula/DividingFormula.java‎code/src/java/pcgen/base/formula/DividingFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/DividingFormula.java
diff --git a/‎…gen/base/formula/MultiplyingFormula.java‎ ‎…gen/base/formula/MultiplyingFormula.java‎code/src/java/pcgen/base/formula/MultiplyingFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/MultiplyingFormula.java b/‎…gen/base/formula/MultiplyingFormula.java‎ ‎…gen/base/formula/MultiplyingFormula.java‎code/src/java/pcgen/base/formula/MultiplyingFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/MultiplyingFormula.java
diff --git a/‎…pcgen/base/formula/ReferenceFormula.java‎ ‎…pcgen/base/formula/ReferenceFormula.java‎code/src/java/pcgen/base/formula/ReferenceFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/ReferenceFormula.java b/‎…pcgen/base/formula/ReferenceFormula.java‎ ‎…pcgen/base/formula/ReferenceFormula.java‎code/src/java/pcgen/base/formula/ReferenceFormula.java renamed to PCGen-Formula/code/src/java/pcgen/base/formula/ReferenceFormula.java
@@ -160,8 +160,10 @@ jobs:
           cache-read-only: false
           cache-overwrite-existing: true
 
-      # Cache cross-platform JDK and JavaFX jmod archives downloaded by
-      # downloadJdk_* / downloadJavaFXMods_* tasks. These are pinned by
+      # Cache the host-platform JDK and JavaFX jmod archives downloaded by
+      # the downloadJdk / downloadJfxMods tasks. Filenames are stamped with
+      # ${hostOs}_${hostArch}, so each runner OS gets its own cache bucket
+      # via the runner-keyed actions/cache key. These are pinned by
       # javaVersion (gradle.properties) and the URL templates (build.gradle),
       # so keying on those file hashes invalidates correctly when we bump
       # the JDK or JavaFX version. We cache only the archives (.tar.gz / .zip)
 
@@ -130,8 +130,10 @@ jobs:
           cache-read-only: false
           cache-overwrite-existing: true
 
-      # Cache cross-platform JDK and JavaFX jmod archives downloaded by
-      # downloadJdk_* / downloadJavaFXMods_* tasks. These are pinned by
+      # Cache the host-platform JDK and JavaFX jmod archives downloaded by
+      # the downloadJdk / downloadJfxMods tasks. Filenames are stamped with
+      # ${hostOs}_${hostArch}, so each runner OS gets its own cache bucket
+      # via the runner-keyed actions/cache key. These are pinned by
       # javaVersion (gradle.properties) and the URL templates (build.gradle),
       # so keying on those file hashes invalidates correctly when we bump
       # the JDK or JavaFX version. We cache only the archives (.tar.gz / .zip)
 
@@ -42,6 +42,12 @@ jobs:
             build/test-results/**/*.xml
             build/test-results/**/*.trx
             build/test-results/**/*.json
+            PCGen-base/build/test-results/**/*.xml
+            PCGen-base/build/test-results/**/*.trx
+            PCGen-base/build/test-results/**/*.json
+            PCGen-Formula/build/test-results/**/*.xml
+            PCGen-Formula/build/test-results/**/*.trx
+            PCGen-Formula/build/test-results/**/*.json
 
       - name: Run Coverage
         run: ./gradlew testCoverage
 
@@ -65,8 +65,7 @@ Always use the wrapper (./gradlew). Java 25 is required; Gradle will fetch depen
   - ./gradlew clean (also triggers cleanPlugins, cleanOutput, cleanJdks, cleanMods, cleanMasterSheets)
 
 Notes
-- Some tasks trigger downloads of JDKs/JavaFX for all platforms (downloadJDKs, extractJDKs, downloadJavaFXMods) or host SDK (downloadJavaFXLocal/extractJavaFXLocal). CI caches build/jre and build/libs.
-- The `jre` task prepares all platform JDKs with JavaFX modules for runtime image creation.
+- jlink/jpackage build only the host platform. The download/extract tasks (downloadJdk, extractJdk, downloadJfxMods, extractJfxMods) target the host OS/arch automatically; the host SDK helper for local dev is downloadJavaFXLocal/extractJavaFXLocal. CI caches build/jre and build/libs.
 - Runtime bundles expect assets in data/, system/, outputsheets/, preview/, vendordata/, homebrewdata/.
 
 ## Running From Source
 
@@ -1,186 +1,93 @@
-/*
- * PCGen formula component build using the Gradle tool. This file specifies the
- * core build tasks and refers to other files in the code/gradle directory for
- * additional tasks for specific output.
- *
- * Developer build: gradle
- * Incremental dev build: gradle build 
- * Full build: gradle all 
- */
-
 plugins {
-    id "ca.coglinc2.javacc" version "3.0.0"
-    id 'java'
-    id 'eclipse'
-    id 'jacoco'
-    id 'ivy-publish'
-    id 'checkstyle'
-    id 'pmd'
-    id 'com.github.ben-manes.versions' version '0.39.0'
-    id 'com.github.spotbugs' version '4.7.5'
+    id 'java-library'
 }
 
 group = 'net.sourceforge.pcgen'
-description = """PCGen formula library"""
-
-defaultTasks 'clean', 'build'
-
-sourceCompatibility = 1.11
-targetCompatibility = 1.11
-
-ext {
-    majorVersion = 1
-    minorVersion = 0
-}
+description = 'PCGen formula library - generic expression parser and solver system'
 
-repositories {
-    mavenCentral()
-    ivy {
-        name "Pcgen Repo"
-        url 'http://pc-gen.org/librepo'
-	allowInsecureProtocol true
+java {
+    toolchain {
+        languageVersion = JavaLanguageVersion.of(25)
     }
+    modularity.inferModulePath = true
 }
 
-dependencies {
-    // Use this if you want to reference your own local pcgen base, alter the 
-    // relative path to what you need.
-    //compile files("../../pcgen-base/PCGen-base/build/libs/PCgen-base-1.0.jar")
-
-    implementation group: 'net.sourceforge.pcgen', name: 'PCGen-base', version:'1.0.220'
-    testImplementation group: 'org.junit.platform', name: 'junit-platform-runner', version: '1.8.0'
-    testImplementation group: 'org.junit.platform', name: 'junit-platform-launcher', version: '1.7.2'
-    testImplementation group: 'org.junit.jupiter', name:  'junit-jupiter-api', version: '5.8.0'
-    testImplementation group: 'org.junit.jupiter', name:  'junit-jupiter-params', version: '5.8.0'
-    testRuntimeOnly group: 'org.junit.jupiter', name: 'junit-jupiter-engine', version: '5.8.0'
-}
+def generatedJjtreeDir = layout.buildDirectory.dir('generated/sources/jjtree').get().asFile
+def generatedJavaccDir = layout.buildDirectory.dir('generated/sources/javacc').get().asFile
 
 sourceSets {
     main {
-        java {
-            srcDirs 'code/src/java'
-        }
-    }
-    javacc {
-        java {
-            srcDirs 'code/src/javacc'
-        }
+        java { srcDirs = ['code/src/java', generatedJjtreeDir, generatedJavaccDir] }
     }
     test {
-        java {
-            srcDirs 'code/src/test'
-        }
-    }
-}
-
-test {
-	useJUnitPlatform()
-	reports {
-        junitXml.enabled = true
-        html.enabled = false
+        java { srcDirs = ['code/src/test'] }
     }
-    systemProperties 'property': 'value'
 }
 
-jacocoTestReport {
-    reports {
-        xml {
-            enabled true // coveralls plugin depends on xml format report
-        }
-
-        html {
-            enabled true
-        }
-    }
-
-    afterEvaluate {
-        getClassDirectories().from(files(classDirectories.files.collect {
-            fileTree(dir: it, exclude: ['**/testsupport/**', '**/Abstract**TestCase', '**/**Test', 'pcgen/base/formula/parse/FormulaParser**', 'pcgen/base/formula/parse/AST**', 'pcgen/base/formula/parse/JJT**', 'pcgen/base/formula/parse/Node**', 'pcgen/base/formula/parse/ParseException**', 'pcgen/base/formula/parse/SimpleCharStream**', 'pcgen/base/formula/parse/Token**'])
-        }))
-    }
-}
-
-compileJjtree {
-    inputDirectory = file('code/src/jjtree')
-    // We consciously choose to put the output in the main repo tree so that 
-    // SimpleNode only exists once...
-    outputDirectory = file('code/src/javacc/')
-    include '**/*.java'
+repositories {
+    mavenCentral()
 }
 
-compileJavacc {
-    inputDirectory = file('code/src/javacc/')
-    outputDirectory = file('code/src/java/')
+configurations {
+    javacc
 }
 
-// Calculate the version number - runs in the parse phase
-allprojects {
-    ext.buildTimestamp = new Date().format('yyyy-MM-dd HH:mm:ss Z')
-    if (System.env.BUILD_NUMBER) {
-        project.version = "$majorVersion.$minorVersion.$System.env.BUILD_NUMBER"
+dependencies {
+    api project(':PCGen-base')
+
+    javacc 'net.java.dev.javacc:javacc:7.0.13'
+
+    testImplementation platform('org.junit:junit-bom:5.11.4')
+    testImplementation 'org.junit.jupiter:junit-jupiter-api'
+    testImplementation 'org.junit.jupiter:junit-jupiter-params'
+    testImplementation 'junit:junit:4.13.2'
+    testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine'
+    testRuntimeOnly 'org.junit.vintage:junit-vintage-engine'
+    testRuntimeOnly 'org.junit.platform:junit-platform-launcher'
+}
+
+// SimpleNode.java and Operator.java in code/src/java are hand-maintained; everything
+// else in pcgen/base/formula/parse is generated at build time. JJTree produces the AST
+// classes, the visitor interfaces, and an annotated .jj grammar; JavaCC then turns the
+// .jj into the parser and token classes.
+tasks.register('jjtree', JavaExec) {
+    def src = file('code/src/jjtree/pcgen/base/formula/parse/formula.jjt')
+    def outDir = new File(generatedJjtreeDir, 'pcgen/base/formula/parse')
+    inputs.file(src)
+    outputs.dir(generatedJjtreeDir)
+    classpath = configurations.javacc
+    mainClass = 'jjtree'
+    args "-OUTPUT_DIRECTORY=${outDir}", src.absolutePath
+    doFirst {
+        generatedJjtreeDir.deleteDir()
+        outDir.mkdirs()
     }
-    else {
-        project.version = "$majorVersion.$minorVersion"
+    doLast {
+        // SimpleNode is hand-maintained in code/src/java; drop JJTree's stub so the
+        // two don't collide on the source path.
+        new File(outDir, 'SimpleNode.java').delete()
     }
 }
 
-task echoVer() doLast {
-    println "${project.name} Version: ${project.version} (${buildTimestamp})"
-}
-
-jar {
-    manifest {
-        attributes 'Implementation-Title': 'PCGenFormulaLibrary', 'Implementation-Version': project.version, 
-            'Built-On': buildTimestamp 
+tasks.register('javacc', JavaExec) {
+    dependsOn 'jjtree'
+    def jjFile = new File(generatedJjtreeDir, 'pcgen/base/formula/parse/formula.jj')
+    def outDir = new File(generatedJavaccDir, 'pcgen/base/formula/parse')
+    inputs.file(jjFile)
+    outputs.dir(generatedJavaccDir)
+    classpath = configurations.javacc
+    mainClass = 'javacc'
+    args "-OUTPUT_DIRECTORY=${outDir}", jjFile.absolutePath
+    doFirst {
+        generatedJavaccDir.deleteDir()
+        outDir.mkdirs()
     }
 }
 
-task sourceJar(type: Jar) {
-    from sourceSets.main.java
-    classifier "sources"
+tasks.named('compileJava') {
+    dependsOn 'javacc'
 }
 
-task javadocJar(type: Jar, dependsOn: javadoc) {
-    classifier = 'javadoc'
-    from javadoc.destinationDir
+tasks.named('test') {
+    useJUnitPlatform()
 }
-
-// Rules for how we publish our artifacts in ivy compliant format
-publishing {
-    repositories {
-        ivy {
-            name "fileRepo"
-            url '/var/www/librepo'
-        }
-    }
-    publications {
-        ivy(IvyPublication) {
-            from components.java
-            configurations {
-                sources {}
-                javadoc {}
-            }
-            artifact(sourceJar) {
-                type "sources"
-                conf "sources"
-            }
-            artifact(javadocJar) {
-                type "javadoc"
-                conf "javadoc"
-            }
-            descriptor.withXml {
-                asNode().info[0].appendNode('description', description)
-            }
-        }
-    }
-}
-
-//We end up with an unneeded task with SpotBugs and JavaCC that causes a failure
-//see: https://github.com/spotbugs/spotbugs-gradle-plugin/issues/70
-tasks.whenTaskAdded {task ->
-    if(task.name.contains("spotbugsJavacc")) {
-        task.enabled = false
-    }
-}
-
-apply from: 'gradle/reporting.gradle'
@@ -0,0 +1,20 @@
+module pcgen.formula {
+    requires transitive pcgen.base;
+
+    exports pcgen.base.formula;
+    exports pcgen.base.formula.analysis;
+    exports pcgen.base.formula.base;
+    exports pcgen.base.formula.exception;
+    exports pcgen.base.formula.factory;
+    exports pcgen.base.formula.function;
+    exports pcgen.base.formula.inst;
+    exports pcgen.base.formula.library;
+    exports pcgen.base.formula.operator.array;
+    exports pcgen.base.formula.operator.bool;
+    exports pcgen.base.formula.operator.generic;
+    exports pcgen.base.formula.operator.number;
+    exports pcgen.base.formula.operator.string;
+    exports pcgen.base.formula.parse;
+    exports pcgen.base.formula.visitor;
+    exports pcgen.base.solver;
+}