Skip to content

docs: add Shell Command Sandbox documentation (RBAC)#84

Open
ashum9 wants to merge 4 commits into
mofa-org:mainfrom
ashum9:issue-67-shell-sandbox
Open

docs: add Shell Command Sandbox documentation (RBAC)#84
ashum9 wants to merge 4 commits into
mofa-org:mainfrom
ashum9:issue-67-shell-sandbox

Conversation

@ashum9
Copy link
Copy Markdown
Contributor

@ashum9 ashum9 commented Mar 14, 2026

Description

This PR adds the missing documentation for the Shell Command Sandbox feature, which is a critical part of the new RBAC system.

The sandbox logic was successfully built into the engine to prevent dangerous commands (like rm -rf / or sudo), but the user-facing tutorials were never updated to reflect how to configure it. This PR fills that gap by adding detailed configuration instructions to TUTORIAL.md and TUTORIAL_CN.md.

Changes Made

  • Explained how the safe_commands list operates as a strict whitelist for shell execution.
  • Provided accurate JSON schema configuration examples under permissions.tools.shell.safe_commands.allowed.
  • Documented the fallback behavior (legacy is_dangerous_command checks) when RBAC is disabled.

Resolves: Closes #67

Copilot AI review requested due to automatic review settings March 14, 2026 17:13
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Shell Command Sandbox (RBAC) documentation to both the English and Chinese tutorial files, describing roles, command whitelisting, and configuration.

Changes:

  • Added a new "Security & Access Control (RBAC)" section documenting roles (Guest/Member/Admin/SuperAdmin) and the shell command sandbox whitelist configuration.
  • Mirrored the same documentation in the Chinese tutorial (TUTORIAL_CN.md).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
TUTORIAL.md Added RBAC and Shell Command Sandbox documentation section
TUTORIAL_CN.md Added equivalent Chinese-language RBAC documentation section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread TUTORIAL_CN.md
Comment thread TUTORIAL.md
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.

feat: Shell Command Sandbox - Whitelist-Based Execution

2 participants