Skip to content

Comments

feat: add OAuth credential setup and gateway output parsing to TUI deploy flow#411

Merged
tejaskash merged 3 commits intoaws:feat/gateway-integrationfrom
aidandaly24:feat/batch-3-tui-deploy-parity
Feb 24, 2026
Merged

feat: add OAuth credential setup and gateway output parsing to TUI deploy flow#411
tejaskash merged 3 commits intoaws:feat/gateway-integrationfrom
aidandaly24:feat/batch-3-tui-deploy-parity

Conversation

@aidandaly24
Copy link
Contributor

@aidandaly24 aidandaly24 commented Feb 23, 2026

Description

The CLI deploy command (agentcore deploy) already handles OAuth credential setup and gateway output parsing, but the TUI deploy flow (the interactive terminal UI) was missing both. This meant users deploying through the TUI wouldn't get their OAuth providers created or their gateway URLs persisted to deployed-state.json.

This PR brings the TUI deploy flow to parity with the CLI:

OAuth credential setup in TUI preflight. The useCdkPreflight hook now runs setupOAuth2Providers during the identity-setup phase, right after API key setup. OAuth credentials (provider ARN, client secret ARN, callback URL) are collected and passed through to the deploy flow.

Gateway output parsing in TUI deploy. After CDK deploy completes, useDeployFlow now reads the MCP spec to get gateway definitions, calls parseGatewayOutputs to extract gateway URLs from CloudFormation outputs, and passes both gateways and OAuth credentials to buildDeployedState so they're persisted correctly.

Without this change, users who deploy via the TUI would have working agents but broken gateway URLs and missing OAuth providers in their deployed state.

Related Issue

Part of the MCP Gateway Phase 1 integration (gateway-integration branch). Task 9b.

Type of Change

  • Bug fix

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

@github-actions github-actions bot added the size/l PR size: L label Feb 23, 2026
@aidandaly24 aidandaly24 force-pushed the feat/batch-3-tui-deploy-parity branch from 9d12d11 to 00c3c7c Compare February 24, 2026 02:46
@github-actions github-actions bot added size/xl PR size: XL and removed size/l PR size: L labels Feb 24, 2026
@aidandaly24 aidandaly24 force-pushed the feat/batch-3-tui-deploy-parity branch from 00c3c7c to 6038460 Compare February 24, 2026 02:54
@github-actions github-actions bot added size/xl PR size: XL and removed size/xl PR size: XL labels Feb 24, 2026
@aidandaly24 aidandaly24 force-pushed the feat/batch-3-tui-deploy-parity branch from 6038460 to 6af88f9 Compare February 24, 2026 17:07
@github-actions github-actions bot added size/s PR size: S and removed size/xl PR size: XL labels Feb 24, 2026
@github-actions github-actions bot added size/m PR size: M and removed size/s PR size: S labels Feb 24, 2026
@tejaskash tejaskash merged commit d655353 into aws:feat/gateway-integration Feb 24, 2026
14 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants