refactor(runner): rename method buildDefinitions in BuiltinActionRegistry

This commit is contained in:
2026-03-17 13:56:25 +08:00
parent 5a717dbdda
commit 4da4e5f161
3 changed files with 6 additions and 6 deletions

View File

@@ -32,14 +32,14 @@ public class BuiltinActionRegistry extends AbstractAgentModule.Standalone {
@Init @Init
public void init() { public void init() {
definitions.clear(); definitions.clear();
for (BuiltinActionDefinition definition : buildDefinitions()) { for (BuiltinActionDefinition definition : buildDefaultActionDefinitions()) {
definitions.put(definition.actionKey(), definition); definitions.put(definition.actionKey(), definition);
} }
actionCapability.registerMetaActions(exportMetaActionInfos()); actionCapability.registerMetaActions(exportMetaActionInfos());
actionCapability.runnerClient().setBuiltinActionRegistry(this); actionCapability.runnerClient().setBuiltinActionRegistry(this);
} }
protected List<BuiltinActionDefinition> buildDefinitions() { protected List<BuiltinActionDefinition> buildDefaultActionDefinitions() {
return List.of(); return List.of();
} }

View File

@@ -890,7 +890,7 @@ public class LocalRunnerClientTest {
LocalRunnerClient client = new LocalRunnerClient(existedMetaActions, executor, tempDir.toString()); LocalRunnerClient client = new LocalRunnerClient(existedMetaActions, executor, tempDir.toString());
BuiltinActionRegistry registry = new BuiltinActionRegistry() { BuiltinActionRegistry registry = new BuiltinActionRegistry() {
@Override @Override
protected List<BuiltinActionDefinition> buildDefinitions() { protected List<BuiltinActionDefinition> buildDefaultActionDefinitions() {
return List.of( return List.of(
definition("echo", buildMetaActionInfo("echo"), params -> params.get("value")) definition("echo", buildMetaActionInfo("echo"), params -> params.get("value"))
); );

View File

@@ -77,7 +77,7 @@ class BuiltinActionRegistryTest {
BuiltinActionRegistry.definition("nil", buildMetaActionInfo("nil"), params -> null) 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("hello", registry.call("builtin::echo", Map.of("value", "hello")));
Assertions.assertEquals("{\"ok\":true}", registry.call("builtin::json", Map.of())); Assertions.assertEquals("{\"ok\":true}", registry.call("builtin::json", Map.of()));
@@ -97,7 +97,7 @@ class BuiltinActionRegistryTest {
throw new IllegalStateException("boom"); throw new IllegalStateException("boom");
}) })
)); ));
registry.getDefinitions().putAll(indexDefinitions(registry.buildDefinitions())); registry.getDefinitions().putAll(indexDefinitions(registry.buildDefaultActionDefinitions()));
IllegalStateException exception = Assertions.assertThrows(IllegalStateException.class, IllegalStateException exception = Assertions.assertThrows(IllegalStateException.class,
() -> registry.call("builtin::boom", Map.of())); () -> registry.call("builtin::boom", Map.of()));
@@ -128,7 +128,7 @@ class BuiltinActionRegistryTest {
} }
@Override @Override
protected List<BuiltinActionDefinition> buildDefinitions() { protected List<BuiltinActionDefinition> buildDefaultActionDefinitions() {
return definitions; return definitions;
} }
} }