Repository navigation
docs(skills): rest-channel records the advisory_report 403 re-test as transient, with the seat retry rule - #30130
Closed
objectstack-fleet[bot] wants to merge 1 commit into
Closed
objectstack-fleet[bot] wants to merge 1 commit into
objectstack-fleet[bot] wants to merge 1 commit into
Conversation
…est as transient and the seat's retry rule The live re-test on #29402 sent a fresh report through the relay with no 403, so the earlier 403 run does not reproduce. The seat reads a later 403 against the two causes security-findings.md already names, retries once on a later day, and never switches account. The advisory_read line folds into the advisory row (its key and --reveal detail already live in security-findings.md and lanes/security.md), so the file stays at its 82-line ceiling. Fixes #29402 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J8VVAnSGyRU8EkF8acg337
9 tasks done
Contributor
Author
|
domain:skills#1 · huangyiirene · session_01J8VVAnMUWA9rufXzFaEh4J Superseded by #30132 · closed unmerged · 2026-11-04T15:22Z This branch's only commit carried a card keyword and a co-author trailer that Generated by Claude Code |
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
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.
Requested by Irene · project thread
Fixes #29402
Clause-②: no
Metadata-protocol: none
The maintainer waived the model-tier mandate while that tier had no quota; this PR was developed below it.
维护者速读
rest-channel.md写侧公告两行重写:L50 并入advisory_read,L51 记advisory_report403 复测读数与席位规则。.claude/**文字,行数不变(82,上限 82),不进任何包,skip-changeset;回滚即 revert。What changes
.claude/skills/pm-dispatch/references/rest-channel.md, write side, two lines rewritten, none added:advisory_readbeside the four advisory write operations and keeps the pointer tosecurity-findings.md. The old L51's detail (one-time key in the seat process, ciphertext back,--revealprints plaintext) already lives insecurity-findings.mdL22 andlanes/security.mdL19, so folding it in loses no rule. "发布归维护者" is likewise held bysecurity-findings.mdL24.advisory_report403 was re-tested and is transient; the seat reads a 403 against the two causessecurity-findings.mdL23 already names (the advisory write grant and the per-account daily limit on private reports), retries once on a later day, and never switches account.security-findings.mdis not edited: the reading does not make L23 wrong, it only says the 11-02 run did not hit either cause as a standing state.Ratchets. Line count 82 → 82 (ceiling 82, the added rule line is paid by folding L51 into L50). Widths: L50 110 bytes, L51 118 bytes, under the 120-byte cap.
Why
The card's last bullet. The 11-02 runs answered HTTP 403 on every
advisory_reportafter the first. On 2026-11-04 a fresh report through the relay succeeded with no 403, and the maintainer confirmed the lane App shows as reporter on it and on the reports of the last two days (card comment 6632996128). The seat needs a written rule for the next 403 so it neither resends into a refusal nor reaches for another account.Verification
Head
93196356, one commit ona52761fd.Local gates on 9319635: 17 derived, 17 run with exit 0, 0 NOT MEASURED (`dispatch-gates --ran`)
node scripts/pm/dispatch-gates.mjs --commandsderived 17 commands for the one changed file. All ran afterpnpm install --frozen-lockfile, each with its exit code captured before any pipe.check:doc-formula-expressionsfirst exited 3 (prerequisite not built); afterturbo run build --filter=@objectstack/formula --filter=@objectstack/lintit exited 0.check:pm-skill-ratchet,check:pm-skill-id-lint,check:skill-frame-sync,check:doc-authoringandcheck-comment-mask-corpus.mjs.--ranprinted:17 derived famil(ies) accounted for — 17 run, 0 NOT-MEASURED (a DERIVED zero — all 17 recorded an exit code and none of them is 3).node scripts/check-empty-changeset.mjs --base origin/main: exit 0. No changeset:.claude/**ships in no package, soskip-changeset.node scripts/check-commit-card-trailers.mjs --range origin/main..HEAD: exit 1, found after the push. The commit message names this card and its co-author trailer carries a model name. The branch is notclaude/issue-*, so AGENTS.md §3 rules out a force-push to reword it. The squash message is taken from this body, so neither line lands onmain.Human Minimum
packages/specholds no runtime logic. (Not touched.)main, one pull request.packages/spectouched: its generated artifacts are regenerated and committed. (Not touched.)content/docs/references/,packages/*/CHANGELOG.mdorcontent/docs/releases/.🤖 Generated with Claude Code
https://claude.ai/code/session_01J8VVAnSGyRU8EkF8acg337