Skip to content

RFC 0001: Provider capability model, health scoring, and fallback policy (Phase 1)#22

Merged
victorquinn merged 10 commits intomainfrom
feat/provider-capabilities-fallback
Aug 15, 2025
Merged

RFC 0001: Provider capability model, health scoring, and fallback policy (Phase 1)#22
victorquinn merged 10 commits intomainfrom
feat/provider-capabilities-fallback

Conversation

@victorquinn
Copy link
Member

This PR introduces the scaffolding for RFC 0001.\n\nChanges:\n- Add provider capability/health/policy types (no behavior change)\n- Barrel export for new types\n- Commit RFC document and CRUSH.md; add .crush to .gitignore\n\nWhy:\n- Establishes the type surface to implement ProviderRegistry, policies, and error normalization as follow-ups.\n\nTest plan:\n- All existing tests pass (no behavior changes).\n\n💘 Generated with Crush

…icy; add CRUSH.md and ignore .crush\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…nd barrel export; no behavior changes\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…nWeather and tests with snapshot; keep behavior unchanged\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…it assertions; ensure CI-friendly\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
@codecov
Copy link

codecov bot commented Aug 15, 2025

Codecov Report

❌ Patch coverage is 80.58252% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.53%. Comparing base (90f8e29) to head (4bf6e22).

Files with missing lines Patch % Lines
src/providers/policy.ts 65.00% 8 Missing and 6 partials ⚠️
src/providers/providerRegistry.ts 91.11% 1 Missing and 3 partials ⚠️
src/providers/openweather/client.ts 80.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
- Coverage   93.42%   90.53%   -2.89%     
==========================================
  Files          13       16       +3     
  Lines         289      391     +102     
  Branches       43       76      +33     
==========================================
+ Hits          270      354      +84     
- Misses         11       18       +7     
- Partials        8       19      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…n capabilities map; wire providers to report outcomes; add tests; no behavior change\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…ability filtering, health tracking, and circuit states; add tests\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…y-then-health (append half-open last), and weighted; add tests; adjust ordering to avoid duplicates; increase coverage\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
… raise coverage and exercise uncovered branches\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…Registry to raise overall coverage above baseline\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
…(capabilities, outcome hooks, registry, policy)\n\n💘 Generated with Crush\nCo-Authored-By: Crush <crush@charm.land>
@victorquinn victorquinn merged commit 5cfbdf1 into main Aug 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant