More common code

[alexander-yevsyukov]

This PR continues making the logging module more platform-neutral. The details are covered by the sections below.

Classes moved under commonMain

• CountingRateLimiter
• DurationRateLimiter
• LogContext
• LogSiteGroupingKey
• RateLimitingPeriod
• RateLimitStatus
• SamplingRateLimiter

Other notable changes

• The ReqursionDepth class was reworked narrowing down the expect/actual to the newly introduced WeekRef class. So, now the ReqursionDepth class class uses the CouroutineContext key Kotlin's feature, which is platform neutral.
• The LoggingScope, LogPerBucketingStrategy, classes, and LogSiteLookup object were reworked to avoid unnecessary expect/ actual arrngament.
• Added expect/actual fun stackForCallerOf() to make the created stack trace platform-neutral.
• Removed expect/actual arrangement for the WithLogging interface as no longer necessary.
• Removed expect/actual arrrangement for the LoggingScope class as no longer necessary.
• Removed expect/actual arrrangement for the LogPerBucketingStrategy class as no longer necessary.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 119 lines in your changes missing coverage. Please review.
✅ Project coverage is 14.59%. Comparing base (6027baf) to head (5fba00f).
⚠️ Report is 39 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #131      +/-   ##
============================================
- Coverage     15.24%   14.59%   -0.65%     
  Complexity      147      147              
============================================
  Files            96      103       +7     
  Lines          2617     2733     +116     
  Branches        365      380      +15     
============================================
  Hits            399      399              
- Misses         2188     2304     +116     
  Partials         30       30              

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

This PR bumps the version from 2.0.0-SNAPSHOT.380 to 2.0.0-SNAPSHOT.390 and introduces significant architectural improvements to the logging library's multi-platform design. The changes move from platform-specific expect/actual declarations to common implementations with platform-specific details where needed, improving code reuse and maintainability.

Key changes include:

• Consolidation of logging scope and recursion depth management into common code using coroutine contexts
• Refactoring of weak reference handling and stack trace utilities to use shared implementations
• Simplification of the logging API by removing platform-specific abstractions

Reviewed Changes

Copilot reviewed 24 out of 31 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
version.gradle.kts, pom.xml	Version bump to 2.0.0-SNAPSHOT.390
logging/src/commonMain/kotlin/io/spine/logging/util/RecursionDepth.kt	Moved from expect/actual to common implementation using CoroutineContext
logging/src/commonMain/kotlin/io/spine/logging/WithLogging.kt	Changed from expect interface to common interface with default implementation
logging/src/commonMain/kotlin/io/spine/logging/WeakRef.kt	Refactored from WithLogging to WeakRef with platform-agnostic API
logging/src/commonMain/kotlin/io/spine/logging/LoggingScope.kt	Moved from expect/actual to common abstract class with concrete implementation
logging/src/commonMain/kotlin/io/spine/logging/KeyPart.kt	Converted from expect/actual to common implementation
logging/src/jvmMain/kotlin/io/spine/logging/WeakRef.kt	JVM-specific implementation using Java WeakReference
logging/src/jvmMain/kotlin/io/spine/logging/LoggingFactory.kt	Variable renaming for clarity

[Copilot]

Pull Request Overview

Copilot reviewed 25 out of 32 changed files in this pull request and generated 6 comments.

[Copilot]

Pull Request Overview

Copilot reviewed 25 out of 32 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

logging/src/commonMain/kotlin/io/spine/logging/WeakRef.kt:1

• There's a typo in the documentation comment. 'WeekRef' should be 'WeakRef'.

/*

[Copilot]

Pull Request Overview

Copilot reviewed 25 out of 32 changed files in this pull request and generated 4 comments.

[Copilot]

Pull Request Overview

Copilot reviewed 25 out of 32 changed files in this pull request and generated 1 comment.