chore: add note and remove amp animation parsing logic

Author: 35C4n0r

README.md

@@ -75,6 +75,10 @@ agentapi server -- goose
 > [!NOTE]
 > When using Codex, Opencode, Copilot, Gemini, Amp or CursorCLI, always specify the agent type explicitly (eg: `agentapi server --type=codex -- codex`), or message formatting may break.
 
+
+> [!NOTE]
+> When using Amp, set `"amp.terminal.animation": false` in `~/.config/amp/settings.json` to disable terminal animations, which can interfere with AgentAPI's message parsing.
+
 An OpenAPI schema is available in [openapi.json](openapi.json).
 
 By default, the server runs on port 3284. Additionally, the server exposes the same OpenAPI schema at http://localhost:3284/openapi.json and the available endpoints in a documentation UI at http://localhost:3284/docs.

lib/termexec/utils.go

@@ -6,31 +6,6 @@ import (
 	"github.com/coder/agentapi/lib/msgfmt"
 )
 
-func calcAmpAnimatedContent(lines []string) (int, bool) {
-	animatedContentEnd := -1
-	firstTextEncountered := false
-	continueRemoving := true
-
-	// search for the first 3 consecutive empty lines after the first text encountered.
-	if len(lines) > 3 {
-		for i := 0; i < len(lines)-3; i++ {
-			if !firstTextEncountered && len(strings.Trim(lines[i], " \n")) != 0 {
-				if strings.HasPrefix(strings.TrimSpace(lines[i]), "┃") {
-					continueRemoving = false
-				}
-				firstTextEncountered = true
-			}
-			if firstTextEncountered && len(strings.Trim(lines[i], " \n")) == 0 && len(strings.Trim(lines[i+1], " \n")) == 0 &&
-				len(strings.Trim(lines[i+2], " \n")) == 0 {
-				animatedContentEnd = i
-				break
-
-			}
-		}
-	}
-	return animatedContentEnd, continueRemoving
-}
-
 func calcOpencodeAnimatedContent(lines []string) (int, bool) {
 	// Skip header lines for Opencode agent type to avoid false positives
 	// The header contains dynamic content (token count, context percentage, cost)
@@ -48,12 +23,11 @@ func removeAnimatedContent(screen string, agentType msgfmt.AgentType) (string, b
 	lines := strings.Split(screen, "\n")
 	animatedContentEnd := -1
 	var continueRemoving bool
-	if agentType == msgfmt.AgentTypeAmp {
-		animatedContentEnd, continueRemoving = calcAmpAnimatedContent(lines)
-	} else if agentType == msgfmt.AgentTypeOpencode {
+	switch agentType {
+	case msgfmt.AgentTypeOpencode:
 		animatedContentEnd, continueRemoving = calcOpencodeAnimatedContent(lines)
-	} else {
-		continueRemoving = false
+	default:
+		return screen, false
 	}
 	return strings.Join(lines[animatedContentEnd+1:], "\n"), continueRemoving
 }