Skip to content

feat(gateway): add interactive invoke console - #1978

Open
aidandaly24 wants to merge 6 commits into
refactorfrom
feat/gateway-invoke-tui
Open

feat(gateway): add interactive invoke console#1978
aidandaly24 wants to merge 6 commits into
refactorfrom
feat/gateway-invoke-tui

Conversation

@aidandaly24

Copy link
Copy Markdown
Contributor

Summary

  • add a persistent POST JSON console for agentcore gateway invoke
  • add Gateway switching, raw path editing, generated/continued sessions, and live response streaming
  • preserve the headless Gateway invoke contract while allowing CLI flags to seed interactive context
  • share the multiline editor with Runtime and keep long payloads within stable terminal rows

UX

  • Ctrl+T switches Gateway and clears Gateway-specific state
  • Ctrl+P edits the Gateway-relative path; blank uses the exact Gateway URL
  • Ctrl+V toggles completed JSON between raw and pretty forms
  • CUSTOM_JWT Gateways proactively require a CLI-provided bearer token
  • Runtime and MCP session metadata continues across requests

Verification

  • 206 pass, 0 fail focused Gateway/Runtime invoke tests
  • 1197 pass, 0 fail full bun test src
  • bun run typecheck
  • bun run lint:check
  • bun run format:check
  • bun run build
  • live TUI validation: NONE, AWS_IAM, CUSTOM_JWT, AUTHENTICATE_ONLY, JWT passthrough, OAuth 3LO, OAuth OBO, inference, Runtime HTTP, MCP aggregation, Lambda, and real SSE streaming

Stack

Depends on #1967. Review this PR against feat/gateway-invoke.

@github-actions github-actions Bot added the size/xl PR size: XL label Aug 11, 2026
@github-actions github-actions Bot added agentcore-harness-reviewing AgentCore Harness review in progress size/xl PR size: XL and removed agentcore-harness-reviewing AgentCore Harness review in progress size/xl PR size: XL labels Aug 11, 2026
@github-actions github-actions Bot removed the size/xl PR size: XL label Aug 12, 2026
@github-actions github-actions Bot added the size/xl PR size: XL label Aug 12, 2026
@aidandaly24
aidandaly24 force-pushed the feat/gateway-invoke-tui branch from 196b394 to 9c11253 Compare August 13, 2026 18:44
@github-actions github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 13, 2026
@github-actions github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 13, 2026
@aidandaly24
aidandaly24 force-pushed the feat/gateway-invoke-tui branch from d312820 to ad207ae Compare August 13, 2026 19:41
Base automatically changed from feat/gateway-invoke to refactor August 13, 2026 19:43
@aidandaly24
aidandaly24 force-pushed the feat/gateway-invoke-tui branch from ad207ae to 858db4d Compare August 13, 2026 19:43
@aidandaly24
aidandaly24 marked this pull request as ready for review August 13, 2026 19:45
@codecov-commenter

codecov-commenter commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.27586% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.98%. Comparing base (4d183dc) to head (e14e884).

Files with missing lines Patch % Lines
src/handlers/gateway/invoke/screen.tsx 97.80% 10 Missing ⚠️
src/handlers/gateway/invoke/index.tsx 98.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           refactor    #1978      +/-   ##
============================================
+ Coverage     96.96%   96.98%   +0.02%     
============================================
  Files           364      366       +2     
  Lines         20758    21304     +546     
============================================
+ Hits          20127    20662     +535     
- Misses          631      642      +11     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

metadata: metadata(response),
state: "streaming",
});
const kind = classifyStreamingResponse(response.contentType);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need the same statusCode !== 204 && statusCode !== 205 clause here that we did in the headless invoke PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Fixed in e14e8848 by exempting HTTP 204 and 205 from the TUI binary/unknown-content guard, matching the headless invoke behavior. I also added screen coverage for both statuses with an empty content type and body; the focused Gateway invoke screen suite passes 31/31.

@aidandaly24
aidandaly24 force-pushed the feat/gateway-invoke-tui branch from 858db4d to 40b3457 Compare August 13, 2026 20:31

@nborges-aws nborges-aws left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xl PR size: XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants