Address review comments

Author: isc-dchui

src/cls/IPM/Main.cls

@@ -2269,7 +2269,7 @@ ClassMethod ShowModulesForRepository(
     set server = ##class(%IPM.Repo.Definition).ServerDefinitionKeyOpen(pRepoName,,.tSC)
     $$$ThrowOnError(tSC)
     if server.%IsA("%IPM.Repo.Filesystem.Definition") {
-        write !,"Last full cache rebuild for '", pRepoName, "' was on ", server.CacheLastRebuilt, " (UTC). Run 'repo -n "_pRepoName_" -rebuild-cache' to refresh the cache.",!
+        write !,"Last full cache rebuild for '", pRepoName, "' was on ", server.CacheLastRebuilt, " (UTC). If needed, run 'repo -n "_pRepoName_" -rebuild-cache' to refresh the cache.",!
     }
 }
 

src/cls/IPM/Repo/Filesystem/Cache.cls

@@ -139,10 +139,14 @@ ClassMethod RootNameVersionOpenValidated(
     try {
         // Open the cache entry
         set cacheObj = ..RootNameVersionOpen(root, name, versionString, concurrency, .status)
-        $$$ThrowOnError(status)
+        if $$$ISERR(status) {
+            quit
+        }
 
         set defObj = ##class(%IPM.Repo.Filesystem.Definition).RootIndexOpen(root, , .status)
-        $$$ThrowOnError(status)
+        if $$$ISERR(status) {
+            quit
+        }
 
         // Check for schema migration (old cache entries without FileMTime)
         set dirPath = ##class(%File).NormalizeFilename(cacheObj.SubDirectory, cacheObj.Root)
@@ -153,7 +157,6 @@ ClassMethod RootNameVersionOpenValidated(
                 set horologTime = ##class(%Library.File).GetFileDateModified(filePath)
                 set currentMTime = $zdatetime(horologTime, 3)
                 do defObj.RefreshCacheEntry(cacheObj, filePath, currentMTime)
-                $$$ThrowOnError(status)
             } else {
                 // File doesn't exist - invalidate cache
                 set status = $$$ERROR($$$GeneralError, "module.xml not found in " _ dirPath)
@@ -178,9 +181,6 @@ ClassMethod RootNameVersionOpenValidated(
         // Compare cached mtime with current mtime
         if (cacheObj.FileMTime '= currentMTime) {
             do defObj.RefreshCacheEntry(cacheObj, filePath, currentMTime)
-            if $$$ISERR(status) {
-                quit
-            }
         }
     } catch ex {
         set status = ex.AsStatus()