Merge pull request #1899 from PR0M3TH3AN/codex/update-requestenablepermissions-logic

Prefer null `extension.enable()` with short timeout and extend explicit-permissions timeout
This commit is contained in:
thePR0M3TH3AN authored and GitHub committed 2026-01-28 15:05:49 -05:00
commit 617095d995
1 file changed
+5 -5
+5 -5
View File
@@ -337,10 +337,10 @@ export async function requestEnablePermissions(
const enableTimeoutMs = getEnableVariantTimeoutMs();
const loginTimeoutMs = getNip07LoginTimeoutMs();
const primaryEnableTimeoutMs = Math.min(15_000, enableTimeoutMs);
const fallbackEnableTimeoutMs = Math.min(
loginTimeoutMs,
const shortEnableTimeoutMs = Math.min(5_000, enableTimeoutMs);
const explicitEnableTimeoutMs = Math.max(
enableTimeoutMs,
Math.min(15_000, loginTimeoutMs),
);
let lastError = null;
@@ -351,11 +351,11 @@ export async function requestEnablePermissions(
const variantTimeoutOverrides =
options === null
? {
timeoutMs: primaryEnableTimeoutMs,
timeoutMs: shortEnableTimeoutMs,
retryMultiplier: 1,
}
: {
timeoutMs: fallbackEnableTimeoutMs,
timeoutMs: explicitEnableTimeoutMs,
retryMultiplier: 1,
};