Experiment: pre-install Seqera AI CLI in training devcontainer#952
Experiment: pre-install Seqera AI CLI in training devcontainer#952pinin4fjords wants to merge 2 commits into
Conversation
Adds a new local devcontainer feature that installs the Seqera AI CLI (npm package `seqera`) from the @dev channel, and a Node.js 20 feature to satisfy its >=18 runtime requirement. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for nextflow-training ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Nextflow linting complete!
💡 Tip: Click filename locations to go directly to that code. View all 53 issuesView formatting changes<...*[Comment body truncated]*
|
I hate to say this (because this would be so convenient!) but no I don't think we should do that. It muddies the waters of what's OSS or not. Perhaps we could pre-build an alternate container that can be selected optionally, just like we're able to select a specific training release version? |
Yeah, don't worry, was mostly figuring out the mechanics how how this might work. |
Summary
Experiment: pre-install the Seqera AI CLI in the training devcontainer so learners can use it without a separate install step.
.devcontainer/local-features/seqera-ai/runsnpm install -g seqera@dev(dev channel; the published stable is marked BETA today).ghcr.io/devcontainers/features/node:1(Node 20) because the prebuilt training image currently has nonode/npm.codespaces-dev/devcontainer.jsononly (the file that builds the publishedghcr.io/nextflow-io/training:latestimage). The production devcontainer.json keeps using that image, so no change needed there.Footprint
npm install -g seqera: ~105 MB unpackedAuthentication happens on first run (
seqera login), not at build time, so the image itself stays unauthenticated.Things to decide before merge
@devchannel was picked for fast iteration. Want to pin tolatest(stable BETA) instead?darwin-{arm64,x64}andlinux-x64prebuilts; if/when ARM Linux training images appear, the install will fail. The install.sh exits gracefully ifnpmis missing, but doesn't yet guard against the missing-arch case.Test plan
seqera --versionandseqera ai --helpto confirm the binary is on PATH🤖 Generated with Claude Code