fix(metadata-protocol): drop internal ADR path from read-only package error#2618
Merged
Merged
Conversation
… error The writable_package_required rejection cited docs/adr/0070-package-first-authoring.md, an internal doc path meaningless to Studio/API users. Rewrite as user-facing remediation guidance instead. Fixes #2615 (framework portion; console-side items tracked separately).
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
📓 Docs Drift CheckNo hand-written docs reference the 1 changed package(s). ✅ |
4 tasks
This was referenced Jul 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Addresses the one framework-side item from #2615 (Studio package-create UX dogfood follow-ups). All other items in that tracking issue are
objectuiconsole-side and out of scope here.writable_package_requiredrejection thrown when authoring runtime-only metadata into a read-only code/installed package cited an internal doc path (See docs/adr/0070-package-first-authoring.md.), which is meaningless to Studio/API callers.Test plan
pnpm --filter @objectstack/metadata-protocol build && pnpm --filter @objectstack/metadata-protocol test— 8/8 passingvitest run test/package-first-authoring.dogfood.test.ts(packages/dogfood) — the golden regression test covering this exact error path — 5/5 passing (asserts onerror.code, not message text, so unaffected by the copy change)🤖 Generated with Claude Code
Generated by Claude Code