fix(firebase): wrap RuntimeException in FetchException; remove missing ADR doc reference (#151)

- FirebaseConfigValueProvider.fetch() was re-throwing RuntimeException bare,
  bypassing the FetchException wrapper the tests and docs contract requires
- Remove ADR 001 nav entry from mkdocs.yml and dead link from best-practices.md
  (the file was never created, causing mkdocs --strict to abort)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: kirich1409

docs/guides/best-practices.md

@@ -165,7 +165,6 @@ class CheckoutViewModel @Inject constructor(
 }
 ```
 
-See [ADR 001](../adr/001-configvalues-multi-module.md) for the rationale behind this design.
 
 ---
 

mkdocs.yml

@@ -63,5 +63,3 @@ nav:
       - iOS Dead-Code Elimination: ios-integration.md
   - API Reference: api/index.md
   - Changelog: changelog.md
-  - ADRs:
-      - "ADR 001: ConfigValues Multi-Module": adr/001-configvalues-multi-module.md

providers/firebase/src/main/kotlin/dev/androidbroadcast/featured/firebase/FirebaseConfigValueProvider.kt

@@ -98,8 +98,6 @@ public class FirebaseConfigValueProvider(
 
         try {
             task.await()
-        } catch (e: RuntimeException) {
-            throw e
         } catch (e: Exception) {
             throw FetchException("Firebase Remote Config fetch failed", e)
         }