docs: mark external oauth hook deprecated

This commit is contained in:
Shakker
2026-04-23 06:46:03 +01:00
committed by Shakker
parent bd4cfe0e7e
commit 0e1407362d
3 changed files with 9 additions and 9 deletions

View File

@@ -797,9 +797,9 @@ export function resolveExternalAuthProfilesWithPlugins(params: {
}
if (!declaredPluginIds.has(plugin.id) && !warnedExternalAuthFallbackPluginIds.has(plugin.id)) {
warnedExternalAuthFallbackPluginIds.add(plugin.id);
// Deprecated compatibility path for plugins that predate the manifest
// contract. Remove this warning with the fallback resolver after the
// externalAuthProviders migration window closes.
// Deprecated compatibility path for plugins that still implement
// resolveExternalOAuthProfiles or omit contracts.externalAuthProviders.
// Remove this warning with the fallback resolver after the migration window.
log.warn(
`Provider plugin "${plugin.id}" uses external auth hooks without declaring contracts.externalAuthProviders. This compatibility fallback is deprecated and will be removed in a future release.`,
);

View File

@@ -137,9 +137,9 @@ export function resolveExternalAuthProfileCompatFallbackPluginIds(params: {
workspaceDir?: string;
env?: PluginLoadOptions["env"];
}): string[] {
// Compatibility fallback for provider plugins that implemented the public
// external auth hook before contracts.externalAuthProviders existed. Remove
// this with the warning path in provider-runtime after the deprecation window.
// Deprecated compatibility fallback for provider plugins that still implement
// resolveExternalOAuthProfiles or omit contracts.externalAuthProviders. Remove
// this with the warning path in provider-runtime after the migration window.
const declaredPluginIds = new Set(resolveExternalAuthProfileProviderPluginIds(params));
const registry = loadProviderManifestRegistry(params);
const normalizedConfig = normalizePluginsConfig(params.config?.plugins);

View File

@@ -1583,9 +1583,9 @@ export type ProviderPlugin = {
| null
| undefined;
/**
* @deprecated Use `resolveExternalAuthProfiles`.
*
* Kept for compatibility with existing provider plugins.
* @deprecated Declare `contracts.externalAuthProviders` in the plugin manifest
* and implement `resolveExternalAuthProfiles` instead. This compatibility hook
* is loaded through a slower fallback path and will be removed in a future release.
*/
resolveExternalOAuthProfiles?: (
ctx: ProviderResolveExternalOAuthProfilesContext,