toolcraft 0.0.309 → 0.0.311

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (26) hide show
  1. package/composition.json +3 -3
  2. package/dist/composition.json +3 -3
  3. package/node_modules/auth-store/README.md +16 -0
  4. package/node_modules/auth-store/dist/encrypted-file-store.d.ts +3 -0
  5. package/node_modules/auth-store/dist/encrypted-file-store.js +7 -0
  6. package/node_modules/auth-store/dist/index.d.ts +1 -0
  7. package/node_modules/auth-store/dist/keychain-store.d.ts +8 -0
  8. package/node_modules/auth-store/dist/keychain-store.js +13 -0
  9. package/node_modules/auth-store/dist/transaction-lock.d.ts +24 -0
  10. package/node_modules/auth-store/dist/transaction-lock.js +152 -0
  11. package/node_modules/auth-store/dist/types.d.ts +2 -0
  12. package/node_modules/tiny-http-mcp-server/dist/composition.json +1 -1
  13. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/README.md +33 -0
  14. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/auth-store-session-store.js +8 -0
  15. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/default-oauth-client-provider.js +202 -185
  16. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/session-transaction.d.ts +6 -0
  17. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/session-transaction.js +42 -0
  18. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/token-endpoint.d.ts +3 -1
  19. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/token-endpoint.js +7 -3
  20. package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/types.d.ts +14 -1
  21. package/node_modules/tiny-http-mcp-server/package.json +1 -1
  22. package/node_modules/tiny-mcp-client/dist/index.d.ts +39 -1
  23. package/node_modules/tiny-mcp-client/dist/index.js +503 -254
  24. package/node_modules/tiny-stdio-mcp-server/dist/composition.json +1 -1
  25. package/node_modules/toolcraft-schema/package.json +1 -1
  26. package/package.json +2 -2
package/composition.json CHANGED
@@ -98,7 +98,7 @@
98
98
  },
99
99
  {
100
100
  "name": "tiny-http-mcp-server",
101
- "version": "0.1.24",
101
+ "version": "0.1.28",
102
102
  "license": "MIT"
103
103
  },
104
104
  {
@@ -113,7 +113,7 @@
113
113
  },
114
114
  {
115
115
  "name": "toolcraft",
116
- "version": "0.0.309",
116
+ "version": "0.0.311",
117
117
  "license": "MIT"
118
118
  },
119
119
  {
@@ -123,7 +123,7 @@
123
123
  },
124
124
  {
125
125
  "name": "toolcraft-schema",
126
- "version": "0.0.309",
126
+ "version": "0.0.311",
127
127
  "license": "MIT"
128
128
  },
129
129
  {
@@ -98,7 +98,7 @@
98
98
  },
99
99
  {
100
100
  "name": "tiny-http-mcp-server",
101
- "version": "0.1.24",
101
+ "version": "0.1.28",
102
102
  "license": "MIT"
103
103
  },
104
104
  {
@@ -113,7 +113,7 @@
113
113
  },
114
114
  {
115
115
  "name": "toolcraft",
116
- "version": "0.0.309",
116
+ "version": "0.0.311",
117
117
  "license": "MIT"
118
118
  },
119
119
  {
@@ -123,7 +123,7 @@
123
123
  },
124
124
  {
125
125
  "name": "toolcraft-schema",
126
- "version": "0.0.309",
126
+ "version": "0.0.311",
127
127
  "license": "MIT"
128
128
  },
129
129
  {
@@ -25,6 +25,22 @@ const value = await store.get(); // "secret-value"
25
25
  await store.delete();
26
26
  ```
27
27
 
28
+ Both built-in backends expose `store.withLock(operation, { signal, timeoutMs })`
29
+ for transactions spanning a read, external operation and write. Independent
30
+ instances and processes serialize the same encrypted file or Keychain
31
+ service/account; unrelated identities proceed independently. The default
32
+ acquisition timeout is 30 seconds. Cancellation during acquisition does not
33
+ release the active owner's lock. Individual `get`, `set` and `delete` calls do
34
+ not implicitly acquire it.
35
+
36
+ Locks use private filesystem claim directories, containing PID/random names
37
+ and numeric tickets without credentials. Dead-owner claims are recovered;
38
+ live claims are never stolen because of age. Empty directories remain so an
39
+ arriving contender cannot race directory removal. Keychain lock storage defaults
40
+ to `~/.auth-store/keychain-locks`; `keychainStore.lock` can select another
41
+ directory or filesystem adapter. Injected encrypted-file adapters need
42
+ `readdir` support when using transactions.
43
+
28
44
  ## Backends
29
45
 
30
46
  | `backendEnvVar` value | Platform | Backend |
@@ -1,9 +1,11 @@
1
1
  import type { SecretStore } from "./types.js";
2
+ import { type SecretStoreLockOptions } from "./transaction-lock.js";
2
3
  export interface MachineIdentity {
3
4
  hostname: string;
4
5
  username: string;
5
6
  }
6
7
  export interface EncryptedFileStoreFileSystem {
8
+ readdir?(path: string): Promise<string[]>;
7
9
  readFile(path: string, encoding: BufferEncoding): Promise<string>;
8
10
  writeFile(path: string, data: string | NodeJS.ArrayBufferView, options?: {
9
11
  encoding?: BufferEncoding;
@@ -40,6 +42,7 @@ export declare class EncryptedFileStore implements SecretStore {
40
42
  private keyPromise;
41
43
  constructor(input: EncryptedFileStoreInput);
42
44
  get(): Promise<string | null>;
45
+ withLock<T>(operation: () => Promise<T>, options?: SecretStoreLockOptions): Promise<T>;
43
46
  set(value: string): Promise<void>;
44
47
  delete(): Promise<void>;
45
48
  private assertCredentialPathHasNoSymbolicLinks;
@@ -3,6 +3,7 @@ import { promises as fs } from "node:fs";
3
3
  import { homedir, hostname, userInfo } from "node:os";
4
4
  import path from "node:path";
5
5
  import { hasOwnErrorCode } from "./error-codes.js";
6
+ import { withSecretStoreFileLock } from "./transaction-lock.js";
6
7
  const derivedKeyCache = new Map();
7
8
  const ENCRYPTION_ALGORITHM = "aes-256-gcm";
8
9
  const ENCRYPTION_VERSION = 1;
@@ -71,6 +72,12 @@ export class EncryptedFileStore {
71
72
  return null;
72
73
  }
73
74
  }
75
+ async withLock(operation, options = {}) {
76
+ await this.assertCredentialPathHasNoSymbolicLinks(`${this.filePath}.lock`);
77
+ if (this.fs.readdir === undefined)
78
+ throw new Error("Secret-store transaction locks require filesystem readdir support");
79
+ return withSecretStoreFileLock(this.fs, `${this.filePath}.lock`, operation, options);
80
+ }
74
81
  async set(value) {
75
82
  await this.assertCredentialPathHasNoSymbolicLinks(this.filePath);
76
83
  const key = await this.getEncryptionKey();
@@ -1,6 +1,7 @@
1
1
  export { createSecretStore } from "./create-secret-store.js";
2
2
  export { EncryptedFileStore } from "./encrypted-file-store.js";
3
3
  export { KeychainStore } from "./keychain-store.js";
4
+ export type { SecretStoreLockOptions, SecretStoreLockFileSystem } from "./transaction-lock.js";
4
5
  export { key, MigratingSecretStore } from "./provider-store.js";
5
6
  export type { SecretStore, StoreBackend, CreateSecretStoreInput, CreateSecretStoreResult } from "./types.js";
6
7
  export type { MachineIdentity, EncryptedFileStoreInput, EncryptedFileStoreFileSystem } from "./encrypted-file-store.js";
@@ -1,4 +1,5 @@
1
1
  import type { SecretStore } from "./types.js";
2
+ import { type SecretStoreLockFileSystem, type SecretStoreLockOptions } from "./transaction-lock.js";
2
3
  export interface KeychainCommandResult {
3
4
  stdout: string;
4
5
  stderr: string;
@@ -12,13 +13,20 @@ export interface KeychainStoreInput {
12
13
  runCommand?: KeychainCommandRunner;
13
14
  service: string;
14
15
  account: string;
16
+ lock?: {
17
+ fs?: SecretStoreLockFileSystem;
18
+ directory?: string;
19
+ };
15
20
  }
16
21
  export declare class KeychainStore implements SecretStore {
17
22
  private readonly runCommand;
18
23
  private readonly service;
19
24
  private readonly account;
25
+ private readonly lockFs;
26
+ private readonly lockDirectory;
20
27
  constructor(input: KeychainStoreInput);
21
28
  get(): Promise<string | null>;
29
+ withLock<T>(operation: () => Promise<T>, options?: SecretStoreLockOptions): Promise<T>;
22
30
  set(value: string): Promise<void>;
23
31
  delete(): Promise<void>;
24
32
  private executeSecurityCommand;
@@ -1,14 +1,23 @@
1
1
  import { spawn } from "node:child_process";
2
+ import { createHash } from "node:crypto";
3
+ import { promises as nodeFs } from "node:fs";
4
+ import { homedir } from "node:os";
5
+ import path from "node:path";
6
+ import { withSecretStoreFileLock } from "./transaction-lock.js";
2
7
  const SECURITY_CLI = "security";
3
8
  const KEYCHAIN_ITEM_NOT_FOUND_EXIT_CODE = 44;
4
9
  export class KeychainStore {
5
10
  runCommand;
6
11
  service;
7
12
  account;
13
+ lockFs;
14
+ lockDirectory;
8
15
  constructor(input) {
9
16
  this.runCommand = input.runCommand ?? runSecurityCommand;
10
17
  this.service = input.service.trim();
11
18
  this.account = input.account.trim();
19
+ this.lockFs = input.lock?.fs ?? nodeFs;
20
+ this.lockDirectory = input.lock?.directory ?? path.join(homedir(), ".auth-store", "keychain-locks");
12
21
  if (this.service.length === 0) {
13
22
  throw new Error("Keychain service must not be empty");
14
23
  }
@@ -26,6 +35,10 @@ export class KeychainStore {
26
35
  }
27
36
  throw createSecurityCliFailure("read secret from macOS Keychain", result);
28
37
  }
38
+ async withLock(operation, options = {}) {
39
+ const identity = createHash("sha256").update(JSON.stringify([this.service, this.account])).digest("hex");
40
+ return withSecretStoreFileLock(this.lockFs, path.join(this.lockDirectory, identity), operation, options);
41
+ }
29
42
  async set(value) {
30
43
  if (value.includes("\n") || value.includes("\r")) {
31
44
  throw new Error("Keychain secrets cannot contain line breaks");
@@ -0,0 +1,24 @@
1
+ export interface SecretStoreLockOptions {
2
+ signal?: AbortSignal;
3
+ timeoutMs?: number;
4
+ }
5
+ export interface SecretStoreLockFileSystem {
6
+ mkdir(path: string, options?: {
7
+ recursive?: boolean;
8
+ mode?: number;
9
+ }): Promise<unknown>;
10
+ readdir(path: string): Promise<string[]>;
11
+ readFile(path: string, encoding: BufferEncoding): Promise<string>;
12
+ writeFile(path: string, value: string, options: {
13
+ encoding: BufferEncoding;
14
+ flag: string;
15
+ mode: number;
16
+ }): Promise<void>;
17
+ rename(from: string, to: string): Promise<void>;
18
+ unlink(path: string): Promise<void>;
19
+ lstat(path: string): Promise<{
20
+ isSymbolicLink(): boolean;
21
+ }>;
22
+ }
23
+ /** Filesystem bakery lock: unique claims allow dead-owner cleanup without deleting a replacement owner's lock. */
24
+ export declare function withSecretStoreFileLock<T>(fs: SecretStoreLockFileSystem, lockDirectory: string, operation: () => Promise<T>, options?: SecretStoreLockOptions): Promise<T>;
@@ -0,0 +1,152 @@
1
+ import { randomUUID } from "node:crypto";
2
+ import path from "node:path";
3
+ import { hasOwnErrorCode } from "./error-codes.js";
4
+ /** Filesystem bakery lock: unique claims allow dead-owner cleanup without deleting a replacement owner's lock. */
5
+ export async function withSecretStoreFileLock(fs, lockDirectory, operation, options = {}) {
6
+ const timeoutMs = options.timeoutMs ?? 30_000;
7
+ if (!Number.isFinite(timeoutMs) || timeoutMs < 0 || timeoutMs > 2_147_483_647)
8
+ throw new Error("Invalid secret-store transaction lock timeout");
9
+ options.signal?.throwIfAborted();
10
+ const deadline = performance.now() + timeoutMs;
11
+ const directory = path.resolve(lockDirectory);
12
+ await assertLockDirectoryPath(fs, directory);
13
+ await fs.mkdir(directory, { recursive: true, mode: 0o700 });
14
+ await assertLockDirectoryPath(fs, directory);
15
+ const name = `${process.pid}-${randomUUID()}.claim`;
16
+ const claimPath = path.join(directory, name);
17
+ const temporaryPath = `${claimPath}.tmp`;
18
+ let claimed = true;
19
+ let temporaryCreated = false;
20
+ let outcome;
21
+ try {
22
+ try {
23
+ await fs.writeFile(claimPath, JSON.stringify({ ticket: null }), { encoding: "utf8", flag: "wx", mode: 0o600 });
24
+ }
25
+ catch (error) {
26
+ if (hasOwnErrorCode(error, "EEXIST"))
27
+ claimed = false;
28
+ throw error;
29
+ }
30
+ const existing = await readClaims(fs, directory, name);
31
+ const ticket = existing.reduce((max, claim) => Math.max(max, claim.ticket ?? 0), 0) + 1;
32
+ if (!Number.isSafeInteger(ticket))
33
+ throw new Error("Secret-store transaction lock ticket overflow");
34
+ temporaryCreated = true;
35
+ try {
36
+ await fs.writeFile(temporaryPath, JSON.stringify({ ticket }), { encoding: "utf8", flag: "wx", mode: 0o600 });
37
+ }
38
+ catch (error) {
39
+ if (hasOwnErrorCode(error, "EEXIST"))
40
+ temporaryCreated = false;
41
+ throw error;
42
+ }
43
+ await fs.rename(temporaryPath, claimPath);
44
+ temporaryCreated = false;
45
+ for (;;) {
46
+ options.signal?.throwIfAborted();
47
+ const peers = await readClaims(fs, directory, name);
48
+ if (!peers.some(peer => peer.ticket === null || peer.ticket < ticket || (peer.ticket === ticket && peer.name < name)))
49
+ break;
50
+ const remaining = deadline - performance.now();
51
+ if (remaining <= 0)
52
+ throw new Error("Timed out waiting for secret-store transaction lock");
53
+ await new Promise((resolve, reject) => {
54
+ const abort = () => { clearTimeout(timer); options.signal?.removeEventListener("abort", abort); reject(options.signal?.reason); };
55
+ const timer = setTimeout(() => { options.signal?.removeEventListener("abort", abort); resolve(); }, Math.min(10, remaining));
56
+ options.signal?.addEventListener("abort", abort, { once: true });
57
+ if (options.signal?.aborted)
58
+ abort();
59
+ });
60
+ }
61
+ options.signal?.throwIfAborted();
62
+ outcome = { result: await operation() };
63
+ }
64
+ catch (error) {
65
+ outcome = { error };
66
+ }
67
+ const cleanup = [];
68
+ for (const target of [...(temporaryCreated ? [temporaryPath] : []), ...(claimed ? [claimPath] : [])]) {
69
+ try {
70
+ await fs.unlink(target);
71
+ }
72
+ catch (error) {
73
+ if (!hasOwnErrorCode(error, "ENOENT"))
74
+ cleanup.push(error);
75
+ }
76
+ }
77
+ if (cleanup.length)
78
+ throw new AggregateError([...("error" in outcome ? [outcome.error] : []), ...cleanup], "Secret-store transaction lock cleanup failed");
79
+ if ("error" in outcome)
80
+ throw outcome.error;
81
+ return outcome.result;
82
+ }
83
+ async function assertNoSymbolicLink(fs, target) {
84
+ try {
85
+ if ((await fs.lstat(target)).isSymbolicLink())
86
+ throw new Error("Refusing secret-store transaction lock through symbolic link");
87
+ }
88
+ catch (error) {
89
+ if (!hasOwnErrorCode(error, "ENOENT"))
90
+ throw error;
91
+ }
92
+ }
93
+ async function assertLockDirectoryPath(fs, directory) {
94
+ const root = path.parse(directory).root;
95
+ const segments = directory.slice(root.length).split(path.sep).filter(Boolean);
96
+ let current = root;
97
+ for (const [index, segment] of segments.entries()) {
98
+ current = path.join(current, segment);
99
+ // Match encrypted credential paths: allow OS root aliases such as macOS /var.
100
+ if (index > 0 || segments.length === 1)
101
+ await assertNoSymbolicLink(fs, current);
102
+ }
103
+ }
104
+ async function readClaims(fs, directory, ownName) {
105
+ const claims = [];
106
+ for (const name of await fs.readdir(directory)) {
107
+ if (name === ownName || !name.endsWith(".claim"))
108
+ continue;
109
+ const pidText = name.slice(0, name.indexOf("-"));
110
+ const pid = Number(pidText);
111
+ if (!Number.isSafeInteger(pid) || pid < 1 || String(pid) !== pidText)
112
+ throw new Error("Malformed secret-store transaction lock owner");
113
+ const target = path.join(directory, name);
114
+ await assertNoSymbolicLink(fs, target);
115
+ let alive = true;
116
+ try {
117
+ process.kill(pid, 0);
118
+ }
119
+ catch (error) {
120
+ alive = !hasOwnErrorCode(error, "ESRCH");
121
+ }
122
+ if (!alive) {
123
+ try {
124
+ await fs.unlink(target);
125
+ }
126
+ catch (error) {
127
+ if (!hasOwnErrorCode(error, "ENOENT"))
128
+ throw error;
129
+ }
130
+ continue;
131
+ }
132
+ let ticket = null;
133
+ let raw;
134
+ try {
135
+ raw = await fs.readFile(target, "utf8");
136
+ }
137
+ catch (error) {
138
+ if (hasOwnErrorCode(error, "ENOENT"))
139
+ continue;
140
+ throw error;
141
+ }
142
+ try {
143
+ const value = JSON.parse(raw);
144
+ if (value !== null && typeof value === "object" && "ticket" in value &&
145
+ typeof value.ticket === "number" && Number.isSafeInteger(value.ticket) && value.ticket > 0)
146
+ ticket = value.ticket;
147
+ }
148
+ catch { /* Incomplete live claims remain in the choosing phase; never steal them by age. */ }
149
+ claims.push({ name, ticket });
150
+ }
151
+ return claims;
152
+ }
@@ -1,11 +1,13 @@
1
1
  import type { EncryptedFileStoreInput } from "./encrypted-file-store.js";
2
2
  import type { KeychainStoreInput } from "./keychain-store.js";
3
+ import type { SecretStoreLockOptions } from "./transaction-lock.js";
3
4
  export interface SecretStore {
4
5
  get(options?: {
5
6
  readOnly?: boolean;
6
7
  }): Promise<string | null>;
7
8
  set(value: string): Promise<void>;
8
9
  delete(): Promise<void>;
10
+ withLock?<T>(operation: () => Promise<T>, options?: SecretStoreLockOptions): Promise<T>;
9
11
  }
10
12
  export type StoreBackend = "file" | "keychain";
11
13
  export interface CreateSecretStoreInput {
@@ -18,7 +18,7 @@
18
18
  },
19
19
  {
20
20
  "name": "tiny-http-mcp-server",
21
- "version": "0.1.24",
21
+ "version": "0.1.28",
22
22
  "license": "MIT"
23
23
  },
24
24
  {
@@ -41,6 +41,7 @@ const verifier = createJwksTokenVerifier({
41
41
  - `mode: "dynamic"` with optional `metadata`
42
42
  - `mode: "static"` with `clientId`, optional `clientSecret`, optional `metadata`
43
43
  - `allowInteractive: false` prevents interactive login while retaining cached tokens and silent refresh
44
+ - `sessionLockTimeoutMs` limits acquisition waits for a session transaction lock (default 30,000 ms; integer from 1 to 2147483647)
44
45
  - `initialGrant: { resource, tokens }` optionally imports an existing Bearer grant for one HTTP resource; requires the original client ID
45
46
  - `browser.openBrowser(url)` optional
46
47
  - `browser.readLine()` optional
@@ -84,6 +85,15 @@ registration, token requests and bounded token-body reads. Cancellation retains
84
85
  its original reason and does not retry authorization. Custom providers should
85
86
  observe the supplied signal and pass it to any work they start.
86
87
 
88
+ `authorizeRequest` may return an owned token snapshot for the request it
89
+ authorized. The HTTP client supplies that snapshot as `presentedTokens`, along
90
+ with the actual request's `requestHeaders`, to `handleUnauthorized`. Providers
91
+ that return `void` remain supported. The native provider compares the rejected
92
+ snapshot with persisted credentials: delayed 401s retry with a newer grant
93
+ without redeeming its refresh token again. A proven current token is refreshed
94
+ on 401 even when the server omits `error="invalid_token"`. Invalid provenance
95
+ fails without quoting token values.
96
+
87
97
  Configure `client.metadata.scope` to request a precise scope set; broader
88
98
  discovery metadata does not override it.
89
99
 
@@ -97,6 +107,29 @@ Input tokens are copied and invalid expiry values fail before authorization.
97
107
 
98
108
  `createAuthStoreSessionStore(options)` accepts the standard `auth-store` config.
99
109
 
110
+ Providers sharing the same `sessionStore` object serialize the complete session
111
+ read, refresh/authorization and persistence transaction for each resource.
112
+ Waiting requests can cancel or time out independently; they cannot release an
113
+ active owner's lock. Custom stores may implement
114
+ `withLock(resource, operation, { signal, timeoutMs })` to serialize the same
115
+ transaction across store instances or processes. The hook must honor acquisition
116
+ cancellation and keep the lock until the operation settles. The timeout bounds
117
+ acquisition, while token and browser operations retain their own deadlines.
118
+ The native `auth-store` session adapter implements this hook for both encrypted
119
+ files and Keychain identities, including across independent processes. Locks
120
+ cover the complete read, refresh/authorization and persisted winner. Dead-owner
121
+ claims are recovered without stealing a live transaction.
122
+
123
+ Before sending a refresh request, the provider persists a tokenless session with
124
+ `refreshState: "pending"`, retaining the original client and discovery binding.
125
+ A successful response replaces it with the rotated grant. A crash, cancellation,
126
+ network disconnect or incomplete response leaves the marker, so another process
127
+ cannot replay a possibly consumed refresh token or revive the initial import.
128
+ Such a session requires fresh authorization. Interactive unauthorized handling
129
+ can recover it; headless requests fail with an explicit unknown-outcome error.
130
+ Only complete OAuth error responses establish a rejected request and allow a
131
+ transient retry or restoration of the original grant. Gateway error pages do not.
132
+
100
133
  ## Environment Variables
101
134
 
102
135
  This package exposes no direct environment variables. When `authStore` is used,
@@ -11,6 +11,12 @@ const DEFAULT_CLIENT_KEYCHAIN_SERVICE = "poe-code-mcp-oauth-clients";
11
11
  const MAX_JS_DATE_MS = 8_640_000_000_000_000;
12
12
  export function createAuthStoreSessionStore(options = {}) {
13
13
  return {
14
+ async withLock(resource, operation, lockOptions) {
15
+ const store = createResourceSecretStore(resource, options);
16
+ if (store.withLock === undefined)
17
+ throw new Error("OAuth secret-store backend does not support transaction locks");
18
+ return store.withLock(operation, lockOptions);
19
+ },
14
20
  async load(resource) {
15
21
  const store = createResourceSecretStore(resource, options);
16
22
  const value = await store.get();
@@ -119,6 +125,8 @@ function isStoredOAuthSession(value) {
119
125
  isNonBlankOwnString(value, "authorizationServer") &&
120
126
  isStoredOAuthClient(getOwnEntry(value, "client")) &&
121
127
  isStoredOAuthDiscovery(getOwnEntry(value, "discovery")) &&
128
+ (getOwnEntry(value, "refreshState") === undefined ||
129
+ (getOwnEntry(value, "refreshState") === "pending" && getOwnEntry(value, "tokens") === undefined)) &&
122
130
  isStoredOAuthTokensOrMissing(getOwnEntry(value, "tokens")));
123
131
  }
124
132
  function isStoredOAuthClient(value) {