mirror of
https://gitee.com/mateos/mateclaw.git
synced 2026-09-13 03:13:41 +08:00
5e6764ecf1
When an agent had any skill bound, the runtime tool gate was silently
hiding @Tool beans that aren't declared in any skill manifest, even
though the global system prompts (SOUL.md / "Web Search Capability" /
"File Reading Guidelines") explicitly tell the LLM these tools are
available. Result: the model would call search / renderDocx / read_file
/ etc., hit "Tool not found", then either give up or fall back to
unhelpful behaviour (e.g. dumping markdown text instead of producing a
.docx download).
This commit:
- Adds universally-promised, agent-wide tools to SYSTEM_LEVEL_TOOLS so
they bypass the manifest restriction: document/media generation
(renderDocx*, image_generate, music_generate, video_generate),
global capability tools the system prompt mentions (search,
browser_use, read_file / write_file / edit_file /
execute_shell_command, detect_file_type, extract_*_text,
readMateClawDoc), skill discovery siblings (listSkillFiles,
listAvailableSkills), and the delegate triplet (delegateToAgent,
delegateParallel, listAvailableAgents).
- Fixes 5 entries in the prior whitelist whose names did not match
any real @Tool bean and were therefore silently dead:
read_workspace_file -> read_workspace_memory_file
write_workspace_file -> write_workspace_memory_file
list_workspace_files -> list_workspace_memory_files
delegate_agent -> delegateToAgent
datetime -> getCurrentDate / getCurrentDateTime / getCurrentTime
Also adds the missing edit_workspace_memory_file.
- In the chat markdown renderer, strips any hallucinated
https?://<host> prefix from /api/v1/files/generated/<id> download
links before building the <a href>. Multiple LLMs have been
observed prepending bogus hosts when echoing tool-returned download
URLs back to the user, breaking the click. One-line defensive
normalization independent of which model is in use.
Verified end-to-end on a previously-broken agent: search / browser_use
/ execute_shell_command / renderDocx all dispatch correctly now and
the final markdown link is a clean same-origin path. 36 whitelist
entries cross-checked against real @Tool method names.
AgentBindingServiceTest green.
28 lines
852 B
Java
28 lines
852 B
Java
package vip.mate.acp.event;
|
|
|
|
/**
|
|
* Lifecycle event for ACP endpoint rows.
|
|
*
|
|
* <p>Published by {@code AcpEndpointService} whenever a row is created,
|
|
* updated, toggled, or deleted. Listened to by
|
|
* {@code AcpSkillBridge} so it can re-sync the auto-bridged virtual
|
|
* skill cards and their wrapper tool registrations without a full
|
|
* application restart.
|
|
*
|
|
* <p>Mirrors the {@code SkillWorkspaceEvent} pattern — a small immutable
|
|
* record carrying just enough context for listeners to fan out.
|
|
*/
|
|
public record AcpEndpointChangedEvent(Long endpointId, String name, Type type) {
|
|
|
|
public enum Type {
|
|
/** Row inserted. */
|
|
CREATED,
|
|
/** Row attributes updated (command/args/env/etc.). */
|
|
UPDATED,
|
|
/** {@code enabled} flag flipped. */
|
|
TOGGLED,
|
|
/** Row deleted. */
|
|
DELETED
|
|
}
|
|
}
|