Skip to content

fix: unknown change slugs fail and write nothing (#207) - #240

Open
androidand wants to merge 3 commits into
mainfrom
fix/207-metadata-needs-existing-change
Open

androidand wants to merge 3 commits into
mainfrom
fix/207-metadata-needs-existing-change

Conversation

@androidand

@androidand androidand commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner
  • claim and release no longer exist, so the phantom-directory path from the report is gone.
  • SaveChangeMetadata refuses a directory that does not exist, so a caller that skips the load-first guard fails instead of creating a stray change directory.
  • Regression tests: the writer itself, and set-stage, set-priority and note for an unknown slug. Each goes red without its guard.
  • Archives the change and updates its design note.

Closes #207

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Deploying specsync with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6830ba0
Status: ✅  Deploy successful!
Preview URL: https://67766e10.specsync.pages.dev
Branch Preview URL: https://fix-207-metadata-needs-exist.specsync.pages.dev

View logs

@androidand androidand changed the title test: unknown change slugs fail and write nothing (#207) fix: unknown change slugs fail and write nothing (#207) Nov 12, 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.

release/claim must not create a phantom change directory

1 participant