Skip to content

Publish to the registry only on a 404, not on any non-200 - #80

Merged
lbesecker195 merged 1 commit into
mainfrom
fix/publish-check-transient
Oct 1, 2026
Merged

lbesecker195 merged 1 commit into
mainfrom
fix/publish-check-transient

Conversation

@lbesecker195

@lbesecker195 lbesecker195 commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Closes #79.

The deploy for #78 went red on publish-mcp-registry with invalid version: cannot publish duplicate version. The listing was fine.

The plan step asks the official registry whether server.json's version already exists. That request got a transient HTTP 500 — it answers 200 now, three times running — and the step treated any non-200 as "not published yet", so the job tried to publish 1.0.1 a second time.

registry answers before after
404 publish publish
200 skip skip
5xx / other publish → fails retry ×4 with backoff, then skip with a warning

Skipping is safe: nothing about the listing changes, and the next deploy checks again. A deploy should never go red because a third party had a bad few seconds.


Pages affected:

  • MCP Harbor -- the registry whose official-registry listing this job maintains.

🤖 Generated with Claude Code

The publish job checks whether server.json's version is already on the
official MCP registry. The registry answered that check with a transient
500, the job read anything but 200 as "not published", and it tried to
publish 1.0.1 again -- failing the run with "duplicate version" though the
listing was fine.

- 404 means publish; 200 means skip
- A 5xx is retried four times with backoff
- If it persists, the run skips publishing with a warning rather than
  failing; the next deploy checks again

Closes #79

---

Pages affected:

- [MCP Harbor](https://ai.mcpharbor.dev/) -- the registry this job lists on the official MCP registry.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@lbesecker195
lbesecker195 merged commit 6f310ef into main Oct 1, 2026
@lbesecker195
lbesecker195 deleted the fix/publish-check-transient branch October 1, 2026 11:24
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.

Registry publish fails the deploy on a transient 500

1 participant