Skip to content

Onboarding: Generate real REST handlers for OpenAPI-discovered agents #2760

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
TalZaccai wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Onboarding: Generate real REST handlers for OpenAPI-discovered agents #2760

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
bdc9cdd
Generate real REST fetch handlers for OpenAPI-discovered agents
TalZaccai Jul 29, 2026
1c2add8
Fix REST handler review issues: $ref path params, GET body, non-http …
TalZaccai Jul 29, 2026
2f2eb39
onboarding: tighten REST generator code comments
TalZaccai Jul 30, 2026
dd9cc3a
Fix REST handler template body under exactOptionalPropertyTypes
TalZaccai Jul 30, 2026
0d02365
Refactor extractOpenApiActions to satisfy complexity ratchet
Copilot Jul 30, 2026
693af70
Type the OpenAPI spec model to clear the lint ratchet
Copilot Jul 30, 2026
8503913
Centralize REST-handler response truncation into agent-sdk helper
Copilot Jul 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions ts/packages/agentSdk/src/helpers/actionHelpers.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -459,3 +459,19 @@ export function actionResultToString(actionResult: ActionResult): string {
return fields.join("\n");
}
}

// Default cap on characters of a tool/action response body echoed back to the
// dispatcher. Conservative v1 default; not yet model-aware — a model-derived
// budget belongs in the host where the selected model is known. Callers should
// reference this instead of inlining a magic number.
export const DEFAULT_MAX_RESPONSE_CHARS = 4000;

// Truncate `text` to at most `maxChars`, appending a marker when clipped.
export function truncateText(
text: string,
maxChars: number = DEFAULT_MAX_RESPONSE_CHARS,
): string {
return text.length > maxChars
? text.slice(0, maxChars) + "\n…(truncated)"
: text;
}
3 changes: 3 additions & 0 deletions ts/packages/agents/onboarding/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@
"clean": "rimraf --glob dist *.tsbuildinfo *.done.build.log",
"prettier": "prettier --check . --ignore-path ../../../.prettierignore",
"prettier:fix": "prettier --write . --ignore-path ../../../.prettierignore",
"test": "npm run test:local",
"test:local": "tsx --test test/*.spec.ts",
"tsc": "tsc -b"
},
"dependencies": {
Expand All @@ -64,6 +66,7 @@
"copyfiles": "^2.4.1",
"prettier": "^3.5.3",
"rimraf": "^6.0.1",
"tsx": "^4.21.0",
"typescript": "~5.4.5",
"ws": "^8.21.1"
},
Expand Down
Loading
Loading