Preserve legacy Redis APIs and integrate current test packages

Author: BenCodez

.mex/context/http.md

@@ -13,6 +13,6 @@ mex:
 
 # HTTP transport boundary
 
-The HTTP transport uses pinned TLS and enrollment; certificate binding authenticates peers, while the envelopes are strictly parsed but not independently signed. Proxy-to-backend outgoing messages are durably queued, and both sides have durable inbound replay fences. Backend-to-proxy outgoing messages remain in memory with a bounded shutdown flush. A network response, queue write, and application acknowledgement are different events. A valid nonempty queue directory reports pending work; invalid or inaccessible queue structure throws `IOException`, which callers must handle conservatively. A restart-visible `RUNNING` callback is neither replayed nor acknowledged because its side effects are uncertain. Source: `SimpleAPI/src/main/java/com/bencodez/simpleapi/servercomm/http/`, `SimpleAPI/src/test/java/com/bencodez/simpleapi/servercomm/http/`.
+The HTTP transport uses pinned TLS and enrollment; certificate binding authenticates peers, while the envelopes are strictly parsed but not independently signed. Proxy-to-backend outgoing messages are durably queued, and both sides have durable inbound replay fences. Backend-to-proxy outgoing messages remain in memory with a bounded shutdown flush. A network response, queue write, and application acknowledgement are different events. A valid nonempty queue directory reports pending work; invalid or inaccessible queue structure throws `IOException`, which callers must handle conservatively. A restart-visible `RUNNING` callback is neither replayed nor acknowledged because its side effects are uncertain. Source: `SimpleAPI/src/main/java/com/bencodez/simpleapi/servercomm/http/`, `SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/servercomm/http/`.
 
 Transport work must keep size bounds, shutdown behavior, and authentication checks together. Inspect current tests for the exact guarantee instead of inferring exactly-once behavior from a queue name.

.mex/patterns/http-transport-change.md

@@ -13,4 +13,4 @@ mex:
 
 # HTTP transport change
 
-Map enqueue, persisted state, send, acknowledgement, restart recovery, and unreadable state before changing completion semantics. Keep TLS/enrollment and bounded execution checks in the same review. See `context/http.md` and current tests under `SimpleAPI/src/test/java/com/bencodez/simpleapi/servercomm/http/`.
+Map enqueue, persisted state, send, acknowledgement, restart recovery, and unreadable state before changing completion semantics. Keep TLS/enrollment and bounded execution checks in the same review. See `context/http.md` and current tests under `SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/servercomm/http/`.

AGENTS.md

@@ -15,6 +15,8 @@ mvn -B -f SimpleAPI/pom.xml test
 mvn -B -f SimpleAPI/pom.xml package
 ```
 
+All test classes and test-only helpers belong under `com.bencodez.simpleapi.tests` (or its subpackages), with matching paths under `SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/`. Keep production package names and visibility unchanged when organizing tests; use narrowly scoped test-only reflection access where an existing test needs implementation internals. Preserve test discovery and both full/shared artifact checks when moving tests.
+
 Confirm current CI and POM settings before relying on these commands. Verify that the package invocation produced both the legacy/full artifact and every configured classifier, especially the `shared` JAR. Run `git diff --check` and confirm tests actually ran.
 
 ## Packaging boundaries

SimpleAPI/pom.xml

@@ -327,7 +327,7 @@
         <dependency>
             <groupId>redis.clients</groupId>
             <artifactId>jedis</artifactId>
-            <version>8.0.1</version>
+            <version>7.5.3</version>
         </dependency>
         <dependency>
             <groupId>org.spongepowered</groupId>

SimpleAPI/src/test/java/com/bencodez/simpleapi/servercomm/redis/RedisHandlerClientConfigTest.java

@@ -1,4 +1,6 @@
-package com.bencodez.simpleapi.build;
+package com.bencodez.simpleapi.tests.build;
+
+
 
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;

…mpleapi/build/BuildInputPinningTest.java …i/tests/build/BuildInputPinningTest.javaSimpleAPI/src/test/java/com/bencodez/simpleapi/build/BuildInputPinningTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/build/BuildInputPinningTest.java SimpleAPI/src/test/java/com/bencodez/simpleapi/build/BuildInputPinningTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/build/BuildInputPinningTest.java

@@ -1,4 +1,6 @@
-package com.bencodez.simpleapi.core;
+package com.bencodez.simpleapi.tests.core;
+
+
 
 import static org.junit.jupiter.api.Assertions.*;
 import java.net.URLClassLoader;

…ez/simpleapi/core/CoreClasspathTest.java …pleapi/tests/core/CoreClasspathTest.javaSimpleAPI/src/test/java/com/bencodez/simpleapi/core/CoreClasspathTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/core/CoreClasspathTest.java SimpleAPI/src/test/java/com/bencodez/simpleapi/core/CoreClasspathTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/core/CoreClasspathTest.java

@@ -1,4 +1,6 @@
-package com.bencodez.simpleapi.file;
+package com.bencodez.simpleapi.tests.file;
+
+import com.bencodez.simpleapi.file.PrivateFilePermissions;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;

…api/file/PrivateFilePermissionsTest.java …sts/file/PrivateFilePermissionsTest.javaSimpleAPI/src/test/java/com/bencodez/simpleapi/file/PrivateFilePermissionsTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/file/PrivateFilePermissionsTest.java SimpleAPI/src/test/java/com/bencodez/simpleapi/file/PrivateFilePermissionsTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/file/PrivateFilePermissionsTest.java

@@ -1,4 +1,6 @@
-package com.bencodez.simpleapi.file.config.configurate;
+package com.bencodez.simpleapi.tests.file.config.configurate;
+
+import com.bencodez.simpleapi.file.config.configurate.ConfigurateConfigView;
 
 import static org.junit.jupiter.api.Assertions.*;
 import java.util.ArrayList;

…nfigurate/ConfigurateConfigViewTest.java …nfigurate/ConfigurateConfigViewTest.javaSimpleAPI/src/test/java/com/bencodez/simpleapi/file/config/configurate/ConfigurateConfigViewTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/file/config/configurate/ConfigurateConfigViewTest.java SimpleAPI/src/test/java/com/bencodez/simpleapi/file/config/configurate/ConfigurateConfigViewTest.java renamed to SimpleAPI/src/test/java/com/bencodez/simpleapi/tests/file/config/configurate/ConfigurateConfigViewTest.java

@@ -1,4 +1,7 @@
-package com.bencodez.simpleapi.file.config.configurate;
+package com.bencodez.simpleapi.tests.file.config.configurate;
+
+import com.bencodez.simpleapi.file.config.configurate.ConfigurateConfigView;
+import com.bencodez.simpleapi.file.config.configurate.YamlConfigDocument;
 
 import static org.junit.jupiter.api.Assertions.*;
 import static org.junit.jupiter.api.Assumptions.assumeTrue;