Skip to content

fix(registry): stop sanity v4 releases failing the nightly publish - #121

Merged
moshest merged 1 commit into
mainfrom
claude/fix-sanity
Aug 25, 2026
Merged

fix(registry): stop sanity v4 releases failing the nightly publish#121
moshest merged 1 commit into
mainfrom
claude/fix-sanity

Conversation

@moshest

@moshest moshest commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

The nightly registry publish failed on Aug 20 and Aug 21:

npm/sanity@4.22.1: Directory not found: /tmp/context-git-dUIB0B/docs
Succeeded: 43  Skipped: 11  Failed: 1

One failed definition exits the whole run non-zero, so all 140 packages were affected.

The run went green again on Aug 22, but nothing had been fixed. publish-all --since 2 only selects versions published in the last two days; once 4.22.1 aged out of that window it stopped being built and the error disappeared. The next v4 release would have failed identically.

Root cause

The definition declared min_version: "3.0.0", but docs/ only exists from v5 onward. Sanity maintains v4, v5 and v6 in parallel and still ships v4 maintenance releases, so any of them hits a missing directory.

Tag Files in docs/ Build
v6.11.0 3 31 sections
v5.31.2 2 23 sections
v4.22.1 0 fails
v3.99.0 0 would fail

Fix

Raise the floor to 5.0.0. Verified locally: 6.11.0 and 5.31.2 build; 4.22.1 now returns No version entry matches 4.22.1 in sanity, and version-check.ts:96 filters unmatched versions out before the build rather than throwing — so the nightly skips it cleanly instead of failing.

Worth a follow-up (not in this PR)

Even where it works, this package is thin: docs/ holds 3 internal developer notes (CORE_CONCEPTS.md, STEGA.md, TELEMETRY.md), not Sanity's user documentation, which lives at sanity.io/docs. By registry/README.md's own rule of thumb — "a few hundred sections means docs_path is right; a handful usually means it's pointing at the wrong directory" — 31 sections says this is the wrong source.

Sanity publishes an llms.txt (https://www.sanity.io/llms.txt returns 200), but registry definitions only support git and zip sources, so pointing at it needs a new source type rather than a config change.

Test plan

  • registry build sanity 6.11.0 → 31 sections
  • registry build sanity 5.31.2 → 23 sections
  • registry build sanity 4.22.1 → cleanly excluded
  • Definition change only; no code touched

Generated by Claude Code

The definition claimed min_version 3.0.0, but sanity's v3 and v4 lines have
no docs/ directory at all. Sanity still ships v4 maintenance releases, and
each one took down the entire nightly run for all 140 packages with
"Directory not found: /tmp/context-git-XXXXXX/docs" — most recently
npm/sanity@4.22.1 on Aug 20 and Aug 21.

The run went green again on Aug 22 only because --since 2 stopped selecting
4.22.1 once it aged out of the window. Nothing was fixed; the next v4 release
would have failed the same way.

docs/ first appears in v5. Verified by building each line:
  v6.11.0 -> 31 sections    v5.31.2 -> 23 sections
  v4.22.1 -> no version entry matches, so version-check.ts filters it out
  before the build rather than throwing.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NBQQpA86yYzwJUiVz8ph2R
@changeset-bot

changeset-bot Bot commented Aug 25, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 76401a8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@moshest
moshest marked this pull request as ready for review August 25, 2026 22:34
@moshest
moshest merged commit 0305b20 into main Aug 25, 2026
3 checks passed
@moshest
moshest deleted the claude/fix-sanity branch August 25, 2026 22:34
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