@sentry/junior 0.125.0 → 0.126.0

package/dist/{chunk-IS4DGI5J.js → chunk-KP7E267S.js}

@@ -9,17 +9,17 @@ import {
   TURN_REASONING_LEVELS,
   getChatConfig,
   modelProfileSchema
-} from "./chunk-KN7QI56S.js";
+} from "./chunk-XLIAQRDI.js";
 import {
   juniorDestinations
 } from "./chunk-VFUK3X5B.js";
-import {
-  normalizeIdentityEmail
-} from "./chunk-MVYNJECP.js";
 import {
   parseSlackChannelReferenceId,
   parseSlackTeamId
 } from "./chunk-F3B2YJ76.js";
+import {
+  normalizeIdentityEmail
+} from "./chunk-JP2ZOSQK.js";
 
 // src/chat/conversations/sql/store.ts
 import { randomUUID as randomUUID2 } from "crypto";
@@ -176,7 +176,7 @@ function normalizeSessionSource(value) {
   if (value.platform === "local") {
     return {
       platform: "local",
-      type: "priv",
+      visibility: "private",
       conversationId: value.conversationId
     };
   }
@@ -186,7 +186,7 @@ function normalizeSessionSource(value) {
   }
   return {
     platform: "slack",
-    type: value.type,
+    visibility: value.visibility,
     teamId: value.teamId,
     channelId: value.channelId,
     threadTs
@@ -294,7 +294,7 @@ function destinationUpsertFromDestination(args) {
       providerTenantId: destination.teamId,
       providerDestinationId: channelId,
       refreshVisibility: args.visibility !== void 0,
-      visibility: args.visibility ?? "private",
+      visibility: args.visibility ?? "unknown",
       ...args.channelName ? { displayName: args.channelName } : {},
       metadata: { platform: "slack" }
     };
@@ -626,7 +626,10 @@ var SqlStore = class {
     if (!row) {
       return void 0;
     }
-    return row.visibility === "public" ? "public" : "private";
+    if (row.visibility === "public" || row.visibility === "private") {
+      return row.visibility;
+    }
+    return void 0;
   }
   /** Serialize all durable mutations for one conversation inside a SQL transaction. */
   async withConversationMutation(conversationId, callback) {
@@ -759,9 +762,9 @@ var SqlStore = class {
       set: {
         kind: sql2`excluded.kind`,
         displayName: sql2`coalesce(excluded.display_name, ${juniorDestinations.displayName})`,
-        // Signal-less writes insert as private but must not clobber an
-        // existing public/private value. Live source signals refresh this
-        // field so converted channels converge on the next message.
+        // Signal-less writes remain unknown and must not clobber an existing
+        // public/private value. Live source signals refresh this field so
+        // converted channels converge on the next message.
         visibility: visibilityUpdate,
         metadata: sql2`coalesce(excluded.metadata_json, ${juniorDestinations.metadata})`,
         updatedAt: sql2`excluded.updated_at`

package/dist/{chunk-2R5MASH3.js → chunk-LCRWDMCH.js}

@@ -1,6 +1,6 @@
 import {
   JUNIOR_VERSION
-} from "./chunk-ZXGBLAYQ.js";
+} from "./chunk-JFW62B3Y.js";
 
 // src/deployment.ts
 var JUNIOR_HEARTBEAT_ROUTE = "/api/internal/heartbeat";

package/dist/{chunk-MYKSUTUV.js → chunk-LIDEU7QS.js}

@@ -2,18 +2,18 @@ import {
   SANDBOX_WORKSPACE_ROOT,
   getStateAdapter,
   sleep
-} from "./chunk-XJYGLTYI.js";
+} from "./chunk-APYVVX2Y.js";
 import {
   toOptionalTrimmed
-} from "./chunk-KN7QI56S.js";
+} from "./chunk-XLIAQRDI.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-NM3PMQS6.js";
+} from "./chunk-GHNO2WUH.js";
 import {
   setSpanAttributes,
   setSpanStatus,
   withSpan
-} from "./chunk-MVYNJECP.js";
+} from "./chunk-JP2ZOSQK.js";
 
 // src/chat/sandbox/snapshot/profile.ts
 import { createHash } from "crypto";

package/dist/{chunk-HT7DSLKL.js → chunk-TUHUIBDX.js}

@@ -30,7 +30,7 @@ import {
   toMcpProviderError,
   toolInvocationDestination,
   uploadFilesToConversation
-} from "./chunk-K6MQWEEJ.js";
+} from "./chunk-4PV73JHH.js";
 import {
   CooperativeTurnYieldError,
   JUNIOR_THREAD_STATE_TTL_MS,
@@ -80,7 +80,7 @@ import {
   stripRuntimeTurnContext,
   trimTrailingAssistantMessages,
   updateConversationStats
-} from "./chunk-7K57FJXR.js";
+} from "./chunk-WRH7DWFO.js";
 import {
   parseSandboxEgressAuthRequiredSignal,
   parseSandboxEgressPermissionDeniedSignal,
@@ -104,10 +104,13 @@ import {
   runNonInteractiveCommand,
   throwSandboxOperationError,
   wrapSandboxSetupError
-} from "./chunk-MYKSUTUV.js";
+} from "./chunk-LIDEU7QS.js";
 import {
   incrementStat
-} from "./chunk-Q3UF3K77.js";
+} from "./chunk-EQMSKORN.js";
+import {
+  parseSlackThreadId
+} from "./chunk-72SDEHFP.js";
 import {
   addAgentTurnUsage,
   hasAgentTurnUsage
@@ -129,7 +132,7 @@ import {
   normalizeSlackConversationId,
   resolveChannelCapabilities,
   withSlackRetries
-} from "./chunk-ODB55JM7.js";
+} from "./chunk-6KWO5DU3.js";
 import {
   KNOWN_CONVERSATION_EVENT_TYPES,
   contextProvenance,
@@ -139,7 +142,7 @@ import {
   instructionActors,
   instructionProvenanceFor,
   sameActorIdentity
-} from "./chunk-IS4DGI5J.js";
+} from "./chunk-KP7E267S.js";
 import {
   SANDBOX_DATA_ROOT,
   SANDBOX_SKILLS_ROOT,
@@ -149,7 +152,7 @@ import {
   sandboxSkillFile,
   sandboxSkillPathResolution,
   sleep
-} from "./chunk-XJYGLTYI.js";
+} from "./chunk-APYVVX2Y.js";
 import {
   ACTIVE_TURN_COMPACTION_SUMMARY_PREFIX,
   COMPACTION_SUMMARY_PREFIX,
@@ -183,10 +186,9 @@ import {
   renderCurrentInstruction,
   resolveGatewayModel,
   unwrapCurrentInstruction
-} from "./chunk-KN7QI56S.js";
+} from "./chunk-XLIAQRDI.js";
 import {
   canExposeConversationPayload,
-  parseSlackThreadId,
   resolveConversationPrivacy,
   runWithConversationPrivacy,
   toCanonicalInputMessage,
@@ -194,16 +196,16 @@ import {
   toGenAiMessagesTraceAttributes,
   toGenAiPayloadMetadata,
   toGenAiPayloadTraceAttributes
-} from "./chunk-S7G34U75.js";
+} from "./chunk-VUBUFT66.js";
 import {
   discoverSkills,
   findSkillByName,
   loadSkillsByName,
   parseSkillInvocation
-} from "./chunk-CAPBVX4L.js";
+} from "./chunk-ACJZZY5W.js";
 import {
   pluginCatalogRuntime
-} from "./chunk-NM3PMQS6.js";
+} from "./chunk-GHNO2WUH.js";
 import {
   CredentialUnavailableError,
   OAuthProviderError,
@@ -211,13 +213,18 @@ import {
   getOAuthProviderErrorAttributes,
   resolveAuthTokenPlaceholder,
   resolvePluginCommandEnv
-} from "./chunk-2U5CLBYF.js";
+} from "./chunk-EFQX6XBW.js";
 import {
   credentialUserSubjectId
 } from "./chunk-W3U3YQYI.js";
 import {
   isUserActor
 } from "./chunk-HZF5P5UN.js";
+import {
+  parseSlackChannelId,
+  parseSlackMessageTs,
+  parseSlackUserId
+} from "./chunk-F3B2YJ76.js";
 import {
   extractGenAiUsageAttributes,
   extractGenAiUsageSummary,
@@ -235,12 +242,7 @@ import {
   summarizeMessageText,
   withLogContext,
   withSpan
-} from "./chunk-MVYNJECP.js";
-import {
-  parseSlackChannelId,
-  parseSlackMessageTs,
-  parseSlackUserId
-} from "./chunk-F3B2YJ76.js";
+} from "./chunk-JP2ZOSQK.js";
 import {
   startInactiveSpan,
   withActiveSpan
@@ -1259,6 +1261,21 @@ var AuthorizationFlowDisabledError = class extends Error {
   }
 };
 
+// src/chat/conversations/destination-visibility.ts
+async function resolveDestinationVisibility(args) {
+  if (args.visibility) {
+    return args.visibility;
+  }
+  if (args.destination.platform === "local") {
+    return "private";
+  }
+  return await getConversationStore().getDestinationVisibility({
+    provider: "slack",
+    providerDestinationId: args.destination.channelId,
+    providerTenantId: args.destination.teamId
+  });
+}
+
 // src/chat/tool-support/action-review-history.ts
 import { z as z3 } from "zod";
 var MAX_VISIBLE_HISTORY_CHARS = 12e3;
@@ -1693,10 +1710,7 @@ function formatAvailableSkillsForPrompt(skills) {
   );
   const sections = [];
   if (autoSelectable.length > 0) {
-    const available = [
-      "<available-skills>",
-      "Scan before answering. Load the most specific matching skill; do not answer from memory when a skill fits. A request that names a skill, plugin, provider, or account matching a skill name is a skill match. If none fits, do not load a skill."
-    ];
+    const available = ["<available-skills>"];
     for (const skill of autoSelectable) {
       available.push(...formatSkillEntry(skill));
     }
@@ -1817,7 +1831,7 @@ var TOOL_CALL_STYLE_RULES = [
 ];
 var SKILL_POLICY_RULES = [
   "- A `<skill>` block in the current user turn is already loaded. Follow its instructions directly and do not call `loadSkill` for that skill.",
-  "- Otherwise, only load skills listed in `<available-skills>`. Never guess or invent a skill name.",
+  "- Otherwise, scan `<available-skills>` before acting. Load the most specific skill whose description matches the request; do not answer from memory when one fits. Only call `loadSkill` with an exact listed `<name>`; if none fits, do not load a skill.",
   "- Load one skill at a time. After `loadSkill`, follow the instructions returned by that tool result."
 ];
 var EXECUTION_CONTRACT_RULES = [
@@ -10775,11 +10789,9 @@ function assertCanQueryConversationEvents(args) {
   if (targetConversationId === accessScope.currentConversationId || targetRootConversationId !== void 0 && targetRootConversationId === accessScope.currentRootConversationId) {
     return;
   }
-  const publicPayloadAllowed = canExposeConversationPayload({
-    conversationId: targetRootConversationId ?? targetConversationId,
-    ...targetVisibility ? { visibility: targetVisibility } : {},
-    ...targetDestination?.platform === "slack" ? { channelId: targetDestination.channelId } : {}
-  });
+  const publicPayloadAllowed = canExposeConversationPayload(
+    targetVisibility ? { visibility: targetVisibility } : {}
+  );
   if (!publicPayloadAllowed) {
     throw new ToolInputError(
       `Conversation events are not accessible: ${targetConversationId}`
@@ -11678,7 +11690,7 @@ function createTools(availableSkills, hooks = {}, context, options = {}) {
     tools.slackCanvasEdit = createSlackCanvasEditTool(state);
     tools.slackCanvasWrite = createSlackCanvasWriteTool(state);
     tools.slackThreadRead = createSlackThreadReadTool(slackContext);
-    if (context.conversationId && slackContext.source.type === "pub") {
+    if (context.conversationId && slackContext.source.visibility === "public") {
       tools.searchConversationHistory = createSlackConversationSearchTool(
         {
           kind: "public_provider_tenant",
@@ -14722,6 +14734,7 @@ function createResumeState(args) {
     channelName: args.channelName,
     conversationId: args.conversationId,
     destination: args.destination,
+    ...args.destinationVisibility ? { destinationVisibility: args.destinationVisibility } : {},
     ...args.dispatchId ? { dispatchId: args.dispatchId } : {},
     source: args.runSource,
     sessionId: args.turnId,
@@ -15409,14 +15422,17 @@ async function executeAgentRun(request, streamFn) {
   if (!request.routing.destination) {
     throw new TypeError("Assistant reply generation requires a destination");
   }
-  const channelId = request.routing.destination.platform === "slack" ? request.routing.destination.channelId : void 0;
+  const destinationVisibility = await resolveDestinationVisibility({
+    destination: request.routing.destination,
+    visibility: request.routing.destinationVisibility
+  });
   const conversationPrivacy = resolveConversationPrivacy({
-    channelId,
-    conversationId: request.conversationId,
-    // Destination visibility is provider-neutral. Slack event context remains
-    // a compatibility fallback for callers that have not projected it yet.
-    visibility: request.routing.destinationVisibility ?? request.routing.slackConversation?.visibility
+    visibility: destinationVisibility
   });
+  const resolvedRequest = destinationVisibility ? {
+    ...request,
+    routing: { ...request.routing, destinationVisibility }
+  } : request;
   const credentialActor = request.routing.credentialContext?.actor;
   const actor = actorFromRouting(request.routing);
   const userActor = actor && "userId" in actor ? actor : void 0;
@@ -15436,9 +15452,9 @@ async function executeAgentRun(request, streamFn) {
   return withLogContext(
     runLogContext,
     () => runWithConversationPrivacy(
-      conversationPrivacy ?? "private",
+      conversationPrivacy,
       () => executeAgentRunInPrivacyContext(
-        request,
+        resolvedRequest,
         conversationPrivacy,
         runLogContext,
         streamFn
@@ -15569,6 +15585,7 @@ async function executeAgentRunInPrivacyContext(request, conversationPrivacy, run
     resume = createResumeState({
       channelName: routing.slackConversation?.name,
       destination: routing.destination,
+      ...routing.destinationVisibility ? { destinationVisibility: routing.destinationVisibility } : {},
       ...routing.dispatch?.id ? { dispatchId: routing.dispatch.id } : {},
       durability,
       getLoadedSkillNames: () => loadedSkillNamesForResume,
@@ -16511,6 +16528,7 @@ export {
   getConfigDefaults,
   shouldEmitDevAgentTrace,
   AuthorizationFlowDisabledError,
+  resolveDestinationVisibility,
   buildSteeringPiMessage,
   SANDBOX_EGRESS_PROXY_PATH,
   parseSandboxEgressCredentialToken,

package/dist/{chunk-S7G34U75.js → chunk-VUBUFT66.js}

@@ -1,128 +1,16 @@
 import {
-  toOptionalString
-} from "./chunk-OB42YVAE.js";
-import {
-  parseSlackChannelId,
-  parseSlackMessageTs,
-  parseSlackTeamId
-} from "./chunk-F3B2YJ76.js";
+  logWarn
+} from "./chunk-JP2ZOSQK.js";
 
 // src/chat/conversation-privacy.ts
 import { AsyncLocalStorage } from "async_hooks";
-
-// src/chat/slack/context.ts
-import { z } from "zod";
-function toTrimmedSlackString(value) {
-  const normalized = toOptionalString(value);
-  return normalized?.trim() || void 0;
-}
-var slackMessageEnvelopeSchema = z.object({
-  raw: z.unknown().optional()
-});
-var rawSlackMessageSchema = z.object({
-  channel: z.unknown().optional(),
-  message: z.unknown().optional(),
-  team: z.unknown().optional(),
-  team_id: z.unknown().optional(),
-  thread_ts: z.unknown().optional(),
-  ts: z.unknown().optional(),
-  user_team: z.unknown().optional()
-});
-var nestedRawSlackMessageSchema = z.object({
-  ts: z.unknown().optional()
-});
-function readSlackRawMessageContext(message) {
-  const envelope = slackMessageEnvelopeSchema.safeParse(message);
-  if (!envelope.success) {
-    return void 0;
-  }
-  const raw = rawSlackMessageSchema.safeParse(envelope.data.raw);
-  if (!raw.success) {
-    return void 0;
-  }
-  const nestedMessage = nestedRawSlackMessageSchema.safeParse(raw.data.message);
-  const channelId = parseSlackChannelId(raw.data.channel);
-  const threadTs = parseSlackMessageTs(raw.data.thread_ts);
-  const messageTs = parseSlackMessageTs(raw.data.ts);
-  const nestedMessageTs = nestedMessage.success && parseSlackMessageTs(nestedMessage.data.ts);
-  const teamId = parseSlackTeamId(raw.data.team_id) ?? parseSlackTeamId(raw.data.team);
-  const authorTeamId = parseSlackTeamId(raw.data.user_team);
-  return {
-    ...channelId ? { channelId } : {},
-    ...threadTs ? { threadTs } : {},
-    ...messageTs ? { messageTs } : {},
-    ...nestedMessageTs ? { nestedMessageTs } : {},
-    ...teamId ? { teamId } : {},
-    ...authorTeamId ? { authorTeamId } : {}
-  };
-}
-function parseSlackThreadId(threadId) {
-  const normalizedThreadId = toTrimmedSlackString(threadId);
-  if (!normalizedThreadId) {
-    return void 0;
-  }
-  const parts = normalizedThreadId.split(":");
-  if (parts.length !== 3 || parts[0] !== "slack") {
-    return void 0;
-  }
-  const channelId = parseSlackChannelId(parts[1]);
-  const threadTs = parseSlackMessageTs(parts[2]);
-  if (!channelId || !threadTs) {
-    return void 0;
-  }
-  return { channelId, threadTs };
-}
-function resolveSlackChannelIdFromThreadId(threadId) {
-  return parseSlackThreadId(threadId)?.channelId;
-}
-function resolveSlackChannelIdFromMessage(message) {
-  const messageChannelId = parseSlackChannelId(
-    message.channelId
-  );
-  if (messageChannelId) {
-    return messageChannelId;
-  }
-  const raw = message.raw;
-  if (raw && typeof raw === "object") {
-    const rawChannel = parseSlackChannelId(
-      raw.channel
-    );
-    if (rawChannel) {
-      return rawChannel;
-    }
-  }
-  const threadId = toTrimmedSlackString(
-    message.threadId
-  );
-  return resolveSlackChannelIdFromThreadId(threadId);
-}
-
-// src/chat/conversation-privacy.ts
 var SAFE_METADATA_KEY_LIMIT = 20;
 var conversationPrivacyStorage = new AsyncLocalStorage();
-function privateNarrowingFromChannelId(channelId) {
-  const normalized = channelId?.trim();
-  if (!normalized) return void 0;
-  return normalized.startsWith("D") || normalized.startsWith("G") ? "private" : void 0;
-}
-function privateNarrowingFromConversationId(conversationId) {
-  const normalized = conversationId?.trim();
-  if (!normalized) return void 0;
-  const slackThread = parseSlackThreadId(normalized);
-  if (slackThread) {
-    return privateNarrowingFromChannelId(slackThread.channelId);
-  }
-  if (normalized.startsWith("slack:")) {
-    return void 0;
-  }
-  return "private";
-}
 function resolveConversationPrivacy(input) {
-  const narrowed = privateNarrowingFromChannelId(input.channelId) ?? privateNarrowingFromConversationId(input.conversationId);
-  if (narrowed === "private") {
-    return "private";
+  if (input.visibility === void 0) {
+    logWarn("conversation.visibility.defaulted");
   }
-  return input.visibility;
+  return input.visibility === "public" ? "public" : "private";
 }
 function canExposeConversationPayload(input) {
   return resolveConversationPrivacy(input) === "public";
@@ -298,10 +186,6 @@ function toGenAiMessagesTraceAttributes(prefix, messages) {
 }
 
 export {
-  readSlackRawMessageContext,
-  parseSlackThreadId,
-  resolveSlackChannelIdFromThreadId,
-  resolveSlackChannelIdFromMessage,
   resolveConversationPrivacy,
   canExposeConversationPayload,
   getCurrentConversationPrivacy,

package/dist/{chunk-7K57FJXR.js → chunk-WRH7DWFO.js}

@@ -16,34 +16,32 @@ import {
   instructionProvenanceFor,
   sanitizePostgresJson,
   withConversationEventLock
-} from "./chunk-IS4DGI5J.js";
+} from "./chunk-KP7E267S.js";
 import {
   getStateAdapter,
   sleep
-} from "./chunk-XJYGLTYI.js";
+} from "./chunk-APYVVX2Y.js";
 import {
   botConfig,
   escapeXml,
   resolveGatewayModel,
   unwrapCurrentInstruction
-} from "./chunk-KN7QI56S.js";
+} from "./chunk-XLIAQRDI.js";
 import {
   toStoredSlackActor
 } from "./chunk-HZF5P5UN.js";
+import {
+  parseSlackMessageTs
+} from "./chunk-F3B2YJ76.js";
 import {
   getActiveTraceId,
+  isRecord,
   logException,
   logWarn,
-  setSpanAttributes
-} from "./chunk-MVYNJECP.js";
-import {
-  isRecord,
+  setSpanAttributes,
   toOptionalNumber,
   toOptionalString
-} from "./chunk-OB42YVAE.js";
-import {
-  parseSlackMessageTs
-} from "./chunk-F3B2YJ76.js";
+} from "./chunk-JP2ZOSQK.js";
 
 // src/chat/turn-context-tag.ts
 var TURN_CONTEXT_TAG = "runtime-turn-context";
@@ -2013,6 +2011,7 @@ async function persistRunningSessionRecord(args) {
       cumulativeDurationMs: latestSessionRecord?.cumulativeDurationMs,
       cumulativeUsage: latestSessionRecord?.cumulativeUsage,
       ...args.destination ?? latestSessionRecord?.destination ? { destination: args.destination ?? latestSessionRecord?.destination } : {},
+      destinationVisibility: args.destinationVisibility,
       ...args.dispatchId ?? latestSessionRecord?.dispatchId ? { dispatchId: args.dispatchId ?? latestSessionRecord?.dispatchId } : {},
       ...args.source ?? latestSessionRecord?.source ? { source: args.source ?? latestSessionRecord?.source } : {},
       sessionId: args.sessionId,
@@ -2152,6 +2151,7 @@ async function persistAuthPauseSessionRecord(args) {
         args.currentUsage
       ),
       ...args.destination ?? latestSessionRecord?.destination ? { destination: args.destination ?? latestSessionRecord?.destination } : {},
+      destinationVisibility: args.destinationVisibility,
       ...args.dispatchId ?? latestSessionRecord?.dispatchId ? { dispatchId: args.dispatchId ?? latestSessionRecord?.dispatchId } : {},
       ...args.source ?? latestSessionRecord?.source ? { source: args.source ?? latestSessionRecord?.source } : {},
       sessionId: args.sessionId,
@@ -2216,6 +2216,7 @@ async function persistContinuationSessionRecord(args) {
         ...args.destination ?? latestSessionRecord?.destination ? {
           destination: args.destination ?? latestSessionRecord?.destination
         } : {},
+        destinationVisibility: args.destinationVisibility,
         ...args.dispatchId ?? latestSessionRecord?.dispatchId ? { dispatchId: args.dispatchId ?? latestSessionRecord?.dispatchId } : {},
         ...args.source ?? latestSessionRecord?.source ? { source: args.source ?? latestSessionRecord?.source } : {},
         sessionId: args.sessionId,
@@ -2243,6 +2244,7 @@ async function persistContinuationSessionRecord(args) {
       cumulativeDurationMs,
       cumulativeUsage,
       ...args.destination ?? latestSessionRecord?.destination ? { destination: args.destination ?? latestSessionRecord?.destination } : {},
+      destinationVisibility: args.destinationVisibility,
       ...args.dispatchId ?? latestSessionRecord?.dispatchId ? { dispatchId: args.dispatchId ?? latestSessionRecord?.dispatchId } : {},
       ...args.source ?? latestSessionRecord?.source ? { source: args.source ?? latestSessionRecord?.source } : {},
       sessionId: args.sessionId,

package/dist/{chunk-KN7QI56S.js → chunk-XLIAQRDI.js}

@@ -1,10 +1,9 @@
 import {
   getCurrentConversationPrivacy,
-  resolveConversationPrivacy,
   toCanonicalInputMessage,
   toCanonicalOutputMessage,
   toGenAiMessagesTraceAttributes
-} from "./chunk-S7G34U75.js";
+} from "./chunk-VUBUFT66.js";
 import {
   extractGenAiUsageAttributes,
   logException,
@@ -12,7 +11,7 @@ import {
   normalizeGenAiFinishReason,
   serializeGenAiAttribute,
   withSpan
-} from "./chunk-MVYNJECP.js";
+} from "./chunk-JP2ZOSQK.js";
 
 // src/chat/config.ts
 import { getModel as getModel2 } from "@earendil-works/pi-ai/compat";
@@ -360,11 +359,7 @@ async function completeText(params) {
   const model = resolveGatewayModel(params.modelId);
   const apiKey = getPiGatewayApiKey();
   const authMode = toOptionalTrimmed(process.env.AI_GATEWAY_API_KEY) ? "api_key" : toOptionalTrimmed(process.env.VERCEL_OIDC_TOKEN) ? "oidc" : "api_key";
-  const privacy = resolveConversationPrivacy({
-    channelId: typeof params.metadata?.channelId === "string" ? params.metadata.channelId : void 0,
-    conversationId: typeof params.metadata?.conversationId === "string" ? params.metadata.conversationId : typeof params.metadata?.threadId === "string" ? params.metadata.threadId : void 0
-  }) ?? getCurrentConversationPrivacy();
-  const effectivePrivacy = privacy ?? "private";
+  const effectivePrivacy = getCurrentConversationPrivacy() ?? "private";
   const messageAttributeMode = params.messageAttributeMode ?? (effectivePrivacy === "public" ? "content" : "metadata");
   const requestMessagesAttribute = messageAttributeMode === "content" ? serializeGenAiAttribute(params.messages.map(toCanonicalInputMessage)) : void 0;
   const systemInstructionsAttribute = params.system && messageAttributeMode === "content" ? serializeGenAiAttribute([{ type: "text", content: params.system }]) : void 0;

package/dist/{chunk-EXD2Z3YR.js → chunk-XVKJJXHF.js}

@@ -1,12 +1,12 @@
 import {
   healthReportSchema
 } from "./chunk-UTL3MHPC.js";
-import {
-  normalizeSlackConversationId
-} from "./chunk-ODB55JM7.js";
 import {
   parseSlackThreadId
-} from "./chunk-S7G34U75.js";
+} from "./chunk-72SDEHFP.js";
+import {
+  normalizeSlackConversationId
+} from "./chunk-6KWO5DU3.js";
 
 // src/handlers/health.ts
 function readHealthReport() {
@@ -56,14 +56,6 @@ function conversationVisibilityFromSlackChannelType(channelType) {
   if (!channelType) return void 0;
   return channelType === "channel" ? "public" : "private";
 }
-function visibilityFromChannelIdPrefix(channelId) {
-  const normalized = normalizeSlackConversationId(channelId);
-  if (!normalized) return void 0;
-  if (normalized.startsWith("D") || normalized.startsWith("G")) {
-    return "private";
-  }
-  return void 0;
-}
 function resolveSlackChannelTypeFromMessage(message) {
   const raw = message.raw;
   if (!raw || typeof raw !== "object") {
@@ -76,7 +68,9 @@ function resolveSlackConversationContext(input) {
   const type = typeFromSlackChannelType(input.channelType) ?? typeFromChannelId(input.channelId, input.channelName);
   if (!type) return void 0;
   const name = normalizeConversationName(type, input.channelName);
-  const visibility = conversationVisibilityFromSlackChannelType(input.channelType) ?? visibilityFromChannelIdPrefix(input.channelId);
+  const visibility = conversationVisibilityFromSlackChannelType(
+    input.channelType
+  );
   return {
     type,
     ...name ? { name } : {},
@@ -85,10 +79,18 @@ function resolveSlackConversationContext(input) {
 }
 function resolveSlackConversationContextFromThreadId(input) {
   const slackThread = parseSlackThreadId(input.threadId);
-  return resolveSlackConversationContext({
+  const context = resolveSlackConversationContext({
     channelId: slackThread?.channelId,
     channelName: input.channelName
   });
+  if (!context || !input.visibility) {
+    return context;
+  }
+  return {
+    ...context,
+    type: input.visibility === "private" && context.type === "public_channel" ? "private_channel" : context.type,
+    visibility: input.visibility
+  };
 }
 function formatSlackConversationTypeLabel(type) {
   if (type === "public_channel") return "Public Channel";