You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
test(registerCommands): pin await semantics with deferred-promise pattern
CodeRabbit nit on the test additions: the two await-asserting tests
for focusInput and toggleAutoApprove only verified the call payload,
so they would still pass if `await` were replaced with `void` in
the handler. Switched both to a deferred-promise pattern that
observes the handler's pending state before the underlying
postMessageToWebview resolves — now a regression that drops the
`await` would be caught at the test boundary.
0 commit comments