Refactor Avatar component to remove deprecation warning

Author: webstackdev

src/components/Avatar/avatars.ts

@@ -6,10 +6,11 @@
  * the import.meta.glob operation only runs once during the module initialization.
  * @example
  * ```typescript
- * import { AvatarManager } from '@components/Avatar/avatars'
+ * import { AvatarManager, normalizeNameToFilename } from '@components/Avatar/avatars'
  *
  * const avatar = AvatarManager.getInstance().getAvatar('kevin-brown')
  * const allAvatars = AvatarManager.getInstance().getAll()
+ * const filename = normalizeNameToFilename('Chris Southam') // 'chris-southam'
  * ```
  */
 
@@ -22,6 +23,24 @@ type ImageMetadata = {
 
 type AvatarMap = Record<string, ImageMetadata>
 
+/**
+ * Normalize a person's name to a filename format
+ * @param name - The person's name (e.g., "Chris Southam")
+ * @returns The normalized filename (e.g., "chris-southam")
+ * @example
+ * ```typescript
+ * normalizeNameToFilename('Chris Southam') // 'chris-southam'
+ * normalizeNameToFilename('Kevin Brown') // 'kevin-brown'
+ * ```
+ */
+export function normalizeNameToFilename(name: string): string {
+  return name
+    .toLowerCase()
+    .trim()
+    .replace(/\s+/g, '-') // Replace spaces with hyphens
+    .replace(/[^a-z0-9-]/g, '') // Remove non-alphanumeric chars except hyphens
+}
+
 class AvatarManagerClass {
   private static instance: AvatarManagerClass
   private readonly avatarMap: Readonly<AvatarMap>
@@ -111,26 +130,3 @@ class AvatarManagerClass {
 // Export singleton instance
 export const AvatarManager = AvatarManagerClass
 
-// Backward-compatible exports for existing code
-const instance = AvatarManager.getInstance()
-
-/**
- * @deprecated Use AvatarManager.getInstance().getAll() instead
- */
-export const avatarMap = instance.getAll()
-
-/**
- * Get avatar image by filename
- * @deprecated Use AvatarManager.getInstance().getAvatar() instead
- */
-export function getAvatarImage(filename: string): ImageMetadata | undefined {
-  return instance.getAvatar(filename)
-}
-
-/**
- * Get all available avatar filenames
- * @deprecated Use AvatarManager.getInstance().getAvailableAvatars() instead
- */
-export function getAvailableAvatars(): readonly string[] {
-  return instance.getAvailableAvatars()
-}

src/components/Avatar/index.astro

@@ -13,7 +13,7 @@
  */
 
 import { Image } from 'astro:assets'
-import { getAvatarImage } from './avatars.ts'
+import { AvatarManager, normalizeNameToFilename } from './avatars.ts'
 
 export interface Props {
   /** The person's name (will be normalized for filename lookup) */
@@ -24,22 +24,10 @@ export interface Props {
 
 const { name, class: className = '' } = Astro.props
 
-/**
- * Normalize a person's name to a filename format
- * "Chris Southam" -> "chris-southam"
- */
-const normalizeNameToFilename = (personName: string): string => {
-  return personName
-    .toLowerCase()
-    .trim()
-    .replace(/\s+/g, '-') // Replace spaces with hyphens
-    .replace(/[^a-z0-9-]/g, '') // Remove non-alphanumeric chars except hyphens
-}
-
 const normalizedFilename = normalizeNameToFilename(name)
 
 // Get the avatar image if it exists
-const avatarImage = getAvatarImage(normalizedFilename)
+const avatarImage = AvatarManager.getInstance().getAvatar(normalizedFilename)
 
 // If no avatar image and no name for fallback, render nothing
 if (!avatarImage && !name) {