Uh oh!
There was an error while loading. Please reload this page.
docs(sharing-service): name the federated phantom-anchor case in the SHARING_NOT_ENABLED list (#8119) - #8217
Merged
os-zhuang merged 1 commit intoAug 12, 2026
Conversation
…SHARING_NOT_ENABLED list (#8119) The "Typical Errors" list enumerates the exact conditions that produce a 422, and #8209 added a fifth: a FEDERATED (ADR-0015 `external`) object whose `owner_id` is the platform's injected anchor rather than a real remote column. Leaving it out is the enforced-but-undocumented inverse of a declared-but-unenforced gap — the runtime now refuses more than the page says. Docs-only; the behaviour itself shipped in #8209. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PEVB6w7D7uCszR9Mw1BL73
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
os-zhuang
marked this pull request as ready for review
August 12, 2026 21:12
Contributor
⛔ merge queue 构建失败 — 先分诊,再决定要不要重排队列构建 31641477795 红了。队列跑的是全量套件(PR 侧 CI 只跑 affected 子集), 失败的 job(日志抽取,best effort):
历史信号:
分诊清单:
Generated by Claude Code · merge-queue-triage workflow (#4859) |
Uh oh!
There was an error while loading. Please reload this page.
os-zhuang
deleted the
claude/issue-8119-sharing-service-doc-phantom-anchor
branch
August 12, 2026 21:34
os-zhuang pushed a commit
that referenced
this pull request
Aug 12, 2026
…pages (#8158) The "Typical Errors" list on `services.sharing` enumerates the exact conditions behind each status, and the ADR-0111 D6 section of the sharing-rules page is where a reader of the rule surface looks. This PR adds a refusal to that surface — an authenticated `manage_sharing` holder whose session resolves no active organization now gets 403 PERMISSION_DENIED on every verb — so both pages say so, including which two callers (system contexts, platform operators) are deliberately unaffected. Same reasoning as #8217: the runtime refusing more than the page says is the enforced-but-undocumented inverse of a declared-but-unenforced gap. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PEVB6w7D7uCszR9Mw1BL73
This was referenced Aug 12, 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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Docs-only follow-up to #8209 (
Part of #8119). One line, one file.Why
content/docs/kernel/runtime-services/sharing-service.mdxhas a "Typical Errors"section that enumerates the exact conditions producing each status. Its
SHARING_NOT_ENABLED(422) entry listed four: public sharing model, noowner_idfield, a bypass object,
controlled_by_parent.#8209 added a fifth — a federated (ADR-0015
external) object whoseowner_idis the platform's injected anchor rather than a real remote column. Leaving it out
is the enforced-but-undocumented inverse of a declared-but-unenforced gap: the
runtime now refuses more than the page says it does, and a reader hitting the new
422 would find no listed condition matching their object.
The docs-drift bot flagged this page on #8209; it was written there but could not be
pushed, because the PR had already been armed and added to the merge queue and a
queued branch cannot be updated. Dequeuing to land a doc line was not the right
trade, so it comes as its own PR — which is the better shape anyway.
The line
Re-read against current
mainbefore applying rather than assuming: #8209 landed thecode but not this line, so the surrounding list is byte-identical to what the wording
was written against and the original text still fits.
Notes
changeset gate disagrees I will add
skip-changesetrather than a changeset.content/docs/releases/is untouched — release-owned and read-only (AGENTS.mdDocumentation Guardrails); the drift bot listed
implementation-status.mdxasadvisory-only and it stays that way.
packages/cli/tsconfig.debt-remeasure.json— acheck:type-check-debtscratch fileswept in by a
git add -Athat ran while the gate was mid-measure. It never reachedmain(that commit was never pushed) and is not in this PR. Filed separately asthe underlying trap: the gate's temp files are not gitignored.
Generated by Claude Code