Skip to content

feat: add code image build to sandbox-image workflow - #314

Merged
ralphbean merged 3 commits into
mainfrom
feat/code-image-workflow
Apr 21, 2026
Merged

feat: add code image build to sandbox-image workflow#314
ralphbean merged 3 commits into
mainfrom
feat/code-image-workflow

Conversation

@ralphbean

Copy link
Copy Markdown
Member

The code agent image (ghcr.io/fullsend-ai/fullsend-code) extends the base sandbox image with Go, gitleaks, and pre-commit. Without a CI build, the image doesn't exist and code agent sandbox creation fails.

Add a build-code job that runs after build-base, passing the freshly built base image via the BASE_IMAGE build arg. Widen the paths trigger to include images/code/**.

The code agent image (ghcr.io/fullsend-ai/fullsend-code) extends the
base sandbox image with Go, gitleaks, and pre-commit. Without a CI
build, the image doesn't exist and code agent sandbox creation fails.

Add a build-code job that runs after build-base, passing the freshly
built base image via the BASE_IMAGE build arg. Widen the paths trigger
to include images/code/**.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Apr 21, 2026

Copy link
Copy Markdown

Site preview

Preview: https://2a4364a3-site.fullsend-ai.workers.dev

Commit: ed815d15e4160861c1a07891cc76181a97f80407

The previous expression fell back to steps.meta.outputs.tags which
is a multi-line string, producing an invalid BASE_IMAGE build-arg.
On PRs the base image isn't pushed, so build-code couldn't pull it.

Replace with a shell step that emits a single reference:
- Push context: immutable digest (name@sha256:...)
- PR context: latest published image from registry

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@ascerra ascerra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We need to delete the build-images.yml job I just created in PR #286
https://github.com/fullsend-ai/fullsend/pull/286/changes#diff-a6ce9b5b0a53d8dc6dbd59d40b23994c3e68a91fac73c0d7eacdf373c95476c9

I was just thinking about working on a PR to combine all of our builds into a single job so I like this approach as long as you remove the other one as a part of this

and now I'm thinking this job shouldn't be names sandbox-image.yml (maybe change to sandbox-images.yml or some other name to describe its building more then just one sandbox image)

…ox-images

Address review feedback from @ascerra: remove the build-images.yml
workflow (from PR #286) and rename sandbox-image.yml to sandbox-images.yml
to reflect that it builds more than one image.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants