fix: show restart hint after any config-modifying steps, not just config_patch
create_agent, bind_channel, setup_identity all modify openclaw.json and need a gateway restart. Previously the hint only showed when config_patch steps ran. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -27,7 +27,7 @@ export function Cook({
|
|||||||
const [resolvedStepList, setResolvedStepList] = useState<ResolvedStep[]>([]);
|
const [resolvedStepList, setResolvedStepList] = useState<ResolvedStep[]>([]);
|
||||||
const [stepStatuses, setStepStatuses] = useState<StepStatus[]>([]);
|
const [stepStatuses, setStepStatuses] = useState<StepStatus[]>([]);
|
||||||
const [stepErrors, setStepErrors] = useState<Record<number, string>>({});
|
const [stepErrors, setStepErrors] = useState<Record<number, string>>({});
|
||||||
const [hasConfigPatch, setHasConfigPatch] = useState(false);
|
const [needsRestart, setNeedsRestart] = useState(false);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
api.listRecipes(recipeSource).then((recipes) => {
|
api.listRecipes(recipeSource).then((recipes) => {
|
||||||
@@ -51,7 +51,7 @@ export function Cook({
|
|||||||
// Auto-skip steps whose template args resolved to empty
|
// Auto-skip steps whose template args resolved to empty
|
||||||
setStepStatuses(steps.map((s) => (s.skippable ? "skipped" : "pending")));
|
setStepStatuses(steps.map((s) => (s.skippable ? "skipped" : "pending")));
|
||||||
setStepErrors({});
|
setStepErrors({});
|
||||||
setHasConfigPatch(steps.some((s) => !s.skippable && s.action === "config_patch"));
|
setNeedsRestart(steps.some((s) => !s.skippable));
|
||||||
setPhase("confirm");
|
setPhase("confirm");
|
||||||
};
|
};
|
||||||
|
|
||||||
@@ -203,7 +203,7 @@ export function Cook({
|
|||||||
{doneCount} step{doneCount !== 1 ? "s" : ""} completed
|
{doneCount} step{doneCount !== 1 ? "s" : ""} completed
|
||||||
{skippedCount > 0 && `, ${skippedCount} skipped`}
|
{skippedCount > 0 && `, ${skippedCount} skipped`}
|
||||||
</p>
|
</p>
|
||||||
{hasConfigPatch && (
|
{needsRestart && (
|
||||||
<p className="text-sm text-muted-foreground mt-1">
|
<p className="text-sm text-muted-foreground mt-1">
|
||||||
Use "Apply Changes" in the sidebar to restart the gateway and activate config changes.
|
Use "Apply Changes" in the sidebar to restart the gateway and activate config changes.
|
||||||
</p>
|
</p>
|
||||||
|
|||||||
Reference in New Issue
Block a user