Conversation
Local Test & Lint Summary
|
noa-lucent
left a comment
There was a problem hiding this comment.
Looks great. Thanks for taking the package all the way to native ESM so platform-server can import it cleanly.
Update
Tests & Lint
|
|
CI is green. This PR addresses Issue #1317 by migrating @agyn/docker-runner to native ESM so @agyn/platform-server can import named exports without runtime errors. Requesting CODEOWNERS review (team: @agynio/humans). Ready to merge once approved. |
|
Updated validation per directive:
|
Local Docker validation (2026-02-21)1. @agyn/docker-runner
2. @agyn/platform-server
Let me know if you want me to iterate further once the runner env/module issue is resolved. |
Local Docker validation (2026-02-21)1. @agyn/docker-runner
2. @agyn/platform-server
Happy to rerun once the docker-runner env module issue is resolved. |
Docker runner Dockerfile update validation (2026-02-21)Build & runtime (agyn/docker-runner)
Platform server end-to-end check (agyn/platform-server)
Repo test/lint guardrailsAll commands pass locally. Let me know if you want me to keep the LiteLLM stub handy or rerun anything else. |
Summary
.jsimport specifiers inside docker-runner sources so the module graph matches the rest of the monorepoTesting
Fixes #1317