mirror of
https://github.com/hanzoai/insights.git
synced 2026-11-03 00:52:32 +00:00
The tree was a debranded snapshot of upstream taken around 2026-02-20, and upstream has moved about five months since. This replaces it with upstream master, mechanically re-derived: bin/debrand does the rename, and the pipeline and squashed migrations are restored from the shipped tree. Not a merge. The previous import replaced the tree wholesale in a single commit, so there is no shared content history to merge against. Reproducing the rename on the new tree is what actually converges. What survives: .hanzo/workflows, the Dockerfiles and bin/docker* entrypoints, compose files, LLM.md, and insights/migrations. The squashed initial migration is load-bearing — the live database records it by name, so upstream's ~918 individual migrations cannot come back. Nine products are dropped as unreferenced. Twenty-one more that we do not run are kept anyway because the core app still imports them; unpicking those is a refactor and does not belong in a rebase. This base does not build yet, and the reasons are not in the rename: upstream now wants Python 3.13.13 against our 3.12.12 and Django 5.2 against our 4.2, and the dependency manifests still name upstream packages that we republish under our own names. Those are decisions, not renames.
319 lines
10 KiB
Python
319 lines
10 KiB
Python
from __future__ import annotations
|
|
|
|
from collections.abc import Iterable
|
|
from datetime import datetime, timedelta
|
|
from typing import TYPE_CHECKING
|
|
|
|
from django.utils.dateparse import parse_datetime
|
|
|
|
import structlog
|
|
|
|
from insights.exceptions_capture import capture_exception
|
|
from insights.insightsql_queries.ai.session_batch_events_query_runner import (
|
|
SessionBatchEventsQueryRunner,
|
|
create_session_batch_events_query,
|
|
)
|
|
from insights.models.comment import Comment
|
|
from insights.models.person.util import get_persons_by_distinct_ids
|
|
from insights.personinsights_client.caller_tag import personinsights_caller_tag
|
|
|
|
if TYPE_CHECKING:
|
|
from insights.models.team import Team
|
|
|
|
from products.conversations.backend.models import Ticket
|
|
|
|
logger = structlog.get_logger(__name__)
|
|
|
|
MAX_CONVERSATION_CHARS = 8000
|
|
MAX_MESSAGES = 50
|
|
MAX_EVENTS_CONTEXT = 30
|
|
MAX_EXCEPTIONS_CONTEXT = 10
|
|
|
|
|
|
def _get_author_label(message: Comment) -> str:
|
|
ctx = message.item_context or {}
|
|
author_type = ctx.get("author_type", "customer")
|
|
is_private = ctx.get("is_private", False)
|
|
|
|
if author_type == "customer":
|
|
return "Customer"
|
|
if is_private:
|
|
return "Support (private note)"
|
|
return "Support"
|
|
|
|
|
|
def format_conversation(ticket: Ticket, messages: Iterable[Comment]) -> str:
|
|
parts: list[str] = []
|
|
|
|
current_url = None
|
|
if ticket.session_context and isinstance(ticket.session_context, dict):
|
|
current_url = ticket.session_context.get("current_url")
|
|
|
|
if current_url:
|
|
parts.append(f"The customer was on the page: {current_url}")
|
|
parts.append("")
|
|
|
|
parts.append("Conversation:")
|
|
|
|
truncated: list[str] = []
|
|
total_messages = 0
|
|
for msg in messages:
|
|
label = _get_author_label(msg)
|
|
content = (msg.content or "").strip()
|
|
if content:
|
|
truncated.append(f"[{label}]: {content}")
|
|
total_messages += 1
|
|
|
|
truncated = truncated[-MAX_MESSAGES:]
|
|
messages_truncated = total_messages > MAX_MESSAGES
|
|
|
|
total_chars = 0
|
|
kept: list[str] = []
|
|
for line in reversed(truncated):
|
|
if total_chars + len(line) > MAX_CONVERSATION_CHARS:
|
|
break
|
|
kept.append(line)
|
|
total_chars += len(line)
|
|
kept.reverse()
|
|
|
|
if messages_truncated or len(kept) < len(truncated):
|
|
parts.append("[Note: Earlier messages were truncated due to length limits]")
|
|
parts.append("")
|
|
|
|
parts.extend(kept)
|
|
return "\n".join(parts)
|
|
|
|
|
|
def _parse_time_window(ticket_created_at: str | None) -> tuple[str, str | None]:
|
|
"""Return (after, before) for a +/-5-minute window around ticket creation."""
|
|
if ticket_created_at:
|
|
created_at = parse_datetime(ticket_created_at)
|
|
if not created_at:
|
|
created_at = datetime.fromisoformat(ticket_created_at.replace("Z", "+00:00"))
|
|
return (created_at - timedelta(minutes=5)).isoformat(), (created_at + timedelta(minutes=5)).isoformat()
|
|
return "-24h", None
|
|
|
|
|
|
def _fetch_session_events(team: Team, session_id: str, ticket_created_at: str | None) -> list[dict]:
|
|
"""Fetch recent events for a session from Datastore."""
|
|
after, before = _parse_time_window(ticket_created_at)
|
|
|
|
query = create_session_batch_events_query(
|
|
session_ids=[session_id],
|
|
select=["event", "timestamp", "properties.$current_url", "properties.$pathname"],
|
|
events_to_ignore=["$feature_flag_called", "$pageleave", "$pageview"],
|
|
after=after,
|
|
before=before,
|
|
max_total_events=MAX_EVENTS_CONTEXT,
|
|
group_by_session=False,
|
|
)
|
|
|
|
runner = SessionBatchEventsQueryRunner(query=query, team=team)
|
|
response = runner.calculate()
|
|
|
|
events = []
|
|
columns = response.columns or []
|
|
for row in response.results or []:
|
|
event_data = dict(zip(columns, row, strict=False))
|
|
events.append(event_data)
|
|
|
|
return events
|
|
|
|
|
|
def _fetch_session_exceptions(team: Team, session_id: str, ticket_created_at: str | None) -> list[dict]:
|
|
"""Fetch exceptions for a session from Datastore."""
|
|
after, before = _parse_time_window(ticket_created_at)
|
|
|
|
query = create_session_batch_events_query(
|
|
session_ids=[session_id],
|
|
select=[
|
|
"event",
|
|
"timestamp",
|
|
"properties.$exception_values",
|
|
"properties.$exception_types",
|
|
"properties.$exception_message",
|
|
"properties.$exception_type",
|
|
"properties.$current_url",
|
|
],
|
|
events_to_ignore=[],
|
|
after=after,
|
|
before=before,
|
|
max_total_events=MAX_EXCEPTIONS_CONTEXT,
|
|
group_by_session=False,
|
|
event="$exception",
|
|
)
|
|
|
|
runner = SessionBatchEventsQueryRunner(query=query, team=team)
|
|
response = runner.calculate()
|
|
|
|
exceptions = []
|
|
columns = response.columns or []
|
|
for row in response.results or []:
|
|
exc_data = dict(zip(columns, row, strict=False))
|
|
exceptions.append(exc_data)
|
|
|
|
return exceptions
|
|
|
|
|
|
def _first_string(value: object) -> str | None:
|
|
if isinstance(value, str):
|
|
return value
|
|
if isinstance(value, list) and value and isinstance(value[0], str):
|
|
return value[0]
|
|
return None
|
|
|
|
|
|
def _format_enhanced_context(
|
|
conversation_text: str,
|
|
events: list[dict],
|
|
exceptions: list[dict],
|
|
) -> str:
|
|
"""Format the conversation with additional technical context."""
|
|
parts = [conversation_text]
|
|
|
|
if exceptions:
|
|
parts.append("\n\nRecent exceptions from the user's session:")
|
|
for exc in exceptions[-MAX_EXCEPTIONS_CONTEXT:]:
|
|
exc_type = (
|
|
_first_string(exc.get("properties.$exception_types"))
|
|
or _first_string(exc.get("$exception_types"))
|
|
or _first_string(exc.get("properties.$exception_type"))
|
|
or _first_string(exc.get("$exception_type"))
|
|
or "Unknown"
|
|
)
|
|
exc_msg = (
|
|
_first_string(exc.get("properties.$exception_values"))
|
|
or _first_string(exc.get("$exception_values"))
|
|
or _first_string(exc.get("properties.$exception_message"))
|
|
or _first_string(exc.get("$exception_message"))
|
|
or "No message"
|
|
)
|
|
url = exc.get("properties.$current_url") or exc.get("$current_url") or ""
|
|
ts = exc.get("timestamp", "")
|
|
parts.append(f"- [{ts}] {exc_type}: {exc_msg} (on {url})")
|
|
|
|
if events:
|
|
parts.append("\n\nRecent events from the user's session:")
|
|
for evt in events[-MAX_EVENTS_CONTEXT:]:
|
|
event_name = evt.get("event", "unknown")
|
|
url = evt.get("properties.$current_url") or evt.get("$current_url") or ""
|
|
ts = evt.get("timestamp", "")
|
|
parts.append(f"- [{ts}] {event_name} (on {url})")
|
|
|
|
return "\n".join(parts)
|
|
|
|
|
|
PERSON_PROPERTY_ALLOWLIST = frozenset(
|
|
{
|
|
"$geoip_country_name",
|
|
"$geoip_city_name",
|
|
"$geoip_time_zone",
|
|
"$browser",
|
|
"$os",
|
|
"$initial_referrer",
|
|
"$initial_referring_domain",
|
|
}
|
|
)
|
|
|
|
PRIORITY_PERSON_PROPERTIES = (
|
|
"email",
|
|
"name",
|
|
"first_name",
|
|
"last_name",
|
|
"plan",
|
|
"company",
|
|
"organization",
|
|
)
|
|
|
|
MAX_PERSON_PROPERTIES = 30
|
|
|
|
|
|
def _load_person_properties(team: Team, distinct_id: str) -> dict:
|
|
try:
|
|
# Only properties are read here, so skip the per-person distinct-id fetch entirely.
|
|
with personinsights_caller_tag("conversations/suggest-person"):
|
|
persons = get_persons_by_distinct_ids(team_id=team.pk, distinct_ids=[distinct_id], distinct_id_limit=0)
|
|
if persons:
|
|
return persons[0].properties or {}
|
|
except Exception:
|
|
capture_exception(additional_properties={"distinct_id": distinct_id})
|
|
return {}
|
|
|
|
|
|
def _format_person_context(properties: dict) -> str:
|
|
if not properties:
|
|
return ""
|
|
|
|
def _is_usable(key: str, value: object) -> bool:
|
|
if value is None or value == "":
|
|
return False
|
|
if key.startswith("$") and key not in PERSON_PROPERTY_ALLOWLIST:
|
|
return False
|
|
return True
|
|
|
|
filtered: dict[str, str] = {}
|
|
|
|
for key in PRIORITY_PERSON_PROPERTIES:
|
|
if key in properties and _is_usable(key, properties[key]):
|
|
filtered[key] = str(properties[key])
|
|
|
|
for key, value in properties.items():
|
|
if len(filtered) >= MAX_PERSON_PROPERTIES:
|
|
break
|
|
if key in filtered:
|
|
continue
|
|
if not _is_usable(key, value):
|
|
continue
|
|
filtered[key] = str(value)
|
|
|
|
if not filtered:
|
|
return ""
|
|
|
|
lines = [f"- {key}: {value}" for key, value in filtered.items()]
|
|
return "\n".join(lines)
|
|
|
|
|
|
def _build_ticket_context(
|
|
ticket: Ticket,
|
|
messages: list[Comment],
|
|
team: Team,
|
|
) -> str:
|
|
"""Build the full context string that gets injected into the agent as a ContextMessage."""
|
|
conversation_text = format_conversation(ticket, messages)
|
|
events: list[dict] = []
|
|
exceptions: list[dict] = []
|
|
|
|
if ticket.session_id:
|
|
try:
|
|
events = _fetch_session_events(team, ticket.session_id, ticket.created_at.isoformat())
|
|
except Exception:
|
|
capture_exception(additional_properties={"ticket_id": str(ticket.id)})
|
|
|
|
try:
|
|
exceptions = _fetch_session_exceptions(team, ticket.session_id, ticket.created_at.isoformat())
|
|
except Exception:
|
|
capture_exception(additional_properties={"ticket_id": str(ticket.id)})
|
|
|
|
context = (
|
|
_format_enhanced_context(conversation_text, events, exceptions) if events or exceptions else conversation_text
|
|
)
|
|
|
|
# Append person context
|
|
person_props = _load_person_properties(team, ticket.distinct_id)
|
|
person_text = _format_person_context(person_props)
|
|
if person_text:
|
|
context += f"\n\nCustomer properties:\n{person_text}"
|
|
|
|
# Ticket metadata
|
|
meta_parts: list[str] = []
|
|
if ticket.channel_source:
|
|
meta_parts.append(f"Channel: {ticket.channel_source}")
|
|
if ticket.status:
|
|
meta_parts.append(f"Status: {ticket.status}")
|
|
if ticket.session_id:
|
|
meta_parts.append(f"Session ID: {ticket.session_id}")
|
|
if meta_parts:
|
|
context += "\n\nTicket metadata:\n" + "\n".join(f"- {p}" for p in meta_parts)
|
|
|
|
return context
|