mirror of
https://github.com/logos-co/logos-test-modules.git
synced 2026-08-27 10:11:12 +00:00
Dario LipicarandClaude Opus 4.8
ab4be29aae
* fix(unit-tests): retarget test harnesses to the qt-split SDK stack The unit-test CMakeLists linked logos-cpp-sdk::logos_sdk, which no longer exists after the Qt split: the Qt developer layer (LogosAPI, provider objects) moved to logos-qt-sdk and the transport/consumer core (types, api_client, token_manager, mock transports) moved into the logos-protocol library. Both harnesses (test-ipc-module, test-ipc-module-new-api) now resolve three roots (LOGOS_CPP_SDK_ROOT via logos_module_context.h, plus LOGOS_QT_SDK_ROOT and LOGOS_PROTOCOL_ROOT), take mock-transport headers from logos-protocol, link logos-qt-sdk::logos_qt_sdk (carries the protocol library transitively) plus logos-cpp-sdk::logos_headers, and in source layouts compile the qt-sdk sources while linking the protocol library instead of compiling the old SDK sources in. flake.nix threads the two new roots into both testBin derivations, reaching the packages through logos-liblogos's inputs like the existing cpp-sdk reference. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * lock: pin extraction-chain branch revs for standalone CI Temporary — drop when the chain PRs merge (re-lock against masters). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * lock: qt-split chain fully merged — pins advance to masters (liblogos 050f2d3, logoscore-cli 519e7d3, module-builder 24cec35) * test(thread-safety): update to liblogos simplified C API (#130) liblogos#130 merged the redundant C API functions: set_modules_dir → add_modules_dir; load_module gained a with_dependencies bool (folding in load_module_with_dependencies); unload_module gained a with_dependents bool. test_thread_safety.cpp still called the pre-#130 signatures, so it failed to compile against current liblogos. Mechanical update — load_module(x) → load_module(x,false), load_module_with_dependencies(x) → load_module(x,true), unload_module(x) → unload_module(x,false). Now compiles; 8/10 tests pass (the residual SIGKILL in ConcurrentLoadSamePlugin is the pre-existing heavy-concurrency runtime instability, not API-related). --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>