Skip to content

Allow app config link to target config by client ID#7630

Draft
gonzaloriestra wants to merge 1 commit into
mainfrom
cli-interactivity/app-config-link
Draft

Allow app config link to target config by client ID#7630
gonzaloriestra wants to merge 1 commit into
mainfrom
cli-interactivity/app-config-link

Conversation

@gonzaloriestra
Copy link
Copy Markdown
Contributor

@gonzaloriestra gonzaloriestra commented May 25, 2026

WHY are these changes introduced?

The interactivity audit found that app config link could not use --client-id and --config together, leaving a prompt in otherwise scripted linking flows.

WHAT is this pull request doing?

  • Allows app config link to pair --client-id with --config.
  • Adds command coverage and refreshes generated command docs.

How to test your changes?

  • pnpm vitest run packages/app/src/cli/commands/app/config/link.test.ts
  • pnpm refresh-manifests

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing, so I've added a changelog entry with pnpm changeset add

Copy link
Copy Markdown
Contributor Author

gonzaloriestra commented May 25, 2026

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant