atiaomar1978-hub commented on PR #25551: URL: https://github.com/apache/camel/pull/25551#issuecomment-5348310170
## Review complete — test coverage assessment _AI-generated on behalf of atiaomar1978-hub_ **Verdict: Request changes** (docs fix required; tests are strong) ### Test coverage score: **8.5 / 10** Bugbot found no critical security bugs. The test suite is one of the better scripting-language additions I've seen in Camel: - ✅ Default vs trusted security modes - ✅ IO / process / class-lookup denial - ✅ Syntax vs evaluation error classification - ✅ List/dict materialization after context close - ✅ Concurrent evaluation (64 threads) - ✅ Resource loading (classpath + file) - ✅ Java DSL + `language:python3:` endpoint + choice/filter routes - ✅ Typed-language SPI compliance **Gaps (non-blocking):** Python `set`/tuple/custom-object return values; YAML DSL route smoke test. **Blocking:** Variables table in `python3-language.adoc` lists `exchange`/`message`/`context` as available but default mode doesn't bind them. Thanks @ramu11 — GraalPy is a great direction for Python 3 support. Fix the docs table and this is ready from a testing/security standpoint. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
