docs: pin enscli/ensskills prerelease install refs to merged snapshot#2252
Conversation
Now that #2242 is merged, point the prerelease install shim at immutable refs from that merge's snapshot publish instead of the floating @next/main: - ENSCLI_ENSSKILLS_NPM_SPEC -> 0.0.0-next-20260603190454 (published snapshot version) - ENSCLI_ENSSKILLS_GIT_REF -> 0eec193 (the main commit it was built from) Also note in the hosted-instance warning that these prerelease refs are functional against the hosted instances running snapshot.sdkVersion. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Warning Review limit reached
More reviews will be available in 54 minutes and 25 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates prerelease pinning documentation and configuration for ChangesPrerelease Snapshot Pinning
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Pins the documentation’s temporary prerelease installation references for enscli/ensskills to the specific snapshot artifacts produced from the now-merged work, replacing floating @next/main refs with immutable version/ref values to keep install snippets stable until the first official lockstep release.
Changes:
- Updated prerelease install constants to a specific snapshot npm version and a specific
maincommit ref. - Adjusted docs prose in integration pages to describe the new “pinned snapshot” behavior.
- Updated the hosted-instance compatibility warning copy to reflect prerelease snapshot installs and hosted compatibility.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/ensnode.io/src/data/enscli-ensskills-prerelease.ts | Replaces floating prerelease refs with pinned snapshot npm spec + commit ref and updates explanatory comments. |
| docs/ensnode.io/src/content/docs/docs/integrate/integration-options/ensskills.mdx | Updates prerelease aside and pinning guidance text to match pinned snapshot behavior. |
| docs/ensnode.io/src/content/docs/docs/integrate/ai-llm.mdx | Updates npx skills quickstart prose to describe pinning to an immutable commit during prerelease. |
| docs/ensnode.io/src/components/molecules/HostedInstanceVersionWarning.astro | Updates hosted-instance warning text to reference prerelease snapshot installs and compatibility statement. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile SummaryThis PR fast-follows #2242 to convert the
Confidence Score: 5/5Safe to merge; the core pin changes are correct and isolated to the prerelease data module. The constant values in enscli-ensskills-prerelease.ts are updated from floating to immutable refs as described, and every downstream doc surface that consumes them is updated consistently. The only noteworthy item is that two Aside blocks in ensskills.mdx — the claude-code agent-key caution and the monorepo scoping note — are removed without explanation, but both were documentation-only and do not affect build output or runtime behavior. docs/ensnode.io/src/content/docs/docs/integrate/integration-options/ensskills.mdx — verify the removal of the claude-code and monorepo Asides is intentional. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[enscli-ensskills-prerelease.ts] -->|ENSCLI_ENSSKILLS_NPM_SPEC\n0.0.0-next-20260603190454| B[ai-llm.mdx]
A -->|ENSCLI_ENSSKILLS_NPM_SPEC| C[ensskills.mdx]
A -->|ENSCLI_ENSSKILLS_NPM_SPEC| D[HostedInstanceVersionWarning.astro]
A -->|ENSCLI_ENSSKILLS_GIT_REF\n0eec193| B
A -->|ENSCLI_ENSSKILLS_GIT_REF| C
B -->|renders npm install snippet| E[Rendered Docs]
B -->|renders npx skills URL| E
C -->|renders npm install snippet| E
C -->|renders npx skills URL| E
D -->|renders prerelease notice| E
Reviews (2): Last reviewed commit: "fix" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
Fast-follow to #2242. Now that it's merged, the docs install snippets for
enscli/ensskillsno longer need to track the floating@nexttag /mainbranch — pin them to the immutable refs that merge's snapshot publish produced.Changes
docs/ensnode.io/src/data/enscli-ensskills-prerelease.ts:ENSCLI_ENSSKILLS_NPM_SPEC→0.0.0-next-20260603190454(the published snapshot version)ENSCLI_ENSSKILLS_GIT_REF→0eec193(themaincommit it was built from, soskills add tree/<ref>is pinned to an immutable commit)Docs prose updated to match (
ai-llm.mdx,ensskills.mdx): the prerelease window now "pins to an immutablemaincommit + exact snapshot version" rather than "tracks@next/main".Hosted-instance warning (
HostedInstanceVersionWarning.astro): notes that these prerelease refs are functional against the hosted instances runningsnapshot.sdkVersion.Still temporary
The
enscli-ensskills-prerelease.tsmodule and theTEMP prereleasemarkers remain — they get removed (snippets reverted tosnapshot.sdkVersion/v${sdkVersion}) at the first official release that publishesenscli/ensskillsin lockstep with the suite.Validation
pnpm -F @docs/ensnode build✓ (113 pages)pnpm lint✓