Skip to content

docs(agentos): sync from rivet-dev/agentos - #39

Open
NathanFlurry wants to merge 1 commit into
mainfrom
docs-sync/agentos
Open

NathanFlurry wants to merge 1 commit into
mainfrom
docs-sync/agentos

Conversation

@NathanFlurry

Copy link
Copy Markdown
Member

Automated docs sync from rivet-dev/agentos@4f0dab7.

Do not edit vendor/agentos/ here. Edit the docs in rivet-dev/agentos and this PR updates itself.

@railway-app

railway-app Bot commented Aug 31, 2026

Copy link
Copy Markdown

🚅 Deployed to the website-pr-39 environment in rivet-website

Service Status Web Updated
website 😴 Sleeping (View Logs) Web Sep 17, 2026 at 2:57 am UTC

@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-39 August 31, 2026 14:21 Destroyed
@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-39 August 31, 2026 14:42 Destroyed
@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-39 August 31, 2026 15:37 Destroyed
@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-39 September 17, 2026 02:07 Destroyed

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 1 medium-severity finding

Reviewed commit 441bb3b.

Comment on lines 39 to +42
const runCommandResult = await vm.process.exec(
"agentos-sandbox run-command --command echo --args 'hello from Docker sandbox'",
);
console.log("Sandbox command:", runCommandResult.stdout.trim());
console.log("Sandbox command:", (runCommandResult.stdout ?? "").trim());

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Sandbox quickstart discards the command output

process.exec() now defaults to output.capture = "none" (the sidecar maps an omitted capture mode to ExecutionOutputCapture::None). Null-coalescing stdout therefore makes this supported quickstart print an empty Sandbox command, and the unchanged call below likewise prints an empty process list. Pass { output: { capture: "all" } } to both exec() calls, as the other updated quickstarts in this diff do.

@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-39 September 17, 2026 02:43 Destroyed
@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-39 September 17, 2026 02:44 Destroyed

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 2 medium-severity findings

Reviewed commit 43bbd7e.

Comment on lines 39 to +42
const runCommandResult = await vm.process.exec(
"agentos-sandbox run-command --command echo --args 'hello from Docker sandbox'",
);
console.log("Sandbox command:", runCommandResult.stdout.trim());
console.log("Sandbox command:", (runCommandResult.stdout ?? "").trim());

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · Sandbox quickstart discards the command output

process.exec() defaults to output.capture = "none"; the sidecar maps an omitted capture mode to ExecutionOutputCapture::None. Null-coalescing stdout therefore makes this quickstart print an empty Sandbox command, and the unchanged call below likewise prints an empty process list. Pass { output: { capture: "all" } } to both exec() calls, as the other updated quickstarts in this diff do.

Comment on lines +33 to +35

To use only your own system prompt, set `ACP_SYSTEM_PROMPT_MODE=replace` on the session's `env`. The built-in Claude Code prompt is dropped, and Claude receives the assembled agentOS prompt instead. Use `skipOsInstructions: true` with `additionalInstructions` to send only your text.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 Medium · System-prompt example bypasses snippet type checking

The agentOS docs convention explicitly forbids inline fenced TypeScript because only examples under examples/ embedded through <CodeSnippet> are compiled during the website build. This new API example can therefore drift without CI detecting it. Move it to a source example (using a named docs region if needed) and embed that region here with <CodeSnippet>.

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.

1 participant