Rename jdt2jar to jtd2jar - #164
Conversation
Mechanical rename of the misspelled schema-to-validator-JAR tool: module directory and artifactId, Java package json.java21.jdt2jar -> json.java21.jtd2jar, class Jdt2Jar -> Jtd2Jar, helper script, GHCR image name (release workflow), and all documentation references. The old image name is frozen on GHCR; jtd2jar/README.md notes the transition. Full reactor verify green with unchanged test counts.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: be60a7c345
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| docker pull ghcr.io/simbo1905/java.util.json.java21/jdt2jar:latest | ||
| docker pull ghcr.io/simbo1905/java.util.json.java21/jdt2jar:2026.05.20 | ||
| docker pull ghcr.io/simbo1905/java.util.json.java21/jtd2jar:latest | ||
| docker pull ghcr.io/simbo1905/java.util.json.java21/jtd2jar:2026.05.20 |
There was a problem hiding this comment.
The documented 2026.05.20 pull uses the new jtd2jar image path, but the note immediately below states that every release through 2026.08.30 was published only under jdt2jar. A user copying this pinned-version example will therefore request a tag that was never published; retain the old repository path for this example or show a tag actually published under the new name.
AGENTS.md reference: AGENTS.md:L31-L35
Useful? React with 👍 / 👎.
| ``` | ||
|
|
||
| See [jdt2jar/README.md](https://github.com/${{ github.repository }}/blob/main/jdt2jar/README.md) for usage. | ||
| See [jtd2jar/README.md](https://github.com/${{ github.repository }}/blob/main/jtd2jar/README.md) for usage. |
There was a problem hiding this comment.
During the first renamed release, the Create GitHub Release with notes step runs before this workflow opens the back-merge PR, so main/jtd2jar/README.md does not yet exist and this usage link is broken until that PR is manually merged. Link to the release tag (for example ${{ github.ref_name }}) rather than main so the documentation is available when the release is published.
AGENTS.md reference: AGENTS.md:L31-L35
Useful? React with 👍 / 👎.
Closes #163
Mechanical rename of the misspelled
jdt2jartool tojtd2jar(JSON Type Definition):jdt2jar/->jtd2jar/, artifactId, parent<module>json.java21.jdt2jar->json.java21.jtd2jar, classJdt2Jar->Jtd2Jar, tests renamedscripts/jdt2jar.sh->scripts/jtd2jar.sh.../jdt2jar->.../jtd2jarjtd2jar/README.mdtransition note: pre-2026.08.30 image tags remain on GHCR under the old name but are frozenVerification: full
mvnd clean verifyBUILD SUCCESS, 0 failures/errors, test counts unchanged (1679 unique).