@hed-hog/core 0.0.398 → 0.0.400

package/src/mcp/tools/core-roles.mcp-tools.ts

@@ -32,7 +32,7 @@ export class CoreRolesMcpTools {
         page: args.page ?? 1,
         pageSize: args.pageSize ?? 20,
         search: args.search ?? '',
-        sortField: args.sortField ?? 'id',
+        ...(args.sortField ? { sortField: args.sortField } : {}),
         sortOrder: (args.sortOrder as any) ?? 'asc',
         fields: '',
       },
@@ -130,7 +130,6 @@ export class CoreRolesMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });
@@ -171,7 +170,6 @@ export class CoreRolesMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });
@@ -219,7 +217,6 @@ export class CoreRolesMcpTools {
         page: args.page ?? 1,
         pageSize: args.pageSize ?? 20,
         search: '',
-        sortField: 'id',
         sortOrder: PageOrderDirection.Asc,
         fields: '',
       },
@@ -264,7 +261,6 @@ export class CoreRolesMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });

package/src/mcp/tools/core-routes.mcp-tools.ts

@@ -31,7 +31,7 @@ export class CoreRoutesMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: args.search ?? '',
-      sortField: args.sortField ?? 'id',
+      ...(args.sortField ? { sortField: args.sortField } : {}),
       sortOrder: (args.sortOrder as any) ?? 'asc',
       fields: '',
     });
@@ -131,7 +131,6 @@ export class CoreRoutesMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });
@@ -172,7 +171,6 @@ export class CoreRoutesMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });

package/src/mcp/tools/core-screens.mcp-tools.ts

@@ -31,7 +31,7 @@ export class CoreScreensMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: args.search ?? '',
-      sortField: args.sortField ?? 'id',
+      ...(args.sortField ? { sortField: args.sortField } : {}),
       sortOrder: (args.sortOrder as any) ?? 'asc',
       fields: '',
     });
@@ -121,7 +121,6 @@ export class CoreScreensMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });
@@ -162,7 +161,6 @@ export class CoreScreensMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: PageOrderDirection.Asc,
       fields: '',
     });

package/src/mcp/tools/core-sessions.mcp-tools.ts

@@ -26,7 +26,6 @@ export class CoreSessionsMcpTools {
         page: args.page ?? 1,
         pageSize: args.pageSize ?? 20,
         search: '',
-        sortField: 'id',
         sortOrder: PageOrderDirection.Asc,
         fields: '',
       },
@@ -53,7 +52,6 @@ export class CoreSessionsMcpTools {
         page: args.page ?? 1,
         pageSize: args.pageSize ?? 20,
         search: '',
-        sortField: 'id',
         sortOrder: PageOrderDirection.Asc,
         fields: '',
       },

package/src/mcp/tools/core-settings.mcp-tools.ts

@@ -74,7 +74,7 @@ export class CoreSettingsMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: args.search ?? '',
-      sortField: args.sortField ?? 'id',
+      ...(args.sortField ? { sortField: args.sortField } : {}),
       sortOrder: (args.sortOrder as any) ?? 'asc',
       fields: '',
     });
@@ -117,7 +117,6 @@ export class CoreSettingsMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: args.search ?? '',
-      sortField: 'id',
       sortOrder: 'asc' as any,
       fields: '',
     });

package/src/mcp/tools/core-users.mcp-tools.ts

@@ -46,7 +46,7 @@ export class CoreUsersMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: args.search ?? '',
-      sortField: args.sortField ?? 'id',
+      ...(args.sortField ? { sortField: args.sortField } : {}),
       sortOrder: (args.sortOrder as any) ?? 'asc',
       fields: '',
     });

package/src/mcp/tools/core-webhooks.mcp-tools.ts

@@ -30,7 +30,7 @@ export class CoreWebhooksMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: args.search ?? '',
-      sortField: args.sortField ?? 'id',
+      ...(args.sortField ? { sortField: args.sortField } : {}),
       sortOrder: (args.sortOrder as any) ?? 'asc',
       fields: '',
     });
@@ -191,7 +191,6 @@ export class CoreWebhooksMcpTools {
       page: args.page ?? 1,
       pageSize: args.pageSize ?? 20,
       search: '',
-      sortField: 'id',
       sortOrder: 'desc' as any,
       fields: '',
     });

package/src/mcp/types/mcp-context.type.ts

@@ -1,6 +1,15 @@
-export type McpContext = {
-  userId: number;
-  sessionId: string;
-  locale: string;
-  toolName: string;
-};
+export type McpContext = {
+  userId: number;
+  sessionId: string;
+  locale: string;
+  toolName: string;
+  /**
+   * Client OAuth externo que originou a chamada (hoje `claude-connector`), ou
+   * null para o caminho interno — chave `hedhog_mcp_`, JWT de login, mcp-chat.
+   *
+   * Vale por dois motivos: a auditoria precisa distinguir uma acao feita pelo
+   * Claude Desktop de uma feita pelo Claude Code, e o teto de permissao so se
+   * aplica a cliente externo.
+   */
+  clientId: string | null;
+};

package/src/mcp-chat/mcp-chat.service.ts

@@ -562,6 +562,10 @@ export class McpChatService {
               sessionId: '',
               locale,
               toolName: originalName,
+              // O chat do admin e caminho interno: quem chama e o proprio
+              // operador, com a tela na frente. Nao ha client OAuth externo, e
+              // por isso o teto do conector nao se aplica aqui.
+              clientId: null,
             };
             try {
               const result = await entry.handler(args, context);
@@ -746,6 +750,10 @@ export class McpChatService {
               sessionId: '',
               locale,
               toolName: originalName,
+              // O chat do admin e caminho interno: quem chama e o proprio
+              // operador, com a tela na frente. Nao ha client OAuth externo, e
+              // por isso o teto do conector nao se aplica aqui.
+              clientId: null,
             };
             try {
               const result = await entry.handler(args ?? {}, context);

package/src/meeting/meeting-provider.registry.ts

@@ -25,6 +25,12 @@ export interface CreateMeetingInput {
    * participante comum.
    */
   coOrganizerUserIds?: number[];
+  /**
+   * E-mails/UPNs adicionais do Entra ID a coorganizar, além dos resolvidos por
+   * {@link coOrganizerUserIds}. Resolvidos pelo provedor — quem não existir no
+   * diretório é apenas ignorado, nunca falha a criação da reunião.
+   */
+  coOrganizerEmails?: string[];
   /** Convidados do evento de calendário. */
   attendeeEmails?: string[];
   /** Local do evento (aula presencial/híbrida com reunião gerenciada). */
@@ -77,6 +83,40 @@ export interface UpdateMeetingDetailsInput {
   description?: string | null;
 }
 
+/**
+ * Pedido de troca dos coorganizadores extras de uma reunião já existente.
+ *
+ * Separado de `createMeeting` pelo mesmo motivo de `updateAttendees`: corrigir
+ * a reunião de uma aula que já foi criada não pode gerar link novo. Só cobre
+ * os coorganizadores extras (por e-mail) — o coorganizador derivado do
+ * instrutor não é tocado por este caminho.
+ */
+export interface UpdateMeetingCoOrganizersInput {
+  sourceModule: string;
+  sourceEntity: string;
+  sourceEntityId: string;
+  coOrganizerEmails: string[];
+}
+
+/**
+ * Estado da tarefa que traz a gravação do provedor para o storage local.
+ *
+ * Descrito em termos genéricos de propósito: a origem precisa saber se a
+ * importação está andando, travou ou vai retentar, mas não pode depender de
+ * como o provedor a executa. Hoje é um job da fila do módulo Microsoft; num
+ * provedor com webhook não haveria job nenhum, e o contrato continua valendo.
+ */
+export interface MeetingRecordingImport {
+  status: 'pending' | 'running' | 'done' | 'failed';
+  attempts: number;
+  maxAttempts: number;
+  lastError: string | null;
+  startedAt: Date | null;
+  finishedAt: Date | null;
+  /** Quando a próxima tentativa acontece, se houver uma agendada. */
+  nextRetryAt: Date | null;
+}
+
 /**
  * Estado de uma reunião, na forma em que o módulo de origem precisa ver.
  *
@@ -107,6 +147,12 @@ export interface MeetingSummary {
    * acompanha a turma.
    */
   attendeeEmails: string[];
+  /**
+   * Coorganizadores extras (por e-mail) atualmente aplicados na reunião, para
+   * o módulo de origem comparar antes de mandar atualizar — mesmo motivo de
+   * `attendeeEmails`. Não inclui o coorganizador derivado do instrutor.
+   */
+  coOrganizerEmails: string[];
   /** Estado da importação da gravação, quando `autoRecord` está ligado. */
   recordingStatus:
     | 'not_requested'
@@ -116,6 +162,60 @@ export interface MeetingSummary {
     | 'failed';
   recordingFileId: number | null;
   lastError: string | null;
+  /**
+   * Quando o provedor procurou a gravação pela última vez, e quantas vezes já
+   * procurou.
+   *
+   * Existe para a origem distinguir "ainda não deu tempo" de "está procurando e
+   * não acha": os dois são `recordingStatus: 'pending'`, e sem isto a tela do
+   * módulo de origem só pode dizer "aguardando", que é a mesma frase por horas.
+   */
+  recordingCheckedAt: Date | null;
+  recordingAttempts: number;
+  recordingImportedAt: Date | null;
+  /**
+   * Janela em que o provedor procura a gravação, já resolvida em instantes.
+   *
+   * Derivada aqui, e não na origem, porque a carência antes da primeira busca e
+   * a largura da janela são conhecimento do provedor — o LMS não deve saber que
+   * o Teams leva 15 minutos para publicar nem que a busca desiste em 48 h.
+   * `null` quando não há gravação pedida ou o provedor não expõe a janela.
+   */
+  recordingSearchStartsAt: Date | null;
+  recordingSearchExpiresAt: Date | null;
+  /**
+   * Andamento do download da gravação, quando o provedor já o iniciou.
+   *
+   * Vem do provedor e não da origem porque só ele sabe como importa — a
+   * alternativa seria o módulo de origem vasculhar a fila de outro módulo por
+   * `source_entity`, que é exatamente o acoplamento que este contrato evita.
+   */
+  recordingImport: MeetingRecordingImport | null;
+}
+
+/**
+ * Pedido do conteúdo de um anexo do chat de uma reunião.
+ *
+ * A origem repassa os identificadores que recebeu no evento do chat; traduzi-los
+ * em chamadas ao provedor é problema de quem implementa.
+ */
+export interface FetchMeetingChatAttachmentInput {
+  sourceModule: string;
+  sourceEntity: string;
+  sourceEntityId: string;
+  providerMessageId: string;
+  providerAttachmentId: string;
+  /** O que veio no evento; alguns provedores só sabem baixar por ele. */
+  contentUrl?: string | null;
+  /** Acima disso o provedor desiste antes de transferir. */
+  maxBytes?: number;
+}
+
+export interface MeetingChatAttachmentContent {
+  buffer: Buffer;
+  filename: string;
+  mimetype: string;
+  size: number;
 }
 
 /**
@@ -146,8 +246,29 @@ export interface IMeetingProvider {
    * link novo. Tolerante a reunião inexistente (mesmo padrão de updateAttendees).
    */
   updateDetails(input: UpdateMeetingDetailsInput): Promise<void>;
+  /**
+   * Troca os coorganizadores extras de uma reunião já criada, sem gerar link
+   * novo. Tolerante a reunião inexistente (mesmo padrão de updateAttendees).
+   */
+  updateCoOrganizers(input: UpdateMeetingCoOrganizersInput): Promise<void>;
   /** Estado das reuniões das origens informadas, para a UI do módulo de origem. */
   findBySource(ref: MeetingSourceRef): Promise<MeetingSummary[]>;
+  /**
+   * Baixa o conteúdo de um anexo do chat da reunião.
+   *
+   * Opcional de propósito: o chat é o terceiro artefato pós-reunião e nem todo
+   * provedor o expõe. Quem chama testa a existência do método antes — assim um
+   * provedor mais antigo continua válido sem alteração.
+   *
+   * Devolve `null` quando o anexo não pôde ser lido (permissão do arquivo,
+   * item removido, tamanho acima do teto). Não é erro: o arquivo do anexo mora
+   * no drive de quem o enviou, não na caixa que organizou a reunião, então
+   * negativa é desfecho esperado e a origem precisa de um plano B em vez de uma
+   * exceção.
+   */
+  fetchChatAttachment?(
+    input: FetchMeetingChatAttachmentInput,
+  ): Promise<MeetingChatAttachmentContent | null>;
 }
 
 /**

package/src/meeting/meeting.constants.ts

@@ -51,3 +51,77 @@ export interface MeetingAttendanceAvailablePayload {
   providerMeetingId?: string | null;
   records: MeetingAttendanceRecord[];
 }
+
+/**
+ * Evento publicado quando o chat de uma reunião foi capturado.
+ *
+ * Terceiro do mesmo desenho, depois de gravação e presença, e pela mesma razão:
+ * o provedor sabe o que foi escrito no chat, mas não sabe o que é uma aula; o
+ * módulo de origem sabe quem é o instrutor, mas não conhece o Graph.
+ */
+export const MEETING_CHAT_AVAILABLE_EVENT = 'microsoft.meeting.chat.available';
+
+/** Arquivo ou imagem que acompanha uma mensagem do chat. */
+export interface MeetingChatAttachment {
+  providerAttachmentId: string;
+  /**
+   * `attachment` é arquivo compartilhado na conversa; `hosted_content` é imagem
+   * colada no corpo da mensagem. A origem costuma querer só o primeiro — print
+   * de tela raramente é material de aula.
+   */
+  kind: 'attachment' | 'hosted_content';
+  name: string | null;
+  contentType: string | null;
+  /**
+   * Endereço do arquivo no provedor. Não é baixável sem token: serve de rastro
+   * e é o que a origem devolve ao provedor para pedir o conteúdo.
+   */
+  contentUrl: string | null;
+}
+
+/** Uma mensagem do chat, na forma em que a origem precisa ver. */
+export interface MeetingChatMessage {
+  providerMessageId: string;
+  /** Mensagem à qual esta responde, quando houver. */
+  replyToId: string | null;
+  /**
+   * Nulo quando o provedor não identificou quem escreveu (convidado anônimo,
+   * aplicativo, ou evento de sistema). A origem decide o que fazer — e quando a
+   * decisão publica conteúdo para terceiros, o certo é descartar: casar por
+   * nome confundiria homônimos.
+   */
+  senderEmail: string | null;
+  senderDisplayName: string;
+  /** ISO-8601. */
+  sentAt: string;
+  /** `message`, `systemEventMessage`, … — como o provedor classificou. */
+  messageType: string;
+  /**
+   * Texto puro, nunca HTML.
+   *
+   * O payload é persistido como jsonb no outbox, e o HTML do Teams é gordo o
+   * bastante para que uma aula inteira não caiba. Quem precisar do original lê
+   * a tabela do provedor.
+   */
+  text: string;
+  attachments: MeetingChatAttachment[];
+}
+
+export interface MeetingChatAvailablePayload {
+  sourceModule: string;
+  sourceEntity: string;
+  sourceEntityId: string;
+  providerMeetingId?: string | null;
+  providerChatId?: string | null;
+  /** Quantas mensagens o chat tem de verdade, mesmo se `messages` foi cortado. */
+  messageCount: number;
+  firstMessageAt: string | null;
+  lastMessageAt: string | null;
+  /** Mensagens em ordem cronológica, limitadas ao teto do evento. */
+  messages: MeetingChatMessage[];
+  /**
+   * `messages` não traz tudo: o chat passou do teto do evento ou da paginação
+   * do provedor. `messageCount` continua sendo o número real.
+   */
+  truncated: boolean;
+}

package/src/menu/menu.service.spec.ts

@@ -74,3 +74,92 @@ describe('MenuService.ensureValidParent', () => {
     await expect(call('en', 5, 10)).rejects.toThrow(BadRequestException);
   });
 });
+
+describe('MenuService.getMenus', () => {
+  let prisma: any;
+  let service: MenuService;
+
+  const row = (id: number, menuId: number | null, name: string) => ({
+    id,
+    menu_id: menuId,
+    order: id,
+    menu_locale: [{ name }],
+  });
+
+  beforeEach(() => {
+    prisma = {
+      user: { count: jest.fn<any>().mockResolvedValue(1) },
+      menu: { findMany: jest.fn<any>().mockResolvedValue([]) },
+    };
+    service = makeService(prisma);
+  });
+
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('lança BadRequestException quando o usuário não existe', async () => {
+    prisma.user.count.mockResolvedValue(0);
+    await expect(service.getMenus('en', 42)).rejects.toThrow(BadRequestException);
+    expect(prisma.menu.findMany).not.toHaveBeenCalled();
+  });
+
+  it('monta a árvore inteira com uma única consulta', async () => {
+    prisma.menu.findMany.mockResolvedValue([
+      row(1, null, 'LMS'),
+      row(2, 1, 'Cursos'),
+      row(3, 2, 'Estrutura'),
+      row(4, null, 'CRM'),
+    ]);
+
+    const tree = await service.getMenus('en', 42);
+
+    // O ponto da correção: uma query para tudo, não uma por nó.
+    expect(prisma.menu.findMany).toHaveBeenCalledTimes(1);
+    expect(prisma.user.count).toHaveBeenCalledTimes(1);
+
+    expect(tree.map((m: any) => m.id)).toEqual([1, 4]);
+    expect(tree[0].name).toBe('LMS');
+    expect(tree[0].menu.map((m: any) => m.id)).toEqual([2]);
+    expect(tree[0].menu[0].menu.map((m: any) => m.id)).toEqual([3]);
+    expect(tree[1].menu).toEqual([]);
+  });
+
+  it('não promove a raiz um menu cujo pai o usuário não acessa', async () => {
+    // O pai (10) não veio do banco porque o usuário não tem o papel dele; o
+    // filho não pode aparecer solto na raiz por causa disso.
+    prisma.menu.findMany.mockResolvedValue([
+      row(1, null, 'LMS'),
+      row(11, 10, 'Filho de menu inacessível'),
+    ]);
+
+    const tree = await service.getMenus('en', 42);
+
+    expect(tree.map((m: any) => m.id)).toEqual([1]);
+  });
+
+  it('respeita a ordem dos irmãos vinda da consulta', async () => {
+    prisma.menu.findMany.mockResolvedValue([
+      row(7, 1, 'Segundo'),
+      row(1, null, 'Raiz'),
+      row(5, 1, 'Primeiro'),
+    ]);
+
+    const tree = await service.getMenus('en', 42);
+
+    expect(tree[0].menu.map((m: any) => m.id)).toEqual([7, 5]);
+  });
+
+  it('retorna apenas a subárvore quando um menuId é informado', async () => {
+    prisma.menu.findMany.mockResolvedValue([
+      row(1, null, 'LMS'),
+      row(2, 1, 'Cursos'),
+      row(3, 2, 'Estrutura'),
+    ]);
+
+    const tree = await service.getMenus('en', 42, 1);
+
+    expect(tree.map((m: any) => m.id)).toEqual([2]);
+    expect(tree[0].menu.map((m: any) => m.id)).toEqual([3]);
+  });
+});

package/src/menu/menu.service.ts

@@ -258,7 +258,6 @@ export class MenuService {
   }
 
   async getMenus(locale: string, userId: number, menuId = 0) {
-
     const userExists = await this.prismaService.user.count({
       where: {
         id: userId,
@@ -271,13 +270,13 @@ export class MenuService {
       );
     }
 
-    if (menuId === 0) {
-      menuId = null;
-    }
-
-    let menu = (await this.prismaService.menu.findMany({
+    // Uma consulta para a árvore inteira, em vez de uma por nó. A versão
+    // recursiva anterior repetia esta busca — e o `user.count` acima — a cada
+    // nível: com os ~260 itens de menu que as libraries registram, eram
+    // centenas de idas ao banco em série e o `/menu/system` levava ~2s a cada
+    // navegação do admin, mesmo quando a resposta terminava em 304.
+    const rows = (await this.prismaService.menu.findMany({
       where: {
-        menu_id: menuId,
         role_menu: {
           some: {
             role: {
@@ -307,13 +306,32 @@ export class MenuService {
       },
     })) as unknown[] as any[];
 
-    menu = menu.map((m) => itemTranslations('menu_locale', m));
+    const childrenByParent = new Map<number | null, any[]>();
+
+    for (const row of rows) {
+      const item = itemTranslations('menu_locale', row) as any;
+      const parentId = item.menu_id ?? null;
+      const siblings = childrenByParent.get(parentId);
 
-    for (let i = 0; i < menu.length; i++) {
-      menu[i].menu = await this.getMenus(locale, userId, menu[i].id);
+      if (siblings) siblings.push(item);
+      else childrenByParent.set(parentId, [item]);
     }
 
-    return menu;
+    // Descer apenas pelos filhos presentes no resultado preserva a regra
+    // anterior: um menu cujo pai o usuário não acessa continua invisível, em
+    // vez de subir para a raiz. Os irmãos já vêm ordenados por `order` da
+    // consulta, e o agrupamento acima mantém essa ordem.
+    const buildTree = (parentId: number | null): any[] => {
+      const children = childrenByParent.get(parentId) ?? [];
+
+      for (const child of children) {
+        child.menu = buildTree(child.id);
+      }
+
+      return children;
+    };
+
+    return buildTree(menuId === 0 ? null : menuId);
   }
 
   async getSystemMenu(locale: string, userId: number) {

package/src/message-template/dto/update-message-template.dto.ts

@@ -0,0 +1,22 @@
+import { IsBoolean, IsObject, IsOptional } from 'class-validator';
+
+/**
+ * Corpo aceito pela edição de um modelo de mensagem.
+ *
+ * `slug` fica de fora de propósito: quem procura o modelo é o código, e
+ * renomeá-lo pelo painel desligaria o aviso em silêncio. O que se edita aqui é o
+ * texto.
+ *
+ * `name` e `description` também não entram: eles moram no registry, declarados
+ * pela library dona do modelo. Deixar editá-los aqui criaria um rótulo que
+ * diverge do que o código diz — e a tela não teria como saber qual vale.
+ */
+export class UpdateMessageTemplateDTO {
+  @IsOptional()
+  @IsBoolean()
+  is_active?: boolean;
+
+  @IsOptional()
+  @IsObject()
+  locale?: Record<string, { body?: string }>;
+}