forked from kimhyungsik/ax_hub_mcp_tool
refactor: remove redundant manual getters and setters in ToolMetadata
This commit is contained in:
@@ -66,21 +66,7 @@ public class ToolMetadata {
|
|||||||
@Builder.Default
|
@Builder.Default
|
||||||
private Boolean requiresApproval = false;
|
private Boolean requiresApproval = false;
|
||||||
|
|
||||||
public boolean isVisible() {
|
|
||||||
return visible != null ? visible : true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public boolean isRegistered() {
|
|
||||||
return isRegistered != null ? isRegistered : true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public void setRegistered(Boolean isRegistered) {
|
|
||||||
this.isRegistered = isRegistered;
|
|
||||||
}
|
|
||||||
|
|
||||||
public boolean isRequiresApproval() {
|
|
||||||
return requiresApproval != null ? requiresApproval : false;
|
|
||||||
}
|
|
||||||
|
|
||||||
// 3. 연동 아키텍처 구분 (DIRECT / MCI_EAI)
|
// 3. 연동 아키텍처 구분 (DIRECT / MCI_EAI)
|
||||||
private String integrationType; // 연동 타입: "DIRECT" 또는 "MCI_EAI"
|
private String integrationType; // 연동 타입: "DIRECT" 또는 "MCI_EAI"
|
||||||
|
|||||||
@@ -66,21 +66,7 @@ public class ToolMetadata {
|
|||||||
@Builder.Default
|
@Builder.Default
|
||||||
private Boolean requiresApproval = false;
|
private Boolean requiresApproval = false;
|
||||||
|
|
||||||
public boolean isVisible() {
|
|
||||||
return visible != null ? visible : true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public boolean isRegistered() {
|
|
||||||
return isRegistered != null ? isRegistered : true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public void setRegistered(Boolean isRegistered) {
|
|
||||||
this.isRegistered = isRegistered;
|
|
||||||
}
|
|
||||||
|
|
||||||
public boolean isRequiresApproval() {
|
|
||||||
return requiresApproval != null ? requiresApproval : false;
|
|
||||||
}
|
|
||||||
|
|
||||||
// 3. 연동 아키텍처 구분 (DIRECT / MCI_EAI)
|
// 3. 연동 아키텍처 구분 (DIRECT / MCI_EAI)
|
||||||
private String integrationType; // 연동 타입: "DIRECT" 또는 "MCI_EAI"
|
private String integrationType; // 연동 타입: "DIRECT" 또는 "MCI_EAI"
|
||||||
|
|||||||
@@ -130,7 +130,7 @@ public class McpRouterController {
|
|||||||
|
|
||||||
if (localTools != null) {
|
if (localTools != null) {
|
||||||
for (ToolMetadata t : localTools) {
|
for (ToolMetadata t : localTools) {
|
||||||
if (!t.isRegistered() && t.isVisible() && !knownTools.contains(t.getToolName())) {
|
if (!Boolean.TRUE.equals(t.getIsRegistered()) && Boolean.TRUE.equals(t.getVisible()) && !knownTools.contains(t.getToolName())) {
|
||||||
activeTools.add(t);
|
activeTools.add(t);
|
||||||
knownTools.add(t.getToolName());
|
knownTools.add(t.getToolName());
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -48,20 +48,6 @@ public class ToolMetadata {
|
|||||||
@Builder.Default
|
@Builder.Default
|
||||||
private Boolean requiresApproval = false;
|
private Boolean requiresApproval = false;
|
||||||
|
|
||||||
public boolean isVisible() {
|
|
||||||
return visible != null ? visible : true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public boolean isRegistered() {
|
|
||||||
return isRegistered != null ? isRegistered : true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public void setRegistered(Boolean isRegistered) {
|
|
||||||
this.isRegistered = isRegistered;
|
|
||||||
}
|
|
||||||
|
|
||||||
public boolean isRequiresApproval() {
|
|
||||||
return requiresApproval != null ? requiresApproval : false;
|
|
||||||
}
|
|
||||||
|
|
||||||
}
|
}
|
||||||
@@ -100,7 +100,7 @@ public class ToolRegistryHeartbeatSender {
|
|||||||
|
|
||||||
boolean isVisible = functionAnnotation.visible();
|
boolean isVisible = functionAnnotation.visible();
|
||||||
meta.setVisible(isVisible);
|
meta.setVisible(isVisible);
|
||||||
meta.setRegistered(isRegister);
|
meta.setIsRegistered(isRegister);
|
||||||
meta.setRequiresApproval(functionAnnotation.requiresApproval());
|
meta.setRequiresApproval(functionAnnotation.requiresApproval());
|
||||||
|
|
||||||
Map<String, String> prompts = new HashMap<>();
|
Map<String, String> prompts = new HashMap<>();
|
||||||
|
|||||||
Reference in New Issue
Block a user