🎭 Playwright E2E Test Results

✅  24 passed

Details

24 tests across 3 suites
 31.4 seconds
 bb95fcc
ℹ️  Test Environment: Ubuntu Latest, Node.js v22.18.0
Browsers: Chromium, Firefox

📊 View Detailed HTML Report (download artifacts)

Hi @halter73 @dend I think this PR which I was about to merge would invalidate the need for this one? Could you take a look before I merge it? Thanks!

@olaservo I approved the other PR, because my main motivation is to fix the resource=undefined issue with the /token request, but I don't think it invalidates the need for this one. Either way, it's wrong to restore the AuthDebuggerState in a way that violates the type definition:

Even if we decide to store the resource in a separate session state key, we should fix the AuthDebuggerState restore behavior so that it creates a valid object. And at that point, I don't know what the advantages of the 30 line change is over the 3 line change when you need to make the 3 line change anyway. Personally, I would probably just merge this PR over the other one for simplicities sake, but they're not mutually exclusive.

Thanks for the explanation @halter73 , looks like there is just a formatting failure in the CI: https://github.com/modelcontextprotocol/inspector/actions/runs/16999856050/job/48199242030?pr=714

I ran prettier --write and pushed the changes to OAuthDebugCallback.tsx