fix(describe): only trust a fresh catalog when auto-detecting type

resolveViaCatalog read the cached catalog unconditionally, so after an edit
(the .mpr mod-time changed) auto-detect could resolve a renamed/deleted name to
a stale type. Gate the catalog lookup on catalogMatchesProject (same MPR path +
unchanged mod-time, mirroring isCacheValid); on a mismatch, fall back to the
authoritative live scan. Verified: fresh catalog stays fast (~4s); a touched
.mpr falls back to the live scan (~8.5s) and still resolves correctly.

Note: mod-time can't detect a project's unsaved in-memory changes (MCP/PED
backend) — that broader catalog-staleness gap is tracked separately.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: ako

cmd/mxcli/cmd_describe.go

@@ -380,6 +380,13 @@ func resolveViaCatalog(projectPath, name string) []string {
 	}
 	defer cat.Close()
 
+	// Only trust a cache that matches the current project file. A stale cache
+	// (the .mpr changed since it was built) could resolve a renamed/deleted name
+	// to the wrong type, so fall back to the authoritative live scan instead.
+	if !catalogMatchesProject(cat, projectPath) {
+		return nil
+	}
+
 	q := "'" + strings.ReplaceAll(name, "'", "''") + "'"
 	var out []string
 	// Documents (the objects union view).
@@ -397,6 +404,34 @@ func resolveViaCatalog(projectPath, name string) []string {
 	return out
 }
 
+// catalogMatchesProject reports whether the cached catalog was built from the
+// current project file and is still up to date — same MPR path and unchanged
+// modification time (Unix seconds, mirroring isCacheValid). Any read failure or
+// mismatch is treated as "not fresh" so the caller live-scans (safe but slower).
+func catalogMatchesProject(cat *catalog.Catalog, projectPath string) bool {
+	info, err := cat.GetCacheInfo()
+	if err != nil {
+		return false
+	}
+	fi, err := os.Stat(projectPath)
+	if err != nil {
+		return false
+	}
+	if info.MprModTime.Unix() != fi.ModTime().Unix() {
+		return false
+	}
+	// Compare absolute paths when both resolve; a path mismatch means the cache
+	// is for a different .mpr in the same directory.
+	if info.MprPath != "" {
+		a, err1 := filepath.Abs(info.MprPath)
+		b, err2 := filepath.Abs(projectPath)
+		if err1 == nil && err2 == nil && a != b {
+			return false
+		}
+	}
+	return true
+}
+
 // resolveViaReader scans the live project for the name. Authoritative but slower
 // than the catalog (it enumerates documents). Covers top-level documents plus
 // entities and associations (including cross-module associations).