Skip to content

fix: install git in Docker runtime image#80

Merged
moshest merged 1 commit intomainfrom
claude/investigate-issue-79-zxHdY
May 6, 2026
Merged

fix: install git in Docker runtime image#80
moshest merged 1 commit intomainfrom
claude/investigate-issue-79-zxHdY

Conversation

@moshest
Copy link
Copy Markdown
Member

@moshest moshest commented May 6, 2026

Summary

  • Fixes Docker: git not installed in image #79: node:22-bookworm-slim does not ship git, so any flow that adds a GitHub URL inside the published Docker image fails with Git clone failed: /bin/sh: 1: git: not found.
  • Installs git (plus ca-certificates for reliable HTTPS clones) in the runtime stage of packages/context/Dockerfile.
  • git is required not just for the initial cloneRepository (packages/context/src/git.ts:212) but also for detectVersion, fetchTagsWithMetadata, and checkoutRef, which all shell out to git.

Test plan

  • pnpm install --frozen-lockfile
  • pnpm lint
  • pnpm build
  • pnpm test (149 + 35 tests passing)
  • Manual: build the Docker image and run context add https://github.com/<owner>/<repo> inside the container to confirm the clone succeeds.

https://claude.ai/code/session_01BNjzMqqmzWbsxPT5HgrXCN


Generated by Claude Code

The runtime stage uses node:22-bookworm-slim, which does not ship git.
Adding GitHub URLs failed at runtime with `Git clone failed: /bin/sh:
1: git: not found`. Install git (and ca-certificates for HTTPS clones)
in the runtime stage so cloneRepository, detectVersion, and
checkoutRef work out of the box.

https://claude.ai/code/session_01BNjzMqqmzWbsxPT5HgrXCN
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 6, 2026

🦋 Changeset detected

Latest commit: aad0243

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@neuledge/context Patch
@neuledge/registry Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@moshest moshest merged commit b5dd83b into main May 6, 2026
3 checks passed
@moshest moshest deleted the claude/investigate-issue-79-zxHdY branch May 6, 2026 06:49
@github-actions github-actions Bot mentioned this pull request May 6, 2026
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.

Docker: git not installed in image

2 participants