Skip to content

test(cli): keep the two-namespace control's tools distinct so the bare-name rule does not judge it - #31742

Merged
objectstack-fleet[bot] merged 1 commit into
mainfrom
claude/project-thread-mjoky7
Nov 9, 2026
Merged

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

Conversation

@objectstack-fleet

Copy link
Copy Markdown
Contributor

Requested by Steve · project thread

Prerequisite for #31661 (it was ejected from the merge queue by this test, queue build 42539489585, Test Core 1/8, triage 6729558102). Test-only; no card of its own, so skip-changeset.

维护者速读

What changes

  • packagesStack(betaNs) gives the beta package's tool its own name when betaNs is not nsn (nsn_other_lookup), so the two-namespace CONTROL shares only the page nsn_home. The shared-namespace fixtures still hold page/nsn_home and tool/nsn_lookup twice, so every refusal assertion is unchanged. The CONTROL's title now says what it holds ("the same page name in two namespaces validates"). A doc comment says why: a tool is also phase 1's bare-name one-holder type, judged environment-wide, so a shared tool across two namespaces is that rule's refusal, not this rule's CONTROL. os validate does not run the bare-name rule, so the CLI CONTROL reads the same either way; only the package-door premise is affected.

Verification

Every run under os-verify-lock, CLI closure rebuilt each time:


Generated by Claude Code

🤖 Generated with Claude Code

https://claude.ai/code/session_01ADHkmQv69sQL5nfT8xuPWs

…e-name rule does not judge it

Claude-Session: https://claude.ai/code/session_01ADHkmQv69sQL5nfT8xuPWs
Co-authored-by: Claude <noreply@anthropic.com>
@objectstack-fleet objectstack-fleet Bot added the skip-changeset PR has no user-facing published change; bypasses the changeset gate label Nov 9, 2026
@github-actions github-actions Bot added the size/s label Nov 9, 2026
@objectstack-fleet

Copy link
Copy Markdown
Contributor Author

domain:server#1 · os-steve · session_01ADHkmPF1skfu2kx89BEXZi

席位 ACCEPT · head 86e164c2d1a5f50b6c5f5a0ceffe24b105ec1255

测试专改(1 个文件,+18/-5,skip-changeset),是 #31661 的前置。

CI 全绿后落地:relay pr_ready + automerge_enable。合入后,席位经 relay 给 #31661 重新开 auto-merge 入队一次。

@github-actions github-actions Bot added the tests label Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

📓 Docs Drift Check

Nothing in this diff resolved to a documentable surface (no symbol, route or SDK anchor derived from 0 changed package(s)), so this run has no opinion about the docs.

What this run could not see

Coarse fallback — 0 page(s) merely mention a changed package (the pre-#9192 predicate, kept for the deliberately-wide backstop): node scripts/docs-audit/affected-docs.mjs --json f2abd4767b32200bcbd866cd34e0b2863f06d148 → packageMentionDocs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant