From 878fe4dc105ab9b2db02ecbe4d3b4353a6fd792c Mon Sep 17 00:00:00 2001
From: slhafzjw <slhafzjw@slhaf.work>
Date: Sat, 25 Apr 2026 13:17:23 +0800
Subject: [PATCH] refactor(action-planner): simplify tendencies state hint
 naming and always follow up on refused candidates

---
 .../partner/module/action/planner/ActionPlanner.java      | 8 ++++----
 .../partner/framework/agent/model/ModelRuntimeRegistry.kt | 1 +
 .../model/provider/openai/OpenAiCompatibleProvider.java   | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/Partner-Core/src/main/java/work/slhaf/partner/module/action/planner/ActionPlanner.java b/Partner-Core/src/main/java/work/slhaf/partner/module/action/planner/ActionPlanner.java
index 3c1afd2b..da83ccf8 100644
--- a/Partner-Core/src/main/java/work/slhaf/partner/module/action/planner/ActionPlanner.java
+++ b/Partner-Core/src/main/java/work/slhaf/partner/module/action/planner/ActionPlanner.java
@@ -42,7 +42,7 @@ import java.util.concurrent.atomic.AtomicBoolean;
 public class ActionPlanner extends AbstractAgentModule.Running<PartnerRunningFlowContext> {
 
     private static final String IMMEDIATE_WATCHER_CRON = "0/5 * * * * ?";
-    private static final String TENDENCIES_EVALUATING_BLOCK_NAME = "action_tendencies_under_evaluation";
+    private static final String TENDENCIES_EVALUATING_BLOCK_NAME = "tendencies_under_evaluation";
 
     private final ActionAssemblyHelper assemblyHelper = new ActionAssemblyHelper();
 
@@ -96,12 +96,12 @@ public class ActionPlanner extends AbstractAgentModule.Running<PartnerRunningFlo
         ));
 
         cognitionCapability.contextWorkspace().register(StateHintContent.createBlock(new StateHintContent(
-                "action_tendencies_under_evaluation",
+                "tendencies_under_evaluation",
                 "Partner is evaluating whether any action tendency should be taken for the latest input."
         ) {
             @Override
             public void fillStateContent(@NotNull Document document, @NotNull Element stateElement) {
-                appendListElement(document, stateElement, "action_tendencies", "tendency", tendencies);
+                appendListElement(document, stateElement, "tendencies", "tendency", tendencies);
             }
         }));
     }
@@ -161,7 +161,7 @@ public class ActionPlanner extends AbstractAgentModule.Running<PartnerRunningFlo
             }
             executeOrSchedule(executableAction);
         }
-        if (hasPassedAction && !refusedTendencies.isEmpty()) {
+        if (!refusedTendencies.isEmpty()) {
             cognitionCapability.initiateTurn("Some candidate action tendencies were refused during evaluation. If any of them were promised to the user, acknowledge that they will not be executed and explain why.", source, getModuleName());
         }
         if (hasPendingConfirmation) {
diff --git a/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/ModelRuntimeRegistry.kt b/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/ModelRuntimeRegistry.kt
index e9f4cdad..fc1cf207 100644
--- a/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/ModelRuntimeRegistry.kt
+++ b/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/ModelRuntimeRegistry.kt
@@ -112,6 +112,7 @@ object ModelRuntimeRegistry : Configurable, ConfigRegistration<ModelRuntimeRegis
         try {
             val parsedConfig = parseJsonConfig(root)
             applyConfig(parsedConfig)
+            log.info("Model config updated")
         } catch (e: Exception) {
             log.error("Error while loading runtime provider config", e)
             baseProvider.clear()
diff --git a/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/provider/openai/OpenAiCompatibleProvider.java b/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/provider/openai/OpenAiCompatibleProvider.java
index 5ae7f4eb..c3cd2c5e 100644
--- a/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/provider/openai/OpenAiCompatibleProvider.java
+++ b/Partner-Framework/src/main/java/work/slhaf/partner/framework/agent/model/provider/openai/OpenAiCompatibleProvider.java
@@ -117,7 +117,7 @@ public class OpenAiCompatibleProvider extends ModelProvider {
             return messages;
         }
 
-        String jsonInstruction = "Return only a valid JSON object that matches the requested response schema.";
+        String jsonInstruction = "Return only a valid JSON object.";
         List<Message> patched = new ArrayList<>(messages.size() + 1);
         boolean merged = false;
         for (Message message : messages) {