-
Notifications
You must be signed in to change notification settings - Fork 1
feat(deva): persist containers and refresh container management #123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,5 +1,4 @@ | ||||||||||
| # syntax=docker/dockerfile:1.4 | ||||||||||
|
|
||||||||||
| # deva.sh - Docker Image | ||||||||||
| # Provides a fully isolated Claude Code environment with sensible development tools | ||||||||||
|
|
||||||||||
|
|
@@ -23,7 +22,7 @@ RUN --mount=type=cache,target=/var/cache/apt,sharing=locked \ | |||||||||
| echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' > /etc/apt/apt.conf.d/keep-cache && \ | ||||||||||
| apt-get update && \ | ||||||||||
| apt-get install -y --no-install-recommends \ | ||||||||||
| ca-certificates curl wget git git-lfs gnupg lsb-release locales sudo \ | ||||||||||
| ca-certificates curl wget git git-lfs gnupg lsb-release locales tzdata sudo \ | ||||||||||
| software-properties-common build-essential pkg-config libssl-dev \ | ||||||||||
| unzip zip bzip2 xz-utils tini gosu less man-db \ | ||||||||||
| python3-dev libffi-dev \ | ||||||||||
|
|
@@ -53,15 +52,32 @@ RUN --mount=type=cache,target=/var/cache/apt,sharing=locked \ | |||||||||
| apt-get install -y --no-install-recommends nodejs && \ | ||||||||||
| apt-get -y clean && rm -rf /var/lib/apt/lists/* | ||||||||||
|
|
||||||||||
| RUN --mount=type=cache,target=/root/.npm,sharing=locked \ | ||||||||||
| npm install -g npm@latest pnpm copilot-api@latest && \ | ||||||||||
| npm cache clean --force | ||||||||||
|
|
||||||||||
| # Install bun runtime before building Copilot API fork | ||||||||||
| RUN curl -fsSL https://bun.sh/install | bash && \ | ||||||||||
| ln -s /root/.bun/bin/bun /usr/local/bin/bun | ||||||||||
|
|
||||||||||
| # Install Copilot API branch with GPT-5 Codex responses support (PR #119 from caozhiyuan fork) | ||||||||||
| # Pinned to specific commit for reproducibility and security | ||||||||||
| ARG COPILOT_API_REPO=https://github.com/caozhiyuan/copilot-api.git | ||||||||||
| ARG COPILOT_API_BRANCH=feature/gpt-5-codex | ||||||||||
| ARG COPILOT_API_COMMIT=HEAD | ||||||||||
|
||||||||||
| ARG COPILOT_API_COMMIT=HEAD | |
| ARG COPILOT_API_COMMIT=abcdef1234567890 |
Copilot
AI
Oct 26, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The pinned commit hash for atlas-cli appears arbitrary and lacks context. Consider documenting why this specific commit was chosen (e.g., last known stable version, specific feature/fix). If this is a security fix or version pin, add a comment explaining it.
| # Install Go tools for Atlassian integration (Confluence/Jira/Bitbucket) | |
| # Install Go tools for Atlassian integration (Confluence/Jira/Bitbucket) | |
| # Pin atlas-cli to commit 5f6a20c4d164bf6fe6f5c60f9ac12dfccf210758 for reproducible builds. | |
| # This commit was chosen as the latest stable version as of 2024-06-01; update if upstream releases a new stable or security fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using
HEADas the default commit introduces non-reproducible builds and potential security risks. Pin to a specific commit SHA to ensure consistent, auditable builds.