@agent-team-foundation/first-tree-hub 0.9.11 → 0.10.0

package/dist/drizzle/0024_display_name_not_null.sql

@@ -0,0 +1,31 @@
+-- Phase 2 of the agent-naming refactor (docs/agent-naming-design.md §4).
+--
+-- Before this migration, `display_name` was nullable and the web layer
+-- silently fell back to `name` for rendering. CLI, server logs, and IM
+-- bridges saw raw NULLs. This migration closes the gap in three steps:
+--
+--   1. Backfill every NULL row with a non-empty label — the agent's `name`
+--      when available, else the tombstone literal "[deleted agent]".
+--      Important: the third branch must NOT be `uuid`, because UUIDs would
+--      surface as human-visible strings in the chat roster and IM bridge.
+--      Only `status = 'deleted'` rows can have both `name` and
+--      `display_name` NULL (see `deleteAgent` in services/agent.ts), which
+--      is why the literal refers to deletion.
+--
+--   2. Add a temporary empty-string DEFAULT so any old server instance
+--      still running during a rolling deploy — i.e. code that doesn't yet
+--      default `display_name` in `createAgent` — can INSERT without
+--      violating the upcoming NOT NULL. The application code treats an
+--      empty string as "no display name" (the Zod read schema accepts it);
+--      a follow-up migration can drop the default once the code is fully
+--      rolled.
+--
+--   3. Promote the column to NOT NULL. Combined with the service-level
+--      default this closes the hole for new rows too.
+
+UPDATE "agents"
+SET "display_name" = COALESCE("display_name", "name", '[deleted agent]')
+WHERE "display_name" IS NULL;
+
+ALTER TABLE "agents" ALTER COLUMN "display_name" SET DEFAULT '';
+ALTER TABLE "agents" ALTER COLUMN "display_name" SET NOT NULL;

package/dist/drizzle/0025_inbox_silent_entries.sql

@@ -0,0 +1,53 @@
+-- Silent inbox entries for missed group-chat context.
+--
+-- Adds `inbox_entries.notify`, defaulting to true so existing rows continue to
+-- behave as "active" deliverables. Group-chat fan-out now writes a row for
+-- every non-sender participant — for `mention_only` participants who aren't
+-- explicitly @mentioned in the triggering message we set `notify = false` so
+-- the row is silently parked. Subsequent active deliveries to the same chat
+-- pick up these silent rows and replay them as preceding context, then
+-- bulk-ack them so they don't get re-replayed.
+--
+-- See proposals/group-chat-ux-improvements §1 (silent inbox).
+--
+-- ──────────────── Indexes ────────────────
+--
+--   idx_inbox_pending_notify — partial index used by pollInbox's claim.
+--     The query is "WHERE inbox_id = ? AND status = 'pending' AND notify = true
+--     ORDER BY created_at LIMIT N FOR UPDATE SKIP LOCKED". Without `notify` in
+--     the index, a chat that accumulates silent rows (mention_only agent in a
+--     chatty group) forces the planner to scan past them before finding the
+--     next trigger. Partial index keeps it bounded.
+--
+--   idx_inbox_chat_silent — used by collectPrecedingContext to walk the
+--     silent rows in a single (inbox, chat) bucket between two triggers.
+--
+-- ──────────────── Operator note ────────────────
+--
+-- Drizzle migrator wraps every migration file in a single transaction (see the
+-- comment block in 0020_unified_user_token.sql), which means we can't use
+-- `CREATE INDEX CONCURRENTLY` here — PG rejects it inside a tx. On a small
+-- `inbox_entries` table the regular `CREATE INDEX` finishes in <1s and is
+-- fine. For a large production table, the runbook is:
+--
+--   1. Stop applying new migrations briefly.
+--   2. Manually run, OUTSIDE a transaction:
+--        CREATE INDEX CONCURRENTLY idx_inbox_pending_notify
+--          ON inbox_entries (inbox_id, created_at)
+--          WHERE status = 'pending' AND notify = true;
+--        CREATE INDEX CONCURRENTLY idx_inbox_chat_silent
+--          ON inbox_entries (inbox_id, chat_id, notify, status);
+--   3. Re-run `pnpm db:migrate`. The `IF NOT EXISTS` clauses below detect
+--      the pre-created indexes and skip them.
+
+ALTER TABLE "inbox_entries"
+	ADD COLUMN "notify" boolean DEFAULT true NOT NULL;
+
+--> statement-breakpoint
+CREATE INDEX IF NOT EXISTS "idx_inbox_pending_notify"
+	ON "inbox_entries" ("inbox_id", "created_at")
+	WHERE status = 'pending' AND notify = true;
+
+--> statement-breakpoint
+CREATE INDEX IF NOT EXISTS "idx_inbox_chat_silent"
+	ON "inbox_entries" ("inbox_id", "chat_id", "notify", "status");

package/dist/drizzle/meta/_journal.json

@@ -169,6 +169,20 @@
       "when": 1777248000000,
       "tag": "0023_clients_org_scoping",
       "breakpoints": true
+    },
+    {
+      "idx": 24,
+      "version": "7",
+      "when": 1777334400000,
+      "tag": "0024_display_name_not_null",
+      "breakpoints": true
+    },
+    {
+      "idx": 25,
+      "version": "7",
+      "when": 1777420800000,
+      "tag": "0025_inbox_silent_entries",
+      "breakpoints": true
     }
   ]
 }

package/dist/{feishu-B2sjp6Z6.mjs → feishu-DEmwoNn_.mjs}

@@ -23,6 +23,21 @@ function extractMentions(content, participants) {
 	}
 	return [...hits];
 }
+/**
+* Return every `@<name>` token that survives the code-stripping / word-
+* boundary gates, regardless of whether it matches a participant. The
+* caller uses this to log unmatched tokens (typos, renamed agents) without
+* polluting the authoritative mention list.
+*/
+function scanMentionTokens(content) {
+	const stripped = stripCode(content);
+	const tokens = [];
+	for (const m of stripped.matchAll(MENTION_REGEX)) {
+		const token = m[1];
+		if (token) tokens.push(token.toLowerCase());
+	}
+	return tokens;
+}
 const adapterPlatformSchema = z.enum([
 	"feishu",
 	"slack",
@@ -204,7 +219,7 @@ function isReservedAgentName(name) {
 const createAgentSchema = z.object({
 	name: z.string().min(1).max(64).regex(AGENT_NAME_REGEX, "Must start with a letter or digit and contain only lowercase letters, digits, hyphens (-), and underscores (_). Max 64 chars.").refine((n) => !isReservedAgentName(n), { message: "That agent name is reserved — pick a different one." }).optional(),
 	type: agentTypeSchema,
-	displayName: z.string().max(200).optional(),
+	displayName: z.string().min(1).max(200).optional(),
 	delegateMention: z.string().max(100).optional(),
 	organizationId: z.string().max(100).optional(),
 	source: agentSourceSchema.optional(),
@@ -215,7 +230,7 @@ const createAgentSchema = z.object({
 });
 const updateAgentSchema = z.object({
 	type: agentTypeSchema.optional(),
-	displayName: z.string().max(200).nullable().optional(),
+	displayName: z.string().min(1).max(200).optional(),
 	delegateMention: z.string().max(100).nullable().optional(),
 	visibility: agentVisibilitySchema.optional(),
 	metadata: z.record(z.string(), z.unknown()).optional(),
@@ -227,7 +242,7 @@ z.object({
 	name: z.string().nullable(),
 	organizationId: z.string(),
 	type: agentTypeSchema,
-	displayName: z.string().nullable(),
+	displayName: z.string(),
 	delegateMention: z.string().nullable(),
 	inboxId: z.string(),
 	status: z.string(),
@@ -254,7 +269,7 @@ const agentPinnedMessageSchema = z.object({
 	type: z.literal("agent:pinned"),
 	agentId: z.string(),
 	name: z.string().nullable(),
-	displayName: z.string().nullable(),
+	displayName: z.string(),
 	agentType: agentTypeSchema
 });
 /**
@@ -450,7 +465,7 @@ const chatParticipantSchema = z.object({
 });
 chatParticipantSchema.extend({
 	name: z.string().nullable(),
-	displayName: z.string().nullable(),
+	displayName: z.string(),
 	type: z.string()
 });
 z.object({
@@ -590,6 +605,23 @@ const inReplyToSnapshotSchema = z.object({
 /** Per-chat participation mode exposed to the recipient runtime. */
 const participantModeSchema = z.enum(["full", "mention_only"]);
 /**
+* Lightweight snapshot of an earlier message in the same chat that the
+* recipient missed (because it was `mention_only` + not @mentioned). Server
+* attaches a list of these to the next active delivery in the chat so the
+* agent's prompt carries enough context to reply meaningfully.
+*
+* Smaller than `messageSchema` on purpose — drops fields that don't help the
+* LLM (replyTo envelopes, source) and aren't safe to leak across recipients.
+*/
+const precedingMessageSchema = z.object({
+	id: z.string(),
+	senderId: z.string(),
+	format: z.string(),
+	content: z.unknown(),
+	metadata: z.record(z.string(), z.unknown()).default({}),
+	createdAt: z.string()
+});
+/**
 * Wire format for messages routed FROM the Hub TO a client runtime.
 *
 * Adds `configVersion` so the client can compare against its locally cached
@@ -605,11 +637,17 @@ const participantModeSchema = z.enum(["full", "mention_only"]);
 *
 * `inReplyToSnapshot` is populated when `inReplyTo` resolves to an existing
 * message; runtime uses it to suppress self-reply echo on direct chats.
+*
+* `precedingMessages` is a (possibly empty) list of older messages in the
+* same chat that this recipient did not previously receive (silent inbox
+* context). The runtime renders them as "earlier in chat" before the
+* triggering message — see proposals/group-chat-ux-improvements §1.
 */
 const clientMessageSchema = messageSchema.extend({
 	configVersion: z.number().int().positive(),
 	recipientMode: participantModeSchema.default("full"),
-	inReplyToSnapshot: inReplyToSnapshotSchema.default(null)
+	inReplyToSnapshot: inReplyToSnapshotSchema.default(null),
+	precedingMessages: z.array(precedingMessageSchema).default([])
 });
 z.enum([
 	"pending",
@@ -1072,4 +1110,4 @@ async function bindFeishuUser(serverUrl, accessToken, agentId, humanAgentId, fei
 	}
 }
 //#endregion
-export { sessionEventSchema as $, createChatSchema as A, isReservedAgentName as B, agentRuntimeConfigPayloadSchema as C, createAdapterConfigSchema as D, connectTokenExchangeSchema as E, dryRunAgentRuntimeConfigSchema as F, paginationQuerySchema as G, loginSchema as H, extractMentions as I, selfServiceFeishuBotSchema as J, refreshTokenSchema as K, imageInlineContentSchema as L, createOrganizationSchema as M, createTaskSchema as N, createAdapterMappingSchema as O, delegateFeishuUserSchema as P, sessionEventMessageSchema as Q, inboxPollQuerySchema as R, agentPinnedMessageSchema as S, clientRegisterSchema as T, messageSourceSchema as U, linkTaskChatSchema as V, notificationQuerySchema as W, sendToAgentSchema as X, sendMessageSchema as Y, sessionCompletionMessageSchema as Z, WS_AUTH_FRAME_TIMEOUT_MS as _, AGENT_NAME_REGEX as a, updateAgentSchema as at, adminUpdateTaskSchema as b, AGENT_STATUSES as c, updateOrganizationSchema as ct, DEFAULT_AGENT_RUNTIME_CONFIG_PAYLOAD as d, wsAuthFrameSchema as dt, sessionReconcileRequestSchema as et, SYSTEM_CONFIG_DEFAULTS as f, TASK_TERMINAL_STATUSES as g, TASK_STATUSES as h, AGENT_BIND_REJECT_REASONS as i, updateAgentRuntimeConfigSchema as it, createMemberSchema as j, createAgentSchema as k, AGENT_TYPES as l, updateSystemConfigSchema as lt, TASK_HEALTH_SIGNALS as m, bindFeishuUser as n, taskListQuerySchema as nt, AGENT_SELECTOR_HEADER as o, updateChatSchema as ot, TASK_CREATOR_TYPES as p, runtimeStateMessageSchema as q, feishu_exports as r, updateAdapterConfigSchema as rt, AGENT_SOURCES as s, updateMemberSchema as st, bindFeishuBot as t, sessionStateMessageSchema as tt, AGENT_VISIBILITY as u, updateTaskStatusSchema as ut, addParticipantSchema as v, agentTypeSchema as w, agentBindRequestSchema as x, adminCreateTaskSchema as y, isRedactedEnvValue as z };
+export { sessionEventMessageSchema as $, createChatSchema as A, isReservedAgentName as B, agentRuntimeConfigPayloadSchema as C, createAdapterConfigSchema as D, connectTokenExchangeSchema as E, dryRunAgentRuntimeConfigSchema as F, paginationQuerySchema as G, loginSchema as H, extractMentions as I, scanMentionTokens as J, refreshTokenSchema as K, imageInlineContentSchema as L, createOrganizationSchema as M, createTaskSchema as N, createAdapterMappingSchema as O, delegateFeishuUserSchema as P, sessionCompletionMessageSchema as Q, inboxPollQuerySchema as R, agentPinnedMessageSchema as S, clientRegisterSchema as T, messageSourceSchema as U, linkTaskChatSchema as V, notificationQuerySchema as W, sendMessageSchema as X, selfServiceFeishuBotSchema as Y, sendToAgentSchema as Z, WS_AUTH_FRAME_TIMEOUT_MS as _, AGENT_NAME_REGEX as a, updateAgentRuntimeConfigSchema as at, adminUpdateTaskSchema as b, AGENT_STATUSES as c, updateMemberSchema as ct, DEFAULT_AGENT_RUNTIME_CONFIG_PAYLOAD as d, updateTaskStatusSchema as dt, sessionEventSchema as et, SYSTEM_CONFIG_DEFAULTS as f, wsAuthFrameSchema as ft, TASK_TERMINAL_STATUSES as g, TASK_STATUSES as h, AGENT_BIND_REJECT_REASONS as i, updateAdapterConfigSchema as it, createMemberSchema as j, createAgentSchema as k, AGENT_TYPES as l, updateOrganizationSchema as lt, TASK_HEALTH_SIGNALS as m, bindFeishuUser as n, sessionStateMessageSchema as nt, AGENT_SELECTOR_HEADER as o, updateAgentSchema as ot, TASK_CREATOR_TYPES as p, runtimeStateMessageSchema as q, feishu_exports as r, taskListQuerySchema as rt, AGENT_SOURCES as s, updateChatSchema as st, bindFeishuBot as t, sessionReconcileRequestSchema as tt, AGENT_VISIBILITY as u, updateSystemConfigSchema as ut, addParticipantSchema as v, agentTypeSchema as w, agentBindRequestSchema as x, adminCreateTaskSchema as y, isRedactedEnvValue as z };

package/dist/index.mjs

@@ -1,6 +1,6 @@
 import "./observability-DV_fQKqV-oxfXX6Z2.mjs";
-import { A as resolveCliInvocation, C as checkServerHealth, D as getClientServiceStatus, E as printResults, F as ClientRuntime, I as handleClientOrgMismatch, K as FirstTreeHubSDK, L as rotateClientIdWithBackup, M as ensurePostgres, N as isDockerAvailable, O as installClientService, P as stopPostgres, R as createOwner, S as checkServerConfig, T as checkWebSocket, V as blank, W as status, b as checkDocker, c as promptMissingFields, d as onboardCheck, f as onboardCreate, g as checkAgentConfigs, h as runMigrations, i as startServer, j as uninstallClientService, k as isServiceSupported, l as formatCheckReport, m as runHomeMigration, o as isInteractive, q as SdkError, s as promptAddAgent, v as checkClientConfig, w as checkServerReachable, x as checkNodeVersion, y as checkDatabase, z as hasUser } from "./core-CuSIXoof.mjs";
+import { A as installClientService, B as createOwner, C as checkNodeVersion, D as checkWebSocket, E as checkServerReachable, F as isDockerAvailable, I as stopPostgres, J as FirstTreeHubSDK, K as status, L as ClientRuntime, M as resolveCliInvocation, N as uninstallClientService, O as printResults, P as ensurePostgres, R as handleClientOrgMismatch, S as checkDocker, T as checkServerHealth, U as blank, V as hasUser, Y as SdkError, _ as runMigrations, b as checkClientConfig, c as promptMissingFields, d as onboardCheck, f as onboardCreate, i as startServer, j as isServiceSupported, k as getClientServiceStatus, l as formatCheckReport, m as runHomeMigration, o as isInteractive, s as promptAddAgent, v as checkAgentConfigs, w as checkServerConfig, x as checkDatabase, z as rotateClientIdWithBackup } from "./core-BgiFGT7Y.mjs";
 import "./logger-core-BTmvdflj-DjW8FM4T.mjs";
 import { a as resolveAccessToken, n as ensureFreshAccessToken, o as resolveServerUrl, r as ensureFreshAdminToken } from "./bootstrap-CtVqQA8a.mjs";
-import { n as bindFeishuUser, t as bindFeishuBot } from "./feishu-B2sjp6Z6.mjs";
+import { n as bindFeishuUser, t as bindFeishuBot } from "./feishu-DEmwoNn_.mjs";
 export { ClientRuntime, FirstTreeHubSDK, SdkError, bindFeishuBot, bindFeishuUser, blank, checkAgentConfigs, checkClientConfig, checkDatabase, checkDocker, checkNodeVersion, checkServerConfig, checkServerHealth, checkServerReachable, checkWebSocket, createOwner, ensureFreshAccessToken, ensureFreshAdminToken, ensurePostgres, formatCheckReport, getClientServiceStatus, handleClientOrgMismatch, hasUser, installClientService, isDockerAvailable, isInteractive, isServiceSupported, onboardCheck, onboardCreate, printResults, promptAddAgent, promptMissingFields, resolveAccessToken, resolveCliInvocation, resolveServerUrl, rotateClientIdWithBackup, runHomeMigration, runMigrations, startServer, status, stopPostgres, uninstallClientService };