Skip to content

docs: explain owned Bash restrictions and host integration failures - #400

Merged
T-Auto merged 2 commits into
mainfrom
docs/host-integration-boundaries
Oct 25, 2026
Merged

T-Auto merged 2 commits into
mainfrom
docs/host-integration-boundaries

Conversation

@Haoming9527

@Haoming9527 Haoming9527 commented Oct 24, 2026 •

Copy link
Copy Markdown
Collaborator

What was wrong

Closes #380 (the requested documentation scope). Refs #346 and #367.

The documentation did not explain why an inherited-tool whitelist can leave the session's own ops_bash visible, or separate missing discovery configuration from a publication stall and a provider tool-change serialization rejection.

Change

Add paired English/Chinese troubleshooting guides and link them from usage. Document the owning shell row's publishBashTool: false switch, preserving other config; identify the host's inherited-versus-owned restriction boundary. Preset-only mounting is explicitly not presented as a verified isolation guarantee.

Explain that DSH/provider adapters own tool_addition message roles, with a bounded fresh-session/static-configuration diagnostic. Add effective-profile/discovery/presentation checks and crosslinks to #383, #390 and #395. The enabled-row stall described by #346 remains an evidence boundary; the guide does not claim that missing-row checks repair it or reproduce Desktop recovery.

Validation

  • Both language guides' local links resolve.
  • Both YAML examples parse through the repository's YAML parser and set the owning row's publishBashTool to false.
  • git diff --check passed.
  • The statements were checked against mountSessionBash, the pinned host's tool view/restriction implementation, actual host metadata/patch APIs, and the issue reports. No runtime implementation changed; no new tests or dependencies.
  • Pull-request CI is green, including Linux, macOS, Windows, Node floor, source-size, tools trial, workflow lint and the required aggregate: https://github.com/T-Auto/dsh-ops/actions/runs/40112861899.

Environment

Documentation-only change, based on main 662a388. It does not alter an installed profile or send provider requests.

Thatgfsj
Thatgfsj previously approved these changes Oct 24, 2026

@Thatgfsj Thatgfsj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

复审无误,批准合并

@T-Auto
T-Auto merged commit b2fb261 into main Oct 25, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[建议] ops_bash 无法被会话级工具白名单(ctx.tools.restrict)遮蔽,与同插件的文件工具不一致

3 participants