Skip to content

fix: preserve existing OpenClaw context engine #756

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
superbirdnine wants to merge 1 commit into
base: main
Choose a base branch
from
+36 −4
Open

fix: preserve existing OpenClaw context engine #756

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 36 additions & 4 deletions scripts/openclaw-setup.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -302,20 +302,46 @@ ensure_plugin_active() {
fi
}

get_context_engine_slot() {
CONFIG_PATH="$CONFIG_PATH" node -e '
const fs = require("fs");
const configPath = process.env.CONFIG_PATH;
try {
const config = JSON.parse(fs.readFileSync(configPath, "utf8"));
const slot = config.plugins?.slots?.contextEngine;
if (typeof slot === "string") process.stdout.write(slot);
} catch (_) {}
' 2>/dev/null || true
}

unset_byterover_context_engine_slot() {
local current_context_engine
current_context_engine=$(get_context_engine_slot)
if [ "$current_context_engine" = "byterover" ]; then
openclaw config unset plugins.slots.contextEngine 2>/dev/null || true
elif [ -n "$current_context_engine" ]; then
info "Preserving existing contextEngine slot: $current_context_engine"
fi
}

remove_existing_byterover_plugin() {
# Remove CLI-installed plugin
openclaw plugins uninstall byterover --force 2>/dev/null || true
# Remove old local plugin files (legacy manual install)
rm -rf "$HOME/.openclaw/extensions/byterover"
# Clean up config entries
openclaw config unset plugins.slots.contextEngine 2>/dev/null || true
# Clean up ByteRover-owned config without disturbing other context engines.
unset_byterover_context_engine_slot
CONFIG_PATH="$CONFIG_PATH" node -e '
const fs = require("fs");
const configPath = process.env.CONFIG_PATH;
try {
const config = JSON.parse(fs.readFileSync(configPath, "utf8"));
const entries = config.plugins?.entries;
if (entries && entries["byterover"]) delete entries["byterover"];
if (config.plugins?.slots?.contextEngine === "byterover") {
delete config.plugins.slots.contextEngine;
}
if (config.plugins?.slots && Object.keys(config.plugins.slots).length === 0) delete config.plugins.slots;
if (Array.isArray(config.plugins?.load?.paths)) {
config.plugins.load.paths = config.plugins.load.paths.filter(p => p !== "~/.openclaw/extensions/byterover");
if (config.plugins.load.paths.length === 0) delete config.plugins.load.paths;
Expand Down Expand Up @@ -368,9 +394,15 @@ configure_context_plugin() {
return
fi

local current_context_engine
current_context_engine=$(get_context_engine_slot)
if [ -n "$current_context_engine" ] && [ "$current_context_engine" != "byterover" ]; then
warn "Context engine slot is currently set to '$current_context_engine'. Installing ByteRover will replace it."
fi

if ! confirm "Install ByteRover Context Plugin?"; then
echo "Uninstalling ByteRover Context Plugin..."
remove_existing_byterover_plugin
echo "Skipping ByteRover Context Plugin setup; removing ByteRover from contextEngine if currently assigned."
unset_byterover_context_engine_slot
echo ""
return
fi
Expand Down