Skip to content

Install VSCode extensions at image build time instead of container init#350

Merged
michaelxiao7 merged 4 commits into
masterfrom
vscode-extensions-df
Mar 30, 2026
Merged

Install VSCode extensions at image build time instead of container init#350
michaelxiao7 merged 4 commits into
masterfrom
vscode-extensions-df

Conversation

@michaelxiao7
Copy link
Copy Markdown
Contributor

Moves Gemini Code Assist and Claude Code extension installation into the Dockerfile

Also added an empty line at the end of install.sh because I noticed that I removed it from my previous PR

PHP-135615

Moves Gemini Code Assist and Claude Code extension installation from a
cont-init script (99-install-extensions) into the Dockerfile RUN step.
This eliminates the s6-setuidgid dependency which fails in Workbench's
restricted Kubernetes environment due to missing CAP_SETGID.

Made-with: Cursor
@michaelxiao7 michaelxiao7 self-assigned this Mar 27, 2026
@michaelxiao7 michaelxiao7 requested review from a team as code owners March 27, 2026 21:09
Comment thread src/vscode/Dockerfile Outdated
Comment on lines +19 to +21

USER root
RUN rm /tmp/geminicodeassist.vsix /tmp/claudecode.vsix
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.

nit: if we do the curling as abc we can also remove all in one RUN block. not highly important but it'll slightly reduce the amount of layers generated

@michaelxiao7 michaelxiao7 force-pushed the vscode-extensions-df branch from 4db7a9a to 1d9a0ca Compare March 30, 2026 16:51
@michaelxiao7 michaelxiao7 force-pushed the vscode-extensions-df branch from 1d9a0ca to cec88e7 Compare March 30, 2026 17:16
@michaelxiao7 michaelxiao7 merged commit b7bf009 into master Mar 30, 2026
3 checks passed
@michaelxiao7 michaelxiao7 deleted the vscode-extensions-df branch March 30, 2026 19:25
PeterSu92 pushed a commit that referenced this pull request May 21, 2026
…it (#350)

Install VSCode extensions at image build time instead of container init

Moves Gemini Code Assist and Claude Code extension installation from a
cont-init script (99-install-extensions) into the Dockerfile RUN step
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