Skip to content

Document connections.list result shape in skills.ts - #1808

Open
baggiiiie wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
baggiiiie:connections-list-result-shape
Open

Document connections.list result shape in skills.ts#1808
baggiiiie wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
baggiiiie:connections-list-result-shape

Conversation

@baggiiiie

Copy link
Copy Markdown
Contributor

issue

when agents call tools.executor.coreTools.connections.list({}) the actual output is

{
  ok: true,
  data: {
    connections: [connection objects],
  }
}

while execute skill doesn't explicitly say where the connection array is inside the {ok , data} envelope.

i have observed that agents almost always tend to write code like:

const result = await tools.executor.coreTools.connections.list({});
for (const conn of result) {
  const slug = conn.integration;
  // something else, e.g., search a tool with name 
}

which gives an error, as conn.integration is invalid.

fix

i've added instructions in skills to document how to deal with results from tools.executor.coreTools.connections.list({}). been running this local build for a week now, have seen significant improvements on this, no mistake at all across 50+ calls! hence im opening this PR

@baggiiiie
baggiiiie force-pushed the connections-list-result-shape branch from 49eb2c0 to 62ffeba Compare August 28, 2026 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant