Issue #58 Fix HttpClient resource management in ApiTracker.java (#79)

* Issue #58 Fix HttpClient resource management in ApiTracker.java

- Replaced static shared HttpClient instance with factory method
- Updated fetchFromUrl() to use try-with-resources for proper resource management
- Updated fetchUpstreamSources() to use try-with-resources for proper resource management
- Ensures HttpClient instances are properly closed after use
- Aligns with Java 21+ AutoCloseable best practices

The changes ensure that HttpClient resources are properly managed using
try-with-resources, preventing potential resource leaks and following
Java best practices for AutoCloseable resources.

* Add try-with-resources guideline to AGENTS.md

- Added bullet point under 'Leverage Java 21+ features' section
- Specifies that try-with-resources must be used for all AutoCloseable resources
- Includes examples like HttpClient and streams
- Ensures proper resource management following Java best practices

* Fix AGENTS.md references and add try-with-resources guideline

- Updated reference to deleted CODING_STYLE.md file
- Changed to indicate this section contains the coding standards directly
- Added try-with-resources bullet point under Java 21+ features
- Ensures documentation is accurate and complete

* Update AGENTS.md commit guidelines to forbid advertising and co-author comments

- Added bullet point prohibiting advertising/promotional content in commit messages
- Added bullet point prohibiting 'Co-Authored-By' comments in commit messages
- Both guidelines added to Commit Requirements section
- Ensures clean, professional commit messages without external advertising
- Maintains proper git attribution through standard author fields

Author: simbo1905

AGENTS.md

@@ -220,6 +220,8 @@ mvn exec:java -pl json-compatibility-suite -Dexec.args="--json"
 - Optionally note unexpected technical details when they are not obvious from the issue itself.
 - Do not report progress or success in the commit message; nothing is final until merged.
 - Every tidy-up commit requires an accompanying issue. If labels are unavailable, title the issue `Tidy Up: ...` and keep the description minimal.
+- **Do not include advertising or promotional content** such as `🤖 Generated with [XXX YYY](https://XXX/YYY)` in commit messages.
+- **Do not add 'Co-Authored-By' comments** to commit messages; keep attribution within the normal git author fields.
 
 ### Pull Requests
 - Describe what was done, not the rationale or implementation details.
@@ -422,7 +424,7 @@ PY
 
 # Java DOP Coding Standards ####################
 
-This file is a Gen AI summary of CODING_STYLE.md to use less tokens of context window. Read the original file for full details.
+This section contains the Java DOP (Data-Oriented Programming) coding standards and guidelines.
 
 IMPORTANT: We do TDD so all code must include targeted unit tests.
 IMPORTANT: Never disable tests written for logic that we are yet to write we do Red-Green-Refactor coding.
@@ -455,6 +457,7 @@ IMPORTANT: Never disable tests written for logic that we are yet to write we do
     * Pattern matching for structural decomposition
     * Sealed classes for exhaustive switches
     * Virtual threads for concurrent processing
+    * **Use try-with-resources for all AutoCloseable resources** (HttpClient, streams, etc.)
 
 ## Package Structure
 

json-java21-api-tracker/src/main/java/io/github/simbo1905/tracker/ApiTracker.java

@@ -64,21 +64,23 @@ enum Nothing implements ApiTracker {}
     // GitHub base URL for upstream sources
     String GITHUB_BASE_URL = "https://raw.githubusercontent.com/openjdk/jdk-sandbox/refs/heads/json/src/java.base/share/classes/";
 
-    // Shared HttpClient instance for efficient resource management
-    HttpClient SHARED_HTTP_CLIENT = HttpClient.newBuilder()
-        .connectTimeout(Duration.ofSeconds(10))
-        .build();
+    // HttpClient factory method for proper resource management
+    static HttpClient createHttpClient() {
+        return HttpClient.newBuilder()
+            .connectTimeout(Duration.ofSeconds(10))
+            .build();
+    }
 
     /// Fetches content from a URL
     static String fetchFromUrl(String url) {
-        try {
+        try (final var httpClient = createHttpClient()) {
             final var request = HttpRequest.newBuilder()
                 .uri(URI.create(url))
                 .timeout(Duration.ofSeconds(30))
                 .GET()
                 .build();
 
-            final var response = SHARED_HTTP_CLIENT.send(request, HttpResponse.BodyHandlers.ofString());
+            final var response = httpClient.send(request, HttpResponse.BodyHandlers.ofString());
 
             if (response.statusCode() == 200) {
                 return response.body();
@@ -227,14 +229,14 @@ static Map<String, String> fetchUpstreamSources(Set<Class<?>> localClasses) {
 
             LOGGER.info("Fetching upstream source: " + url);
 
-            try {
+            try (final var httpClient = createHttpClient()) {
                 final var request = HttpRequest.newBuilder()
                     .uri(URI.create(url))
                     .timeout(Duration.ofSeconds(30))
                     .GET()
                     .build();
 
-                final var response = SHARED_HTTP_CLIENT.send(request, HttpResponse.BodyHandlers.ofString());
+                final var response = httpClient.send(request, HttpResponse.BodyHandlers.ofString());
 
                 if (response.statusCode() == 200) {
                     final var body = response.body();