Skip to content

chore(scripts): route-envelope ledger note names the enveloped-refusal relay - #28013

Merged
objectstack-fleet[bot] merged 1 commit into
mainfrom
claude/project-thread-kwainq
Oct 30, 2026
Merged

objectstack-fleet[bot] merged 1 commit into
mainfrom
claude/project-thread-kwainq

Conversation

@objectstack-fleet

Copy link
Copy Markdown
Contributor

domain:server#4 · os-elon-musk · session_016WV4d1Np3hU7RQnA13YZcL

Refs #27738

维护者速读

Clause-②: no — a comment and a ledger note string change; no gate verdict, count or runtime path moves.
Metadata-protocol: none

What changes

  • The actions.ts entry's note now reads "the non-2xx relay (apiErrorResponse for an enveloped refusal, else deps.error, spread)".
  • The block comment above it says the enveloped refusal is built by apiErrorResponse, which takes no headers either, and is spread the same way.

Verification

  • Not run locally: this worktree has no node_modules, and check-route-envelope.mjs imports typescript (it exits 3, prerequisite not met, measuring nothing). CI's lint job runs check:route-envelope and ESLint on this head.
  • check-empty-changeset --base origin/main and check-commit-card-trailers --range origin/main..HEAD: exit 0. No changeset: no package changes.

🤖 Generated with Claude Code

https://claude.ai/code/session_016WV4d1Np3hU7RQnA13YZcL

…l relay

The actions.ts ledger note said the api forward's non-2xx relay spreads
deps.error. Since the enveloped refusal is built by apiErrorResponse,
the note and its comment name both builders. handBuilt stays 3.

Co-authored-by: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016WV4d1Np3hU7RQnA13YZcL
@objectstack-fleet

Copy link
Copy Markdown
Contributor Author

domain:server#4 · os-elon-musk · session_016WV4d1Np3hU7RQnA13YZcL

维护者速读

席位意见:ACCEPT。

席位核对

Head-sha: f8037ab21f728df75a8603dbb3c4f44ad5e02cc9

  • Shape: one ledger note string and one block comment in scripts/check-route-envelope.mjs; before: "the non-2xx relay (deps.error spread)"; after: "the non-2xx relay (apiErrorResponse for an enveloped refusal, else deps.error, spread)". The comment adds that apiErrorResponse takes no headers either and is spread the same way. check-governed-merges --pr 28013: NOT governed, 5 changed lines.
  • Premise measured: on main after PR fix(runtime): a forwarded api action keeps the target's whole refusal, and decision outputs pass the param check #27915, packages/runtime/src/domains/actions.ts builds an enveloped non-2xx refusal with apiErrorResponse and the rest with deps.error, both spread with the relayed headers.
  • Gate check-runs on the head: Lint · repo & tooling gates success; Check Changeset failed before skip-changeset was applied (nothing is released) and its re-run after the label is the current result; mergeable_state clean.
  • Contract review: not owed (comment and note text in a repo script; no package, spec or protocol surface).
  • Accepted deviations: none. The PR body says the gate was not run locally (no node_modules in the worktree); CI ran it.
  • Landing: under the maintainer's landing grant for this seat term (seat 4 thread, 2026-10-29 22:04Z): pr_ready + automerge_enable through the relay.

@objectstack-fleet
objectstack-fleet Bot marked this pull request as ready for review October 30, 2026 09:06
@objectstack-fleet
objectstack-fleet Bot enabled auto-merge October 30, 2026 09:06
@objectstack-fleet
objectstack-fleet Bot added this pull request to the merge queue Oct 30, 2026
Merged via the queue into main with commit fb3fe52 Oct 30, 2026
46 of 47 checks passed
@objectstack-fleet
objectstack-fleet Bot deleted the claude/project-thread-kwainq branch October 30, 2026 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs skip-changeset PR has no user-facing published change; bypasses the changeset gate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant