mirror of
https://github.com/openclaw/openclaw.git
synced 2026-05-08 12:21:24 +00:00
* Agents: add subagent orchestration controls
* Agents: add subagent orchestration controls (WIP uncommitted changes)
* feat(subagents): add depth-based spawn gating for sub-sub-agents
* feat(subagents): tool policy, registry, and announce chain for nested agents
* feat(subagents): system prompt, docs, changelog for nested sub-agents
* fix(subagents): prevent model fallback override, show model during active runs, and block context overflow fallback
Bug 1: When a session has an explicit model override (e.g., gpt/openai-codex),
the fallback candidate logic in resolveFallbackCandidates silently appended the
global primary model (opus) as a backstop. On reinjection/steer with a transient
error, the session could fall back to opus which has a smaller context window
and crash. Fix: when storedModelOverride is set, pass fallbacksOverride ?? []
instead of undefined, preventing the implicit primary backstop.
Bug 2: Active subagents showed 'model n/a' in /subagents list because
resolveModelDisplay only read entry.model/modelProvider (populated after run
completes). Fix: fall back to modelOverride/providerOverride fields which are
populated at spawn time via sessions.patch.
Bug 3: Context overflow errors (prompt too long, context_length_exceeded) could
theoretically escape runEmbeddedPiAgent and be treated as failover candidates
in runWithModelFallback, causing a switch to a model with a smaller context
window. Fix: in runWithModelFallback, detect context overflow errors via
isLikelyContextOverflowError and rethrow them immediately instead of trying the
next model candidate.
* fix(subagents): track spawn depth in session store and fix announce routing for nested agents
* Fix compaction status tracking and dedupe overflow compaction triggers
* fix(subagents): enforce depth block via session store and implement cascade kill
* fix: inject group chat context into system prompt
* fix(subagents): always write model to session store at spawn time
* Preserve spawnDepth when agent handler rewrites session entry
* fix(subagents): suppress announce on steer-restart
* fix(subagents): fallback spawned session model to runtime default
* fix(subagents): enforce spawn depth when caller key resolves by sessionId
* feat(subagents): implement active-first ordering for numeric targets and enhance task display
- Added a test to verify that subagents with numeric targets follow an active-first list ordering.
- Updated `resolveSubagentTarget` to sort subagent runs based on active status and recent activity.
- Enhanced task display in command responses to prevent truncation of long task descriptions.
- Introduced new utility functions for compacting task text and managing subagent run states.
* fix(subagents): show model for active runs via run record fallback
When the spawned model matches the agent's default model, the session
store's override fields are intentionally cleared (isDefault: true).
The model/modelProvider fields are only populated after the run
completes. This left active subagents showing 'model n/a'.
Fix: store the resolved model on SubagentRunRecord at registration
time, and use it as a fallback in both display paths (subagents tool
and /subagents command) when the session store entry has no model info.
Changes:
- SubagentRunRecord: add optional model field
- registerSubagentRun: accept and persist model param
- sessions-spawn-tool: pass resolvedModel to registerSubagentRun
- subagents-tool: pass run record model as fallback to resolveModelDisplay
- commands-subagents: pass run record model as fallback to resolveModelDisplay
* feat(chat): implement session key resolution and reset on sidebar navigation
- Added functions to resolve the main session key and reset chat state when switching sessions from the sidebar.
- Updated the `renderTab` function to handle session key changes when navigating to the chat tab.
- Introduced a test to verify that the session resets to "main" when opening chat from the sidebar navigation.
* fix: subagent timeout=0 passthrough and fallback prompt duplication
Bug 1: runTimeoutSeconds=0 now means 'no timeout' instead of applying 600s default
- sessions-spawn-tool: default to undefined (not 0) when neither timeout param
is provided; use != null check so explicit 0 passes through to gateway
- agent.ts: accept 0 as valid timeout (resolveAgentTimeoutMs already handles
0 → MAX_SAFE_TIMEOUT_MS)
Bug 2: model fallback no longer re-injects the original prompt as a duplicate
- agent.ts: track fallback attempt index; on retries use a short continuation
message instead of the full original prompt since the session file already
contains it from the first attempt
- Also skip re-sending images on fallback retries (already in session)
* feat(subagents): truncate long task descriptions in subagents command output
- Introduced a new utility function to format task previews, limiting their length to improve readability.
- Updated the command handler to use the new formatting function, ensuring task descriptions are truncated appropriately.
- Adjusted related tests to verify that long task descriptions are now truncated in the output.
* refactor(subagents): update subagent registry path resolution and improve command output formatting
- Replaced direct import of STATE_DIR with a utility function to resolve the state directory dynamically.
- Enhanced the formatting of command output for active and recent subagents, adding separators for better readability.
- Updated related tests to reflect changes in command output structure.
* fix(subagent): default sessions_spawn to no timeout when runTimeoutSeconds omitted
The previous fix (75a791106) correctly handled the case where
runTimeoutSeconds was explicitly set to 0 ("no timeout"). However,
when models omit the parameter entirely (which is common since the
schema marks it as optional), runTimeoutSeconds resolved to undefined.
undefined flowed through the chain as:
sessions_spawn → timeout: undefined (since undefined != null is false)
→ gateway agent handler → agentCommand opts.timeout: undefined
→ resolveAgentTimeoutMs({ overrideSeconds: undefined })
→ DEFAULT_AGENT_TIMEOUT_SECONDS (600s = 10 minutes)
This caused subagents to be killed at exactly 10 minutes even though
the user's intent (via TOOLS.md) was for subagents to run without a
timeout.
Fix: default runTimeoutSeconds to 0 (no timeout) when neither
runTimeoutSeconds nor timeoutSeconds is provided by the caller.
Subagent spawns are long-running by design and should not inherit the
600s agent-command default timeout.
* fix(subagent): accept timeout=0 in agent-via-gateway path (second 600s default)
* fix: thread timeout override through getReplyFromConfig dispatch path
getReplyFromConfig called resolveAgentTimeoutMs({ cfg }) with no override,
always falling back to the config default (600s). Add timeoutOverrideSeconds
to GetReplyOptions and pass it through as overrideSeconds so callers of the
dispatch chain can specify a custom timeout (0 = no timeout).
This complements the existing timeout threading in agentCommand and the
cron isolated-agent runner, which already pass overrideSeconds correctly.
* feat(model-fallback): normalize OpenAI Codex model references and enhance fallback handling
- Added normalization for OpenAI Codex model references, specifically converting "gpt-5.3-codex" to "openai-codex" before execution.
- Updated the `resolveFallbackCandidates` function to utilize the new normalization logic.
- Enhanced tests to verify the correct behavior of model normalization and fallback mechanisms.
- Introduced a new test case to ensure that the normalization process works as expected for various input formats.
* feat(tests): add unit tests for steer failure behavior in openclaw-tools
- Introduced a new test file to validate the behavior of subagents when steer replacement dispatch fails.
- Implemented tests to ensure that the announce behavior is restored correctly and that the suppression reason is cleared as expected.
- Enhanced the subagent registry with a new function to clear steer restart suppression.
- Updated related components to support the new test scenarios.
* fix(subagents): replace stop command with kill in slash commands and documentation
- Updated the `/subagents` command to replace `stop` with `kill` for consistency in controlling sub-agent runs.
- Modified related documentation to reflect the change in command usage.
- Removed legacy timeoutSeconds references from the sessions-spawn-tool schema and tests to streamline timeout handling.
- Enhanced tests to ensure correct behavior of the updated commands and their interactions.
* feat(tests): add unit tests for readLatestAssistantReply function
- Introduced a new test file for the `readLatestAssistantReply` function to validate its behavior with various message scenarios.
- Implemented tests to ensure the function correctly retrieves the latest assistant message and handles cases where the latest message has no text.
- Mocked the gateway call to simulate different message histories for comprehensive testing.
* feat(tests): enhance subagent kill-all cascade tests and announce formatting
- Added a new test to verify that the `kill-all` command cascades through ended parents to active descendants in subagents.
- Updated the subagent announce formatting tests to reflect changes in message structure, including the replacement of "Findings:" with "Result:" and the addition of new expectations for message content.
- Improved the handling of long findings and stats in the announce formatting logic to ensure concise output.
- Refactored related functions to enhance clarity and maintainability in the subagent registry and tools.
* refactor(subagent): update announce formatting and remove unused constants
- Modified the subagent announce formatting to replace "Findings:" with "Result:" and adjusted related expectations in tests.
- Removed constants for maximum announce findings characters and summary words, simplifying the announcement logic.
- Updated the handling of findings to retain full content instead of truncating, ensuring more informative outputs.
- Cleaned up unused imports in the commands-subagents file to enhance code clarity.
* feat(tests): enhance billing error handling in user-facing text
- Added tests to ensure that normal text mentioning billing plans is not rewritten, preserving user context.
- Updated the `isBillingErrorMessage` and `sanitizeUserFacingText` functions to improve handling of billing-related messages.
- Introduced new test cases for various scenarios involving billing messages to ensure accurate processing and output.
- Enhanced the subagent announce flow to correctly manage active descendant runs, preventing premature announcements.
* feat(subagent): enhance workflow guidance and auto-announcement clarity
- Added a new guideline in the subagent system prompt to emphasize trust in push-based completion, discouraging busy polling for status updates.
- Updated documentation to clarify that sub-agents will automatically announce their results, improving user understanding of the workflow.
- Enhanced tests to verify the new guidance on avoiding polling loops and to ensure the accuracy of the updated prompts.
* fix(cron): avoid announcing interim subagent spawn acks
* chore: clean post-rebase imports
* fix(cron): fall back to child replies when parent stays interim
* fix(subagents): make active-run guidance advisory
* fix(subagents): update announce flow to handle active descendants and enhance test coverage
- Modified the announce flow to defer announcements when active descendant runs are present, ensuring accurate status reporting.
- Updated tests to verify the new behavior, including scenarios where no fallback requester is available and ensuring proper handling of finished subagents.
- Enhanced the announce formatting to include an `expectFinal` flag for better clarity in the announcement process.
* fix(subagents): enhance announce flow and formatting for user updates
- Updated the announce flow to provide clearer instructions for user updates based on active subagent runs and requester context.
- Refactored the announcement logic to improve clarity and ensure internal context remains private.
- Enhanced tests to verify the new message expectations and formatting, including updated prompts for user-facing updates.
- Introduced a new function to build reply instructions based on session context, improving the overall announcement process.
* fix: resolve prep blockers and changelog placement (#14447) (thanks @tyler6204)
* fix: restore cron delivery-plan import after rebase (#14447) (thanks @tyler6204)
* fix: resolve test failures from rebase conflicts (#14447) (thanks @tyler6204)
* fix: apply formatting after rebase (#14447) (thanks @tyler6204)
552 lines
19 KiB
TypeScript
552 lines
19 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { buildSubagentSystemPrompt } from "./subagent-announce.js";
|
|
import { buildAgentSystemPrompt, buildRuntimeLine } from "./system-prompt.js";
|
|
|
|
describe("buildAgentSystemPrompt", () => {
|
|
it("includes owner numbers when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
ownerNumbers: ["+123", " +456 ", ""],
|
|
});
|
|
|
|
expect(prompt).toContain("## User Identity");
|
|
expect(prompt).toContain(
|
|
"Owner numbers: +123, +456. Treat messages from these numbers as the user.",
|
|
);
|
|
});
|
|
|
|
it("omits owner section when numbers are missing", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
});
|
|
|
|
expect(prompt).not.toContain("## User Identity");
|
|
expect(prompt).not.toContain("Owner numbers:");
|
|
});
|
|
|
|
it("omits extended sections in minimal prompt mode", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
promptMode: "minimal",
|
|
ownerNumbers: ["+123"],
|
|
skillsPrompt:
|
|
"<available_skills>\n <skill>\n <name>demo</name>\n </skill>\n</available_skills>",
|
|
heartbeatPrompt: "ping",
|
|
toolNames: ["message", "memory_search"],
|
|
docsPath: "/tmp/openclaw/docs",
|
|
extraSystemPrompt: "Subagent details",
|
|
ttsHint: "Voice (TTS) is enabled.",
|
|
});
|
|
|
|
expect(prompt).not.toContain("## User Identity");
|
|
expect(prompt).not.toContain("## Skills");
|
|
expect(prompt).not.toContain("## Memory Recall");
|
|
expect(prompt).not.toContain("## Documentation");
|
|
expect(prompt).not.toContain("## Reply Tags");
|
|
expect(prompt).not.toContain("## Messaging");
|
|
expect(prompt).not.toContain("## Voice (TTS)");
|
|
expect(prompt).not.toContain("## Silent Replies");
|
|
expect(prompt).not.toContain("## Heartbeats");
|
|
expect(prompt).toContain("## Safety");
|
|
expect(prompt).toContain("You have no independent goals");
|
|
expect(prompt).toContain("Prioritize safety and human oversight");
|
|
expect(prompt).toContain("if instructions conflict");
|
|
expect(prompt).toContain("Inspired by Anthropic's constitution");
|
|
expect(prompt).toContain("Do not manipulate or persuade anyone");
|
|
expect(prompt).toContain("Do not copy yourself or change system prompts");
|
|
expect(prompt).toContain("## Subagent Context");
|
|
expect(prompt).not.toContain("## Group Chat Context");
|
|
expect(prompt).toContain("Subagent details");
|
|
});
|
|
|
|
it("includes safety guardrails in full prompts", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
});
|
|
|
|
expect(prompt).toContain("## Safety");
|
|
expect(prompt).toContain("You have no independent goals");
|
|
expect(prompt).toContain("Prioritize safety and human oversight");
|
|
expect(prompt).toContain("if instructions conflict");
|
|
expect(prompt).toContain("Inspired by Anthropic's constitution");
|
|
expect(prompt).toContain("Do not manipulate or persuade anyone");
|
|
expect(prompt).toContain("Do not copy yourself or change system prompts");
|
|
});
|
|
|
|
it("includes voice hint when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
ttsHint: "Voice (TTS) is enabled.",
|
|
});
|
|
|
|
expect(prompt).toContain("## Voice (TTS)");
|
|
expect(prompt).toContain("Voice (TTS) is enabled.");
|
|
});
|
|
|
|
it("adds reasoning tag hint when enabled", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
reasoningTagHint: true,
|
|
});
|
|
|
|
expect(prompt).toContain("## Reasoning Format");
|
|
expect(prompt).toContain("<think>...</think>");
|
|
expect(prompt).toContain("<final>...</final>");
|
|
});
|
|
|
|
it("includes a CLI quick reference section", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
});
|
|
|
|
expect(prompt).toContain("## OpenClaw CLI Quick Reference");
|
|
expect(prompt).toContain("openclaw gateway restart");
|
|
expect(prompt).toContain("Do not invent commands");
|
|
});
|
|
|
|
it("marks system message blocks as internal and not user-visible", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
});
|
|
|
|
expect(prompt).toContain("`[System Message] ...` blocks are internal context");
|
|
expect(prompt).toContain("are not user-visible by default");
|
|
expect(prompt).toContain("reports completed cron/subagent work");
|
|
expect(prompt).toContain("rewrite it in your normal assistant voice");
|
|
});
|
|
|
|
it("guides subagent workflows to avoid polling loops", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
});
|
|
|
|
expect(prompt).toContain("Completion is push-based: it will auto-announce when done.");
|
|
expect(prompt).toContain("Do not poll `subagents list` / `sessions_list` in a loop");
|
|
});
|
|
|
|
it("lists available tools when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
toolNames: ["exec", "sessions_list", "sessions_history", "sessions_send"],
|
|
});
|
|
|
|
expect(prompt).toContain("Tool availability (filtered by policy):");
|
|
expect(prompt).toContain("sessions_list");
|
|
expect(prompt).toContain("sessions_history");
|
|
expect(prompt).toContain("sessions_send");
|
|
});
|
|
|
|
it("preserves tool casing in the prompt", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
toolNames: ["Read", "Exec", "process"],
|
|
skillsPrompt:
|
|
"<available_skills>\n <skill>\n <name>demo</name>\n </skill>\n</available_skills>",
|
|
docsPath: "/tmp/openclaw/docs",
|
|
});
|
|
|
|
expect(prompt).toContain("- Read: Read file contents");
|
|
expect(prompt).toContain("- Exec: Run shell commands");
|
|
expect(prompt).toContain(
|
|
"- If exactly one skill clearly applies: read its SKILL.md at <location> with `Read`, then follow it.",
|
|
);
|
|
expect(prompt).toContain("OpenClaw docs: /tmp/openclaw/docs");
|
|
expect(prompt).toContain(
|
|
"For OpenClaw behavior, commands, config, or architecture: consult local docs first.",
|
|
);
|
|
});
|
|
|
|
it("includes docs guidance when docsPath is provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
docsPath: "/tmp/openclaw/docs",
|
|
});
|
|
|
|
expect(prompt).toContain("## Documentation");
|
|
expect(prompt).toContain("OpenClaw docs: /tmp/openclaw/docs");
|
|
expect(prompt).toContain(
|
|
"For OpenClaw behavior, commands, config, or architecture: consult local docs first.",
|
|
);
|
|
});
|
|
|
|
it("includes workspace notes when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
workspaceNotes: ["Reminder: commit your changes in this workspace after edits."],
|
|
});
|
|
|
|
expect(prompt).toContain("Reminder: commit your changes in this workspace after edits.");
|
|
});
|
|
|
|
it("includes user timezone when provided (12-hour)", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
userTimezone: "America/Chicago",
|
|
userTime: "Monday, January 5th, 2026 — 3:26 PM",
|
|
userTimeFormat: "12",
|
|
});
|
|
|
|
expect(prompt).toContain("## Current Date & Time");
|
|
expect(prompt).toContain("Time zone: America/Chicago");
|
|
});
|
|
|
|
it("includes user timezone when provided (24-hour)", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
userTimezone: "America/Chicago",
|
|
userTime: "Monday, January 5th, 2026 — 15:26",
|
|
userTimeFormat: "24",
|
|
});
|
|
|
|
expect(prompt).toContain("## Current Date & Time");
|
|
expect(prompt).toContain("Time zone: America/Chicago");
|
|
});
|
|
|
|
it("shows timezone when only timezone is provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
userTimezone: "America/Chicago",
|
|
userTimeFormat: "24",
|
|
});
|
|
|
|
expect(prompt).toContain("## Current Date & Time");
|
|
expect(prompt).toContain("Time zone: America/Chicago");
|
|
});
|
|
|
|
it("hints to use session_status for current date/time", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/clawd",
|
|
userTimezone: "America/Chicago",
|
|
});
|
|
|
|
expect(prompt).toContain("session_status");
|
|
expect(prompt).toContain("current date");
|
|
});
|
|
|
|
// The system prompt intentionally does NOT include the current date/time.
|
|
// Only the timezone is included, to keep the prompt stable for caching.
|
|
// See: https://github.com/moltbot/moltbot/commit/66eec295b894bce8333886cfbca3b960c57c4946
|
|
// Agents should use session_status or message timestamps to determine the date/time.
|
|
// Related: https://github.com/moltbot/moltbot/issues/1897
|
|
// https://github.com/moltbot/moltbot/issues/3658
|
|
it("does NOT include a date or time in the system prompt (cache stability)", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/clawd",
|
|
userTimezone: "America/Chicago",
|
|
userTime: "Monday, January 5th, 2026 — 3:26 PM",
|
|
userTimeFormat: "12",
|
|
});
|
|
|
|
// The prompt should contain the timezone but NOT the formatted date/time string.
|
|
// This is intentional for prompt cache stability — the date/time was removed in
|
|
// commit 66eec295b. If you're here because you want to add it back, please see
|
|
// https://github.com/moltbot/moltbot/issues/3658 for the preferred approach:
|
|
// gateway-level timestamp injection into messages, not the system prompt.
|
|
expect(prompt).toContain("Time zone: America/Chicago");
|
|
expect(prompt).not.toContain("Monday, January 5th, 2026");
|
|
expect(prompt).not.toContain("3:26 PM");
|
|
expect(prompt).not.toContain("15:26");
|
|
});
|
|
|
|
it("includes model alias guidance when aliases are provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
modelAliasLines: [
|
|
"- Opus: anthropic/claude-opus-4-5",
|
|
"- Sonnet: anthropic/claude-sonnet-4-5",
|
|
],
|
|
});
|
|
|
|
expect(prompt).toContain("## Model Aliases");
|
|
expect(prompt).toContain("Prefer aliases when specifying model overrides");
|
|
expect(prompt).toContain("- Opus: anthropic/claude-opus-4-5");
|
|
});
|
|
|
|
it("adds ClaudeBot self-update guidance when gateway tool is available", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
toolNames: ["gateway", "exec"],
|
|
});
|
|
|
|
expect(prompt).toContain("## OpenClaw Self-Update");
|
|
expect(prompt).toContain("config.apply");
|
|
expect(prompt).toContain("update.run");
|
|
});
|
|
|
|
it("includes skills guidance when skills prompt is present", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
skillsPrompt:
|
|
"<available_skills>\n <skill>\n <name>demo</name>\n </skill>\n</available_skills>",
|
|
});
|
|
|
|
expect(prompt).toContain("## Skills");
|
|
expect(prompt).toContain(
|
|
"- If exactly one skill clearly applies: read its SKILL.md at <location> with `read`, then follow it.",
|
|
);
|
|
});
|
|
|
|
it("appends available skills when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
skillsPrompt:
|
|
"<available_skills>\n <skill>\n <name>demo</name>\n </skill>\n</available_skills>",
|
|
});
|
|
|
|
expect(prompt).toContain("<available_skills>");
|
|
expect(prompt).toContain("<name>demo</name>");
|
|
});
|
|
|
|
it("omits skills section when no skills prompt is provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
});
|
|
|
|
expect(prompt).not.toContain("## Skills");
|
|
expect(prompt).not.toContain("<available_skills>");
|
|
});
|
|
|
|
it("renders project context files when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
contextFiles: [
|
|
{ path: "AGENTS.md", content: "Alpha" },
|
|
{ path: "IDENTITY.md", content: "Bravo" },
|
|
],
|
|
});
|
|
|
|
expect(prompt).toContain("# Project Context");
|
|
expect(prompt).toContain("## AGENTS.md");
|
|
expect(prompt).toContain("Alpha");
|
|
expect(prompt).toContain("## IDENTITY.md");
|
|
expect(prompt).toContain("Bravo");
|
|
});
|
|
|
|
it("ignores context files with missing or blank paths", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
contextFiles: [
|
|
{ path: undefined as unknown as string, content: "Missing path" },
|
|
{ path: " ", content: "Blank path" },
|
|
{ path: "AGENTS.md", content: "Alpha" },
|
|
],
|
|
});
|
|
|
|
expect(prompt).toContain("# Project Context");
|
|
expect(prompt).toContain("## AGENTS.md");
|
|
expect(prompt).toContain("Alpha");
|
|
expect(prompt).not.toContain("Missing path");
|
|
expect(prompt).not.toContain("Blank path");
|
|
});
|
|
|
|
it("adds SOUL guidance when a soul file is present", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
contextFiles: [
|
|
{ path: "./SOUL.md", content: "Persona" },
|
|
{ path: "dir\\SOUL.md", content: "Persona Windows" },
|
|
],
|
|
});
|
|
|
|
expect(prompt).toContain(
|
|
"If SOUL.md is present, embody its persona and tone. Avoid stiff, generic replies; follow its guidance unless higher-priority instructions override it.",
|
|
);
|
|
});
|
|
|
|
it("summarizes the message tool when available", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
toolNames: ["message"],
|
|
});
|
|
|
|
expect(prompt).toContain("message: Send messages and channel actions");
|
|
expect(prompt).toContain("### message tool");
|
|
expect(prompt).toContain("respond with ONLY: NO_REPLY");
|
|
});
|
|
|
|
it("includes runtime provider capabilities when present", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
runtimeInfo: {
|
|
channel: "telegram",
|
|
capabilities: ["inlineButtons"],
|
|
},
|
|
});
|
|
|
|
expect(prompt).toContain("channel=telegram");
|
|
expect(prompt).toContain("capabilities=inlineButtons");
|
|
});
|
|
|
|
it("includes agent id in runtime when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
runtimeInfo: {
|
|
agentId: "work",
|
|
host: "host",
|
|
os: "macOS",
|
|
arch: "arm64",
|
|
node: "v20",
|
|
model: "anthropic/claude",
|
|
},
|
|
});
|
|
|
|
expect(prompt).toContain("agent=work");
|
|
});
|
|
|
|
it("includes reasoning visibility hint", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
reasoningLevel: "off",
|
|
});
|
|
|
|
expect(prompt).toContain("Reasoning: off");
|
|
expect(prompt).toContain("/reasoning");
|
|
expect(prompt).toContain("/status shows Reasoning");
|
|
});
|
|
|
|
it("builds runtime line with agent and channel details", () => {
|
|
const line = buildRuntimeLine(
|
|
{
|
|
agentId: "work",
|
|
host: "host",
|
|
repoRoot: "/repo",
|
|
os: "macOS",
|
|
arch: "arm64",
|
|
node: "v20",
|
|
model: "anthropic/claude",
|
|
defaultModel: "anthropic/claude-opus-4-5",
|
|
},
|
|
"telegram",
|
|
["inlineButtons"],
|
|
"low",
|
|
);
|
|
|
|
expect(line).toContain("agent=work");
|
|
expect(line).toContain("host=host");
|
|
expect(line).toContain("repo=/repo");
|
|
expect(line).toContain("os=macOS (arm64)");
|
|
expect(line).toContain("node=v20");
|
|
expect(line).toContain("model=anthropic/claude");
|
|
expect(line).toContain("default_model=anthropic/claude-opus-4-5");
|
|
expect(line).toContain("channel=telegram");
|
|
expect(line).toContain("capabilities=inlineButtons");
|
|
expect(line).toContain("thinking=low");
|
|
});
|
|
|
|
it("describes sandboxed runtime and elevated when allowed", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
sandboxInfo: {
|
|
enabled: true,
|
|
workspaceDir: "/tmp/sandbox",
|
|
containerWorkspaceDir: "/workspace",
|
|
workspaceAccess: "ro",
|
|
agentWorkspaceMount: "/agent",
|
|
elevated: { allowed: true, defaultLevel: "on" },
|
|
},
|
|
});
|
|
|
|
expect(prompt).toContain("Your working directory is: /workspace");
|
|
expect(prompt).toContain(
|
|
"For read/write/edit/apply_patch, file paths resolve against host workspace: /tmp/openclaw.",
|
|
);
|
|
expect(prompt).toContain("Sandbox container workdir: /workspace");
|
|
expect(prompt).toContain("Sandbox host workspace: /tmp/sandbox");
|
|
expect(prompt).toContain("You are running in a sandboxed runtime");
|
|
expect(prompt).toContain("Sub-agents stay sandboxed");
|
|
expect(prompt).toContain("User can toggle with /elevated on|off|ask|full.");
|
|
expect(prompt).toContain("Current elevated level: on");
|
|
});
|
|
|
|
it("includes reaction guidance when provided", () => {
|
|
const prompt = buildAgentSystemPrompt({
|
|
workspaceDir: "/tmp/openclaw",
|
|
reactionGuidance: {
|
|
level: "minimal",
|
|
channel: "Telegram",
|
|
},
|
|
});
|
|
|
|
expect(prompt).toContain("## Reactions");
|
|
expect(prompt).toContain("Reactions are enabled for Telegram in MINIMAL mode.");
|
|
});
|
|
});
|
|
|
|
describe("buildSubagentSystemPrompt", () => {
|
|
it("includes sub-agent spawning guidance for depth-1 orchestrator when maxSpawnDepth >= 2", () => {
|
|
const prompt = buildSubagentSystemPrompt({
|
|
childSessionKey: "agent:main:subagent:abc",
|
|
task: "research task",
|
|
childDepth: 1,
|
|
maxSpawnDepth: 2,
|
|
});
|
|
|
|
expect(prompt).toContain("## Sub-Agent Spawning");
|
|
expect(prompt).toContain("You CAN spawn your own sub-agents");
|
|
expect(prompt).toContain("sessions_spawn");
|
|
expect(prompt).toContain("`subagents` tool");
|
|
expect(prompt).toContain("announce their results back to you automatically");
|
|
expect(prompt).toContain("Do NOT repeatedly poll `subagents list`");
|
|
});
|
|
|
|
it("does not include spawning guidance for depth-1 leaf when maxSpawnDepth == 1", () => {
|
|
const prompt = buildSubagentSystemPrompt({
|
|
childSessionKey: "agent:main:subagent:abc",
|
|
task: "research task",
|
|
childDepth: 1,
|
|
maxSpawnDepth: 1,
|
|
});
|
|
|
|
expect(prompt).not.toContain("## Sub-Agent Spawning");
|
|
expect(prompt).not.toContain("You CAN spawn");
|
|
});
|
|
|
|
it("includes leaf worker note for depth-2 sub-sub-agents", () => {
|
|
const prompt = buildSubagentSystemPrompt({
|
|
childSessionKey: "agent:main:subagent:abc:subagent:def",
|
|
task: "leaf task",
|
|
childDepth: 2,
|
|
maxSpawnDepth: 2,
|
|
});
|
|
|
|
expect(prompt).toContain("## Sub-Agent Spawning");
|
|
expect(prompt).toContain("leaf worker");
|
|
expect(prompt).toContain("CANNOT spawn further sub-agents");
|
|
});
|
|
|
|
it("uses 'parent orchestrator' label for depth-2 agents", () => {
|
|
const prompt = buildSubagentSystemPrompt({
|
|
childSessionKey: "agent:main:subagent:abc:subagent:def",
|
|
task: "leaf task",
|
|
childDepth: 2,
|
|
maxSpawnDepth: 2,
|
|
});
|
|
|
|
expect(prompt).toContain("spawned by the parent orchestrator");
|
|
expect(prompt).toContain("reported to the parent orchestrator");
|
|
});
|
|
|
|
it("uses 'main agent' label for depth-1 agents", () => {
|
|
const prompt = buildSubagentSystemPrompt({
|
|
childSessionKey: "agent:main:subagent:abc",
|
|
task: "orchestrator task",
|
|
childDepth: 1,
|
|
maxSpawnDepth: 2,
|
|
});
|
|
|
|
expect(prompt).toContain("spawned by the main agent");
|
|
expect(prompt).toContain("reported to the main agent");
|
|
});
|
|
|
|
it("defaults to depth 1 and maxSpawnDepth 1 when not provided", () => {
|
|
const prompt = buildSubagentSystemPrompt({
|
|
childSessionKey: "agent:main:subagent:abc",
|
|
task: "basic task",
|
|
});
|
|
|
|
// Should not include spawning guidance (default maxSpawnDepth is 1, depth 1 is leaf)
|
|
expect(prompt).not.toContain("## Sub-Agent Spawning");
|
|
expect(prompt).toContain("spawned by the main agent");
|
|
});
|
|
});
|