cat << 'EOF' | git apply
diff --git a/src/libs/subsonic/impl/payloads/ClientInfo.cpp b/src/libs/subsonic/impl/payloads/ClientInfo.cpp
--- a/src/libs/subsonic/impl/payloads/ClientInfo.cpp
+++ b/src/libs/subsonic/impl/payloads/ClientInfo.cpp
@@ -187,17 +187,40 @@
         }
     } // namespace
 
+    /**
+     * Parses a ClientInfo object from a JSON input stream.
+     *
+     * This function reads the entire input stream, interprets it as a JSON object,
+     * and extracts all required and optional ClientInfo attributes.
+     *
+     * Expected keys include:
+     *   - name: Name of the client (string, mandatory)
+     *   - ... (other expected fields, depending on implementation)
+     *
+     * Throws:
+     *   - BadParameterGenericError if required fields are missing or malformed,
+     *     or if parsing fails.
+     *
+     * Parameters:
+     *   is: Input stream containing JSON client info
+     *
+     * Returns:
+     *   A populated ClientInfo structure.
+     */
     ClientInfo parseClientInfoFromJson(std::istream& is)
     {
         ClientInfo res;
 
+        // Read the entire input stream into a string for parsing
         const std::string msgBody{ std::istreambuf_iterator<char>{ is }, std::istreambuf_iterator<char>{} };
 
         try
         {
             Wt::Json::Object root;
+            // Parse JSON from string into root object; throws on malformed JSON
             Wt::Json::parse(msgBody, root);
 
+            // Extract mandatory "name" field from JSON
             res.name = parseMandatoryValue<std::string>(root, "name");
             res.platform = parseMandatoryValue<std::string>(root, "platform");
             {
@@ -316,9 +328,11 @@
         }
         catch (const Wt::WException& error)
         {
+            // Wrap and rethrow any JSON parsing or field access exceptions as BadParameterGenericError
             throw BadParameterGenericError{ "ClientInfo", error.what() };
         }
 
+        // Return the populated ClientInfo object
         return res;
     }
 } // namespace lms::api::subsonic
\ No newline at end of file
EOF

cat << 'EOF' | git apply
diff --git a/src/libs/subsonic/test/ClientInfo.cpp b/src/libs/subsonic/test/ClientInfo.cpp
--- a/src/libs/subsonic/test/ClientInfo.cpp
+++ b/src/libs/subsonic/test/ClientInfo.cpp
@@ -27,7 +27,24 @@
 {
     TEST(ClientInfo, basic)
     {
-        // Example as in https://opensubsonic.netlify.app/docs/payloads/clientinfo/
+        /**
+         * This test verifies parsing and validation of the ClientInfo structure
+         * according to the Subsonic API specification. It uses a comprehensive
+         * JSON payload, representative of a real client profile, with multiple
+         * nested arrays (direct play profiles, transcoding profiles, codec profiles).
+         * Each field and object is checked to ensure correct deserialization and
+         * interpretation of values.
+         *
+         * Key checks include:
+         *  - Name and platform extraction.
+         *  - Maximum audio/transcoding bitrates.
+         *  - Direct play profiles, including container, codec, protocol, and channel handling.
+         *  - Transcoding profiles validation.
+         *  - Codec profile parsing and limitations validation.
+        */
+
+        // JSON test data taken from Subsonic payload documentation:
+        // https://opensubsonic.netlify.app/docs/payloads/clientinfo/
         std::istringstream iss{ R"({
   "name": "Play:1",
   "platform": "Sonos",
EOF

cat << 'EOF' | git apply
diff --git a/src/libs/subsonic/test/TranscodeDecision.cpp b/src/libs/subsonic/test/TranscodeDecision.cpp
--- a/src/libs/subsonic/test/TranscodeDecision.cpp
+++ b/src/libs/subsonic/test/TranscodeDecision.cpp
@@ -90,10 +90,28 @@
         }
     } // namespace
 
+    /**
+     * Test suite for TranscodeDecision logic.
+     *
+     * This test exercises the logic for deciding whether to transcode or allow direct play
+     * of audio streams given a wide variety of client profiles, media formats, and codec/container
+     * properties. The testCases array captures various scenarios, including:
+     *   - Direct play (no transcoding required)
+     *   - Scenarios that trigger audio transcoding for bitrate, codec, or container mismatches
+     *   - Failure cases where playback is not possible due to unhandled formats or restrictions
+     *
+     * Each test case specifies a ClientInfo object (playback device capabilities),
+     * the source audio properties, and the expected transcode decision (direct play, transcode, or failure).
+     * 
+     * The utility function processTests iterates each scenario, checks that the computed result matches
+     * the expected result, and provides index information for easier debugging.
+     *
+     * This comprehensive parameterized test helps ensure stability as format and device support evolves.
+     */
     TEST(TranscodeDecision, directPlay)
     {
         const TestCase testCases[]{
-            // Direct play
+            // === Direct play: scenarios that should allow play without transcoding ===
             {
                 .clientInfo = {
                     .name = "TestClient",
EOF

cat << 'EOF' | git apply
diff --git a/src/libs/subsonic/impl/endpoints/transcoding/TranscodeDecision.cpp b/src/libs/subsonic/impl/endpoints/transcoding/TranscodeDecision.cpp
--- a/src/libs/subsonic/impl/endpoints/transcoding/TranscodeDecision.cpp
+++ b/src/libs/subsonic/impl/endpoints/transcoding/TranscodeDecision.cpp
@@ -328,7 +328,7 @@
                 valueToCheck = static_cast<unsigned>(source.sampleRate);
                 break;
             case Limitation::Type::AudioProfile:
-                // TODO;
+                // TODO: Handle AudioProfile limitation
                 break;
             case Limitation::Type::AudioBitdepth:
                 if (source.bitsPerSample)
EOF