This repository was archived by the owner on Mar 18, 2026. It is now read-only.
fix: merge wallet persistence PR, fix safety issues, update docs - #19
Merged
Conversation
Clean up old planning directories from previous quest system. Phase 1 of repo-cleanup quest. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Prevents data loss if cp fails during wallet/moltbook migration. Previously, cp failure (|| true) would be followed by rm, deleting the source before we knew the copy succeeded. Now checks cp exit code and only removes source if copy succeeded. If copy fails, preserves original directory and logs warning. Addresses safety concern in PR #18. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Pin @aibtc/mcp-server@1.22.2 (was @latest, previously @1.14.2) - Pin OpenClaw base image to v2026.2.2 (was :latest) - Sync vps-setup.sh inline Dockerfile with repo Dockerfile - Add entrypoint.sh creation to vps-setup.sh - Add entrypoint.sh to CI shellcheck Closes #17 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ty procedures Add comprehensive documentation covering: - Symlink persistence model explanation (entrypoint.sh → volume mount) - Pre-update backup procedures with mnemonic/seed phrase reminder - Enhanced update procedures (full rebuild vs quick restart) - Post-update health checks and verification steps - Data safety warnings and best practices This addresses the documentation gap for the new wallet persistence model introduced in PR #18, ensuring users understand how their data persists across Docker rebuilds and how to safely manage updates. Relates to issue #16 (update docs). Co-Authored-By: Claude <noreply@anthropic.com>
…ipts - Move hadolint ignore comment to line immediately before RUN instruction - Add SHELL directive, comments, and formatting to vps-setup.sh inline Dockerfile - Add SHELL directive, comments, and formatting to local-setup.sh inline Dockerfile - Ensures all three Dockerfiles have consistent structure and pass hadolint Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the container/runtime setup to persist wallet and Moltbook credentials across Docker rebuilds, improves migration safety in the entrypoint, bumps the MCP server version, expands update documentation, and cleans up stale planning artifacts.
Changes:
- Add/standardize an
entrypoint.shpersistence/migration mechanism (symlink into~/.openclawvolume) and include it in CI ShellCheck. - Pin the base image to
ghcr.io/openclaw/openclaw:v2026.2.2and bump@aibtc/mcp-serverto1.22.2in the canonical Dockerfile (and VPS setup). - Expand README “Updating” docs and remove tracked
.planning/files.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
vps-setup.sh |
Adds generation of entrypoint.sh, updates inline Dockerfile to pinned base + MCP version, and uses ENTRYPOINT to enable persistence. |
local-setup.sh |
Updates inline Dockerfile (but currently not kept in sync with the repo Dockerfile’s pinned base/version). |
entrypoint.sh |
Adds safer migration behavior (only rm -rf after successful copy) and symlink-based persistence. |
README.md |
Documents persistence model, backup/update procedure, and post-update health checks. |
Dockerfile |
Pins base image and MCP server version; installs gh CLI dependencies; wires in entrypoint. |
.github/workflows/ci.yml |
Adds entrypoint.sh to ShellCheck in CI. |
.planning/... |
Removes stale planning docs from the repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Pin mcporter to v0.7.3 instead of @latest (supply chain safety) - Add guards for existing non-directory paths before symlink creation - Fix local-setup.sh: pin base image to v2026.2.2, pin package versions - Sync local-setup.sh entrypoint with error-checked migration pattern - Fix README: clarify workspace persists via volume mount, not symlink Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
rm)@aibtc/mcp-serverfrom v1.14.2 to v1.22.2 (closes @aibtc/mcp-server pinned at v1.21.1 — needs update mechanism to latest (v1.22.2+) #17)vps-setup.shandlocal-setup.shinline Dockerfiles with repo Dockerfileentrypoint.shto CI shellcheck workflow.planning/tracked files (closes remove .planning directory #13)Test plan
entrypoint.sh,local-setup.sh,vps-setup.sh)🤖 Generated with Claude Code