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.
- package/composition.json +3 -3
- package/dist/composition.json +3 -3
- package/node_modules/auth-store/README.md +16 -0
- package/node_modules/auth-store/dist/encrypted-file-store.d.ts +3 -0
- package/node_modules/auth-store/dist/encrypted-file-store.js +7 -0
- package/node_modules/auth-store/dist/index.d.ts +1 -0
- package/node_modules/auth-store/dist/keychain-store.d.ts +8 -0
- package/node_modules/auth-store/dist/keychain-store.js +13 -0
- package/node_modules/auth-store/dist/transaction-lock.d.ts +24 -0
- package/node_modules/auth-store/dist/transaction-lock.js +152 -0
- package/node_modules/auth-store/dist/types.d.ts +2 -0
- package/node_modules/tiny-http-mcp-server/dist/composition.json +1 -1
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/README.md +33 -0
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/auth-store-session-store.js +8 -0
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/default-oauth-client-provider.js +202 -185
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/session-transaction.d.ts +6 -0
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/session-transaction.js +42 -0
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/token-endpoint.d.ts +3 -1
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/token-endpoint.js +7 -3
- package/node_modules/tiny-http-mcp-server/node_modules/mcp-oauth/dist/client/types.d.ts +14 -1
- package/node_modules/tiny-http-mcp-server/package.json +1 -1
- package/node_modules/tiny-mcp-client/dist/index.d.ts +39 -1
- package/node_modules/tiny-mcp-client/dist/index.js +503 -254
- package/node_modules/tiny-stdio-mcp-server/dist/composition.json +1 -1
- package/node_modules/toolcraft-schema/package.json +1 -1
- 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.
|
|
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.
|
|
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.
|
|
126
|
+
"version": "0.0.311",
|
|
127
127
|
"license": "MIT"
|
|
128
128
|
},
|
|
129
129
|
{
|
package/dist/composition.json
CHANGED
|
@@ -98,7 +98,7 @@
|
|
|
98
98
|
},
|
|
99
99
|
{
|
|
100
100
|
"name": "tiny-http-mcp-server",
|
|
101
|
-
"version": "0.1.
|
|
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.
|
|
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.
|
|
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 {
|
|
@@ -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) {
|