Merge test into main: Content Agent Slack bot

[recoup-coding-agent]

Merges the content-agent feature (PR #342) from test into main after review, QA, and CI verification.

Summary by CodeRabbit

• New Features

  • Introduced content agent for creating video content with Slack integration.
  • Added webhook callback endpoints to process content generation status updates and post results to threads.

• Refactor

  • Extracted platform routing logic into a reusable factory for consistent webhook handling across agents.
  • Centralized agent state management and thread resolution utilities for shared use.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
recoup-api	Ready	Preview	Mar 26, 2026 5:21pm

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6c61eae4-5dec-4855-80eb-4cb308ecb620

📥 Commits

Reviewing files that changed from the base of the PR and between fdb09e0 and e6beccf.

⛔ Files ignored due to path filters (5)

• lib/agents/content/__tests__/handleContentAgentCallback.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/agents/content/__tests__/isContentAgentConfigured.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/agents/content/__tests__/validateContentAgentEnv.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/coding-agent/__tests__/getThread.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**
• lib/coding-agent/__tests__/handlePRCreated.test.ts is excluded by !**/*.test.*, !**/__tests__/** and included by lib/**

📒 Files selected for processing (20)

• app/api/coding-agent/[platform]/route.ts
• app/api/content-agent/[platform]/route.ts
• app/api/content-agent/callback/route.ts
• lib/agents/content/bot.ts
• lib/agents/content/handleContentAgentCallback.ts
• lib/agents/content/handlers/registerHandlers.ts
• lib/agents/content/handlers/registerOnNewMention.ts
• lib/agents/content/handlers/registerOnSubscribedMessage.ts
• lib/agents/content/isContentAgentConfigured.ts
• lib/agents/content/types.ts
• lib/agents/content/validateContentAgentCallback.ts
• lib/agents/content/validateContentAgentEnv.ts
• lib/agents/createAgentState.ts
• lib/agents/createPlatformRoutes.ts
• lib/agents/getThread.ts
• lib/coding-agent/bot.ts
• lib/coding-agent/getThread.ts
• lib/coding-agent/handleCodingAgentCallback.ts
• lib/coding-agent/handlePRCreated.ts
• lib/trigger/triggerPollContentRun.ts

---

📝 Walkthrough

Walkthrough

This PR refactors the coding-agent webhook routing to use a reusable createPlatformRoutes factory, extracts shared utilities for agent initialization and thread management, and introduces a complete content-agent bot system with webhook handlers, event listeners, and callback processing tied to a Trigger.dev polling workflow.

Changes

Cohort / File(s)	Summary
Coding-agent webhook routing refactor app/api/coding-agent/[platform]/route.ts	Replaced inline GET/POST handlers with createPlatformRoutes({ getBot: () => codingAgentBot }) factory call, delegating platform routing, initialization, and waitUntil behavior to shared utility.
Content-agent webhook routes app/api/content-agent/[platform]/route.ts, app/api/content-agent/callback/route.ts	New webhook endpoints for platform-specific events (GET/POST via createPlatformRoutes) and callback POST handler with secret verification, bot initialization, and state machine callback processing.
Shared platform routing factory lib/agents/createPlatformRoutes.ts	New factory generating parameterized GET/POST handlers with platform validation, optional config readiness checks, Slack URL verification, bot initialization, and waitUntil wrapping.
Shared agent state & utilities lib/agents/createAgentState.ts, lib/agents/getThread.ts	Centralized agent state creation (Redis-backed, shared logger) and thread instantiation by parsing adapter:channel:thread identifiers; getThread extracted from coding-agent for reuse.
Content-agent bot setup lib/agents/content/bot.ts, lib/agents/content/types.ts, lib/agents/content/isContentAgentConfigured.ts, lib/agents/content/validateContentAgentEnv.ts	New content-agent bot singleton, typed ContentAgentThreadState (status: running/completed/failed/timeout plus metadata), environment validation, and configuration checks.
Content-agent event handlers lib/agents/content/handlers/registerHandlers.ts, lib/agents/content/handlers/registerOnNewMention.ts, lib/agents/content/handlers/registerOnSubscribedMessage.ts	Handler registration and event logic: on new mention, resolves artist, triggers batch content creation, and starts polling via Trigger.dev; on subscribed message, posts reminder if agent still running.
Content-agent callback & validation lib/agents/content/handleContentAgentCallback.ts, lib/agents/content/validateContentAgentCallback.ts	Callback handler with secret verification, state machine dispatch (completed/failed/timeout), message formatting with video results, and Zod validation schema for incoming callback payloads.
Trigger integration lib/trigger/triggerPollContentRun.ts	New wrapper calling tasks.trigger("poll-content-run", payload) with run IDs and callback thread ID.
Coding-agent handler import updates lib/coding-agent/handleCodingAgentCallback.ts, lib/coding-agent/handlePRCreated.ts, lib/coding-agent/bot.ts, lib/coding-agent/getThread.ts	Updated imports to use centralized @/lib/agents/getThread with generic type parameter; simplified bot state setup via createAgentState("coding-agent"); removed now-redundant local getThread.

Sequence Diagram

sequenceDiagram
    participant Client as Slack/External
    participant Router as Content-agent Router
    participant Handler as Event Handler
    participant Bot as Content-agent Bot
    participant State as Redis State
    participant Trigger as Trigger.dev
    participant Callback as Callback Handler

    Client->>Router: POST /api/content-agent/[platform]
    Router->>Bot: initialize()
    Bot->>State: connect & load state
    Router->>Handler: dispatch webhook event (onNewMention)
    Handler->>Handler: resolve artist, fetch repo
    Handler->>Trigger: triggerCreateContent (batch times)
    Trigger-->>Handler: collect runIds
    Handler->>State: set state (status: running, runIds)
    Handler->>Trigger: triggerPollContentRun
    Trigger-->>Handler: polling task created
    
    par Polling Loop
        Trigger->>Callback: POST /api/content-agent/callback (poll results)
        Callback->>State: load thread state
        Callback->>Callback: validate status (completed/failed/timeout)
        alt Status: completed
            Callback->>Callback: format video results
            Callback->>Bot: post message to thread
            Callback->>State: mark thread completed
        else Status: failed or timeout
            Callback->>Bot: post error message
            Callback->>State: mark thread failed/timeout
        end
        Callback-->>Trigger: { status: ok }
    end

    Handler->>Client: acknowledgment posted (running)
    Client->>Bot: subscribed message (while running)
    Bot->>Handler: onSubscribedMessage callback
    Handler->>State: check state
    Handler->>Client: reply reminder message

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

The changes span multiple architectural layers (routing factory, bot setup, event handlers, callback state machine) with heterogeneous logic density. The new createPlatformRoutes factory requires careful review of parameter validation and async waitUntil behavior. Content-agent handlers involve state machine transitions, external API integration (Trigger.dev, artist resolution), and message formatting. Coding-agent refactoring is simpler but must verify import consolidation didn't break type safety. The diff mixes new feature code, refactored patterns, and dependency injection updates—each requiring separate reasoning paths.

Possibly related PRs

• feat: Recoup Content Agent Slack bot #342: Directly related—adds the content-agent bot system, createPlatformRoutes factory, and triggerPollContentRun in parallel.
• feat: shared Redis PR state for cross-platform coding agent #265: Related—modifies coding-agent handlers (handleCodingAgentCallback.ts, handlePRCreated.ts) alongside state changes; shares the import consolidation pattern.
• agent: @U0AJM7X8FBR API - we want to expand our current slack integration to in #290: Related—also refactors coding-agent webhook routing and bot adapter code; overlaps on route parameterization and handler dispatch.

Suggested reviewers

• sweetmantech

---

🤖 From two agents, a pattern blooms—
Shared routes and state across the rooms,
Callbacks dance with Trigger's beat,
Platform webhooks now compete,
Architecture clean and sound! ✨

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}