Changes for a python code interpreter #12

Merged
pptx704 merged 5 commits from feat/python-code-interpreter into dev 2026-04-07 20:18:06 +00:00
Member

Added env expansion, sandbox env fetching and configurable healthcheck

Created a basic recipefile and healthcheck for a minimal python interpreter with jupyter

Added env expansion, sandbox env fetching and configurable healthcheck Created a basic recipefile and healthcheck for a minimal python interpreter with jupyter
tksadik92 added 1 commit 2026-04-06 19:23:53 +00:00
healthchecks

Fix ENV instructions to expand $VAR references at set time using the
current env state, preventing self-referencing values like
PATH=/opt/venv/bin:$PATH from producing recursive expansions. Remove
expandEnv from shellPrefix to avoid double expansion.

Fetch sandbox environment variables via `env` before recipe execution
so ENV steps resolve against actual runtime values from the base
template image.

Replace hardcoded healthcheck timing with a Dockerfile-like flag parser
supporting --interval, --timeout, --start-period, and --retries. Add
start-period grace window and bounded retry counting to
waitForHealthcheck.

Add python-interpreter-v0-beta recipe and healthcheck files.
pptx704 added 1 commit 2026-04-06 20:45:56 +00:00
pptx704 requested changes 2026-04-07 06:50:30 +00:00
@ -59,0 +123,4 @@
// environment variable name (alphanumeric or underscore)
func isNameChar(c byte) bool {
return (c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z') ||
(c >= '0' && c <= '9') || c == '_'
Owner

Use regex here

Use regex here
pptx704 marked this conversation as resolved
tksadik92 added 1 commit 2026-04-07 16:57:06 +00:00
Updated recipefile with test script to check code execution with state
management
pptx704 added 1 commit 2026-04-07 19:35:57 +00:00
pptx704 added 1 commit 2026-04-07 20:15:03 +00:00
- Fix envRegex: remove spurious (\$)? group that swallowed $$$, handle ${}
- wrenn-init.sh: add || true to networking commands under set -e, remove dead code
- waitForHealthcheck: use context deadline for unlimited retries instead of implicit 100 cap
- Make parseSandboxEnv a package-level function (unused receiver)
- Fix WrappedCommand test: map iteration order dependency, pre-expand env values
- Fix error wrapping: %v → %w per project conventions
- test-jupyter-kernel.py: move import to top-level, fix misleading comment
pptx704 changed title from WIP: Added necessary changes for a python code interpreter to Changes for a python code interpreter 2026-04-07 20:17:58 +00:00
pptx704 merged commit 2737288a2b into dev 2026-04-07 20:18:06 +00:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wrenn/wrenn#12
No description provided.