Dmod_VerifyAllApisSignatures: treat already-connected outputs as valid

[Copilot]

Dmod_VerifyAllApisSignatures reported an error for any output entry that wasn't a valid signature string — including entries that had already been connected (i.e., replaced with a live function pointer by Dmod_ConnectApi). This caused false negatives when verifying a partially- or fully-connected module.

Changes

src/system/public/dmod_dmf_api.c

• Added static helper FindConnectedFunctionSignature(void*) that searches for a pointer in the system builtin inputs (Dmod_BuiltinInputApi) and all loaded module inputs (Dmod_Contexts[i]->Inputs), returning the matched entry's signature string (or NULL if not found).
• Updated the output-entry loop in Dmod_VerifyAllApisSignatures: an entry that fails Dmod_ApiSignature_IsValid is now passed to the helper before being treated as an error.
  • Match found → VERBOSE log with the actual signature of the connected function; result stays true.
  • No match → existing ERROR path, result = false.

if( !Dmod_ApiSignature_IsValid( Context->Outputs.OutputSection->Entries[i] ) )
{
    const char* connectedSignature = FindConnectedFunctionSignature( Context->Outputs.OutputSection->Entries[i] );
    if( connectedSignature != NULL )
    {
        DMOD_LOG_VERBOSE("Output API at index %zu is already connected: %s\n", i, connectedSignature);
    }
    else
    {
        DMOD_LOG_ERROR("Invalid API signature in output API: %s\n", ...);
        result = false;
    }
}

tests/system/public/tests_dmod_verify_apis.cpp (new)

Eight unit tests covering: null context, empty sections, valid signatures, output already connected via builtin inputs, output already connected via loaded-module inputs, unknown pointer (must fail), invalid input signature (must fail), valid input signature.

tests/system/public/CMakeLists.txt

Registered the new test file.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Dodać weryfikowanie czy API jest poprawnie połączone w Dmod_VerifyAllApisSignatures</issue_title>
<issue_description>Aktualnie funkcja Dmod_VerifyAllApisSignatures sprawdza, czy sygnatury są poprawne - ale dobrze by było w przypadku, gdy zawartość nie wygląda na poprawną sygnaturę, sprawdzić czy to czasem nie jest już podłączone API - czyli porównać pointery z dostępnymi funkcjami w systemowych inputach i załadowanych modułach</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Dodać weryfikowanie czy API jest poprawnie połączone w Dmod_VerifyAllApisSignatures #288

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[JohnAmadis]

@copilot popraw to - przerób tę funkcję tak, aby zwracała string z sygnaturą i użyj jej w logach

[JohnAmadis]

@copilot zamiast zwracac po prostu true/false przerob te funkcje tak, aby zwracala sygnature jesli cos znajdzie

[Copilot]

Done in dc13b3c. Renamed to FindConnectedFunctionSignature — now returns const char* (the matched signature) or NULL if nothing is found.

[JohnAmadis]

tutaj bys mogl drukowac sygnature tej funkcji, ktora jest podlaczona uzywajac wynikow tego przeszukiwania, bo samo drukowanie informacji, ze ten pointer jest funkcja niewiele pomaga. Ponadto to nie powinien byc warning, tylko zwykle info/verbose, bo to nie jest nic zlego

[Copilot]

Fixed in dc13b3c. The log now prints the actual signature returned by FindConnectedFunctionSignature, and the level is DMOD_LOG_VERBOSE.

[Copilot]

@copilot popraw to - przerób tę funkcję tak, aby zwracała string z sygnaturą i użyj jej w logach

Refactored in dc13b3c — FindConnectedFunctionSignature returns the signature string (or NULL), and that string is used directly in the DMOD_LOG_VERBOSE call.