Skip to content

feat(tak): consolidate TAK integration phases into single delivery#2061

Open
krital wants to merge 10 commits intodevelopmentfrom
codex/feat-tak
Open

feat(tak): consolidate TAK integration phases into single delivery#2061
krital wants to merge 10 commits intodevelopmentfrom
codex/feat-tak

Conversation

@krital
Copy link
Contributor

@krital krital commented Feb 20, 2026

Summary

  • Consolidates TAK phase 1-5 work into one branch/PR.
  • Aligns TAK integration with MAPS Extension + ProtocolImplFactory patterns.
  • Finalizes MAPS-managed transport-only behavior for TAK stream path.
  • Includes CoT XML support (required), protobuf framing/codec support, and multicast additions from prior phases.
  • Includes style alignment pass (Lombok getters in config + TAK extension cleanup).

Consolidation Notes

  • Removed standalone socket fallback transport classes and related standalone reconnect/transport tests.
  • Removed use_maps_transport toggle from TAK config/tests.
  • Kept TAK codec/framing/extension coverage and MQTT protocol publish/subscribe integration coverage.

Validation Performed

  • mvn -q -Dtest=io.mapsmessaging.network.protocol.impl.tak.TakExtensionTest,io.mapsmessaging.network.protocol.impl.tak.codec.CotXmlCodecTest,io.mapsmessaging.network.protocol.impl.tak.codec.TakProtobufCodecTest,io.mapsmessaging.network.protocol.impl.tak.framing.TakStreamFramerTest,io.mapsmessaging.network.protocol.impl.tak.framing.TakStreamFramerXmlTest test
  • mvn -q -Dtest=io.mapsmessaging.network.protocol.impl.mqtt.SimpleBufferBasedMQTTIT,io.mapsmessaging.network.protocol.impl.mqtt5.SimpleBufferBasedMQTT5IT,io.mapsmessaging.network.protocol.impl.tak.codec.CotXmlCodecTest,io.mapsmessaging.network.protocol.impl.tak.codec.TakProtobufCodecTest,io.mapsmessaging.network.protocol.impl.tak.framing.TakStreamFramerTest,io.mapsmessaging.network.protocol.impl.tak.framing.TakStreamFramerXmlTest,io.mapsmessaging.network.protocol.impl.tak.TakExtensionTest test

Scope Note

  • This PR does not claim a full-repo mvn test pass; the commands above are targeted validation for TAK + relevant MQTT integration paths.

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