Skip to content

chore: Add support for rtk - #262

Merged
cleithner-comcast merged 2 commits into
mainfrom
cleith/dev/rtkSupport
Aug 6, 2026
Merged

chore: Add support for rtk#262
cleithner-comcast merged 2 commits into
mainfrom
cleith/dev/rtkSupport

Conversation

@cleithner-comcast

Copy link
Copy Markdown
Contributor

Add support in the devcontainer for rtk and mount the user's ~/.claude directory. To use rtk automatically in vs code copilot chat, enable "chat.useClaudeHooks" in your vsc settings and reload (only works on new sessions). This change assumes the user has rtk installed and initialized globally. It does not run rtk init in the project.

Add support in the devcontainer for rtk and mount the user's ~/.claude
directory. To use rtk automatically in vs code copilot chat, enable
"chat.useClaudeHooks" in your vsc settings and reload (only works on new
sessions). This change assumes the user has rtk installed and
initialized globally. It does not run rtk init in the project.
tleacmcsa
tleacmcsa previously approved these changes Aug 6, 2026
Comment thread .devcontainer/devcontainer.json
kfundecmcsa
kfundecmcsa previously approved these changes Aug 6, 2026

Copilot AI 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.

Pull request overview

This PR updates the VS Code devcontainer environment to support using rtk inside the container and to surface the developer’s local Claude configuration by bind-mounting ~/.claude into the container user’s home directory.

Changes:

  • Install rtk during devcontainer image build under the non-root builder user.
  • Bind-mount the host ~/.claude directory into the container to enable Claude hooks/config usage.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docker/Dockerfile.devcontainer Adds a user-scoped rtk installation step intended for devcontainer usage.
.devcontainer/devcontainer.json Adds a bind mount for the host ~/.claude directory into the container user’s home.

Comment thread docker/Dockerfile.devcontainer Outdated
Comment thread docker/Dockerfile.devcontainer Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@cleithner-comcast
cleithner-comcast dismissed stale reviews from kfundecmcsa and tleacmcsa via 608221b August 6, 2026 15:21
Copilot AI review requested due to automatic review settings August 6, 2026 15:21

Copilot AI 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.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Suppressed comments (2)

docker/Dockerfile.devcontainer:65

  • curl ... | sh can mask download failures because the RUN step’s exit code will be sh’s status (and /bin/sh doesn’t enable pipefail). If the fetch fails, sh may exit 0 after reading an empty stream, leaving rtk uninstalled while the image build still succeeds.
RUN curl -fsSL https://raw.githubusercontent.com/rtk-ai/rtk/refs/heads/master/install.sh | sh

.devcontainer/devcontainer.json:36

  • This bind mount will fail devcontainer startup for developers who don’t already have ${HOME}/.claude on the host (Dev Containers typically uses --mount type=bind, which errors when the source path doesn’t exist). Either ensure it’s created by the host-side init script or document the requirement here so startup failures are self-explanatory.
        "source=${localEnv:HOME}/.ssh,target=/home/${localEnv:USER}/.ssh,type=bind",
        "source=${localEnv:HOME}/.claude,target=/home/${localEnv:USER}/.claude,type=bind"
    ],

@cleithner-comcast

Copy link
Copy Markdown
Contributor Author

This PR focuses on the preferred install path of a user setting up rtk on the host globally for various tools (could be copilot, claude, opencode, gemini, whatever). If we prefer a local-scoped install instead, I can do that. It just means that token saving data will get wiped each container rebuild and gains won't contribute to user-wide data.

@cleithner-comcast
cleithner-comcast merged commit 615f3ec into main Aug 6, 2026
13 checks passed
@cleithner-comcast
cleithner-comcast deleted the cleith/dev/rtkSupport branch August 6, 2026 16:07
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants