openclaw - 💡(How to fix) Fix sessions_spawn with runtime='acp' fails: spawnedBy is only supported for subagent:* sessions [2 comments, 2 participants]

1. Error returned: spawnedBy is only supported for subagent:* sessions Also fails without agentId (relying on acp.defaultAgent). Same error.

Fix Plan

To resolve the issue, we need to modify the sessions_spawn function to handle acp runtime sessions without setting spawnedBy. Here are the steps:

• Modify the sessions_spawn function to check if the runtime is acp and handle it separately.
• Create a new session key in the format agent:<agentId>:acp:<uuid> and route it to the acpx backend.

Example Code

// Modified sessions_spawn function
function sessions_spawn(options) {
  if (options.runtime === 'acp') {
    // Create a new session key
    const sessionKey = `agent:${options.agentId || 'gemini'}:acp:${uuidv4()}`;
    // Route the session to the acpx backend
    return acpxBackend.createSession(sessionKey, options.task, options.label);
  } else {
    // Handle other runtimes as before
    // ...
  }
}

// acpxBackend.createSession function
function createSession(sessionKey, task, label) {
  // Create a new session in the acpx backend
  // ...
  return {
    sessionKey,
    task,
    label,
  };
}

Verification

To verify that the fix worked, follow these steps:

• Restart the gateway
• Call sessions_spawn with runtime: "acp" and verify that a new session is created in the acpx backend
• Check the gateway logs to ensure that the spawnedBy error is no longer present

Extra Tips

• Make sure to update the acpx plugin to the latest version to ensure compatibility with the modified sessions_spawn function.
• Test the modified sessions_spawn function with different agentId and runtime options to ensure that it handles all scenarios correctly.