Keep RMCP refresh TODO repository-neutral

This commit is contained in:
Steven Lee
2026-07-07 16:36:42 +00:00
parent 5ab71eac2c
commit d1fe648cdc

View File

@@ -184,8 +184,8 @@ impl OAuthPersistor {
// variant. Match RMCP's own request path for now so rejected refresh tokens
// prompt reauthorization instead of surfacing as generic MCP startup failures.
// This can also prompt reauthorization after a transient failure.
// TODO: Once modelcontextprotocol/rust-sdk#963 is available in the RMCP version
// used by Codex, map only its typed refresh-token rejection error here.
// TODO: When RMCP exposes a typed distinction for refresh-token rejection,
// map only that definitive rejection to `AuthorizationRequired` here.
warn!(
error = %error,
"MCP OAuth refresh failed; reauthorization required by RMCP compatibility policy"