feat: add local backup CLI#40163
Conversation
openclaw-barnacle
bot
added
docs
Greptile SummaryThis PR adds a first-class Key concern: The The self-inclusion guard, overwrite prevention, dry-run mode, and JSON output suppression in the banner are all implemented correctly. Confidence Score: 3/5
Last reviewed commit: 20c606d |
![greptile-apps[bot]](proxy.php?url=https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](proxy.php?url=https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 92f02cb482
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@joshavant @gumadeiras this touches core CLI command wiring, local state/storage handling, and DX/docs around destructive flows. Could one of you take a review when you have a slot? What is in scope now:
What is intentionally out of scope:
Targeted coverage is in the PR body. If the shape looks right, this should be ready to merge after review. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8e661396e9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0f32d5a73d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 71955ffe5b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
gumadeiras
force-pushed
the
codex/backup-cli
branch
from
71955ff to
c301597
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a4be741e5f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
gumadeiras
force-pushed
the
codex/backup-cli
branch
from
93ff652 to
ed46625
Compare
|
Merged via squash.
Thanks @shichangs! |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ed46625ae2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Merged via squash. Prepared head SHA: ed46625 Co-authored-by: shichangs <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: ed46625 Co-authored-by: shichangs <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
上游更新摘要（abb8f6310 → bda63c3，164 commits）： ### 新功能 - ACP: 新增 resumeSessionId 支持 ACP session 恢复（openclaw#41847） - CLI: 新增 openclaw backup create/verify 本地状态归档命令（openclaw#40163） - Talk: 新增 talk.silenceTimeoutMs 配置项，可自定义静默超时（openclaw#39607） - ACP Provenance: 新增 ACP 入站溯源元数据和回执注入（openclaw#40473） - Brave 搜索: 新增 llm-context 模式，返回 AI 精炼摘要（openclaw#33383） - browser.relayBindHost: Chrome relay 可绑定非 loopback 地址（WSL2 支持）（openclaw#39364） - node-pending-work: 新增 node.pending.pull/ack RPC 接口 - Telegram: 新增 exec-approvals 处理器，支持 Telegram 内命令执行审批 - Mattermost: 新增 target-resolution，修复 markdown 保留和 DM media 上传 - MS Teams: 修复 Bot Framework General channel 对话 ID 兼容性（openclaw#41838） - secrets/runtime-web-tools: 全新 web runtime secrets 工具模块 - cron: 新增 store-migration，isolated-agent 直送核心通道，delivery failure notify - TUI: 自动检测浅色终端主题（COLORFGBG），支持 OPENCLAW_THEME 覆盖（openclaw#38636） ### 修复 - macOS: launchd 重启前重启已禁用服务，修复 openclaw update 卡死问题 - Telegram DM: 按 agent 去重入站 DM，防止同一条消息触发重复回复（openclaw#40519） - Matrix DM: 修复 m.direct homeserver 检测，保留房间绑定优先级（openclaw#19736） - 飞书: 清理插件发现缓存，修复 onboarding 安装后重复弹窗（openclaw#39642） - config/runtime snapshots: 修复 config 写入后 secret 快照丢失问题（openclaw#37313） - browser/CDP: 修复 ws:// CDP URL 反向代理和 wildcard 地址重写 - agents/failover: 识别 Bedrock tokens per day 限额为 rate limit ### 版本 - ACPX 0.1.16 - iOS/macOS 版本号更新 - Android: 精简后台权限 构建验证：待执行
Merged via squash. Prepared head SHA: ed46625 Co-authored-by: shichangs <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: ed46625 Co-authored-by: shichangs <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: ed46625 Co-authored-by: shichangs <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: ed46625 Co-authored-by: shichangs <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
2 participants
Summary
resetanduninstallcan remove local state, credentials, sessions, and workspaces, so recoverability depended on manual operator steps.openclaw backup create,openclaw backup verify, optionalopenclaw backup create --verify, docs, archive manifest validation, atomic archive writes, and backup guidance in destructive flows.backup restoreyet.Change Type (select all)
Scope (select all touched areas)
Linked Issue/PR
User-visible / Behavior Changes
openclaw backup createcommand to archive local state/config/credentials/workspace into a timestamped.tar.gz.openclaw backup verify <archive>command to validate backup structure and embeddedmanifest.json.openclaw backup create --verifyoption to self-check an archive immediately after writing it.resetanduninstallnow recommend runningopenclaw backup createbefore state-destructive flows.Security Impact (required)
Yes/No) YesYes/No) NoYes/No) NoYes/No) YesYes/No) YesYes, explain risk + mitigation:This adds a local CLI capability that reads user state/config/workspace paths and writes a backup tarball. Mitigations:
backup verifyvalidates manifest/payload consistencyRepro + Verification
Environment
Steps
openclaw backup create --output <dir>..tar.gzand embeddedmanifest.json.openclaw backup verify <archive>.openclaw reset --dry-runoropenclaw uninstall --dry-runand confirm backup guidance is shown for state-destructive scopes.Expected
Actual
Evidence
Human Verification (required)
What you personally verified (not just CI), and how:
resetanduninstallfor state-destructive flows--verifytsc, due to unrelated existing failures outside this PRReview Conversations
Compatibility / Migration
Yes/No) YesYes/No) NoYes/No) NoFailure Recovery (if this breaks)
backupcommandsRisks and Mitigations
Testing
pnpm exec vitest run src/commands/backup.atomic.test.ts src/commands/backup.test.ts src/commands/backup-verify.test.ts src/commands/reset.test.ts src/commands/uninstall.test.ts src/cli/program/register.backup.test.ts src/cli/program/command-registry.test.ts src/cli/program/preaction.test.ts src/cli/program/register.maintenance.test.tspnpm exec tsc -p tsconfig.json --noEmitwas attempted earlier, but the repo currently fails on unrelated pre-existing errors inextensions/tlonand existing agent/provider tests