Skip to content

Conversation

@abelonogov-ld
Copy link
Contributor

@abelonogov-ld abelonogov-ld commented Dec 6, 2025

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Provide a clear and concise description of what you expect to happen.

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context about the pull request here.


Note

Introduce identify hooks and integrate them into all identify flows with optional timeout handling, adding IdentifySeriesContext and extending the Hook protocol.

  • Hooks:
    • Extend Hook with beforeIdentify/afterIdentify stages (default no-ops) and docs.
    • Add IdentifySeriesContext for identify hook context.
  • LDClient:
    • Add hook execution for identify via LDClientIdentifyHook.swift (executeWithIdentifyHooks, _identifyHooked).
    • Route all identify(...) overloads through _identifyHooked; timeout variant now uses hook-aware path.
    • Make _identify(...) accessible for reuse (was private).
  • Evaluations:
    • Minor refactor in evaluateWithHooks to early-exit with guard when no hooks.

Written by Cursor Bugbot for commit f30300b. This will update automatically on new commits. Configure here.

@abelonogov-ld abelonogov-ld requested a review from a team as a code owner December 6, 2025 01:27
@abelonogov-ld abelonogov-ld changed the title Identify hooks feat" Identify hooks Dec 6, 2025
@abelonogov-ld abelonogov-ld changed the title feat" Identify hooks feat: Identify hooks Dec 6, 2025
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.

2 participants