Skip to content

Create devcontainer - #211

Merged
abkfenris merged 5 commits into
mainfrom
devcontainer
Jul 10, 2026
Merged

Create devcontainer#211
abkfenris merged 5 commits into
mainfrom
devcontainer

Conversation

@abkfenris

Copy link
Copy Markdown
Collaborator

For Codespaces and safer local agent usage.

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

Javascript size report

Size
Base (main) 1118.8 KB
PR 1148.0 KB
Delta +29.2 KB (+2.6%)

⚠️ Size increased +2.6% (within 5% threshold).

For Codespaces and safer local agent usage.
The official ghcr.io/anthropics/devcontainer-features/claude-code
feature installs itself as root via npm, and root can't resolve
mise's user-scoped shims for node/npm (mise reports "npm is not a
valid shim"). Install the Claude Code CLI ourselves in the same
Dockerfile layer as `mise install`, where mise is already correctly
activated as vscode.

Also stop symlinking individual mise-managed binaries into
/usr/local/bin — npm's own bin script computes sibling paths like
../lib/node_modules relative to its real directory, which breaks
through a single-file symlink. Symlink the containing install
directories instead and put those on PATH, which works for any
caller (root included) since the kernel resolves the ".." after the
symlink itself.

Verified end-to-end: rust_test, py_test (3.12-3.14), js_build,
js_test, and js_test_wasm (headless Chrome via wasm-pack) all pass
inside the built container.
…build

The wasm_size job checks out github.base_ref into a nested base/
checkout and builds from base/js. mise's version resolution stops at
the nearest ancestor config, and main's mise.toml (pre-migration)
only declares node — so rust/wasm-pack are invisible from base/js and
the shim errors "No version is set for shim: wasm-pack".

Write the already-installed versions to mise's global config right
after installing them (mise's own suggested remedy), which is
consulted as a fallback below the nearest project config.

Confirmed via CI run 29113346137: Build npm package, Build demo, and
Test all passed; only WASM size check failed, exactly as diagnosed.
@abkfenris
abkfenris marked this pull request as ready for review July 10, 2026 19:15
@abkfenris
abkfenris merged commit dbcd091 into main Jul 10, 2026
21 checks passed
@abkfenris
abkfenris deleted the devcontainer branch July 10, 2026 19:15
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.

1 participant