feat: trust a resolved conversation as fully handled by support - #85
Merged
Conversation
A resolved conversation, first time or Nth time, no longer triggers classification/investigation on its own -- the resolved-conversation loop in crisp-classify.mjs now just records "session X seen resolved at time T" into the new state/resolved-seen.json, nothing more. Investigation still happens via three paths: - Manual !tg-autopilot investigate note (unchanged, always overrides). - A conversation reopening (going active again) after a prior resolve -- investigated using only messages after that recorded resolve time. If it gets resolved again before a run ever catches it in that in-between window, it's never investigated for that reopen at all: the resolved-conversation loop just records the newer resolve time on its next pass and there's nothing left to catch. Intentional, not a gap -- a second resolve is trusted the same as the first. - The existing stale-12h-and-never-resolved auto-escalation path, unchanged. Updates PHASE2-SETUP.md § 4c and the README's feature table to describe the new policy, and adds state/resolved-seen.json to crisp-triage.yml's committed state files. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
New policy: a resolved conversation (first time or Nth time) is trusted as fully handled by support and no longer triggers classification/investigation on its own. The resolved-conversation loop in
crisp-classify.mjsnow only records "session X seen resolved at time T" into a newstate/resolved-seen.json.Investigation still happens via three paths:
!tg-autopilot investigatenote — unchanged, always overrides, sees the full transcript.crisp-triagerun ever catches it in that in-between window, it's never investigated for that reopen — the resolved-conversation loop just records the newer resolve time and there's nothing left to catch. This is intentional: a second resolve is trusted the same as the first.Also updates
PHASE2-SETUP.md § 4cand the README's feature table to describe the new policy accurately, and addsstate/resolved-seen.jsontocrisp-triage.yml's committed state files.Test plan
node --checkpassescrisp-triageruns' step summaries for sane counts (resolved conversations fetched vs. actually escalated)