docs: document agent identity files and backup/restore guidance#456
docs: document agent identity files and backup/restore guidance#456phanisaimunipalli wants to merge 2 commits intoNVIDIA:mainfrom
Conversation
Add reference page for SOUL.md, USER.md, and IDENTITY.md — the workspace files OpenClaw creates inside the sandbox. Documents their purpose, exact location (/sandbox/.openclaw/workspace/), persistence behavior (survive restarts, lost on gateway destroy), and manual backup/restore steps using openshell sandbox download/upload. Closes NVIDIA#434.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughA new reference page documents OpenClaw agent identity files (SOUL.md, USER.md, IDENTITY.md), their exact workspace paths, persistence semantics, and CLI backup/restore steps; Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
docs/reference/agent-identity-files.md (3)
21-21: Align H1 withtitle.pagefrontmatter value.
# Agent Identity Filesdoes not matchtitle.page(NemoClaw Agent Identity Files — SOUL.md, USER.md, and IDENTITY.md).
Please make these identical.As per coding guidelines, "H1 heading matches the
title.pagefrontmatter value."🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/reference/agent-identity-files.md` at line 21, Update the document's top-level H1 so it exactly matches the title.page frontmatter value; replace the current `# Agent Identity Files` H1 with `# NemoClaw Agent Identity Files — SOUL.md, USER.md, and IDENTITY.md` (ensure punctuation and casing match the frontmatter), and verify the `title.page` frontmatter remains unchanged so the H1 and frontmatter are identical.
27-33: Add an introductory sentence immediately under each H2 before table/list content.These sections start directly with table/list content.
Add a one-line intro sentence after each heading.As per coding guidelines, "Sections use H2 and H3, each starting with an introductory sentence."
Also applies to: 59-66, 120-124
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/reference/agent-identity-files.md` around lines 27 - 33, Add a one-line introductory sentence immediately under each H2 heading before any table or list content (for example the "The Identity Files" H2 that currently jumps straight into the table listing SOUL.md, USER.md, IDENTITY.md), e.g. a brief sentence describing the section purpose; apply the same fix to the other H2 sections noted in the review (the H2s at the other referenced ranges) so every H2 is followed by a single-line intro before any tables or lists.
24-24: Use active voice for persistence behavior sentence.This line uses passive voice ("are lost when the gateway is destroyed").
Rewrite in active voice.As per coding guidelines, "Active voice required. Flag passive constructions."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/reference/agent-identity-files.md` at line 24, Change the passive clause in the persistence behavior sentence to active voice: replace "These files persist across sandbox restarts but are lost when the gateway is destroyed." with an active construction such as "These files persist across sandbox restarts, but the gateway destroys them." Edit the sentence labeled "persistence behavior" (the line starting "These files persist across sandbox restarts...") accordingly to use the active verb "destroys."
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/reference/agent-identity-files.md`:
- Around line 42-44: The fenced code blocks in
docs/reference/agent-identity-files.md lack language identifiers; update each
triple-backtick fence shown (the single-line block containing
"/sandbox/.openclaw/workspace/" and the multi-line block listing
SOUL.md/USER.md/IDENTITY.md, plus the similar block at the later 48-52 section)
to include an explicit language tag such as text (e.g., change ``` to ```text)
so the markdown linting and formatting pass; ensure both opening fences are
updated while leaving the fenced content unchanged.
---
Nitpick comments:
In `@docs/reference/agent-identity-files.md`:
- Line 21: Update the document's top-level H1 so it exactly matches the
title.page frontmatter value; replace the current `# Agent Identity Files` H1
with `# NemoClaw Agent Identity Files — SOUL.md, USER.md, and IDENTITY.md`
(ensure punctuation and casing match the frontmatter), and verify the
`title.page` frontmatter remains unchanged so the H1 and frontmatter are
identical.
- Around line 27-33: Add a one-line introductory sentence immediately under each
H2 heading before any table or list content (for example the "The Identity
Files" H2 that currently jumps straight into the table listing SOUL.md, USER.md,
IDENTITY.md), e.g. a brief sentence describing the section purpose; apply the
same fix to the other H2 sections noted in the review (the H2s at the other
referenced ranges) so every H2 is followed by a single-line intro before any
tables or lists.
- Line 24: Change the passive clause in the persistence behavior sentence to
active voice: replace "These files persist across sandbox restarts but are lost
when the gateway is destroyed." with an active construction such as "These files
persist across sandbox restarts, but the gateway destroys them." Edit the
sentence labeled "persistence behavior" (the line starting "These files persist
across sandbox restarts...") accordingly to use the active verb "destroys."
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 73554c83-73b2-4536-bbe9-5d44415c6072
📒 Files selected for processing (2)
docs/index.mddocs/reference/agent-identity-files.md
- Match H1 exactly to title.page frontmatter value - Rewrite passive voice to active (gateway destroys them) - Add introductory sentence under each H2 before tables - Add text language tag to bare fenced code blocks
|
Thanks for taking the time to document the workspace files OpenClaw creates inside the sandbox, this will definitely help with understanding persistence behavior and backup/restore procedures. |
| SPDX-License-Identifier: Apache-2.0 | ||
| --> | ||
|
|
||
| # NemoClaw Agent Identity Files — SOUL.md, USER.md, and IDENTITY.md |
There was a problem hiding this comment.
| # NemoClaw Agent Identity Files — SOUL.md, USER.md, and IDENTITY.md | |
| # Agent Identity Files |
| | `SOUL.md` | The agent's core personality, tone, and behavioral rules. | | ||
| | `USER.md` | Preferences, context, and facts the agent learns about you over time. | | ||
| | `IDENTITY.md` | The agent's name, role description, and self-presentation. | |
There was a problem hiding this comment.
Would you add links to the corresponding identity file doc page in OpenClaw, such as https://docs.openclaw.ai/reference/templates/IDENTITY for IDENTITY.md?
| These files persist across sandbox restarts, but the gateway destroys them when it is torn down. | ||
| This page describes the files, where they are stored, and how to back them up. | ||
|
|
||
| ## The Identity Files |
There was a problem hiding this comment.
| ## The Identity Files | |
| ## Identity Files Created by OpenClaw |
Add concept page explaining all six workspace files (SOUL.md, USER.md, IDENTITY.md, AGENTS.md, MEMORY.md, memory/) with links to upstream OpenClaw template docs. Add how-to page for manual and scripted backup/restore using openshell sandbox download/upload. Add data-loss warning to the destroy command reference. Add Workspace section to docs navigation. Closes #434 Supersedes #456 and #476
* docs: document workspace file persistence and backup/restore guidance Add concept page explaining all six workspace files (SOUL.md, USER.md, IDENTITY.md, AGENTS.md, MEMORY.md, memory/) with links to upstream OpenClaw template docs. Add how-to page for manual and scripted backup/restore using openshell sandbox download/upload. Add data-loss warning to the destroy command reference. Add Workspace section to docs navigation. Closes #434 Supersedes #456 and #476 * fix: address CodeRabbit review on PR #663 - Rename "Related Topics" to "Next Steps" in both doc pages - Fail hard on chmod instead of suppressing with 2>/dev/null * fix: minor edit * fix: nit --------- Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>
) * docs: document workspace file persistence and backup/restore guidance Add concept page explaining all six workspace files (SOUL.md, USER.md, IDENTITY.md, AGENTS.md, MEMORY.md, memory/) with links to upstream OpenClaw template docs. Add how-to page for manual and scripted backup/restore using openshell sandbox download/upload. Add data-loss warning to the destroy command reference. Add Workspace section to docs navigation. Closes NVIDIA#434 Supersedes NVIDIA#456 and NVIDIA#476 * fix: address CodeRabbit review on PR NVIDIA#663 - Rename "Related Topics" to "Next Steps" in both doc pages - Fail hard on chmod instead of suppressing with 2>/dev/null * fix: minor edit * fix: nit --------- Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>
Summary
Adds a reference page for the three workspace files OpenClaw creates inside the sandbox:
SOUL.md,USER.md, andIDENTITY.md./sandbox/.openclaw/workspace/)openshell sandbox download/uploadCloses #434.
Pages added
docs/reference/agent-identity-files.md(new)docs/index.md(nav entry added)Summary by CodeRabbit