Issue #145 mechanical package move jdk.sandbox to jdk.incubator

Mirror the upstream move of the JSON API from the jdk-sandbox into the
jdk.incubator.json module as a pure token swap, no API changes:

- git mv of jdk/sandbox source and test trees to jdk/incubator in
  json-java21 and json-compatibility-suite (34 files, history preserved)
- global sed jdk.sandbox -> jdk.incubator and jdk/sandbox -> jdk/incubator
  across java, md, html, yml, py, pom and properties files in all modules
- sync-process docs in json-java21/AGENTS.md and .github/copilot-instructions.md
  now name the new packages; api-tracker still maps
  jdk.incubator.java.util.json to the upstream raw path jdk/incubator/json

Verify: mvn -B clean verify -DskipITs=false -> BUILD SUCCESS, 1355 tests,
0 failures, 0 errors, 0 skipped (matches the CI test-count assertion).

Author: simbo1905

.github/copilot-instructions.md

@@ -81,8 +81,8 @@ return switch (jsonValue) {
 ```
 
 ### Project Architecture
-- Core package: `jdk.sandbox.java.util.json`
-- Internal implementation: `jdk.sandbox.internal.util.json`
+- Core package: `jdk.incubator.java.util.json`
+- Internal implementation: `jdk.incubator.internal.util.json`
 - JSON Schema validator in separate module
 - Use appropriate logging configuration per module
 

AGENTS.md

@@ -151,15 +151,15 @@ See `README.md` for user-facing commands. When running locally as an agent, use
 - `json-java21-api-tracker`: API evolution tracking utilities.
 - `json-compatibility-suite`: JSON Test Suite compatibility validation.
 - `json-java21-jtd`: JSON Type Definition (JTD) validator based on RFC 8927.
-- `json-java21-jsonpath`: JsonPath query engine over `jdk.sandbox.java.util.json` values.
+- `json-java21-jsonpath`: JsonPath query engine over `jdk.incubator.java.util.json` values.
 
 Only when you are asked to work on a specific module, start by reading that module's `README.md`, then its `AGENTS.md`.
 
 These modules are treated as separate subsystems; do not read their docs unless you are actively working on them. They are not interlinked and each depends only on the core `json-java21` API.
 
 ### Core Components
 
-#### Public API (`jdk.sandbox.java.util.json`)
+#### Public API (`jdk.incubator.java.util.json`)
 - `Json`: Static utilities for parsing, formatting, and conversion.
 - `JsonValue`: Sealed root interface for all JSON types.
 - `JsonObject`: JSON objects (key-value pairs).
@@ -169,14 +169,14 @@ These modules are treated as separate subsystems; do not read their docs unless
 - `JsonBoolean`: JSON booleans.
 - `JsonNull`: JSON null.
 
-IMPORTANT: This API **MUST NOT** deviate from the upstream jdk.sandbox repo which is will track.  
+IMPORTANT: This API **MUST NOT** deviate from the upstream jdk.incubator repo which is will track.  
 
-#### Internal Implementation (`jdk.sandbox.internal.util.json`)
+#### Internal Implementation (`jdk.incubator.internal.util.json`)
 - `JsonParser`: Recursive descent JSON parser.
 - `Json*Impl`: Immutable implementations of `Json*` types.
 - `Utils`: Internal utilities and factory methods.
 
-IMPORTANT: Bugs in upstream-derived core logic MUST be fixed upstream. Do not patch `jdk.sandbox.*` sources in this repo unless the user explicitly agrees (for example, to carry a temporary local backport while the upstream fix is in progress).
+IMPORTANT: Bugs in upstream-derived core logic MUST be fixed upstream. Do not patch `jdk.incubator.*` sources in this repo unless the user explicitly agrees (for example, to carry a temporary local backport while the upstream fix is in progress).
 
 Only bugs in local, non-upstream code (for example, backporting shims/polyfills or other modules in this repo) should be fixed here using normal TDD.
 
@@ -480,7 +480,7 @@ flowchart LR
 python3 - <<'PY'
 import os, sys, re
 src = 'updates/2025-09-04/upstream/jdk.internal.util.json'
-dst = 'json-java21/src/main/java/jdk/sandbox/internal/util/json'
+dst = 'json-java21/src/main/java/jdk/incubator/internal/util/json'
 def xform(text):
     # old old python3 stuff here
 print('OK')

README.md

@@ -17,7 +17,7 @@ This repo is organized into the following modules:
 | `json-java21-jtd` | JTD (RFC 8927) stack-machine interpreter — ideal for infrequent config parsing and one-time validation | 21+ |
 | `json-java21-jtd-codegen` | Bytecode code generator for JTD schemas — ahead-of-time compiled validators for repeated hot-path validation | 24+ (auto-skipped on JDK 21) |
 | `jdt2jar` | CLI + distroless container to pre-compile JTD schemas into standalone validator JARs (eliminates JDK 24+ runtime requirement) | 24+ (auto-skipped on JDK 21) |
-| `json-java21-jsonpath` | JsonPath query engine over `jdk.sandbox.java.util.json` values (Goessner-style: filters, slices, recursive descent, unions) | 21+ |
+| `json-java21-jsonpath` | JsonPath query engine over `jdk.incubator.java.util.json` values (Goessner-style: filters, slices, recursive descent, unions) | 21+ |
 | `json-compatibility-suite` | JSON Test Suite conformance reporter (tests against [nst/JSONTestSuite](https://github.com/nst/JSONTestSuite)) | 21+ |
 | `json-java21-api-tracker` | Daily upstream API drift detector — fetches OpenJDK sandbox sources, compares public API signatures, reports differences | 25+ |
 
@@ -31,7 +31,7 @@ To try the examples from this README, build the project and run the standalone e
 
 ```bash
 ./mvnw package
-java -cp ./json-java21/target/test-classes/:./json-java21/target/classes/ jdk.sandbox.java.util.json.examples.ReadmeExamples
+java -cp ./json-java21/target/test-classes/:./json-java21/target/classes/ jdk.incubator.java.util.json.examples.ReadmeExamples
 ```
 
 ## API Overview
@@ -307,7 +307,7 @@ The JSON compatibitlity tests in this repo suggest 99% conformance with a leadin
 
 ### CI: Upstream API Tracking
 
-The `daily-api-tracker.yml` workflow runs daily at 02:00 UTC: it fetches the upstream `jdk.incubator.json` sources from the [jdk-sandbox `json` branch](https://github.com/openjdk/jdk-sandbox/tree/json) HEAD and compares public API signatures against the local `jdk.sandbox.java.util.json` classes. When they differ it creates a fingerprint-deduplicated "API drift detected" issue; reports are uploaded as workflow artifacts (`target/api-tracker/`) with 90-day retention. The check can also be run locally:
+The `daily-api-tracker.yml` workflow runs daily at 02:00 UTC: it fetches the upstream `jdk.incubator.json` sources from the [jdk-sandbox `json` branch](https://github.com/openjdk/jdk-sandbox/tree/json) HEAD and compares public API signatures against the local `jdk.incubator.java.util.json` classes. When they differ it creates a fingerprint-deduplicated "API drift detected" issue; reports are uploaded as workflow artifacts (`target/api-tracker/`) with 90-day retention. The check can also be run locally:
 
 ```bash
 $(command -v mvnd || command -v mvn || command -v ./mvnw) -pl json-java21-api-tracker exec:java \
@@ -361,7 +361,7 @@ Per **RFC 8927 (JSON Typedef)**, the empty schema `{}` is the **empty form** and
 
 ```java
 import json.java21.jtd.Jtd;
-import jdk.sandbox.java.util.json.*;
+import jdk.incubator.java.util.json.*;
 
 JsonValue schema = Json.parse("{\"properties\":{\"name\":{\"type\":\"string\"}}}");
 JsonValue data = Json.parse("{\"name\":\"Alice\"}");
@@ -400,7 +400,7 @@ This repo also includes a JsonPath query engine (module `json-java21-jsonpath`),
 https://goessner.net/articles/JsonPath/
 
 ```java
-import jdk.sandbox.java.util.json.*;
+import jdk.incubator.java.util.json.*;
 import json.java21.jsonpath.JsonPath;
 import json.java21.jsonpath.JsonPathStreams;
 

index.html

@@ -179,7 +179,7 @@ <h3>Maven</h3>
 </code></pre>
 
                 <h3>Parse and access</h3>
-                <pre><code>import jdk.sandbox.java.util.json.*;
+                <pre><code>import jdk.incubator.java.util.json.*;
 
 JsonObject obj = (JsonObject) Json.parse("{\"name\":\"Alice\",\"age\":30}");
 
@@ -199,7 +199,7 @@ <h2>What’s Included</h2>
         </div>
 
         <h2>Record Mapping (explicit, typed)</h2>
-        <pre><code>import jdk.sandbox.java.util.json.*;
+        <pre><code>import jdk.incubator.java.util.json.*;
 import java.util.*;
 
 record User(String name, String email, boolean active) {}
@@ -224,7 +224,7 @@ <h2>Record Mapping (explicit, typed)</h2>
         <h2>Run the README Examples</h2>
         <pre><code>mvn package
 java -cp ./json-java21/target/java.util.json-*.jar:./json-java21/target/test-classes \
-  jdk.sandbox.java.util.json.examples.ReadmeExamples</code></pre>
+  jdk.incubator.java.util.json.examples.ReadmeExamples</code></pre>
         <p class="meta">Replace <code>*</code> with the actual version number from the JAR filename.</p>
 
         <h2>JSON Test Suite Compatibility</h2>
@@ -237,7 +237,7 @@ <h2>JSON Test Suite Compatibility</h2>
         <h2>JSON Type Definition (JTD) Validator</h2>
         <p class="meta">This repo includes an incubating JTD validator (RFC 8927) in module <code>json-java21-jtd</code>. Per RFC 8927, the empty schema <code>{}</code> accepts all JSON instances.</p>
         <pre><code>import json.java21.jtd.Jtd;
-import jdk.sandbox.java.util.json.*;
+import jdk.incubator.java.util.json.*;
 
 JsonValue schema = Json.parse("""
   {

jdt2jar/src/main/java/json/java21/jdt2jar/Jdt2Jar.java

@@ -1,6 +1,6 @@
 package json.java21.jdt2jar;
 
-import jdk.sandbox.java.util.json.Json;
+import jdk.incubator.java.util.json.Json;
 import json.java21.jtd.codegen.JtdCodegen;
 
 import java.io.ByteArrayOutputStream;
@@ -158,8 +158,8 @@ private static void copyJarEntries(JarOutputStream out, Set<String> written, Pat
   }
 
   private static boolean shouldCopyRuntime(String path) {
-    return (path.startsWith("jdk/sandbox/java/util/json/")
-        || path.startsWith("jdk/sandbox/internal/util/json/")
+    return (path.startsWith("jdk/incubator/java/util/json/")
+        || path.startsWith("jdk/incubator/internal/util/json/")
         || path.startsWith("json/java21/jtd/"))
         && !path.startsWith("json/java21/jtd/codegen/")
         || path.startsWith("json/java21/jtd/codegen/JtdValidator.class")
@@ -202,7 +202,7 @@ private static void writeSourceFile(Path sourcePath, Options options) throws IOE
     final var source = """
         package %s;
 
-        import jdk.sandbox.java.util.json.JsonValue;
+        import jdk.incubator.java.util.json.JsonValue;
         import json.java21.jtd.JtdValidationResult;
         import json.java21.jdt2jar.runtime.ValidatorMain;
 

jdt2jar/src/main/java/json/java21/jdt2jar/runtime/ValidatorMain.java

@@ -1,10 +1,10 @@
 package json.java21.jdt2jar.runtime;
 
-import jdk.sandbox.java.util.json.Json;
-import jdk.sandbox.java.util.json.JsonObject;
-import jdk.sandbox.java.util.json.JsonParseException;
-import jdk.sandbox.java.util.json.JsonString;
-import jdk.sandbox.java.util.json.JsonValue;
+import jdk.incubator.java.util.json.Json;
+import jdk.incubator.java.util.json.JsonObject;
+import jdk.incubator.java.util.json.JsonParseException;
+import jdk.incubator.java.util.json.JsonString;
+import jdk.incubator.java.util.json.JsonValue;
 import json.java21.jtd.JtdValidationResult;
 
 import java.io.IOException;
@@ -120,8 +120,8 @@ private static String readResourceString(String name) throws IOException {
 
   private static JsonObject toJson(JtdValidationResult result) {
     return JsonObject.of(java.util.Map.of(
-        "valid", jdk.sandbox.java.util.json.JsonBoolean.of(result.isValid()),
-        "errors", jdk.sandbox.java.util.json.JsonArray.of(result.errors().stream()
+        "valid", jdk.incubator.java.util.json.JsonBoolean.of(result.isValid()),
+        "errors", jdk.incubator.java.util.json.JsonArray.of(result.errors().stream()
             .map(error -> JsonObject.of(java.util.Map.of(
                 "instancePath", JsonString.of(error.instancePath()),
                 "schemaPath", JsonString.of(error.schemaPath()))))

json-compatibility-suite/pom.xml

@@ -73,7 +73,7 @@
                 <artifactId>exec-maven-plugin</artifactId>
                 <version>3.4.1</version>
                 <configuration>
-                    <mainClass>jdk.sandbox.compatibility.JsonCompatibilitySummary</mainClass>
+                    <mainClass>jdk.incubator.compatibility.JsonCompatibilitySummary</mainClass>
                     <includePluginDependencies>false</includePluginDependencies>
                 </configuration>
             </plugin>

…patibility/JsonCompatibilitySummary.java …patibility/JsonCompatibilitySummary.javajson-compatibility-suite/src/main/java/jdk/sandbox/compatibility/JsonCompatibilitySummary.java renamed to json-compatibility-suite/src/main/java/jdk/incubator/compatibility/JsonCompatibilitySummary.java json-compatibility-suite/src/main/java/jdk/sandbox/compatibility/JsonCompatibilitySummary.java renamed to json-compatibility-suite/src/main/java/jdk/incubator/compatibility/JsonCompatibilitySummary.java

@@ -1,11 +1,11 @@
-package jdk.sandbox.compatibility;
+package jdk.incubator.compatibility;
 
-import jdk.sandbox.java.util.json.Json;
-import jdk.sandbox.java.util.json.JsonArray;
-import jdk.sandbox.java.util.json.JsonObject;
-import jdk.sandbox.java.util.json.JsonString;
-import jdk.sandbox.java.util.json.JsonNumber;
-import jdk.sandbox.java.util.json.JsonParseException;
+import jdk.incubator.java.util.json.Json;
+import jdk.incubator.java.util.json.JsonArray;
+import jdk.incubator.java.util.json.JsonObject;
+import jdk.incubator.java.util.json.JsonString;
+import jdk.incubator.java.util.json.JsonNumber;
+import jdk.incubator.java.util.json.JsonParseException;
 
 import java.nio.charset.MalformedInputException;
 import java.nio.charset.StandardCharsets;

…box/compatibility/RobustCharDecoder.java …tor/compatibility/RobustCharDecoder.javajson-compatibility-suite/src/main/java/jdk/sandbox/compatibility/RobustCharDecoder.java renamed to json-compatibility-suite/src/main/java/jdk/incubator/compatibility/RobustCharDecoder.java json-compatibility-suite/src/main/java/jdk/sandbox/compatibility/RobustCharDecoder.java renamed to json-compatibility-suite/src/main/java/jdk/incubator/compatibility/RobustCharDecoder.java

@@ -1,4 +1,4 @@
-package jdk.sandbox.compatibility;
+package jdk.incubator.compatibility;
 
 import java.nio.ByteBuffer;
 import java.nio.CharBuffer;

…patibility/DownloadVerificationTest.java …patibility/DownloadVerificationTest.javajson-compatibility-suite/src/test/java/jdk/sandbox/compatibility/DownloadVerificationTest.java renamed to json-compatibility-suite/src/test/java/jdk/incubator/compatibility/DownloadVerificationTest.java json-compatibility-suite/src/test/java/jdk/sandbox/compatibility/DownloadVerificationTest.java renamed to json-compatibility-suite/src/test/java/jdk/incubator/compatibility/DownloadVerificationTest.java

@@ -1,4 +1,4 @@
-package jdk.sandbox.compatibility;
+package jdk.incubator.compatibility;
 
 import org.junit.jupiter.api.Test;
 import java.nio.file.Files;