Skip to content

Commit e48b869

Browse files
DrJKLclaudeampagent
authored
refactor: remove unused OAuthConsentDecision type (#13277)
The `OAuthConsentDecision` type alias had no references anywhere — the function `submitOAuthConsentDecision` is not annotated with it, and nothing imports it. Removed. The other type/enum candidates fallow flagged were verified as NOT dead and left untouched: - `NodeBadgeMode.None/ShowAll/HideBuiltIn` — consumed via `Object.values` across `coreSettings`, `apiSchema` (zod enum), and the badge composables. - `src/types/index.ts` barrel and litegraph types — public/extension surface. - `generatedManagerTypes` `paths`/`webhooks`/`$defs`/`operations` — openapi-codegen output, not hand-editable. Gates: typecheck, knip pass (type-only change). Stacked PR. Base: `drjkl/cleanup-dead-exports`. --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com> Co-authored-by: Amp <amp@ampcode.com>
1 parent 8b48d05 commit e48b869

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

src/platform/cloud/oauth/oauthApi.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,6 @@ export type OAuthConsentDecisionParams = {
5151
expectedRedirectUri?: string
5252
}
5353

54-
export type OAuthConsentDecision = (
55-
params: OAuthConsentDecisionParams
56-
) => Promise<void>
57-
5854
// Schemes that execute in our origin if navigated. Never navigable,
5955
// regardless of what the backend returns. Everything else is governed
6056
// by binding to the challenge's registered redirect_uri — no per-client

0 commit comments

Comments
 (0)