commit ac492c9
Author: Tomelia1999 <76162525+Tomelia1999@users.noreply.github.com>
Date:   Wed Dec 31 13:57:00 2025 +0200

    fix: Strict timeout statemachine (#2160)

    * update the state machine to check if global context timer is done

    * unit test

    * adding timeout verification in the begining of the stetMachine

    ---------

    Co-authored-by: Nim Rod <nimrod.teich@gmail.com>

commit 3c21210
Author: Nim Rod <nimrod.teich@gmail.com>
Date:   Wed Dec 31 13:56:47 2025 +0200

    fix: Hiesen tests failing randomly  (#2166)

    * fix: only add WebSocket URLs for API interfaces that support them

    REST API only supports HTTP/HTTPS, not WebSocket. Previously,
    CreateChainLibMocks was unconditionally adding ws:// URLs for all
    non-gRPC interfaces, causing test failures with:
      ERR URL scheme should be (http/https), got: ws apiInterface=rest

    Now WebSocket URLs are only added for JsonRPC and TendermintRPC
    interfaces which actually support WebSocket connections.

    * fix: set Enabled: true for test Endpoints to prevent provider blocking

    When creating Endpoint structs in tests without setting Enabled: true,
    the Go zero-value (false) causes all endpoints to be considered
    disabled, immediately blocking providers with:
      'purging provider after all endpoints are disabled'

    This fix adds Enabled: true to all Endpoint configurations in:
    - rpcconsumer_server_test.go
    - rpcsmartrouter_server_test.go
    - rpcsmartrouter_test.go

commit ae6d527
Author: Nim Rod <nimrod.teich@gmail.com>
Date:   Wed Dec 31 12:33:58 2025 +0200

    fix: add type assertion checks to satisfy forcetypeassert linter (#2164)

commit cf3ac0d
Author: Nim Rod <nimrod.teich@gmail.com>
Date:   Wed Dec 31 11:59:59 2025 +0200

    fix: use dynamic ports in tests to avoid address conflicts (#2162)

    Replace hardcoded port 54321 with dynamically allocated ports using
    net.Listen on :0 to prevent 'address already in use' errors when
    tests run in parallel or when ports are occupied.

    Changes:
    - Add getFreePort() helper function to both test files
    - Update createRpcConsumer() and createRpcSmartRouter() to use dynamic ports

commit f87a634
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Wed Dec 31 11:59:49 2025 +0200

    feat(processingCtx): default processing ctx timeout can be set through flag (#2161)

commit 77c514f
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Tue Dec 30 16:48:11 2025 +0200

    perf: add early return optimization to LavaFormatLog (#2155)

commit f3e59ef
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Tue Dec 30 16:47:55 2025 +0200

    perf: implement double-check locking in cacheAddonAddresses (#2156)

    * perf: implement double-check locking in cacheAddonAddresses

    Replace simple write lock with double-check locking pattern to reduce
    lock contention in cacheAddonAddresses. This allows multiple concurrent
    readers on cache hits (fast path) while maintaining thread safety for
    cache misses (write path).

    Performance improvement:
    - Cache hits: RLock only (allows concurrent access)
    - Cache misses: RLock check → upgrade to Lock → double-check → populate

    This significantly reduces lock wait times when the cache is already
    populated, which is the common case during steady-state operation.

    * refactor: enhance cacheAddonAddresses with improved double-check locking

    Updated the cacheAddonAddresses method to utilize a more efficient double-check locking pattern. This change reduces lock contention by allowing concurrent reads on cache hits while ensuring thread safety during cache misses. The implementation includes cloning the extensions to prevent aliasing and optimizes the cache population process. This refactor aims to improve performance during steady-state operations.

    ---------

    Co-authored-by: avitenzer <tenzer@clara.co.uk>

commit f466762
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Tue Dec 30 16:16:42 2025 +0200

    fix(smart-router): skip block gap warning when expectedBH is MaxInt64 (#2157)

* Adding a fix for redundent retries when pairingListEmpty

* Squashed commit of the following:

commit ac492c9
Author: Tomelia1999 <76162525+Tomelia1999@users.noreply.github.com>
Date:   Wed Dec 31 13:57:00 2025 +0200

    fix: Strict timeout statemachine (#2160)

    * update the state machine to check if global context timer is done

    * unit test

    * adding timeout verification in the begining of the stetMachine

    ---------

    Co-authored-by: Nim Rod <nimrod.teich@gmail.com>

commit 3c21210
Author: Nim Rod <nimrod.teich@gmail.com>
Date:   Wed Dec 31 13:56:47 2025 +0200

    fix: Hiesen tests failing randomly  (#2166)

    * fix: only add WebSocket URLs for API interfaces that support them

    REST API only supports HTTP/HTTPS, not WebSocket. Previously,
    CreateChainLibMocks was unconditionally adding ws:// URLs for all
    non-gRPC interfaces, causing test failures with:
      ERR URL scheme should be (http/https), got: ws apiInterface=rest

    Now WebSocket URLs are only added for JsonRPC and TendermintRPC
    interfaces which actually support WebSocket connections.

    * fix: set Enabled: true for test Endpoints to prevent provider blocking

    When creating Endpoint structs in tests without setting Enabled: true,
    the Go zero-value (false) causes all endpoints to be considered
    disabled, immediately blocking providers with:
      'purging provider after all endpoints are disabled'

    This fix adds Enabled: true to all Endpoint configurations in:
    - rpcconsumer_server_test.go
    - rpcsmartrouter_server_test.go
    - rpcsmartrouter_test.go

commit ae6d527
Author: Nim Rod <nimrod.teich@gmail.com>
Date:   Wed Dec 31 12:33:58 2025 +0200

    fix: add type assertion checks to satisfy forcetypeassert linter (#2164)

commit cf3ac0d
Author: Nim Rod <nimrod.teich@gmail.com>
Date:   Wed Dec 31 11:59:59 2025 +0200

    fix: use dynamic ports in tests to avoid address conflicts (#2162)

    Replace hardcoded port 54321 with dynamically allocated ports using
    net.Listen on :0 to prevent 'address already in use' errors when
    tests run in parallel or when ports are occupied.

    Changes:
    - Add getFreePort() helper function to both test files
    - Update createRpcConsumer() and createRpcSmartRouter() to use dynamic ports

commit f87a634
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Wed Dec 31 11:59:49 2025 +0200

    feat(processingCtx): default processing ctx timeout can be set through flag (#2161)

commit 77c514f
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Tue Dec 30 16:48:11 2025 +0200

    perf: add early return optimization to LavaFormatLog (#2155)

commit f3e59ef
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Tue Dec 30 16:47:55 2025 +0200

    perf: implement double-check locking in cacheAddonAddresses (#2156)

    * perf: implement double-check locking in cacheAddonAddresses

    Replace simple write lock with double-check locking pattern to reduce
    lock contention in cacheAddonAddresses. This allows multiple concurrent
    readers on cache hits (fast path) while maintaining thread safety for
    cache misses (write path).

    Performance improvement:
    - Cache hits: RLock only (allows concurrent access)
    - Cache misses: RLock check → upgrade to Lock → double-check → populate

    This significantly reduces lock wait times when the cache is already
    populated, which is the common case during steady-state operation.

    * refactor: enhance cacheAddonAddresses with improved double-check locking

    Updated the cacheAddonAddresses method to utilize a more efficient double-check locking pattern. This change reduces lock contention by allowing concurrent reads on cache hits while ensuring thread safety during cache misses. The implementation includes cloning the extensions to prevent aliasing and optimizes the cache population process. This refactor aims to improve performance during steady-state operations.

    ---------

    Co-authored-by: avitenzer <tenzer@clara.co.uk>

commit f466762
Author: NadavLevi <30565624+NadavLevi@users.noreply.github.com>
Date:   Tue Dec 30 16:16:42 2025 +0200

    fix(smart-router): skip block gap warning when expectedBH is MaxInt64 (#2157)

---------

Co-authored-by: avitenzer <tenzer@clara.co.uk>

The early return optimization in #2155 broke the contract that LavaFormat*
functions always return an error. When log level was disabled and no original
error was passed, it returned nil instead of an error.

This caused issues like:
  return nil, utils.LavaFormatTrace("error message")
to return (nil, nil) instead of (nil, error), leading to nil pointer panics
in code that relied on the error return.

Changes:
- utils/lavalog.go: Return fmt.Errorf based on description when log level
  disabled and no original error
- protocol/parser/parser.go: Add defensive nil checks as safety net

The early return optimization in #2155 broke the contract that LavaFormat*
functions always return an error. When log level was disabled and no original
error was passed, it returned nil instead of an error.

This caused issues like:
  return nil, utils.LavaFormatTrace("error message")
to return (nil, nil) instead of (nil, error), leading to nil pointer panics
in code that relied on the error return.

Changes:
- utils/lavalog.go: Return fmt.Errorf based on description when log level
  disabled and no original error
- protocol/parser/parser.go: Add defensive nil checks as safety net