pi-quiver 6.0.1 → 6.1.0

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/CHANGELOG.md CHANGED
@@ -8,6 +8,11 @@ Published to npm as `pi-quiver` (`pi install npm:pi-quiver`). Pushing a
8
8
  via OIDC trusted publishing. The release helper at
9
9
  `.agents/skills/release/scripts/release.sh` cuts the tag; CI publishes.
10
10
 
11
+ ## v6.1.0 - 2026-09-15
12
+
13
+ - `slack`: tokens also resolve from a fixed per-user file - `$XDG_CONFIG_HOME/pi-quiver/.env` or `~/.config/pi-quiver/.env` (Linux/macOS), `%APPDATA%\pi-quiver\.env` (Windows) - after process env, the repo `.env`, and the primary checkout's `.env`. Every rung now falls through when it lacks the key; a repo `.env` holding only the bot token no longer blocks the user token. Breaking: an unreadable `.env` at any rung propagates the raw filesystem error instead of collapsing to `missing_token`; the `missing_token` message now lists every checked path (#22).
14
+ - `slack`: every `channel` parameter accepts `@name` or a user ID (`U...`/`W...`) and targets that person's DM - `resolveChannel` opens it via `conversations.open` (needs `im:write`; `slack_thread` DM reads need `im:history`) and tools echo the resulting `D...`. Cached display/real-name aliases are trusted only after a `slack_cache_refresh` snapshot (#20).
15
+
11
16
  ## v6.0.1 - 2026-09-13
12
17
 
13
18
  - `session-name` Herdr sink tolerates herdr-ntfy-notify's armed marker: exactly one leading `* ` on the live label no longer counts as a human rename, is preserved on every rename and on the shutdown restore, and its removal keeps the claim (#19).
package/README.md CHANGED
@@ -70,7 +70,7 @@ A 300 KB changelog page never touches your context window - you get a preview an
70
70
  | `extensions/sword-header.ts` | `/builtin-header` | Themed ASCII startup header replacing pi's default logo. OFF by default. |
71
71
  | `extensions/fast-mode.ts` | `/fast` | Inject Anthropic fast-mode (`speed: "fast"` + `anthropic-beta: fast-mode-2026-02-01`) into every Claude Opus 4.8 / Opus 5 request, any thinking level. `--fast` flag + `/fast [on\|off\|status]`. OFF by default. |
72
72
  | `extensions/provider-stall-watchdog.ts` | - | Opt-in provider-stall recovery, in two tiers: a pre-first-event deadline (`firstEventMs`, 20s) on every provider request in every mode, and the mid-stream pair (warn at 2 min, recover at 4 min) in TUI runs only. Policy D offers each stall to Pi's retry loop until the stall retry budget (`maxStallRetries`, default = `retry.maxRetries`) is exhausted. OFF by default. |
73
- | `extensions/slack.ts` | `slack_search`, `slack_thread`, `slack_post`, `slack_update`, `slack_delete`, `slack_pin`, `slack_upload`, `slack_cache_refresh` | Context-safe Slack search/threads/posting with dual `user`/`bot` token identities, a workspace-keyed channel/user name->ID cache, fetch-style output size gating, and a transactional headline+detail-thread announce protocol with a documented recovery path. OFF by default. Behavior lives in `lib/slack-core.ts` and `lib/slack-cache.ts`. |
73
+ | `extensions/slack.ts` | `slack_search`, `slack_thread`, `slack_post`, `slack_update`, `slack_delete`, `slack_pin`, `slack_upload`, `slack_cache_refresh` | Context-safe Slack search/threads/posting with dual `user`/`bot` token identities, a workspace-keyed channel/user name->ID cache, DM targets by `@name` / user ID, fetch-style output size gating, and a transactional headline+detail-thread announce protocol with a documented recovery path. OFF by default. Behavior lives in `lib/slack-core.ts` and `lib/slack-cache.ts`. |
74
74
 
75
75
  Full routing rules, size-gate mechanics, and config: [doc/fetch.md](doc/fetch.md), [doc/doc-to-md.md](doc/doc-to-md.md), [doc/slack.md](doc/slack.md).
76
76
 
@@ -280,7 +280,7 @@ Operational notes:
280
280
  | `botTokenEnv` | `SLACK_BOT_TOKEN` | Env var name holding the bot token. |
281
281
  | `uploadThresholdChars` | `4000` | Link-collapsed length above which an announce/thread detail body is delivered as a file upload instead of inline text. |
282
282
 
283
- Each setting can also be overridden per-process via `PI_QUIVER_SLACK_ENABLED`, `PI_QUIVER_SLACK_CACHE_PATH`, `PI_QUIVER_SLACK_USER_TOKEN_ENV`, `PI_QUIVER_SLACK_BOT_TOKEN_ENV`, and `PI_QUIVER_SLACK_UPLOAD_THRESHOLD_CHARS` - applied on top of the resolved `settings.json` layers, same override rung the extension's config resolver defines. `userTokenCommand` and `userTokenCommandTimeoutSeconds` are settings-only: pi executes the argv directly, without a shell, on every user-identity Slack tool call. For example, macOS Keychain can supply the token with `"userTokenCommand": ["security", "find-generic-password", "-s", "slack-user-token", "-w"]`. Its stdout is the token; empty output, nonzero exit, or timeout is a sanitized hard error and never falls back to `userTokenEnv`. Without the command, tokens resolve per call from process env and then `.env` (or the primary checkout's, for a worktree with none). Bot resolution is unchanged. Restart pi after changing Slack settings because the extension captures them at session start. Full reference incl. cache layering, the announce protocol, and the `search.messages`/`conversations.replies` throttle caveats: [doc/slack.md](doc/slack.md).
283
+ Each setting can also be overridden per-process via `PI_QUIVER_SLACK_ENABLED`, `PI_QUIVER_SLACK_CACHE_PATH`, `PI_QUIVER_SLACK_USER_TOKEN_ENV`, `PI_QUIVER_SLACK_BOT_TOKEN_ENV`, and `PI_QUIVER_SLACK_UPLOAD_THRESHOLD_CHARS` - applied on top of the resolved `settings.json` layers, same override rung the extension's config resolver defines. `userTokenCommand` and `userTokenCommandTimeoutSeconds` are settings-only: pi executes the argv directly, without a shell, on every user-identity Slack tool call. For example, macOS Keychain can supply the token with `"userTokenCommand": ["security", "find-generic-password", "-s", "slack-user-token", "-w"]`. Its stdout is the token; empty output, nonzero exit, or timeout is a sanitized hard error and never falls back to `userTokenEnv`. Without the command, tokens resolve per call from process env, then the repo `.env`, then the primary checkout's `.env` (linked worktrees), then a per-user `~/.config/pi-quiver/.env` (`$XDG_CONFIG_HOME` / `%APPDATA%` aware); a file lacking the key falls through to the next rung - see [doc/slack.md](doc/slack.md). Bot resolution is unchanged. Restart pi after changing Slack settings because the extension captures them at session start. Full reference incl. cache layering, the announce protocol, and the `search.messages`/`conversations.replies` throttle caveats: [doc/slack.md](doc/slack.md).
284
284
 
285
285
  ### doc_to_md settings
286
286
 
@@ -269,9 +269,9 @@ export default function slackExtension(pi: ExtensionAPI) {
269
269
  label: "Slack Thread",
270
270
  promptSnippet: "Read all replies in a Slack thread",
271
271
  description:
272
- 'Read a Slack thread via conversations.replies. Always uses the "user" identity (no `as` param). Provide either `channel` (#name or channel ID; user @names not accepted) plus `ts`, or a `permalink` (parsed for channel+ts). Paginates by cursor until Slack reports no more replies or a cap of 50 pages / 5,000 messages is hit; the result carries a `complete` flag and a resumable `next_cursor` when capped. Caveat: since 2025-05-29, conversations.replies is rate-limited to ~1 request/minute (limit capped at 15) for apps that are neither Marketplace-listed nor classified internal - hitting that throttle mid-pagination returns the messages collected so far plus a caveat and a resumable cursor instead of spinning. Output is size-gated like slack_search.',
272
+ 'Read a Slack thread via conversations.replies. Always uses the "user" identity (no `as` param). Provide either `channel` (#name, channel ID, @name, or user ID (DM)) plus `ts`, or a `permalink` (parsed for channel+ts). Paginates by cursor until Slack reports no more replies or a cap of 50 pages / 5,000 messages is hit; the result carries a `complete` flag and a resumable `next_cursor` when capped. Caveat: since 2025-05-29, conversations.replies is rate-limited to ~1 request/minute (limit capped at 15) for apps that are neither Marketplace-listed nor classified internal - hitting that throttle mid-pagination returns the messages collected so far plus a caveat and a resumable cursor instead of spinning. Output is size-gated like slack_search.',
273
273
  parameters: Type.Object({
274
- channel: Type.Optional(Type.String({ description: "#name or channel ID (user @names not accepted)" })),
274
+ channel: Type.Optional(Type.String({ description: "#name, channel ID, @name, or user ID (DM)" })),
275
275
  ts: Type.Optional(Type.String({ description: "Thread parent timestamp" })),
276
276
  permalink: Type.Optional(Type.String({ description: "A Slack message permalink URL to parse channel+ts from" })),
277
277
  cursor: Type.Optional(Type.String({ description: "Resume pagination from a next_cursor returned by a prior capped call" })),
@@ -279,7 +279,8 @@ export default function slackExtension(pi: ExtensionAPI) {
279
279
  async execute(_toolCallId, params, signal) {
280
280
  return guarded(async () => {
281
281
  const { deps, cacheCtx } = await resolveCall("user", cfg, ctx, signal, repoRoot);
282
- const channel = params.channel !== undefined ? await resolveChannel(params.channel, cacheCtx) : undefined;
282
+ const channel =
283
+ params.permalink === undefined && params.channel !== undefined ? await resolveChannel(params.channel, cacheCtx) : undefined;
283
284
  const result = await readThread({ channel, ts: params.ts, permalink: params.permalink, cursor: params.cursor }, deps);
284
285
  return {
285
286
  content: [{ type: "text" as const, text: threadResultText(result) }],
@@ -296,10 +297,10 @@ export default function slackExtension(pi: ExtensionAPI) {
296
297
  label: "Slack Post",
297
298
  promptSnippet: "Post a Slack message, reply, or headline+detail announcement",
298
299
  description:
299
- "Post a Slack message via chat.postMessage, as `as: \"user\"` or `as: \"bot\"`. `channel` accepts #name or a channel ID (user @names not accepted). Plain post: `text` and/or `blocks` (Block Kit JSON, passed through unvalidated). Threaded reply: also set `thread_ts` - no headline is ever emitted, `thread_body` (or `text`) becomes the reply body. Announce mode: set `thread_body` WITHOUT `thread_ts` - posts a short single-line `text` headline, then posts `thread_body` as the first threaded reply in the same call; if `thread_body`'s rendered length exceeds the configured uploadThresholdChars (default 4000), it is delivered as a threaded file upload instead. Recovery: re-invoke with `thread_ts` set (never re-omit it) to post only into the existing thread - a second headline is never sent. On detail-delivery failure the headline is marked \"detail pending\" and the detail is saved to a temp file; the error names the path. `unfurl_links`/`unfurl_media` apply to this post only, are omitted when unset (Slack's default stands), and slack_update cannot change unfurling after the fact.",
300
+ "Post a Slack message via chat.postMessage, as `as: \"user\"` or `as: \"bot\"`. `channel` accepts #name, channel ID, @name, or user ID (DM). Plain post: `text` and/or `blocks` (Block Kit JSON, passed through unvalidated). Threaded reply: also set `thread_ts` - no headline is ever emitted, `thread_body` (or `text`) becomes the reply body. Announce mode: set `thread_body` WITHOUT `thread_ts` - posts a short single-line `text` headline, then posts `thread_body` as the first threaded reply in the same call; if `thread_body`'s rendered length exceeds the configured uploadThresholdChars (default 4000), it is delivered as a threaded file upload instead. Recovery: re-invoke with `thread_ts` set (never re-omit it) to post only into the existing thread - a second headline is never sent. On detail-delivery failure the headline is marked \"detail pending\" and the detail is saved to a temp file; the error names the path. `unfurl_links`/`unfurl_media` apply to this post only, are omitted when unset (Slack's default stands), and slack_update cannot change unfurling after the fact.",
300
301
  parameters: Type.Object({
301
302
  as: IDENTITY,
302
- channel: Type.String({ description: "#name or channel ID (user @names not accepted)" }),
303
+ channel: Type.String({ description: "#name, channel ID, @name, or user ID (DM)" }),
303
304
  text: Type.Optional(Type.String({ description: "Message text, or the announce headline when thread_body is set" })),
304
305
  blocks: Type.Optional(Type.Array(Type.Unknown(), { description: "Block Kit JSON array, passed through unvalidated" })),
305
306
  thread_ts: Type.Optional(Type.String({ description: "Reply into this existing thread instead of posting a new headline" })),
@@ -378,10 +379,10 @@ export default function slackExtension(pi: ExtensionAPI) {
378
379
  label: "Slack Update",
379
380
  promptSnippet: "Edit an existing Slack message",
380
381
  description:
381
- 'Edit a message via chat.update, as `as: "user"` or `as: "bot"`. `channel` accepts #name or a channel ID (user @names not accepted). Only the identity that originally posted the message can edit it (Slack constraint; surfaced as an error otherwise). Accepts `text` and/or `blocks` (Block Kit JSON, unvalidated).',
382
+ 'Edit a message via chat.update, as `as: "user"` or `as: "bot"`. `channel` accepts #name, channel ID, @name, or user ID (DM). Only the identity that originally posted the message can edit it (Slack constraint; surfaced as an error otherwise). Accepts `text` and/or `blocks` (Block Kit JSON, unvalidated).',
382
383
  parameters: Type.Object({
383
384
  as: IDENTITY,
384
- channel: Type.String({ description: "#name or channel ID (user @names not accepted)" }),
385
+ channel: Type.String({ description: "#name, channel ID, @name, or user ID (DM)" }),
385
386
  ts: Type.String({ description: "Timestamp of the message to edit" }),
386
387
  text: Type.Optional(Type.String()),
387
388
  blocks: Type.Optional(Type.Array(Type.Unknown(), { description: "Block Kit JSON array, passed through unvalidated" })),
@@ -408,10 +409,10 @@ export default function slackExtension(pi: ExtensionAPI) {
408
409
  label: "Slack Delete",
409
410
  promptSnippet: "Delete a Slack message",
410
411
  description:
411
- 'Delete a message via chat.delete, as `as: "user"` or `as: "bot"`. `channel` accepts #name or a channel ID (user @names not accepted). Only the identity that originally posted the message can delete it (Slack constraint; surfaced as an error otherwise).',
412
+ 'Delete a message via chat.delete, as `as: "user"` or `as: "bot"`. `channel` accepts #name, channel ID, @name, or user ID (DM). Only the identity that originally posted the message can delete it (Slack constraint; surfaced as an error otherwise).',
412
413
  parameters: Type.Object({
413
414
  as: IDENTITY,
414
- channel: Type.String({ description: "#name or channel ID (user @names not accepted)" }),
415
+ channel: Type.String({ description: "#name, channel ID, @name, or user ID (DM)" }),
415
416
  ts: Type.String({ description: "Timestamp of the message to delete" }),
416
417
  }),
417
418
  async execute(_toolCallId, params, signal) {
@@ -434,10 +435,10 @@ export default function slackExtension(pi: ExtensionAPI) {
434
435
  label: "Slack Pin",
435
436
  promptSnippet: "Pin a Slack message to its channel",
436
437
  description:
437
- 'Pin a message via pins.add, as `as: "user"` or `as: "bot"`. `channel` accepts #name or a channel ID (user @names not accepted). Slack errors are mapped: already_pinned, not_pinnable (this message type cannot be pinned), too_many_pins (the channel hit Slack\'s pin limit).',
438
+ 'Pin a message via pins.add, as `as: "user"` or `as: "bot"`. `channel` accepts #name, channel ID, @name, or user ID (DM). Slack errors are mapped: already_pinned, not_pinnable (this message type cannot be pinned), too_many_pins (the channel hit Slack\'s pin limit).',
438
439
  parameters: Type.Object({
439
440
  as: IDENTITY,
440
- channel: Type.String({ description: "#name or channel ID (user @names not accepted)" }),
441
+ channel: Type.String({ description: "#name, channel ID, @name, or user ID (DM)" }),
441
442
  ts: Type.String({ description: "Timestamp of the message to pin" }),
442
443
  }),
443
444
  async execute(_toolCallId, params, signal) {
@@ -460,10 +461,10 @@ export default function slackExtension(pi: ExtensionAPI) {
460
461
  label: "Slack Upload",
461
462
  promptSnippet: "Upload a file to a Slack channel or thread",
462
463
  description:
463
- 'Upload a file to Slack (getUploadURLExternal -> upload -> completeUploadExternal), as `as: "user"` or `as: "bot"`. `channel` accepts #name or a channel ID (user @names not accepted). `path` is an absolute path or resolved relative to the current working directory; a missing file errors before any network call. `filename` defaults to the path\'s basename. Optional `title`, `thread_ts` (attach to an existing thread), and `initial_comment`.',
464
+ 'Upload a file to Slack (getUploadURLExternal -> upload -> completeUploadExternal), as `as: "user"` or `as: "bot"`. `channel` accepts #name, channel ID, @name, or user ID (DM). `path` is an absolute path or resolved relative to the current working directory; a missing file errors before any network call. `filename` defaults to the path\'s basename. Optional `title`, `thread_ts` (attach to an existing thread), and `initial_comment`.',
464
465
  parameters: Type.Object({
465
466
  as: IDENTITY,
466
- channel: Type.String({ description: "#name or channel ID (user @names not accepted)" }),
467
+ channel: Type.String({ description: "#name, channel ID, @name, or user ID (DM)" }),
467
468
  path: Type.String({ description: "Absolute path, or a path relative to the current working directory" }),
468
469
  filename: Type.Optional(Type.String({ description: "Defaults to the basename of path" })),
469
470
  title: Type.Optional(Type.String()),
@@ -119,13 +119,21 @@ function toUserEntry(u: SlackUser): UserEntry {
119
119
  };
120
120
  }
121
121
 
122
+ /** DM conversations are opened per identity: Slack returns the existing D... for a repeat call, so this is idempotent and safe to retry. */
123
+ async function openDm(userId: string, ctx: CacheCtx): Promise<string> {
124
+ const data = await ctx.apiCall("conversations.open", ctx.token, { users: userId }, { retry: true, signal: ctx.signal });
125
+ const channel = data.channel as { id?: unknown } | undefined;
126
+ if (typeof channel?.id !== "string") {
127
+ throw new SlackError("unexpected_response", 'conversations.open returned an unexpected response: missing "channel.id".');
128
+ }
129
+ return channel.id;
130
+ }
131
+
122
132
  export async function resolveChannel(input: string, ctx: CacheCtx): Promise<string> {
123
133
  if (RAW_CHANNEL_ID.test(input)) return input;
124
- if (input.startsWith("@")) {
125
- throw new SlackError(
126
- "invalid_channel",
127
- `"${input}" looks like a user name, which is not accepted in a channel position (opening a DM is out of scope).`,
128
- );
134
+ if (RAW_USER_ID.test(input) || input.startsWith("@")) {
135
+ const userId = await resolveUser(input, ctx);
136
+ return openDm(userId, ctx);
129
137
  }
130
138
  const name = stripPrefix(input);
131
139
 
@@ -189,22 +197,26 @@ export async function resolveUser(input: string, ctx: CacheCtx): Promise<string>
189
197
  const byUsername = cached.users[name];
190
198
  if (byUsername) return byUsername.id;
191
199
 
192
- const displayMatches = Object.values(cached.users).filter((u) => u.display_name === name);
193
- if (displayMatches.length === 1) return displayMatches[0].id;
194
- if (displayMatches.length > 1) {
195
- throw new SlackError(
196
- "ambiguous_user",
197
- `Multiple users have display name "${name}": ${displayMatches.map((u) => u.id).join(", ")}.`,
198
- );
199
- }
200
+ // Same gate as resolveMentions' aliasTrusted: an alias match is conclusive only against a
201
+ // complete workspace snapshot; a partial cache may hold a different person with that alias.
202
+ if (cached.snapshot_at !== undefined) {
203
+ const displayMatches = Object.values(cached.users).filter((u) => u.display_name === name);
204
+ if (displayMatches.length === 1) return displayMatches[0].id;
205
+ if (displayMatches.length > 1) {
206
+ throw new SlackError(
207
+ "ambiguous_user",
208
+ `Multiple users have display name "${name}": ${displayMatches.map((u) => u.id).join(", ")}.`,
209
+ );
210
+ }
200
211
 
201
- const realMatches = Object.values(cached.users).filter((u) => u.real_name === name);
202
- if (realMatches.length === 1) return realMatches[0].id;
203
- if (realMatches.length > 1) {
204
- throw new SlackError(
205
- "ambiguous_user",
206
- `Multiple users have real name "${name}": ${realMatches.map((u) => u.id).join(", ")}.`,
207
- );
212
+ const realMatches = Object.values(cached.users).filter((u) => u.real_name === name);
213
+ if (realMatches.length === 1) return realMatches[0].id;
214
+ if (realMatches.length > 1) {
215
+ throw new SlackError(
216
+ "ambiguous_user",
217
+ `Multiple users have real name "${name}": ${realMatches.map((u) => u.id).join(", ")}.`,
218
+ );
219
+ }
208
220
  }
209
221
  }
210
222
 
package/lib/slack-core.ts CHANGED
@@ -191,14 +191,25 @@ export function parseEnvFile(content: string): Map<string, string> {
191
191
  return result;
192
192
  }
193
193
 
194
- function readEnvFile(dir: string): Map<string, string> | undefined {
194
+ export function userConfigEnvPath(
195
+ env: Record<string, string | undefined>,
196
+ platform: NodeJS.Platform = process.platform,
197
+ ): string | undefined {
198
+ if (platform === "win32") {
199
+ return env.APPDATA ? join(env.APPDATA, "pi-quiver", ".env") : undefined;
200
+ }
201
+ if (env.XDG_CONFIG_HOME) return join(env.XDG_CONFIG_HOME, "pi-quiver", ".env");
202
+ if (env.HOME) return join(env.HOME, ".config", "pi-quiver", ".env");
203
+ return undefined;
204
+ }
205
+
206
+ function readEnvFile(path: string): Map<string, string> | undefined {
195
207
  try {
196
- return parseEnvFile(readFileSync(join(dir, ".env"), "utf8"));
208
+ return parseEnvFile(readFileSync(path, "utf8"));
197
209
  } catch (err) {
198
- // Deliberate: a genuinely absent .env degrades to "token not found", not a thrown error.
210
+ // Deliberate: a genuinely absent .env degrades to "token not found". Anything else (permissions,
211
+ // a directory at that path) is a misconfiguration the caller must see, so it propagates raw.
199
212
  if ((err as NodeJS.ErrnoException)?.code === "ENOENT") return undefined;
200
- // Present but unreadable (e.g. permissions): re-thrown so callers treat it as present-but-empty,
201
- // not absent - this must NOT authorize the primary-checkout fallback.
202
213
  throw err;
203
214
  }
204
215
  }
@@ -208,45 +219,29 @@ export function resolveToken(
208
219
  cfg: SlackConfig,
209
220
  env: Record<string, string | undefined>,
210
221
  repoRoot: string,
222
+ platform: NodeJS.Platform = process.platform,
211
223
  ): string {
212
224
  const envVar = identity === "user" ? cfg.userTokenEnv : cfg.botTokenEnv;
213
225
 
214
226
  const fromEnv = env[envVar];
215
227
  if (fromEnv) return fromEnv;
216
228
 
217
- let repoEnvFile: Map<string, string> | undefined;
218
- let repoEnvUnreadable = false;
219
- try {
220
- repoEnvFile = readEnvFile(repoRoot);
221
- } catch {
222
- repoEnvUnreadable = true;
223
- }
229
+ const candidates = [join(repoRoot, ".env")];
230
+ const primaryRoot = primaryCheckoutRoot(repoRoot);
231
+ if (primaryRoot !== undefined && primaryRoot !== repoRoot) candidates.push(join(primaryRoot, ".env"));
232
+ const userPath = userConfigEnvPath(env, platform);
233
+ if (userPath !== undefined) candidates.push(userPath);
224
234
 
225
- if (repoEnvFile) {
226
- // An empty (after quote-strip) .env value is treated as missing, not a usable empty token.
227
- const value = repoEnvFile.get(envVar);
235
+ for (const path of candidates) {
236
+ const parsed = readEnvFile(path);
237
+ // An empty (after quote-strip) value is treated as missing, not a usable empty token.
238
+ const value = parsed?.get(envVar);
228
239
  if (value) return value;
229
- } else if (!repoEnvUnreadable) {
230
- // File-level fallback only, per spec: this only triggers when the worktree root has no .env at
231
- // all (ENOENT); an existing-but-unreadable local .env blocks the fallback just like an existing
232
- // one that lacks this key.
233
- const primaryRoot = primaryCheckoutRoot(repoRoot);
234
- if (primaryRoot && primaryRoot !== repoRoot) {
235
- // Best-effort: any error reading the primary .env (missing or otherwise) just means no fallback.
236
- let primaryEnvFile: Map<string, string> | undefined;
237
- try {
238
- primaryEnvFile = readEnvFile(primaryRoot);
239
- } catch {
240
- primaryEnvFile = undefined;
241
- }
242
- const value = primaryEnvFile?.get(envVar);
243
- if (value) return value;
244
- }
245
240
  }
246
241
 
247
242
  throw new SlackError(
248
243
  "missing_token",
249
- `No Slack ${identity} token: env var ${envVar} is empty and no .env entry was found.`,
244
+ `No Slack ${identity} token: env var ${envVar} is empty and no entry found in ${candidates.join(", ")}.`,
250
245
  );
251
246
  }
252
247
 
@@ -279,11 +274,12 @@ export async function resolveCredential(
279
274
  cfg: SlackConfig,
280
275
  env: Record<string, string | undefined>,
281
276
  repoRoot: string,
277
+ platform: NodeJS.Platform = process.platform,
282
278
  ): Promise<string> {
283
279
  if (identity === "user" && cfg.userTokenCommand) {
284
280
  return runCredentialCommand(cfg.userTokenCommand, Math.ceil(cfg.userTokenCommandTimeoutSeconds * 1000));
285
281
  }
286
- return resolveToken(identity, cfg, env, repoRoot);
282
+ return resolveToken(identity, cfg, env, repoRoot, platform);
287
283
  }
288
284
 
289
285
  /**
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "pi-quiver",
3
- "version": "6.0.1",
3
+ "version": "6.1.0",
4
4
  "description": "Personal pack of Pi coding-agent extensions: context-safe fetch, doc_to_md PDF/DOCX/PPTX-to-Markdown conversion, session naming, a themed ASCII startup header, Opus 4.8 fast mode, and a provider-stall watchdog.",
5
5
  "author": "Jacek Juraszek",
6
6
  "license": "MIT",