Fix broken Java literalEval implementation in concoredocker.java

[GaneshPatil7517]

Fixes #228

Problem: concoredocker.java has a fundamentally broken literalEval() that uses simple String.split(',') — this fails on any input containing commas inside quoted strings, nested structures, or escape sequences. All Java-based Docker functionality is broken.

Solution: Replace with a proper recursive descent parser and fix all related methods.

Changes to concoredocker.java:

• Recursive descent literalEval() parser replacing the broken split-based implementation. Correctly handles:
  • Nested dicts/lists/tuples
  • Quoted strings containing commas, colons, and escape sequences
  • Scientific notation (1.5e3)
  • Booleans (True/False) and None
  • Trailing commas
• simtime changed from int to double — preserves fractional values like 0.5
• delay changed from 1ms to 1000ms — matches Python's time.sleep(1)
• maxRetries = 5 added to read() — prevents infinite blocking (matches Python)
• Thread.currentThread().interrupt() in all InterruptedException handlers
• toPythonLiteral() serializer for write() — outputs True/False/None (not Java's true/false/null)
• write() accepts List in addition to legacy Object[]
• initVal() returns List<Object> with double simtime
• Error handling in parseFile() and defaultMaxTime() for malformed input
• Bounds check on sparams before charAt()

New file TestLiteralEval.java:

• 20 test methods with 38 assertions covering all parser functionality
• All tests pass

Testing:

javac concoredocker.java TestLiteralEval.java && java TestLiteralEval
# === Results: 38 passed, 0 failed out of 38 tests ===

[Copilot]

Pull request overview

Fixes the Java concoredocker Python-literal parsing/serialization used for file-based Docker IPC by replacing the previous split-based logic with a recursive-descent parser and aligning timing/retry behavior with the Python implementation.

Changes:

• Replaced literalEval() with a recursive-descent parser supporting nested containers, strings/escapes, numbers (incl. scientific notation), booleans, and None.
• Updated IPC behavior: delay to 1000ms, simtime to double, added bounded retries in read(), and added a Python-literal serializer for write().
• Added a Java test harness TestLiteralEval.java to validate parser behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.

File	Description
concoredocker.java	Implements new literal parser, updates read/write semantics, simtime handling, retry logic, and serialization.
TestLiteralEval.java	Adds parser-focused regression tests for common and nested literal forms.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[GaneshPatil7517]

Hey @pradeeban sir review this when you get the time...

[pradeeban]

@GaneshPatil7517 Looks like you are changing the Python implementation also with this PR. I did not look carefully on what is changed in the python implementation. If those are some other important fixes, please create a separate PR for that.

Pls keep this fixing Java a strictly java-only PR. That way, I can merge it without much consideration.
Only the two java files should be in the PR as far as I understand. Please recreate a separate PR with just the Java implementation changed.