romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(render): simplify permission handling with set +e/-e toggle

Browse Source 
- Use for loop to try directories in order
- Temporarily disable set -e for permission tests
- Write config file directly to test writability
- More robust error handling that works with set -e
This commit is contained in:
Ojus Save 2026-01-26 14:05:40 -08:00
parent 041e279427
commit c23dd3e3e3
1 changed files with 37 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

77
scripts/render-start.sh
View File

@ -1,6 +1,5 @@
#!/bin/sh #!/bin/sh
# Render startup script - creates config and starts gateway # Render startup script - creates config and starts gateway
# Note: We use set -e but handle permission errors gracefully
set -e set -e
echo "=== Render startup script ===" echo "=== Render startup script ==="
@ -13,44 +12,11 @@ if [ -z "${HOME}" ]; then
echo "Warning: HOME not set, using ${HOME}" echo "Warning: HOME not set, using ${HOME}"
fi fi
# Determine config directory with fallback chain # Determine config directory - try in order until one works
# 1. Try CLAWDBOT_STATE_DIR if set # 1. CLAWDBOT_STATE_DIR if set
# 2. Try /data/.clawdbot (Render persistent disk) # 2. /data/.clawdbot (Render persistent disk)
# 3. Fall back to $HOME/.clawdbot (always writable by node user) # 3. $HOME/.clawdbot (always writable by node user)
CONFIG_DIR=""
if [ -n "${CLAWDBOT_STATE_DIR}" ]; then
if mkdir -p "${CLAWDBOT_STATE_DIR}" 2>/dev/null && touch "${CLAWDBOT_STATE_DIR}/.test" 2>/dev/null && rm -f "${CLAWDBOT_STATE_DIR}/.test" 2>/dev/null; then
CONFIG_DIR="${CLAWDBOT_STATE_DIR}"
echo "Using CLAWDBOT_STATE_DIR: ${CONFIG_DIR}"
else
echo "Warning: ${CLAWDBOT_STATE_DIR} is not writable"
fi
fi
if [ -z "${CONFIG_DIR}" ]; then
if mkdir -p "/data/.clawdbot" 2>/dev/null && touch "/data/.clawdbot/.test" 2>/dev/null && rm -f "/data/.clawdbot/.test" 2>/dev/null; then
CONFIG_DIR="/data/.clawdbot"
echo "Using /data/.clawdbot: ${CONFIG_DIR}"
else
echo "Warning: /data/.clawdbot is not writable"
fi
fi
if [ -z "${CONFIG_DIR}" ]; then
# Final fallback: use HOME (always writable by node user)
CONFIG_DIR="${HOME}/.clawdbot"
echo "Using fallback: ${CONFIG_DIR}"
fi
CONFIG_FILE="${CONFIG_DIR}/clawdbot.json"
echo "Config dir: ${CONFIG_DIR}"
echo "Config file: ${CONFIG_FILE}"
# Create config directory (should succeed now)
mkdir -p "${CONFIG_DIR}"
# Config content
CONFIG_CONTENT='{ CONFIG_CONTENT='{
"gateway": { "gateway": {
"mode": "local", "mode": "local",
@ -61,8 +27,39 @@ CONFIG_CONTENT='{
} }
}' }'
# Write config file CONFIG_DIR=""
echo "${CONFIG_CONTENT}" > "${CONFIG_FILE}" CONFIG_FILE=""
# Try each directory in order
for dir in "${CLAWDBOT_STATE_DIR:-}" "/data/.clawdbot" "${HOME}/.clawdbot"; do
if [ -z "${dir}" ]; then
continue
fi
# Temporarily disable set -e for this test
set +e
mkdir -p "${dir}" 2>/dev/null
if echo "${CONFIG_CONTENT}" > "${dir}/clawdbot.json" 2>/dev/null; then
set -e
CONFIG_DIR="${dir}"
CONFIG_FILE="${dir}/clawdbot.json"
echo "Successfully wrote config to: ${CONFIG_FILE}"
break
fi
set -e
done
# Ensure we have a config directory (should always succeed with HOME fallback)
if [ -z "${CONFIG_DIR}" ]; then
CONFIG_DIR="${HOME}/.clawdbot"
CONFIG_FILE="${CONFIG_DIR}/clawdbot.json"
mkdir -p "${CONFIG_DIR}"
echo "${CONFIG_CONTENT}" > "${CONFIG_FILE}"
echo "Using fallback: ${CONFIG_FILE}"
fi
echo "Config dir: ${CONFIG_DIR}"
echo "Config file: ${CONFIG_FILE}"
echo "=== Config written ===" echo "=== Config written ==="
echo "=== ${CONFIG_FILE}: ===" echo "=== ${CONFIG_FILE}: ==="