a5fcecb3f6
This PR adds a Captain Assistant **Overview** page to show some KPI metrics (conversations handled, auto-resolution, handoff, hours saved, reopen-after-resolve, conversation depth) with trend deltas vs the previous window, a real knowledge card, and a lazily-loaded, cached LLM welcome summary. ### Highlights - **Two contextual banners** on the overview: - **Inbox banner** — prompts the user to connect an inbox when the assistant has none, so it can actually do work. - **Coverage banner** — warns when FAQ coverage is below 85% with more than 100 responses pending review, linking straight to the pending queue. Dismissal persists per-assistant for 24h via localStorage. - **Batched stats builder** (`Captain::AssistantStatsBuilder`) computes both windows in single FILTER-aggregated scans to cut round trips, behind new `stats`/`summary` endpoints. - **Cards included but intentionally left dummy / not rendered yet:** `ResponseQualityCard` (flagged responses) and `CreditUsageCard` (credit usage + daily chart). Credits are an account-wide counter with no per-assistant or daily history, so there is no real data to back them yet; they ship in the codebase but are not wired into the page. ### Index migration - Replaces `index_messages_on_sender_type_and_sender_id` with `index_messages_on_sender_and_created` `(sender_type, sender_id, created_at)`. - **Why it helps:** the per-assistant windowed lookups filter `sender_*` *and* a `created_at` range. The old 2-column index matched every lifetime row for the assistant and filtered the time slice at the heap (~89% of rows discarded); adding `created_at` as a range column lets Postgres scan only the window, and fixes the row-count estimate so the planner picks a hash join over a nested loop on `reporting_events`. - **Why dropping the old index is safe:** the new index is a left-prefix superset `(sender_type, sender_id, ...)`, so every query the old one served is still served. No code references it by name, and dropping it keeps write amplification on `messages` neutral. Built/dropped with `CONCURRENTLY` and `if_not_exists`/`if_exists` guards. ## Preview <img width="2572" height="1754" alt="CleanShot 2026-06-29 at 22 38 51@2x" src="https://github.com/user-attachments/assets/3798d09e-7850-48e4-b2cd-508533f15cea" /> ## Banners #### Inbox connect alert <img width="2178" height="612" alt="CleanShot 2026-06-30 at 14 26 55@2x" src="https://github.com/user-attachments/assets/373c371c-bb7d-4291-a0f9-620673078302" /> #### Coverage alert <img width="2178" height="612" alt="CleanShot 2026-06-30 at 14 25 41@2x" src="https://github.com/user-attachments/assets/e12d6308-11b6-4ba2-88a2-8a3077dd3e8f" /> --------- Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com>
88 lines
2.4 KiB
Vue
88 lines
2.4 KiB
Vue
<script setup>
|
|
import { computed } from 'vue';
|
|
import { useRoute } from 'vue-router';
|
|
import { useI18n } from 'vue-i18n';
|
|
|
|
const props = defineProps({
|
|
knowledge: {
|
|
type: Object,
|
|
default: () => ({ approved: 0, pending: 0, documents: 0, coverage: 0 }),
|
|
},
|
|
});
|
|
|
|
const { t } = useI18n();
|
|
const route = useRoute();
|
|
|
|
const approvedPct = computed(() => props.knowledge.coverage ?? 0);
|
|
|
|
const linkTo = routeName => ({
|
|
name: routeName,
|
|
params: {
|
|
accountId: route.params.accountId,
|
|
assistantId: route.params.assistantId,
|
|
},
|
|
});
|
|
|
|
const stats = computed(() => [
|
|
{
|
|
key: 'approved',
|
|
value: props.knowledge.approved,
|
|
label: t('CAPTAIN.OVERVIEW.KNOWLEDGE.APPROVED'),
|
|
to: linkTo('captain_assistants_responses_index'),
|
|
},
|
|
{
|
|
key: 'pending',
|
|
value: props.knowledge.pending,
|
|
label: t('CAPTAIN.OVERVIEW.KNOWLEDGE.PENDING'),
|
|
to: linkTo('captain_assistants_responses_pending'),
|
|
},
|
|
{
|
|
key: 'documents',
|
|
value: props.knowledge.documents,
|
|
label: t('CAPTAIN.OVERVIEW.KNOWLEDGE.DOCUMENTS'),
|
|
to: linkTo('captain_assistants_documents_index'),
|
|
},
|
|
]);
|
|
</script>
|
|
|
|
<template>
|
|
<div
|
|
class="flex flex-col gap-4 p-5 border rounded-xl bg-n-solid-1 border-n-weak"
|
|
>
|
|
<div class="flex items-center justify-between">
|
|
<span class="text-sm font-medium text-n-slate-12">
|
|
{{ $t('CAPTAIN.OVERVIEW.KNOWLEDGE.TITLE') }}
|
|
</span>
|
|
<span class="text-sm tabular-nums text-n-slate-11">
|
|
{{ $t('CAPTAIN.OVERVIEW.KNOWLEDGE.COVERAGE', { pct: approvedPct }) }}
|
|
</span>
|
|
</div>
|
|
<div class="w-full h-2 overflow-hidden rounded-full bg-n-alpha-2">
|
|
<div
|
|
class="h-full rounded-full bg-n-brand"
|
|
:style="{ width: `${approvedPct}%` }"
|
|
/>
|
|
</div>
|
|
<div class="grid grid-cols-3 gap-3">
|
|
<RouterLink
|
|
v-for="stat in stats"
|
|
:key="stat.key"
|
|
:to="stat.to"
|
|
class="flex flex-col gap-1 group/stat"
|
|
>
|
|
<span class="text-xl font-semibold tabular-nums text-n-slate-12">
|
|
{{ stat.value }}
|
|
</span>
|
|
<span
|
|
class="inline-flex items-center gap-1 text-xs transition-colors text-n-slate-11 group-hover/stat:text-n-slate-12"
|
|
>
|
|
{{ stat.label }}
|
|
<span
|
|
class="transition-opacity opacity-0 i-lucide-arrow-up-right size-3 group-hover/stat:opacity-100"
|
|
/>
|
|
</span>
|
|
</RouterLink>
|
|
</div>
|
|
</div>
|
|
</template>
|