mirror of
https://github.com/tiennm99/goclaw.git
synced 2026-10-12 18:16:09 +00:00
viettranx
4a692d51d8
SignFileURLs now heals legacy session data at delivery time: - Dedup double /v1/files/v1/files/ prefix from old mutation bug - Strip stale ?ft= from markdown link display text - Re-sign all URLs with fresh tokens (never skip) File serving: use workspace root for ft= auth fallback search (no tenant context available with signed tokens). Deduplicate ContentSuffix media by basename to prevent duplicate file links when agent text already references the same file. Frontend dedup as defense-in-depth for legacy data.
69 lines
2.0 KiB
Go
69 lines
2.0 KiB
Go
package agent
|
|
|
|
import (
|
|
"path/filepath"
|
|
"strings"
|
|
)
|
|
|
|
// deduplicateMediaSuffix filters ContentSuffix lines, removing any whose file
|
|
// basename already appears in the main content. This prevents duplicate file
|
|
// references when the agent's own text already includes the same media.
|
|
//
|
|
// ContentSuffix format (from mediaToMarkdown):
|
|
//
|
|
// \n\n
|
|
// [filename.md](/v1/files/path/to/filename.md)
|
|
func deduplicateMediaSuffix(content, suffix string) string {
|
|
lines := strings.Split(suffix, "\n")
|
|
var kept []string
|
|
for _, line := range lines {
|
|
if line == "" {
|
|
kept = append(kept, line)
|
|
continue
|
|
}
|
|
// Extract basename from the URL in markdown link/image: ](url) or ](url)
|
|
base := extractLinkBasename(line)
|
|
if base != "" && containsMediaRef(content, base) {
|
|
continue // already referenced in content
|
|
}
|
|
kept = append(kept, line)
|
|
}
|
|
result := strings.Join(kept, "\n")
|
|
// If only empty lines remain, return empty string.
|
|
if strings.TrimSpace(result) == "" {
|
|
return ""
|
|
}
|
|
return result
|
|
}
|
|
|
|
// containsMediaRef checks if content contains a URL reference to a file by basename.
|
|
// Requires URL context (slash before, paren or query after) to avoid false positives
|
|
// matching prose text like "I saved output.png".
|
|
func containsMediaRef(content, basename string) bool {
|
|
return strings.Contains(content, "/"+basename+")") ||
|
|
strings.Contains(content, "/"+basename+"?") ||
|
|
strings.Contains(content, "/"+basename+"\n") ||
|
|
strings.HasSuffix(content, "/"+basename)
|
|
}
|
|
|
|
// extractLinkBasename extracts the filename (basename) from a markdown link or
|
|
// image line. Handles: , [text](url).
|
|
func extractLinkBasename(line string) string {
|
|
// Find the URL between ]( and )
|
|
idx := strings.Index(line, "](")
|
|
if idx < 0 {
|
|
return ""
|
|
}
|
|
urlStart := idx + 2
|
|
urlEnd := strings.LastIndex(line, ")")
|
|
if urlEnd <= urlStart {
|
|
return ""
|
|
}
|
|
url := line[urlStart:urlEnd]
|
|
// Strip query params before taking basename.
|
|
if qIdx := strings.Index(url, "?"); qIdx > 0 {
|
|
url = url[:qIdx]
|
|
}
|
|
return filepath.Base(url)
|
|
}
|