Skip to content

Implemented Telemetry, Cache, DefaultImplementation. Refactored components#7

Open
rcrdortiz wants to merge 2 commits intomainfrom
add/code-genration-cache-telemetry
Open

Implemented Telemetry, Cache, DefaultImplementation. Refactored components#7
rcrdortiz wants to merge 2 commits intomainfrom
add/code-genration-cache-telemetry

Conversation

@rcrdortiz
Copy link
Copy Markdown
Owner

@rcrdortiz rcrdortiz commented Apr 27, 2025

Implemented:

  • Telemetry
  • Cache
  • DefaultImplmentation

Refactored:

  • Container
  • AnnotationReader
  • Build chain

rcrdortiz and others added 2 commits April 27, 2025 22:26
…Implementation annotation. Refactored the container.
…x minor issues across codebase.

Rewrote Klist and Kmap to use deferred pipeline evaluation with short-circuit support. Added KlistTest and KmapTest. Updated AnnotationService, Container, and various storage/telemetry classes with minor fixes and improvements. All 268 tests pass.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

❌ Patch coverage is 75.22236% with 195 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.74%. Comparing base (94d91b8) to head (50ba638).

Files with missing lines Patch % Lines
src/Cache/InMemoryCacheService.php 0.00% 41 Missing ⚠️
src/Code/AnnotationCodeGenerator.php 0.00% 35 Missing ⚠️
src/Telemetry/EchoTelemetryService.php 0.00% 33 Missing ⚠️
src/Collection/Klist.php 95.42% 16 Missing ⚠️
src/Annotation/AnnotationService.php 26.31% 14 Missing ⚠️
src/Cache/Annotation/CacheAnnotationHandler.php 0.00% 14 Missing ⚠️
src/Telemetry/Annotation/RecordTimingHandler.php 0.00% 12 Missing ⚠️
src/Container/DependencyResolver.php 68.18% 7 Missing ⚠️
src/Storage/Connection/Mysqli/MysqliStatement.php 0.00% 4 Missing ⚠️
src/Collection/Kmap.php 97.95% 3 Missing ⚠️
... and 8 more

❌ Your patch check has failed because the patch coverage (75.22%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project check has failed because the head coverage (74.74%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main       #7      +/-   ##
============================================
+ Coverage     72.29%   74.74%   +2.44%     
- Complexity      346      566     +220     
============================================
  Files            44       55      +11     
  Lines          1025     1548     +523     
============================================
+ Hits            741     1157     +416     
- Misses          284      391     +107     
Files with missing lines Coverage Δ
src/Annotation/Annotation.php 100.00% <ø> (ø)
src/Annotation/AnnotationReader.php 100.00% <100.00%> (ø)
src/Annotation/MethodExecutionAnnotation.php 0.00% <ø> (ø)
src/Code/MethodCodeGenerator.php 91.89% <ø> (-0.42%) ⬇️
src/Container/Annotation/Inject.php 0.00% <ø> (ø)
src/Container/CircularReferenceGuard.php 100.00% <100.00%> (ø)
src/Container/Container.php 100.00% <100.00%> (+14.47%) ⬆️
...c/MethodExecution/Builder/MethodExecutionProxy.php 100.00% <100.00%> (ø)
src/Reflection/ReflectionUtils.php 98.59% <100.00%> (-0.02%) ⬇️
src/Repository/Handler/RepositoryBuildHandler.php 87.14% <100.00%> (ø)
... and 19 more

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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