@pinet/slack-bridge 0.2.2 → 0.2.6

package/dist/prompts/broker/default.md

@@ -10,9 +10,9 @@ IF ASKED TO CODE: Refuse politely and immediately delegate. Say: "I'm the broker
 
 ALLOWED — These are your responsibilities: (1) Route messages between humans and agents. (2) Check `pinet action=agents` for idle workers and delegate tasks via `pinet action=send`. (3) Coordinate GitHub issues/PRs and request reviews, but do NOT launch local review subagents from the broker. (4) Monitor agent health via the RALPH loop. (5) Relay status updates, answer questions about system state, and coordinate workflows. (6) Use bash for read-only inspection and lightweight GitHub coordination: git log, git status, gh pr list, gh pr view, ls, cat — never for code changes or implementation work. (7) Use tmux only to launch repo-scoped Pinet follower workers when capacity is missing.
 
-PRIORITIZED ISSUE GATE: Do not start, assign, or broadcast extension changes unless the request names a GitHub issue/PR and carries clear maintainer priority/approval. Do not self-start from open issue lists. If unclear, stop and ask.
+GUGU91/PINET PRIORITIZED ISSUE GATE: For work in `gugu91/pinet`, do not start, assign, or broadcast changes unless the request names a GitHub issue/PR and carries clear maintainer priority/approval. Do not self-start from that repository's open issue lists. If unclear, stop and ask. This gate is specific to `gugu91/pinet`; unrelated repositories, including `tmustier/pi-extensions`, follow their own instructions and normal requester authority.
 
-DELEGATE, THEN TRACK: Do not perform task triage yourself beyond checking explicit routing requirements already present in the request: repo, issue/PR number, maintainer approval, branch/worktree, and worker availability. If required routing facts are missing, ask; otherwise assign promptly.
+DELEGATE, THEN TRACK: Do not perform task triage yourself beyond checking explicit routing requirements already present in the request: repo, and, where that repo's instructions require them, issue/PR number and maintainer approval, plus branch/worktree and worker availability. If required routing facts are missing, ask; otherwise assign promptly.
 
 PM MODE AWARENESS: For complex or coordination-heavy maintainer-delegated tasks, offer PM mode instead of assuming it. PM mode is consent-gated: ask/confirm before enabling unless the maintainer explicitly requested PM-style coordination. When enabled, assign an accountable follower as PM/coordinator; that follower nominates an implementation lead, delegates implementation, coordinates blockers/status, performs second-pass review, and coordinates merge readiness. The broker remains coordination-only and must not implement or launch local subagents. Use durable lane metadata (`pinet action=lanes`) so PM-mode role/lane state is inspectable by RALPH/status flows. A `detached` lane means human/manual supervision; keep it visible, but do not auto-reassign it as normal broker-managed work without explicit human/broker action.
 
@@ -26,7 +26,7 @@ When a human asks for work to be done, ALWAYS check `pinet action=agents` for id
 
 If a repo instruction says to use the `code-reviewer` subagent, treat that as work for the owning connected worker in the same repo to run locally and summarize back — never the broker itself.
 
-When delegating, include: task, issue/PR numbers, maintainer priority/approval, repo/branch/worktree setup, and where to report back (Slack thread_ts).
+When delegating, include: task, repo/branch/worktree setup, where to report back (Slack thread_ts), and any issue/PR numbers or maintainer priority/approval required by the target repository's instructions.
 
 If no repo-matched workers are available and new capacity is needed, you may spin up a worker as broker infrastructure: create a tmux session in the target repo, launch `pi` with `PINET_BROKER_MANAGED=1 PINET_BROKER_AGENT_ID=<current-broker-agent-id> PINET_LAUNCH_SOURCE=broker-tmux PINET_TMUX_SESSION=<session>`, run `/pinet follow`, wait for the worker in `pinet action=agents`, then delegate via `pinet action=send`. These env vars let the broker safely distinguish broker-managed follower PIDs from unrelated local processes before ghost reaping. NEVER do the work yourself or cross-route to another repo as a fallback.
 

package/dist/prompts/broker/tmux.md

@@ -10,9 +10,9 @@ IF ASKED TO CODE: Refuse politely and immediately delegate. Say: "I'm the broker
 
 ALLOWED — These are your responsibilities: (1) Route messages between humans and agents. (2) Check `pinet action=agents` for idle workers and delegate tasks via `pinet action=send`. (3) Coordinate GitHub issues/PRs and request reviews, but do NOT launch local review subagents from the broker. (4) Monitor agent health via the RALPH loop. (5) Relay status updates, answer questions about system state, and coordinate workflows. (6) Use bash for read-only inspection and lightweight GitHub coordination: git log, git status, gh pr list, gh pr view, ls, cat — never for code changes or implementation work. (7) Use tmux only for broker-managed follower lifecycle operations: launch repo-scoped workers, inspect recorded broker-managed sessions, and close safely mapped sessions after their workers have exited; never manipulate unrelated or unmapped tmux sessions.
 
-PRIORITIZED ISSUE GATE: Do not start, assign, or broadcast extension changes unless the request names a GitHub issue/PR and carries clear maintainer priority/approval. Do not self-start from open issue lists. If unclear, stop and ask.
+GUGU91/PINET PRIORITIZED ISSUE GATE: For work in `gugu91/pinet`, do not start, assign, or broadcast changes unless the request names a GitHub issue/PR and carries clear maintainer priority/approval. Do not self-start from that repository's open issue lists. If unclear, stop and ask. This gate is specific to `gugu91/pinet`; unrelated repositories, including `tmustier/pi-extensions`, follow their own instructions and normal requester authority.
 
-DELEGATE, THEN TRACK: Do not perform task triage yourself beyond checking explicit routing requirements already present in the request: repo, issue/PR number, maintainer approval, branch/worktree, and worker availability. If required routing facts are missing, ask; otherwise assign promptly.
+DELEGATE, THEN TRACK: Do not perform task triage yourself beyond checking explicit routing requirements already present in the request: repo, and, where that repo's instructions require them, issue/PR number and maintainer approval, plus branch/worktree and worker availability. If required routing facts are missing, ask; otherwise assign promptly.
 
 PM MODE AWARENESS: For complex or coordination-heavy maintainer-delegated tasks, offer PM mode instead of assuming it. PM mode is consent-gated: ask/confirm before enabling unless the maintainer explicitly requested PM-style coordination. When enabled, assign an accountable follower as PM/coordinator; that follower nominates an implementation lead, delegates implementation, coordinates blockers/status, performs second-pass review, and coordinates merge readiness. The broker remains coordination-only and must not implement or launch local subagents. Use durable lane metadata (`pinet action=lanes`) so PM-mode role/lane state is inspectable by RALPH/status flows. A `detached` lane means human/manual supervision; keep it visible, but do not auto-reassign it as normal broker-managed work without explicit human/broker action.
 
@@ -34,7 +34,7 @@ When a human asks for work to be done, ALWAYS check `pinet action=agents` for id
 
 If a repo instruction says to use the `code-reviewer` subagent, treat that as work for the owning connected worker in the same repo to run locally and summarize back — never the broker itself.
 
-When delegating, include: task, issue/PR numbers, maintainer priority/approval, repo/branch/worktree setup, acceptance criteria, relevant files or docs, and where to report back (Slack thread_ts or Pinet thread).
+When delegating, include: task, repo/branch/worktree setup, acceptance criteria, relevant files or docs, where to report back (Slack thread_ts or Pinet thread), and any issue/PR numbers or maintainer priority/approval required by the target repository's instructions.
 
 If no repo-matched workers are available and new capacity is needed, you may spin up a worker as broker infrastructure: create a tmux session in the target repo, launch `pi` with `PINET_BROKER_MANAGED=1 PINET_BROKER_AGENT_ID=<current-broker-agent-id> PINET_LAUNCH_SOURCE=broker-tmux PINET_TMUX_SESSION=<session>`, run `/pinet follow`, wait for the worker in `pinet action=agents`, then delegate via `pinet action=send`. NEVER do the work yourself or cross-route to another repo as a fallback.
 

package/dist/runtime-agent-context.d.ts

@@ -56,9 +56,9 @@ export interface RuntimeAgentContextDeps {
     getExtensionContext: () => ExtensionContext | null;
     persistState: () => void;
     updateBadge: () => void;
-    getGitContext: () => Promise<GitContext>;
+    getGitContext: (signal?: AbortSignal) => Promise<GitContext>;
     /** Optional test hook for live branch/dirty probing. */
-    getDynamicGitState?: (cwd: string) => Promise<GitDynamicState>;
+    getDynamicGitState?: (cwd: string, signal?: AbortSignal) => Promise<GitDynamicState>;
 }
 export interface RuntimeAgentContext {
     isUserAllowed: (userId: string) => boolean;
@@ -79,7 +79,7 @@ export interface RuntimeAgentContext {
     snapshotReloadableRuntime: () => ReloadableRuntimeSnapshot;
     restoreReloadableRuntime: (snapshot: ReloadableRuntimeSnapshot) => void;
     detectProjectTools: (repoRoot: string, cwd: string) => string[];
-    getAgentMetadata: (role?: RuntimeAgentRole) => Promise<Record<string, unknown>>;
+    getAgentMetadata: (role?: RuntimeAgentRole, signal?: AbortSignal) => Promise<Record<string, unknown>>;
     asStringValue: (value: unknown) => string | undefined;
     getMeshRoleFromMetadata: (metadata: Record<string, unknown> | undefined, fallback?: RuntimeAgentRole) => RuntimeAgentRole;
     buildSkinMetadata: (metadata: Record<string, unknown> | undefined, personality: string, statusVocabulary?: PinetSkinStatusVocabulary) => Record<string, unknown>;

package/dist/runtime-agent-context.js

@@ -185,18 +185,20 @@ export function createRuntimeAgentContext(deps) {
         tools.add("git");
         return [...tools].sort();
     }
-    const dynamicGitProbe = deps.getDynamicGitState ?? ((cwd) => probeGitDynamic(cwd));
-    async function getAgentMetadata(role = "worker") {
-        const gitContext = await deps.getGitContext();
+    const dynamicGitProbe = deps.getDynamicGitState ??
+        ((cwd, signal) => probeGitDynamic(cwd, undefined, signal));
+    async function getAgentMetadata(role = "worker", signal) {
+        const gitContext = await deps.getGitContext(signal);
         const { cwd, repo, repoRoot } = gitContext;
         const resolvedRepoRoot = repoRoot ?? cwd;
         const tools = detectProjectTools(resolvedRepoRoot, cwd);
         const scope = buildSlackCompatibilityScope();
         let dynamic;
         try {
-            dynamic = await dynamicGitProbe(cwd);
+            dynamic = await dynamicGitProbe(cwd, signal);
         }
         catch {
+            signal?.throwIfAborted();
             dynamic = { probeFailed: true };
         }
         const branch = dynamic.branch;
@@ -215,12 +217,16 @@ export function createRuntimeAgentContext(deps) {
         const skinAssignment = resolveSkinAssignment(role, getIdentitySeedForRole(role));
         const brokerManaged = role === "worker" && process.env.PINET_BROKER_MANAGED === "1";
         const parentAgentId = process.env.PINET_PARENT_AGENT_ID?.trim() || undefined;
+        const launchSource = process.env.PINET_LAUNCH_SOURCE?.trim() || "broker-tmux";
+        const herdrLaunch = launchSource === "broker-herdr" || launchSource === "subtree-broker-herdr";
+        const tmuxSession = process.env.PINET_TMUX_SESSION?.trim() || undefined;
         const brokerManagedMetadata = brokerManaged
             ? {
                 brokerManaged: true,
                 brokerManagedBy: process.env.PINET_BROKER_AGENT_ID?.trim() || undefined,
-                launchSource: process.env.PINET_LAUNCH_SOURCE?.trim() || "broker-tmux",
-                tmuxSession: process.env.PINET_TMUX_SESSION?.trim() || undefined,
+                launchSource,
+                runtimeKind: herdrLaunch ? "herdr" : "tmux",
+                ...(!herdrLaunch && tmuxSession ? { runtimeLocator: tmuxSession, tmuxSession } : {}),
                 brokerManagedAt: new Date().toISOString(),
                 ...(parentAgentId ? { parentAgentId, pinetParentAgentId: parentAgentId } : {}),
                 ...(process.env.PINET_ROOT_AGENT_ID?.trim()

package/dist/runtime-mode.js

@@ -16,8 +16,12 @@ export function isBrokerManagedFollowerLaunch(env = process.env) {
         return false;
     if (env.PINET_LAUNCH_SOURCE === "broker-tmux")
         return true;
+    if (env.PINET_LAUNCH_SOURCE === "broker-herdr")
+        return true;
     if (env.PINET_LAUNCH_SOURCE === "subtree-broker-tmux")
         return true;
+    if (env.PINET_LAUNCH_SOURCE === "subtree-broker-herdr")
+        return true;
     return Boolean(env.PINET_SOCKET_PATH?.trim());
 }
 export function resolveSlackBridgeStartupRuntimeMode(settings, options = {}) {

package/dist/session-ui-runtime.js

@@ -43,6 +43,7 @@ export function createSessionUiRuntime(deps) {
     let terminalInputUnsubscribe = null;
     let deferredDrainTimer = null;
     let extCtx = null;
+    let statusState = "ok";
     function getExtensionContext() {
         return extCtx;
     }
@@ -50,7 +51,7 @@ export function createSessionUiRuntime(deps) {
         extCtx = ctx;
     }
     function updateBadge() {
-        if (!extCtx?.hasUI)
+        if (statusState !== "ok" || !extCtx?.hasUI)
             return;
         const t = extCtx.ui.theme;
         const n = deps.getInboxLength();
@@ -60,6 +61,7 @@ export function createSessionUiRuntime(deps) {
         extCtx.ui.setStatus("slack-bridge", label);
     }
     function setExtStatus(ctx, state) {
+        statusState = state;
         if (!ctx.hasUI)
             return;
         extCtx = ctx;

package/dist/single-player-runtime.d.ts

@@ -59,7 +59,7 @@ export interface SinglePlayerRuntimeDeps {
     beginThreadStatus?: (channelId: string, threadTs: string, status: string) => Promise<void>;
 }
 export interface SinglePlayerRuntime {
-    connect: (ctx: ExtensionContext) => Promise<void>;
+    connect: (ctx: ExtensionContext, signal?: AbortSignal) => Promise<void>;
     disconnect: () => Promise<void>;
     getBotUserId: () => string | null;
     isConnected: () => boolean;

package/dist/single-player-runtime.js

@@ -356,7 +356,8 @@ export function createSinglePlayerRuntime(deps) {
         },
     };
     return {
-        async connect(ctx) {
+        async connect(ctx, signal) {
+            signal?.throwIfAborted();
             shuttingDown = false;
             const socket = new SlackSocketModeClient({
                 slack: deps.slack,
@@ -371,9 +372,14 @@ export function createSinglePlayerRuntime(deps) {
                         deps.setExtStatus(ctx, "reconnecting");
                     }
                 },
-                onError: (error) => {
-                    if (!isAbortError(error)) {
-                        console.error(`[slack-bridge] Slack access: ${deps.formatError(error)}`);
+                onError: (error, source) => {
+                    if (isAbortError(error))
+                        return;
+                    if (source === "connection") {
+                        deps.setExtStatus(ctx, "error");
+                    }
+                    else {
+                        ctx.ui.notify(`Slack event failed: ${deps.formatError(error)}`, "error");
                     }
                 },
                 onThreadStarted: (event) => onThreadStarted(event),
@@ -398,7 +404,17 @@ export function createSinglePlayerRuntime(deps) {
                 onInteractive: (event) => queueInteractiveInboxEvent(event, ctx),
             });
             slackSocket = socket;
-            await socket.connect();
+            const abortConnect = () => {
+                void socket.disconnect().catch(() => undefined);
+            };
+            signal?.addEventListener("abort", abortConnect, { once: true });
+            try {
+                await socket.connect();
+                signal?.throwIfAborted();
+            }
+            finally {
+                signal?.removeEventListener("abort", abortConnect);
+            }
         },
         async disconnect() {
             shuttingDown = true;

package/dist/slack-access.d.ts

@@ -1,4 +1,5 @@
 import type { RuntimeScopeCarrier } from "@pinet/transport-core";
+import { type SlackMessageFileMetadata } from "./slack-message-context.js";
 import { type SlackInteractiveInboxEvent } from "./slack-block-kit.js";
 import { type ThreadOwnerHint } from "./broker/router.js";
 export { SLACK_SOCKET_DELIVERY_DEDUP_MAX_SIZE, SLACK_SOCKET_DELIVERY_DEDUP_TTL_MS, } from "./slack-socket-dedup.js";
@@ -70,6 +71,10 @@ export declare function extractAppHomeOpened(evt: Record<string, unknown>): Pars
  * Classification result for an incoming message event.
  * Uses a discriminated union so TypeScript narrows fields when relevant is true.
  */
+export interface SlackMessageClassificationMetadata extends Record<string, unknown> {
+    slackSubtype?: string;
+    slackFiles?: SlackMessageFileMetadata[];
+}
 export type MessageClassification = {
     relevant: false;
 } | {
@@ -81,7 +86,7 @@ export type MessageClassification = {
     isDM: boolean;
     isChannelMention: boolean;
     messageTs: string;
-    metadata?: Record<string, unknown>;
+    metadata?: SlackMessageClassificationMetadata;
 };
 /**
  * Classify an incoming Slack message event. Determines whether the
@@ -157,6 +162,7 @@ export interface ResolveSlackThreadOwnerHintInput {
 }
 export declare function resolveSlackThreadOwnerHint(input: ResolveSlackThreadOwnerHintInput): Promise<ThreadOwnerHint | null>;
 export declare const RECONNECT_DELAY_MS = 5000;
+export type SlackSocketErrorSource = "connection" | "event";
 export interface SlackSocketModeClientConfig {
     slack: SlackCall;
     botToken: string;
@@ -167,7 +173,7 @@ export interface SlackSocketModeClientConfig {
     abortAndWait?: () => Promise<void>;
     onOpen?: () => void;
     onReconnectScheduled?: () => void;
-    onError?: (error: unknown) => void;
+    onError?: (error: unknown, source: SlackSocketErrorSource) => void;
     onThreadStarted?: (event: ParsedThreadStarted) => Promise<void> | void;
     onThreadContextChanged?: (event: ParsedThreadContextChanged) => Promise<void> | void;
     onMessage?: (event: Record<string, unknown>) => Promise<void> | void;
@@ -184,7 +190,9 @@ export declare class SlackSocketModeClient {
     private readonly config;
     private botUserId;
     private ws;
+    private pendingSocket;
     private reconnectTimer;
+    private connecting;
     private shuttingDown;
     constructor(config: SlackSocketModeClientConfig);
     getBotUserId(): string | null;

package/dist/slack-access.js

@@ -330,7 +330,9 @@ export class SlackSocketModeClient {
     config;
     botUserId = null;
     ws = null;
+    pendingSocket = null;
     reconnectTimer = null;
+    connecting = false;
     shuttingDown = false;
     constructor(config) {
         this.config = config;
@@ -343,11 +345,22 @@ export class SlackSocketModeClient {
     }
     async connect() {
         this.shuttingDown = false;
-        if (this.config.resolveBotUserIdOnConnect ?? true) {
-            const auth = await this.config.slack("auth.test", this.config.botToken);
-            this.botUserId = typeof auth.user_id === "string" ? auth.user_id : null;
+        const authPromise = (this.config.resolveBotUserIdOnConnect ?? true)
+            ? this.config.slack("auth.test", this.config.botToken)
+            : Promise.resolve(null);
+        const socketResponsePromise = this.config.slack("apps.connections.open", this.config.appToken);
+        void socketResponsePromise.catch(() => undefined);
+        try {
+            const auth = await authPromise;
+            if (auth) {
+                this.botUserId = typeof auth.user_id === "string" ? auth.user_id : null;
+            }
+            await this.connectSocketMode(socketResponsePromise);
+        }
+        catch (error) {
+            await this.disconnect().catch(() => undefined);
+            throw error;
         }
-        await this.connectSocketMode();
     }
     async disconnect() {
         this.shuttingDown = true;
@@ -357,51 +370,78 @@ export class SlackSocketModeClient {
         }
         try {
             this.ws?.close();
+            this.pendingSocket?.close();
         }
         catch {
             /* ignore close errors */
         }
         this.ws = null;
+        this.pendingSocket = null;
         await this.config.abortAndWait?.();
     }
-    async connectSocketMode() {
-        if (this.shuttingDown)
+    async connectSocketMode(pendingResponse) {
+        if (this.shuttingDown || this.connecting)
             return;
+        this.connecting = true;
+        let shouldReconnect = false;
         try {
-            const response = await this.config.slack("apps.connections.open", this.config.appToken);
-            this.ws = new WebSocket(response.url);
-            this.ws.addEventListener("open", () => {
+            const response = await (pendingResponse ??
+                this.config.slack("apps.connections.open", this.config.appToken));
+            if (this.shuttingDown)
+                return;
+            const previous = this.ws;
+            const socket = new WebSocket(response.url);
+            this.pendingSocket = socket;
+            socket.addEventListener("open", () => {
+                if (this.pendingSocket !== socket || this.shuttingDown)
+                    return;
+                this.pendingSocket = null;
+                this.ws = socket;
+                previous?.close();
                 this.config.onOpen?.();
             });
-            this.ws.addEventListener("message", (event) => {
-                void this.handleFrame(String(event.data)).catch((error) => {
-                    this.config.onError?.(error);
+            socket.addEventListener("message", (event) => {
+                void this.handleFrame(socket, String(event.data)).catch((error) => {
+                    if (!this.shuttingDown) {
+                        this.config.onError?.(error, "event");
+                    }
                 });
             });
-            this.ws.addEventListener("close", () => {
-                if (!this.shuttingDown) {
+            socket.addEventListener("close", () => {
+                if (this.pendingSocket === socket) {
+                    this.pendingSocket = null;
+                    this.scheduleReconnect();
+                }
+                else if (this.ws === socket) {
+                    this.ws = null;
                     this.scheduleReconnect();
                 }
             });
-            this.ws.addEventListener("error", () => {
+            socket.addEventListener("error", () => {
                 /* close fires after error — handled there */
             });
         }
         catch (error) {
-            if (!isAbortError(error)) {
-                this.config.onError?.(error);
+            if (!this.shuttingDown && !isAbortError(error)) {
+                this.config.onError?.(error, "connection");
             }
+            shouldReconnect = !this.shuttingDown;
+        }
+        finally {
+            this.connecting = false;
+        }
+        if (shouldReconnect) {
             this.scheduleReconnect();
         }
     }
-    async handleFrame(raw) {
+    async handleFrame(socket, raw) {
         if (this.shuttingDown)
             return;
         const envelope = parseSocketFrame(raw);
         if (!envelope)
             return;
-        if (envelope.envelopeId) {
-            this.ws?.send(JSON.stringify({ envelope_id: envelope.envelopeId }));
+        if (envelope.envelopeId && socket.readyState === WebSocket.OPEN) {
+            socket.send(JSON.stringify({ envelope_id: envelope.envelopeId }));
         }
         const dedupKey = envelope.dedupKey ?? null;
         try {
@@ -412,7 +452,9 @@ export class SlackSocketModeClient {
                 this.config.dedup?.add(dedupKey);
             }
             if (envelope.type === "disconnect") {
-                this.scheduleReconnect();
+                if (this.ws === socket) {
+                    this.scheduleReconnect();
+                }
                 return;
             }
             if (envelope.interactivePayload) {
@@ -480,7 +522,7 @@ export class SlackSocketModeClient {
         }
     }
     scheduleReconnect() {
-        if (this.shuttingDown || this.reconnectTimer)
+        if (this.shuttingDown || this.connecting || this.pendingSocket || this.reconnectTimer)
             return;
         this.config.onReconnectScheduled?.();
         this.reconnectTimer = setTimeout(() => {

package/dist/slack-block-kit.d.ts

@@ -1,3 +1,4 @@
+import { type DecodedSlackModalPrivateMetadata } from "./slack-modals.js";
 export type SlackBlock = Record<string, unknown>;
 export interface SlackNormalizedBlockAction {
     actionId: string;
@@ -10,13 +11,54 @@ export interface SlackNormalizedBlockAction {
     parsedValue?: unknown;
     actionTs?: string;
 }
+export interface SlackSanitizedBlockAction extends Record<string, unknown> {
+    actionId: string;
+    blockId: string | null;
+    text: string | null;
+    type: string | null;
+    style: string | null;
+    value: string | null;
+    parsedValue: SlackNormalizedBlockAction["parsedValue"];
+    actionTs: string | null;
+}
+export interface SlackBlockActionMetadata extends Record<string, unknown> {
+    kind: "slack_block_action";
+    triggerId: string | null;
+    viewId: string | null;
+    callbackId: string | null;
+    viewHash: string | null;
+    blockId: string | null;
+    actionId: string;
+    value: string | null;
+    parsedValue: SlackNormalizedBlockAction["parsedValue"];
+    actionText: string | null;
+    channel: string;
+    threadTs: string;
+    messageTs: string | null;
+    modalPrivateMetadata: DecodedSlackModalPrivateMetadata["value"];
+    actions: SlackSanitizedBlockAction[];
+}
+export type SlackViewSubmissionStateValue = SlackBlock;
+export type SlackViewSubmissionStateValues = Record<string, Record<string, SlackViewSubmissionStateValue>>;
+export interface SlackViewSubmissionMetadata extends Record<string, unknown> {
+    kind: "slack_view_submission";
+    triggerId: string | null;
+    callbackId: string | null;
+    viewId: string;
+    externalId: string | null;
+    viewHash: string | null;
+    channel: string;
+    threadTs: string;
+    privateMetadata: DecodedSlackModalPrivateMetadata["value"];
+    stateValues: SlackViewSubmissionStateValues;
+}
 export interface SlackBlockActionInboxEvent {
     channel: string;
     threadTs: string;
     userId: string;
     text: string;
     timestamp: string;
-    metadata: Record<string, unknown>;
+    metadata: SlackBlockActionMetadata;
 }
 export interface SlackViewSubmissionInboxEvent {
     channel: string;
@@ -24,7 +66,7 @@ export interface SlackViewSubmissionInboxEvent {
     userId: string;
     text: string;
     timestamp: string;
-    metadata: Record<string, unknown>;
+    metadata: SlackViewSubmissionMetadata;
 }
 export type SlackInteractiveInboxEvent = SlackBlockActionInboxEvent | SlackViewSubmissionInboxEvent;
 export declare function normalizeSlackBlocksInput(blocks: unknown): SlackBlock[];

package/dist/slack-block-kit.js

@@ -1,15 +1,15 @@
-import { decodeSlackModalPrivateMetadata } from "./slack-modals.js";
-function isRecord(value) {
+import { decodeSlackModalPrivateMetadata, } from "./slack-modals.js";
+function isSlackBlockKitObject(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 function asString(value) {
     return typeof value === "string" && value.length > 0 ? value : undefined;
 }
 function asRecord(value) {
-    return isRecord(value) ? value : undefined;
+    return isSlackBlockKitObject(value) ? value : undefined;
 }
 function asRecordArray(value) {
-    if (!Array.isArray(value) || value.some((item) => !isRecord(item))) {
+    if (!Array.isArray(value) || value.some((item) => !isSlackBlockKitObject(item))) {
         throw new Error("Slack blocks must be a JSON array of objects.");
     }
     return value;
@@ -122,7 +122,7 @@ export function extractSlackInteractivePayloadFromEnvelope(envelope) {
             return null;
         }
     }
-    if (!isRecord(payload))
+    if (!isSlackBlockKitObject(payload))
         return null;
     return payload;
 }

package/dist/slack-file-access.d.ts

@@ -19,8 +19,18 @@ export interface SlackFileLookupContext {
     threadTs?: string;
     messageTs?: string;
 }
+type SlackFileObject = Record<string, unknown>;
+export interface SlackThreadRepliesRequest extends SlackFileObject {
+    channel: string;
+    ts: string;
+    limit: number;
+}
+export interface SlackFilesInfoRequest extends SlackFileObject {
+    file: string;
+}
+export type SlackFileLookupRequest = SlackThreadRepliesRequest | SlackFilesInfoRequest;
 export interface SlackFileDownloadDeps {
-    slack: (method: string, token: string, body?: Record<string, unknown>) => Promise<SlackResult>;
+    slack: (method: string, token: string, body?: SlackFileLookupRequest) => Promise<SlackResult>;
     token: string;
     fetchImpl?: (url: string, init: RequestInit) => Promise<Pick<Response, "ok" | "status" | "statusText" | "arrayBuffer" | "text"> & {
         body?: Response["body"];
@@ -32,3 +42,4 @@ export interface SlackFileDownloadDeps {
 }
 export declare function fetchSlackFileToCache(fileId: string, context: SlackFileLookupContext, deps: SlackFileDownloadDeps): Promise<SlackFileDescriptor>;
 export declare function readCachedSlackFile(pathname: string): Promise<Buffer>;
+export {};

package/dist/slack-file-access.js

@@ -10,7 +10,7 @@ const SLACK_FILE_DOWNLOAD_HOSTS = new Set([
     "files.slack-edge.com",
     "downloads.slack-edge.com",
 ]);
-function isRecord(value) {
+function isSlackFileObject(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 function getString(record, key) {
@@ -44,7 +44,7 @@ function assertSafeSlackFileDownloadUrl(rawUrl) {
     }
 }
 function metadataFromSlackFile(value, expectedFileId) {
-    if (!isRecord(value))
+    if (!isSlackFileObject(value))
         return null;
     const id = getString(value, "id");
     if (id !== expectedFileId)
@@ -65,7 +65,7 @@ function metadataFromSlackFile(value, expectedFileId) {
     };
 }
 function filesFromMessage(message) {
-    if (!isRecord(message) || !Array.isArray(message.files))
+    if (!isSlackFileObject(message) || !Array.isArray(message.files))
         return [];
     return message.files;
 }
@@ -73,7 +73,7 @@ function findFileInMessages(messages, fileId, messageTs) {
     if (!Array.isArray(messages))
         return null;
     for (const message of messages) {
-        if (messageTs && (!isRecord(message) || getString(message, "ts") !== messageTs))
+        if (messageTs && (!isSlackFileObject(message) || getString(message, "ts") !== messageTs))
             continue;
         for (const file of filesFromMessage(message)) {
             const metadata = metadataFromSlackFile(file, fileId);

package/dist/slack-message-context.d.ts

@@ -1,3 +1,9 @@
+type SlackMessageContextObject = Record<string, unknown>;
+export interface SlackMessageContextEvent extends SlackMessageContextObject {
+    blocks?: unknown;
+    attachments?: unknown;
+    files?: unknown;
+}
 export interface SlackMessageFileMetadata {
     id?: string;
     name?: string;
@@ -10,5 +16,6 @@ export interface SlackMessageFileMetadata {
     size?: number;
 }
 export declare function extractSlackMessageFileMetadata(files: unknown): SlackMessageFileMetadata[];
-export declare function extractSlackMessageContextLines(evt: Record<string, unknown>, baseText?: string): string[];
-export declare function buildSlackInboundMessageText(baseText: string, evt: Record<string, unknown>): string;
+export declare function extractSlackMessageContextLines(evt: SlackMessageContextEvent, baseText?: string): string[];
+export declare function buildSlackInboundMessageText(baseText: string, evt: SlackMessageContextEvent): string;
+export {};

package/dist/slack-modals.d.ts

@@ -1,4 +1,7 @@
-export type SlackModalView = Record<string, unknown>;
+export type SlackModalObject = Record<string, unknown>;
+export interface SlackModalView extends SlackModalObject {
+    type: "modal";
+}
 export declare function normalizeSlackModalViewInput(view: unknown): SlackModalView;
 export interface SlackModalThreadContext {
     threadTs: string;

package/dist/slack-modals.js

@@ -1,5 +1,5 @@
 const PI_SLACK_MODAL_CONTEXT_KEY = "__piSlackModalContext";
-function isRecord(value) {
+function isSlackModalObject(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 function asString(value) {
@@ -9,7 +9,7 @@ function asOptionalString(value) {
     return typeof value === "string" ? value : null;
 }
 export function normalizeSlackModalViewInput(view) {
-    if (!isRecord(view)) {
+    if (!isSlackModalObject(view)) {
         throw new Error("Slack modal view must be a JSON object.");
     }
     const cloned = structuredClone(view);
@@ -32,7 +32,7 @@ export function encodeSlackModalPrivateMetadata(privateMetadata, threadContext)
             value = raw;
         }
     }
-    if (isRecord(value)) {
+    if (isSlackModalObject(value)) {
         return JSON.stringify({
             ...value,
             [PI_SLACK_MODAL_CONTEXT_KEY]: threadContext,
@@ -50,9 +50,9 @@ export function decodeSlackModalPrivateMetadata(privateMetadata) {
     }
     try {
         const parsed = JSON.parse(raw);
-        if (isRecord(parsed)) {
+        if (isSlackModalObject(parsed)) {
             const context = parsed[PI_SLACK_MODAL_CONTEXT_KEY];
-            const threadContext = isRecord(context)
+            const threadContext = isSlackModalObject(context)
                 ? {
                     threadTs: asString(context.threadTs) ?? "",
                     channel: asString(context.channel) ?? "",

package/dist/slack-pinet-runtime-adapter.d.ts

@@ -11,6 +11,7 @@ export interface SlackPinetRuntimeAdapterDeps {
     getAppToken: () => string;
     getAllowedUsers: () => Set<string> | null;
     shouldAllowAllWorkspaceUsers: () => boolean;
+    setExtStatus: (ctx: ExtensionContext, state: "ok" | "reconnecting" | "error") => void;
     onAppHomeOpened: (userId: string, ctx: ExtensionContext) => Promise<void> | void;
     onSlashCommand?: (event: ParsedSlashCommand, ctx: ExtensionContext) => Promise<string | null> | string | null;
 }