fix: return null for missing replacements to allow cache hits

Author: 9romise

src/providers/diagnostics/rules/replacement.ts

@@ -19,8 +19,7 @@ function generateMessage(replacement: ModuleReplacement) {
 
 export const checkReplacement: DiagnosticRule = async (dep) => {
   const replacement = await getReplacement(dep.name)
-  // Fallback for cache compatibility (LRUCache rejects null/undefined)
-  if (!replacement || !('type' in replacement))
+  if (!replacement)
     return
 
   return {

src/utils/api/replacement.ts

@@ -9,11 +9,11 @@ export type {
   ModuleReplacement,
 }
 
-export const getReplacement = memoize<string, Promise<ModuleReplacement>>(async (name) => {
+export const getReplacement = memoize<string, Promise<ModuleReplacement | null>>(async (name) => {
   logger.info(`Fetching replacements for ${name}`)
   const encodedName = encodePackageName(name)
 
-  const result = await ofetch<ModuleReplacement>(`${NPMX_DEV_API}/replacements/${encodedName}`)
+  const result = await ofetch<ModuleReplacement | undefined>(`${NPMX_DEV_API}/replacements/${encodedName}`) ?? null
   logger.info(`Fetched replacements for ${name}`)
 
   return result