Generate templated sources under main/java-templates

This path avoids the sources getting wiped out during `mvn clean`,
since they are not generated during the maven build.

This patch also moves the generated WASM build under src/main/wasm
since it is really a source file and not a test file. It will not
be included in the built artifact.

Author: headius

.github/workflows/documentation.yml

@@ -36,7 +36,7 @@ jobs:
     - name: Check C coverage
       run: doxygen Doxyfile
     - name: Check Java coverage
-      run: javadoc -Xdoclint:all,-missing -d doc/java -sourcepath java/api/target/generated-sources/java/:java/api/src/main/java org.ruby_lang.prism
+      run: javadoc -Xdoclint:all,-missing -d doc/java -sourcepath java/api/src/main/java-templates/:java/api/src/main/java org.ruby_lang.prism
     - name: Generate Rust documentation
       run: |
         bundle exec rake cargo:build

.github/workflows/github-pages.yml

@@ -51,7 +51,7 @@ jobs:
       - name: Build with Doxygen
         run: doxygen Doxyfile
       - name: Build with JavaDoc
-        run: javadoc -Xdoclint:all,-missing -d doc/java -sourcepath java/api/target/generated-sources/java/:java/api/src/main/java org.ruby_lang.prism
+        run: javadoc -Xdoclint:all,-missing -d doc/java -sourcepath java/api/src/main/java-templates/:java/api/src/main/java org.ruby_lang.prism
       - name: Build with rustdoc
         run: |
           bundle exec rake cargo:build

.github/workflows/java-wasm-bindings.yml

@@ -55,4 +55,4 @@ jobs:
       - uses: actions/upload-artifact@v7
         with:
           name: prism.wasm
-          path: java/wasm/src/test/resources/prism.wasm
+          path: java/wasm/src/main/wasm/prism.wasm

.gitignore

@@ -42,7 +42,7 @@ out.svg
 /java/org/ruby_lang/prism/AbstractNodeVisitor.java
 /java/org/ruby_lang/prism/Loader.java
 /java/org/ruby_lang/prism/Nodes.java
-/java/wasm/src/test/resources/prism.wasm
+/java/wasm/src/main/wasm/prism.wasm
 /lib/prism/compiler.rb
 /lib/prism/dispatcher.rb
 /lib/prism/dot_visitor.rb

Makefile

@@ -31,7 +31,7 @@ all: shared static
 shared: build/libprism.$(SOEXT)
 static: build/libprism.a
 wasm: javascript/src/prism.wasm
-java-wasm: java/wasm/src/test/resources/prism.wasm
+java-wasm: java/wasm/src/main/wasm/prism.wasm
 
 build/libprism.$(SOEXT): $(SHARED_OBJECTS)
 	$(ECHO) "linking $@ with $(CC)"
@@ -51,7 +51,7 @@ javascript/src/prism.wasm: Makefile $(SOURCES) $(HEADERS)
 		-Oz -g0 -flto -fdata-sections -ffunction-sections \
 		-o $@ $(SOURCES)
 
-java/wasm/src/test/resources/prism.wasm: Makefile $(SOURCES) $(HEADERS)
+java/wasm/src/main/wasm/prism.wasm: Makefile $(SOURCES) $(HEADERS)
 	$(ECHO) "building $@"
 	$(Q) $(MAKEDIRS) $(@D)
 	$(Q) $(WASI_SDK_PATH)/bin/clang \

docs/configuration.md

@@ -8,9 +8,9 @@ A lot of code in prism's repository is templated from a single configuration fil
 * `include/prism/node_new.h` - for defining the functions that create the nodes in C
 * `javascript/src/deserialize.js` - for defining how to deserialize the nodes in JavaScript
 * `javascript/src/nodes.js` - for defining the nodes in JavaScript
-* `java/api/target/generated-sources/java/org/ruby_lang/prism/AbstractNodeVisitor.java` - for defining the visitor interface for the nodes in Java
-* `java/api/target/generated-sources/java/org/ruby_lang/prism/Loader.java` - for defining how to deserialize the nodes in Java
-* `java/api/target/generated-sources/java/org/ruby_lang/prism/Nodes.java` - for defining the nodes in Java
+* `java/api/src/main/java-templates/org/ruby_lang/prism/AbstractNodeVisitor.java` - for defining the visitor interface for the nodes in Java
+* `java/api/src/main/java-templates/org/ruby_lang/prism/Loader.java` - for defining how to deserialize the nodes in Java
+* `java/api/src/main/java-templates/org/ruby_lang/prism/Nodes.java` - for defining the nodes in Java
 * `lib/prism/compiler.rb` - for defining the compiler for the nodes in Ruby
 * `lib/prism/dispatcher.rb` - for defining the dispatch visitors for the nodes in Ruby
 * `lib/prism/dot_visitor.rb` - for defining the dot visitor for the nodes in Ruby

java/.gitignore

@@ -1,4 +1,7 @@
+api/src/main/java-templates
 api/target
 native/target
+wasm/src/main/wasm
 wasm/target
+target
 .idea

java/README.md

@@ -18,7 +18,7 @@ Sources under `api` are generated from templates in `../templates`. Those source
 $ PRISM_EXCLUDE_PRETTYPRINT=1 PRISM_SERIALIZE_ONLY_SEMANTICS_FIELDS=1 bundle exec rake templates
 ```
 
-The files go under `api/target/generated-sources/java` and will be removed with `mvn clean`.
+The files are generated under `api/src/main/java-templates` and will not be removed with `mvn clean`.
 
 ### WASM build of Prism
 
@@ -28,7 +28,7 @@ The `wasm` project needs a WASM build of Prism to be generated with the followin
 $ make java-wasm WASI_SDK_PATH=<path to WASI sdk>
 ```
 
-The files go under `wasm/target/generated-sources/wasm` and will be removed with `mvn clean`.
+The build is generated under `wasm/src/main/wasm` and will not be removed with `mvn clean`.
 
 ### Build and install
 

java/api/pom.xml

@@ -27,7 +27,7 @@
             </goals>
             <configuration>
               <sources>
-                <source>target/generated-sources/java</source>
+                <source>src/main/java-templates</source>
               </sources>
             </configuration>
           </execution>

java/wasm/pom.xml

@@ -79,8 +79,8 @@
             </goals>
             <configuration>
               <name>org.ruby_lang.prism.wasm.PrismParser</name>
-              <wasmFile>src/test/resources/prism.wasm</wasmFile>
               <moduleInterface>org.ruby_lang.prism.wasm.Prism</moduleInterface>
+              <wasmFile>src/main/wasm/prism.wasm</wasmFile>
             </configuration>
           </execution>
         </executions>