Add in-memory file access and improve parsing robustness

Introduces InMemoryFileAccess for ZIP archives, refactors S3FileAccessV2 for better cache and root path handling, and adds new tests for in-memory file access. Improves XML and encoding detection, SCORM version detection, and deserialization logic for several types. Enhances error handling and parsing robustness across multiple modules, and updates dependencies in pom.xml for logging and AWS SDK support.

Author: jcputney

pom.xml

@@ -1,4 +1,6 @@
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  xmlns="http://maven.apache.org/POM/4.0.0"
+  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
   <modelVersion>4.0.0</modelVersion>
 
   <groupId>dev.jcputney</groupId>
@@ -55,7 +57,9 @@
     <version.jmh>1.37</version.jmh>
     <version.jqwik>1.9.3</version.jqwik>
     <version.junit>5.13.4</version.junit>
+    <version.log4j-to-slf4j>2.25.2</version.log4j-to-slf4j>
     <version.logback>1.5.18</version.logback>
+    <version.mockito>5.18.0</version.mockito>
     <version.org.json>20250517</version.org.json>
     <version.plugin.central-publishing>0.8.0</version.plugin.central-publishing>
     <version.plugin.compiler>3.14.0</version.plugin.compiler>
@@ -71,7 +75,6 @@
     <version.slf4j>2.0.17</version.slf4j>
     <version.testcontainers>1.21.3</version.testcontainers>
     <version.threeten>1.8.0</version.threeten>
-    <version.mockito>5.18.0</version.mockito>
   </properties>
 
   <dependencyManagement>
@@ -154,17 +157,32 @@
     <dependency>
       <groupId>software.amazon.awssdk</groupId>
       <artifactId>s3</artifactId>
-      <scope>provided</scope>
+      <scope>compile</scope>
+      <optional>true</optional>
     </dependency>
     <dependency>
       <groupId>software.amazon.awssdk</groupId>
       <artifactId>auth</artifactId>
-      <scope>provided</scope>
+      <scope>compile</scope>
+      <optional>true</optional>
     </dependency>
     <dependency>
       <groupId>software.amazon.awssdk</groupId>
       <artifactId>aws-core</artifactId>
-      <scope>provided</scope>
+      <scope>compile</scope>
+      <optional>true</optional>
+    </dependency>
+    <dependency>
+      <groupId>software.amazon.awssdk</groupId>
+      <artifactId>sso</artifactId>
+      <scope>test</scope>
+      <optional>true</optional>
+    </dependency>
+    <dependency>
+      <groupId>software.amazon.awssdk</groupId>
+      <artifactId>ssooidc</artifactId>
+      <scope>test</scope>
+      <optional>true</optional>
     </dependency>
 
     <!-- Test Dependencies -->
@@ -258,6 +276,12 @@
       <version>${version.ascii-table}</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.apache.poi</groupId>
+      <artifactId>poi-ooxml</artifactId>
+      <version>5.2.5</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.assertj</groupId>
       <artifactId>assertj-core</artifactId>
@@ -267,15 +291,23 @@
 
 
     <!-- Logging Dependencies -->
+    <dependency>
+      <groupId>org.apache.logging.log4j</groupId>
+      <artifactId>log4j-to-slf4j</artifactId>
+      <version>${version.log4j-to-slf4j}</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
       <version>${version.slf4j}</version>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>ch.qos.logback</groupId>
       <artifactId>logback-classic</artifactId>
       <version>${version.logback}</version>
+      <scope>test</scope>
     </dependency>
   </dependencies>
 
@@ -288,7 +320,7 @@
           <version>${version.plugin.release}</version>
           <configuration>
             <arguments>-ntp</arguments>
-            <scmCommentPrefix />
+            <scmCommentPrefix/>
             <tagNameFormat>v@{project.version}</tagNameFormat>
             <releaseProfiles>central</releaseProfiles>
           </configuration>
@@ -436,10 +468,12 @@
                 <configuration>
                   <finalName>benchmarks</finalName>
                   <transformers>
-                    <transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
+                    <transformer
+                      implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
                       <mainClass>org.openjdk.jmh.Main</mainClass>
                     </transformer>
-                    <transformer implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer" />
+                    <transformer
+                      implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer"/>
                   </transformers>
                   <filters>
                     <filter>

src/main/java/dev/jcputney/elearning/parser/impl/AbstractS3FileAccess.java

@@ -116,19 +116,33 @@ public abstract class AbstractS3FileAccess implements FileAccess {
   protected AbstractS3FileAccess(String bucketName, String rootPath) {
     this.bucketName = bucketName;
     this.executorService = Executors.newFixedThreadPool(10);
+    initializeRootPath(rootPath);
+  }
 
+  private void initializeRootPath(String rootPath) {
     String processedPath = rootPath;
     if (processedPath == null) {
       processedPath = "";
     }
 
-    // Lazy initialization of root path detection
+    if (processedPath.endsWith("/")) {
+      processedPath = processedPath.substring(0, processedPath.length() - 1);
+    }
+
     this.rootPath = processedPath;
+  }
 
-    String localRootPath = this.rootPath;
-    if (localRootPath.endsWith("/")) {
-      this.rootPath = localRootPath.substring(0, localRootPath.length() - 1);
-    }
+  protected final void reconfigureRootPath(String newRootPath) {
+    clearAllCaches();
+    initializeRootPath(newRootPath);
+  }
+
+  private void clearAllCaches() {
+    fileExistsCache.clear();
+    directoryListCache.clear();
+    smallFileCache.clear();
+    fileSizeCache.clear();
+    allFilesCache = null;
   }
 
   /**
@@ -148,8 +162,11 @@ public boolean fileExistsInternal(String path) {
     // If we have the allFilesCache populated, use it to determine existence
     List<String> allFiles = allFilesCache;
     if (allFiles != null) {
-      String fullFilePath = fullPath(path);
-      boolean exists = allFiles.contains(fullFilePath);
+      boolean exists = allFiles.contains(path);
+      if (!exists) {
+        // SDK-specific implementations may cache absolute keys; fall back to that form
+        exists = allFiles.contains(fullPath(path));
+      }
       fileExistsCache.put(path, exists);
       return exists;
     }
@@ -582,4 +599,4 @@ protected long getCachedFileSize(String path) {
    * @return The detected internal root directory
    */
   protected abstract String detectInternalRootDirectory(String rootPath);
-}
+}