From 4da4e5f16166432892049696c3545f116d1783d3 Mon Sep 17 00:00:00 2001
From: slhafzjw <slhafzjw@slhaf.work>
Date: Tue, 17 Mar 2026 13:56:25 +0800
Subject: [PATCH] refactor(runner): rename method buildDefinitions in
 BuiltinActionRegistry

---
 .../modules/action/builtin/BuiltinActionRegistry.java       | 4 ++--
 .../partner/core/action/runner/LocalRunnerClientTest.java   | 2 +-
 .../modules/action/builtin/BuiltinActionRegistryTest.java   | 6 +++---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Partner-Core/src/main/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistry.java b/Partner-Core/src/main/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistry.java
index de52d4ce..0f401be8 100644
--- a/Partner-Core/src/main/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistry.java
+++ b/Partner-Core/src/main/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistry.java
@@ -32,14 +32,14 @@ public class BuiltinActionRegistry extends AbstractAgentModule.Standalone {
     @Init
     public void init() {
         definitions.clear();
-        for (BuiltinActionDefinition definition : buildDefinitions()) {
+        for (BuiltinActionDefinition definition : buildDefaultActionDefinitions()) {
             definitions.put(definition.actionKey(), definition);
         }
         actionCapability.registerMetaActions(exportMetaActionInfos());
         actionCapability.runnerClient().setBuiltinActionRegistry(this);
     }
 
-    protected List<BuiltinActionDefinition> buildDefinitions() {
+    protected List<BuiltinActionDefinition> buildDefaultActionDefinitions() {
         return List.of();
     }
 
diff --git a/Partner-Core/src/test/java/work/slhaf/partner/core/action/runner/LocalRunnerClientTest.java b/Partner-Core/src/test/java/work/slhaf/partner/core/action/runner/LocalRunnerClientTest.java
index f2e84a81..e564f257 100644
--- a/Partner-Core/src/test/java/work/slhaf/partner/core/action/runner/LocalRunnerClientTest.java
+++ b/Partner-Core/src/test/java/work/slhaf/partner/core/action/runner/LocalRunnerClientTest.java
@@ -890,7 +890,7 @@ public class LocalRunnerClientTest {
             LocalRunnerClient client = new LocalRunnerClient(existedMetaActions, executor, tempDir.toString());
             BuiltinActionRegistry registry = new BuiltinActionRegistry() {
                 @Override
-                protected List<BuiltinActionDefinition> buildDefinitions() {
+                protected List<BuiltinActionDefinition> buildDefaultActionDefinitions() {
                     return List.of(
                             definition("echo", buildMetaActionInfo("echo"), params -> params.get("value"))
                     );
diff --git a/Partner-Core/src/test/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistryTest.java b/Partner-Core/src/test/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistryTest.java
index 05eceae5..9b47699a 100644
--- a/Partner-Core/src/test/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistryTest.java
+++ b/Partner-Core/src/test/java/work/slhaf/partner/module/modules/action/builtin/BuiltinActionRegistryTest.java
@@ -77,7 +77,7 @@ class BuiltinActionRegistryTest {
                 BuiltinActionRegistry.definition("nil", buildMetaActionInfo("nil"), params -> null)
         ));
 
-        registry.getDefinitions().putAll(indexDefinitions(registry.buildDefinitions()));
+        registry.getDefinitions().putAll(indexDefinitions(registry.buildDefaultActionDefinitions()));
 
         Assertions.assertEquals("hello", registry.call("builtin::echo", Map.of("value", "hello")));
         Assertions.assertEquals("{\"ok\":true}", registry.call("builtin::json", Map.of()));
@@ -97,7 +97,7 @@ class BuiltinActionRegistryTest {
                     throw new IllegalStateException("boom");
                 })
         ));
-        registry.getDefinitions().putAll(indexDefinitions(registry.buildDefinitions()));
+        registry.getDefinitions().putAll(indexDefinitions(registry.buildDefaultActionDefinitions()));
 
         IllegalStateException exception = Assertions.assertThrows(IllegalStateException.class,
                 () -> registry.call("builtin::boom", Map.of()));
@@ -128,7 +128,7 @@ class BuiltinActionRegistryTest {
         }
 
         @Override
-        protected List<BuiltinActionDefinition> buildDefinitions() {
+        protected List<BuiltinActionDefinition> buildDefaultActionDefinitions() {
             return definitions;
         }
     }