AI-Session-Id: 095253e8-7e1c-4578-9779-bf96395021cf
AI-Tool: claude-code
AI-Model: unknown

AI-Session-Id: 095253e8-7e1c-4578-9779-bf96395021cf
AI-Tool: claude-code
AI-Model: unknown

…e access

Make STANDARD_TYPE and RULE_BASED_TYPE public so they are accessible
from parsing-commons once that module is extracted.

Move 18 DTOs from client/dtos, ParsedCondition + ParserUtils from
engine/experiments, and Labels from engine/evaluator into a new
parsing-commons module. Add targeting-engine as its only dependency.
Add parsing-commons dep to client/pom.xml and register module in root pom.xml.

Audited pluggable-storage, redis-wrapper, and testing modules.
None import the moved DTOs, ParserUtils, ParsedCondition, or Labels
directly — no additional dependency changes required.

…g-commons

The git mv deletions were not staged in the original PR 1.2 commit due to
a stash/pop mishap. This commit removes the 21 files from client/ that were
moved to parsing-commons.

…entCacheInMemoryImp

Replace com.google.common.collect.Maps.newConcurrentMap() with
new ConcurrentHashMap<>() so this class can be extracted to
storage-commons without a Guava dependency.

Move 3 RuleBasedSegment cache interfaces, 1 in-memory impl, RuleBasedSegmentParser,
ParsedRuleBasedSegment, and RuleBasedSegment DTO into new storage-commons module.
Depends on parsing-commons and slf4j. Add storage-commons dep to client/pom.xml.

Audited pluggable-storage, redis-wrapper, and testing modules.
None import RuleBasedSegment cache interfaces, ParsedRuleBasedSegment,
RuleBasedSegmentParser, or RuleBasedSegment DTO directly.
No additional dependency changes required.

- Add ExecutorFactory, TelemetryListener, DefinitionsCacheConsumer interfaces
- Refactor SegmentFetcherImp and SegmentSynchronizationTaskImp to use them
- Remove Guava dependencies from segment classes
- Update SplitFactoryImpl and all affected tests

- New segment-commons module with 16 files moved from client:
  - engine/segments: SegmentFetcher, SegmentChangeFetcher, SegmentFetcherImp,
    SegmentImp, SegmentSynchronizationTask, SegmentSynchronizationTaskImp,
    ExecutorFactory, TelemetryListener
  - storages: SegmentCache, SegmentCacheCommons, SegmentCacheConsumer,
    SegmentCacheProducer, DefinitionsCacheConsumer
  - storages/memory: SegmentCacheInMemoryImpl
  - client/dtos: SegmentChange
  - engine/common: FetchOptions
  - engine/experiments: SyncTask
- Remove Guava from SegmentCacheInMemoryImpl and FetchOptions
- Add segment-commons dependency to client/pom.xml

Full build (excluding pre-existing testing/App.java failure) passes:
- pluggable-storage and redis-wrapper compile with segment-commons transitive deps
- SplitFactoryImpl wiring was completed in PR 3.1