Skip to content

Conversation

@mishushakov
Copy link
Member

@mishushakov mishushakov commented Oct 18, 2025

Note

Replaces the Dockerfile workflow with a Python/Poetry-based E2B Template, updates CI to build/push via buildx and build the E2B template, and adds necessary project/config files.

  • Template build system:
    • Replace template/Dockerfile, template/e2b.Dockerfile, and template/e2b.toml with programmatic E2B Template in template/template.py plus build scripts (build_dev.py, build_prod.py, build_docker.py).
    • Add Poetry project (template/pyproject.toml, template/poetry.lock, template/.python-version) and README instructions.
    • Add desktop/config assets under template/files/ (Firefox policies/config, Chrome desktop entry, VS Code settings, XFCE config).
  • CI/CD (.github/workflows/template.yml):
    • Set up Python 3.12 and Poetry; install deps.
    • Generate Dockerfile via build_docker.py and pipe to docker buildx build to push desktop:latest.
    • Build E2B template via build_prod.py using E2B_API_KEY.
    • Remove npm-based E2B CLI steps.
  • Repo hygiene:
    • Ignore .ruff_cache/ in .gitignore.

Written by Cursor Bugbot for commit eded58d. This will update automatically on new commits. Configure here.

@mishushakov mishushakov requested a review from mlejva as a code owner October 18, 2025 14:35
cursor[bot]

This comment was marked as outdated.

@mishushakov mishushakov removed the request for review from mlejva October 18, 2025 14:36
cursor[bot]

This comment was marked as outdated.

@mishushakov
Copy link
Member Author

mishushakov commented Oct 18, 2025

PR is currently borked due to an issue in RUN command appending root to package commands and COPY not taking into account the user

)

# Template with user and workdir set
template_with_user_workdir = template.set_user("user").set_workdir("/home/user")
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need this because earlier we set the user to root and workdir to /
this will set the final user and workdir to the Sandbox

Copy link
Member Author

@mishushakov mishushakov Oct 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can probably remove this and just append user=root on every command and copy (broken now)

@mishushakov
Copy link
Member Author

tested on local with current template

Screenshot 2025-10-18 at 17 59 20

@dobrac dobrac self-assigned this Oct 19, 2025
@mishushakov mishushakov merged commit 507f2eb into main Oct 29, 2025
5 of 6 checks passed
@mishushakov mishushakov deleted the templates-sdk branch October 29, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants