Skip to content

Use splat hashes for editor overwrite and publish flows#912

Merged
abstrakt8 merged 1 commit into
mainfrom
codex/list-editor-api-endpoints
May 27, 2026
Merged

Use splat hashes for editor overwrite and publish flows#912
abstrakt8 merged 1 commit into
mainfrom
codex/list-editor-api-endpoints

Conversation

@abstrakt8

Copy link
Copy Markdown
Contributor

Summary

  • Remove editor reliance on internal splat ids for overwrite, settings, and republish flows
  • Use hash consistently when loading existing scene settings and republishing
  • Stop passing overwriteId through the publish settings dialog

Testing

  • npm run lint passed
  • npm run build passed

@abstrakt8 abstrakt8 marked this pull request as ready for review May 27, 2026 09:30
@abstrakt8 abstrakt8 requested a review from Copilot May 27, 2026 09:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the publish/overwrite flow to use splat hashes instead of internal splat IDs, aligning editor overwrite, settings, and republish behavior around the hash identifier.

Changes:

  • Removed Scene.id and PublishSettings.overwriteId.
  • Updated settings fetch/update and republish calls to use overwriteHash.
  • Stopped passing overwriteId from the publish settings dialog.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/publish.ts Switches publish, republish, and settings flows from internal IDs to hashes.
src/ui/publish-settings-dialog.ts Resolves publish settings with overwriteHash only.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@abstrakt8 abstrakt8 merged commit d349d5d into main May 27, 2026
3 checks passed
@abstrakt8 abstrakt8 deleted the codex/list-editor-api-endpoints branch May 27, 2026 09:36
Re-qi pushed a commit to Re-qi/ReSplat that referenced this pull request Jun 21, 2026
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.

3 participants