Skip to content

fix: clarify get_container transport event include #362

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

Merged
dodeja merged 3 commits into from
Sep 10, 2026
+6 −1
Merged

fix: clarify get_container transport event include #362

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4c23bbb
fix: clarify container transport event include
cursoragent Sep 10, 2026
b0cf08c
test: assert container event summary description
cursoragent Sep 10, 2026
4651389
chore: format MCP schema expectation
cursoragent Sep 10, 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
5 changes: 5 additions & 0 deletions packages/mcp/src/protocol-compat.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,11 @@ describe('MCP protocol compatibility', () => {
maxLength: 128,
description: expect.stringMatching(/never pass conversation text/i),
});
expect(
toolSchemas.get('get_container')?.properties?.include?.description,
).toMatch(
/transport_events: Event summary.*get_container_transport_events/,
);

for (const name of [
'list_shipments',
Expand Down
2 changes: 1 addition & 1 deletion packages/mcp/src/server.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1105,7 +1105,7 @@ export function createTerminal49McpServer(
"Optional related data to include. Default: ['shipment'] covers most use cases. " +
'• shipment: Routing, BOL, line, ref numbers (lightweight, always useful) ' +
'• pod_terminal: Terminal name, location, availability (lightweight, needed for demurrage questions) ' +
'• transport_events: Full event history, rail tracking (heavy 50-100 events, use for journey/timeline questions)',
'• transport_events: Event summary (count, rail event count, and latest event); use get_container_transport_events for the full timeline',
),
}),
outputSchema: z.object({}).passthrough(),
Expand Down
Loading