Skip to content

ci: pin the dx CLI to a build with the hardened api-key path - #32

Merged
wittjosiah merged 1 commit into
mainfrom
claude/plugin-excalidraw-pr8-cleanup-hb0ebu
Aug 9, 2026
Merged

ci: pin the dx CLI to a build with the hardened api-key path#32
wittjosiah merged 1 commit into
mainfrom
claude/plugin-excalidraw-pr8-cleanup-hb0ebu

Conversation

@wittjosiah

Copy link
Copy Markdown
Member

Summary

Bumps the pinned DX_CLI_PACKAGE default 1edc570d7b0a3b (dxos main after dxos/dxos#12528 merged) in release.yml and setup-publisher.yml.

The new build carries the review-hardening the previous pin predates:

  • PDS session resolved before the immutable bundle upload — a publish that can't authenticate its record writes fails before burning the version.
  • Rejected api key is terminal (no useless retry loop).
  • 409 on re-upload of an existing version resolves to the existing bundle URL, keeping registry_only re-runs idempotent — validated live: run logged Version already in registry: …/tictactoe/0.10.1/manifest.json and rewrote the records cleanly.
  • Raw endpoints (putBlob) honor the api key too.

Pin still drops entirely once a CLI with these fixes ships to npm.

Changeset

None — CI-only change.


Generated by Claude Code

dxos/dxos#12528 landed: the CLI now resolves the PDS session before the
immutable upload, treats a rejected api key as terminal, and tolerates the 409
on an already-uploaded version — the previous pin predates all of that.

Co-Authored-By: Claude Opus 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01SZ4ZdaPo9erkh5yF4eXh3Z
@wittjosiah
wittjosiah marked this pull request as ready for review August 9, 2026 15:55
@wittjosiah
wittjosiah added this pull request to the merge queue Aug 9, 2026
Merged via the queue into main with commit 9fdcc92 Aug 9, 2026
3 checks passed
@wittjosiah
wittjosiah deleted the claude/plugin-excalidraw-pr8-cleanup-hb0ebu branch August 9, 2026 15:56
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