feat: add Crystal toolchain #55

Merged
alex merged 3 commits from pi/issue54-1791943921739 into master 2026-10-14 02:31:15 +00:00
Contributor

Fixes #54

Adds the Crystal toolchain to the runner image using the floating docker.io/crystallang/crystal:1 tag.

Copied from the Crystal image into the build target:

  • /usr/bin/crystal (statically linked)
  • /usr/bin/shards
  • /usr/lib/crystal (libgc.a)
  • /usr/share/crystal (stdlib sources)

A crystal --version check was added to the version-print step.

Fixes #54 Adds the Crystal toolchain to the runner image using the floating `docker.io/crystallang/crystal:1` tag. Copied from the Crystal image into the build target: - `/usr/bin/crystal` (statically linked) - `/usr/bin/shards` - `/usr/lib/crystal` (`libgc.a`) - `/usr/share/crystal` (stdlib sources) A `crystal --version` check was added to the version-print step.
Co-authored-by: alex <alex@forge.l3x.in>
alex requested changes 2026-10-14 02:15:10 +00:00
Dismissed
alex left a comment

/pi update the pr

/pi update the pr
Containerfile Outdated
@ -119,6 +124,11 @@ COPY --from=golangci /usr/bin/golangci-lint /usr/local/bin/golangci-lint
COPY --from=deno /deno /usr/local/bin/deno
ENV DENO_DIR=/deno-dir
COPY --from=crystal /usr/bin/crystal /usr/local/bin/crystal
Owner

we should add a copy crystal comment here

we should add a copy crystal comment here
alex marked this conversation as resolved
Author
Contributor

No prompt found - cannot proceed


View action run | Model: zai/glm-5.3-flash (thinking: low) | Time: 0s | Action v2.30.0-develop (develop @ 9df4fde)

<!-- pi-coding-agent-comment --> No prompt found - cannot proceed --- [View action run](https://forge.l3x.in/alex/runner-image/actions/runs/169) | Model: zai/glm-5.3-flash (thinking: low) | Time: 0s | Action v2.30.0-develop (develop @ 9df4fde)
Owner

/pi I added a review, patch is good but lacks a "copy crystal" comment header, fix it and push changes to pr

/pi I added a review, patch is good but lacks a "copy crystal" comment header, fix it and push changes to pr
Author
Contributor

Done. Added the missing header comment above the Crystal COPY block in Containerfile, matching the existing style:

#################
# Copy Crystal  #
#################
COPY --from=crystal /usr/bin/crystal /usr/local/bin/crystal
...

Pushed as commit 3353c9d to pi/issue54-1791943921739, so PR #55 is updated.


View action run | Model: zai/glm-5.3-flash (thinking: low) | Time: 1m 2s | Tokens: 51.8K | Pi SDK v1.0.4 | Action v2.30.0-develop (develop @ 9df4fde)

<!-- pi-coding-agent-comment --> Done. Added the missing header comment above the Crystal `COPY` block in `Containerfile`, matching the existing style: ```dockerfile ################# # Copy Crystal # ################# COPY --from=crystal /usr/bin/crystal /usr/local/bin/crystal ... ``` Pushed as commit `3353c9d` to `pi/issue54-1791943921739`, so PR #55 is updated. --- [View action run](https://forge.l3x.in/alex/runner-image/actions/runs/173) | Model: zai/glm-5.3-flash (thinking: low) | Time: 1m 2s | Tokens: 51.8K | Pi SDK v1.0.4 | Action v2.30.0-develop (develop @ 9df4fde)
Owner

/pi rebase on head, resolve conflicts, push changes

/pi rebase on head, resolve conflicts, push changes
forgejo-actions force-pushed pi/issue54-1791943921739 from 3353c9d2d9 to e477852217 2026-10-14 02:24:47 +00:00 Compare
Author
Contributor

Rebased PR #55 onto current master and force-pushed.

  • Resolved the conflicts caused by the Deno removal on master: the Deno COPY/ENV DENO_DIR block and deno --version check were dropped, keeping only the Crystal additions (COPY --from=crystal ... block and RUN crystal --version).
  • Squashed the follow-up "add 'Copy Crystal' header" commit into the main commit since the header was already included in the resolved conflict.
  • Branch pi/issue54-1791943921739 now contains a single commit e477852 feat: add Crystal toolchain on top of b365ca6 chore: remove Deno from runner image.

View action run | Model: zai/glm-5.3-flash (thinking: low) | Time: 2m 26s | Tokens: 113.4K | Cost: $0.01 | Pi SDK v1.0.4 | Action v2.30.0-develop (develop @ 9df4fde)

<!-- pi-coding-agent-comment --> Rebased PR #55 onto current `master` and force-pushed. - Resolved the conflicts caused by the Deno removal on master: the Deno `COPY`/`ENV DENO_DIR` block and `deno --version` check were dropped, keeping only the Crystal additions (`COPY --from=crystal ...` block and `RUN crystal --version`). - Squashed the follow-up "add 'Copy Crystal' header" commit into the main commit since the header was already included in the resolved conflict. - Branch `pi/issue54-1791943921739` now contains a single commit `e477852 feat: add Crystal toolchain` on top of `b365ca6 chore: remove Deno from runner image`. --- [View action run](https://forge.l3x.in/alex/runner-image/actions/runs/176) | Model: zai/glm-5.3-flash (thinking: low) | Time: 2m 26s | Tokens: 113.4K | Cost: $0.01 | Pi SDK v1.0.4 | Action v2.30.0-develop (develop @ 9df4fde)
alex approved these changes 2026-10-14 02:30:52 +00:00
alex merged commit 5b0cda664a into master 2026-10-14 02:31:15 +00:00
alex deleted branch pi/issue54-1791943921739 2026-10-14 02:31:15 +00:00
alex referenced this pull request from a commit 2026-10-14 02:31:17 +00:00
Sign in to join this conversation.
No reviewers
No labels
dependencies
No milestone
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
alex/runner-image!55
No description provided.