Refactor project structure and fix small code issues

.editorconfig

@@ -12,3 +12,4 @@ trim_trailing_whitespace = true
 # IDE0060: Remove unused parameter
 dotnet_diagnostic.IDE0060.severity = warning
 configure_await_analysis_mode = library
+resharper_inheritdoc_consider_usage_highlighting = none

.gitignore

@@ -324,3 +324,4 @@ ASALocalRun/
 .artifacts
 
 .tests
+.tokensave

Directory.Packages.props

@@ -1,31 +1,28 @@
 <Project>
-
-    <PropertyGroup>
-        <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
-    </PropertyGroup>
-
-    <ItemGroup>
-        <PackageVersion Include="CreativeCoders.CakeBuild" Version="6.7.1"/>
-        <PackageVersion Include="CreativeCoders.Cli.Core" Version="6.7.0"/>
-        <PackageVersion Include="CreativeCoders.Cli.Hosting" Version="6.7.0"/>
-        <PackageVersion Include="CreativeCoders.Configuration" Version="6.7.0"/>
-        <PackageVersion Include="CreativeCoders.Core" Version="6.7.0"/>
-        <PackageVersion Include="CreativeCoders.Net.JsonRpc" Version="6.7.0"/>
-        <PackageVersion Include="CreativeCoders.Net.XmlRpc" Version="6.7.0"/>
-        <PackageVersion Include="CreativeCoders.SysConsole.Cli.Actions" Version="6.7.0"/>
-        <PackageVersion Include="coverlet.collector" Version="8.0.1"/>
-        <PackageVersion Include="Devlooped.CredentialManager" Version="2.7.0"/>
-        <PackageVersion Include="FakeItEasy" Version="9.0.1"/>
-        <PackageVersion Include="AwesomeAssertions" Version="9.4.0"/>
-        <PackageVersion Include="JetBrains.Annotations" Version="2025.2.4"/>
-        <PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="10.0.6"/>
-        <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="10.0.6"/>
-        <PackageVersion Include="Microsoft.Extensions.Http" Version="10.0.6"/>
-        <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.4.0"/>
-        <PackageVersion Include="Spectre.Console" Version="0.54.0"/>
-        <PackageVersion Include="xunit" Version="2.9.3"/>
-        <PackageVersion Include="xunit.runner.visualstudio" Version="3.1.5"/>
-        <PackageVersion Include="XunitXml.TestLogger" Version="8.0.0"/>
-    </ItemGroup>
-
-</Project>
+  <PropertyGroup>
+    <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
+  </PropertyGroup>
+  <ItemGroup>
+    <PackageVersion Include="CreativeCoders.CakeBuild" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.Cli.Core" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.Cli.Hosting" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.Configuration" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.Core" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.Net.JsonRpc" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.Net.XmlRpc" Version="6.7.2" />
+    <PackageVersion Include="CreativeCoders.SysConsole.Cli.Actions" Version="6.7.2" />
+    <PackageVersion Include="coverlet.collector" Version="8.0.1" />
+    <PackageVersion Include="Devlooped.CredentialManager" Version="2.7.0" />
+    <PackageVersion Include="FakeItEasy" Version="9.0.1" />
+    <PackageVersion Include="AwesomeAssertions" Version="9.4.0" />
+    <PackageVersion Include="JetBrains.Annotations" Version="2025.2.4" />
+    <PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="10.0.6" />
+    <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="10.0.6" />
+    <PackageVersion Include="Microsoft.Extensions.Http" Version="10.0.6" />
+    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.4.0" />
+    <PackageVersion Include="Spectre.Console" Version="0.55.2" />
+    <PackageVersion Include="xunit" Version="2.9.3" />
+    <PackageVersion Include="xunit.runner.visualstudio" Version="3.1.5" />
+    <PackageVersion Include="XunitXml.TestLogger" Version="8.0.0" />
+  </ItemGroup>
+</Project>

HomeMatic.sln

@@ -19,6 +19,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "__global", "__global", "{73
 		GitVersion.yml = GitVersion.yml
 		Directory.Packages.props = Directory.Packages.props
 		.github\dependabot.yml = .github\dependabot.yml
+		.editorconfig = .editorconfig
 	EndProjectSection
 EndProject
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Cli", "Cli", "{B79F3B3E-C9CE-4629-ADE3-B1659AF9C673}"
@@ -74,6 +75,11 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "CreativeCoders.HomeMatic.To
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "CreativeCoders.HomeMatic.Tools.Cli.Commands", "source\Tools\Cli\CreativeCoders.HomeMatic.Tools.Cli.Commands\CreativeCoders.HomeMatic.Tools.Cli.Commands.csproj", "{822ECD72-5DB0-4637-B794-CE27B02827AC}"
 EndProject
+Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "__docs", "__docs", "{24644358-1979-4A95-A64C-C686A71E2AF1}"
+	ProjectSection(SolutionItems) = preProject
+		docs\HomeMatic-XmlRpc.md = docs\HomeMatic-XmlRpc.md
+	EndProjectSection
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU

build/BuildContext.cs

@@ -6,7 +6,6 @@
 using CreativeCoders.CakeBuild.Tasks.Templates.Settings;
 using CreativeCoders.Core;
 using CreativeCoders.Core.Collections;
-using CreativeCoders.Core.IO;
 using JetBrains.Annotations;
 
 namespace Build;

source/CreativeCoders.HomeMatic.Core/CcuDeviceUri.cs

@@ -1,20 +1,50 @@
 using System.Diagnostics.CodeAnalysis;
+using CreativeCoders.HomeMatic.XmlRpc;
 
 namespace CreativeCoders.HomeMatic.Core;
 
+/// <summary>
+/// Represents a URI that uniquely identifies a device on a specific CCU.
+/// </summary>
 [ExcludeFromCodeCoverage]
 public class CcuDeviceUri
 {
+    /// <summary>
+    /// Gets the host name or IP address of the CCU.
+    /// </summary>
+    /// <value>The network host name or IP address of the CCU.</value>
     public required string CcuHost { get; init; }
 
+    /// <summary>
+    /// Gets the logical name of the CCU.
+    /// </summary>
+    /// <value>The human-readable name of the CCU. The default is an empty string.</value>
     public string CcuName { get; init; } = string.Empty;
 
+    /// <summary>
+    /// Gets the kind of device addressed by this URI.
+    /// </summary>
+    /// <value>One of the enumeration values that specifies the device kind.</value>
     public required CcuDeviceKind Kind { get; init; }
 
+    /// <summary>
+    /// Gets the device address within the CCU.
+    /// </summary>
+    /// <value>The device or channel address.</value>
     public required string Address { get; init; }
 
+    /// <summary>
+    /// Gets the preferred display name for the CCU host.
+    /// </summary>
+    /// <value>
+    /// The value of <see cref="CcuName"/> if it is not empty; otherwise, the value of <see cref="CcuHost"/>.
+    /// </value>
     public string HostDisplayName => string.IsNullOrWhiteSpace(CcuName) ? CcuHost : CcuName;
 
+    /// <summary>
+    /// Returns a string representation of this URI in the form <c>{Kind}://{CcuHost}/{Address}</c>.
+    /// </summary>
+    /// <returns>A string representation of the device URI.</returns>
     public override string ToString()
     {
         return $"{Kind}://{CcuHost}/{Address}";

source/CreativeCoders.HomeMatic.Core/CcuLogLevel.cs

@@ -1,15 +1,45 @@
-using JetBrains.Annotations;
+using JetBrains.Annotations;
 
 namespace CreativeCoders.HomeMatic.Core;
 
+/// <summary>
+/// Specifies the severity of log entries produced by the CCU.
+/// </summary>
 [PublicAPI]
 public enum CcuLogLevel
 {
+    /// <summary>
+    /// All log entries are included.
+    /// </summary>
     All = 0,
+
+    /// <summary>
+    /// Detailed diagnostic messages used for debugging.
+    /// </summary>
     Debug = 1,
+
+    /// <summary>
+    /// Informational messages that describe normal operation.
+    /// </summary>
     Info = 2,
+
+    /// <summary>
+    /// Notable events that do not indicate a problem.
+    /// </summary>
     Notice = 3,
+
+    /// <summary>
+    /// Conditions that may indicate a potential problem.
+    /// </summary>
     Warning = 4,
+
+    /// <summary>
+    /// Errors that affect the current operation.
+    /// </summary>
     Error = 5,
+
+    /// <summary>
+    /// Fatal errors that prevent further operation.
+    /// </summary>
     FatalError = 6
-}
+}

source/CreativeCoders.HomeMatic.Core/CreativeCoders.HomeMatic.Core.csproj

@@ -12,4 +12,8 @@
     <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" />
     </ItemGroup>
 
+    <ItemGroup>
+        <ProjectReference Include="..\CreativeCoders.HomeMatic.XmlRpc\CreativeCoders.HomeMatic.XmlRpc.csproj" />
+    </ItemGroup>
+
 </Project>

source/CreativeCoders.HomeMatic.Core/Devices/CcuParameterDescription.cs

@@ -1,29 +1,76 @@
 using System.Collections.Generic;
-using CreativeCoders.HomeMatic.Core.Parameters;
+using CreativeCoders.HomeMatic.XmlRpc.Parameters;
 
 namespace CreativeCoders.HomeMatic.Core.Devices;
 
+/// <summary>
+/// Describes the metadata of a single HomeMatic parameter as reported by the CCU.
+/// </summary>
 public class CcuParameterDescription
 {
+    /// <summary>
+    /// Gets the identifier of the parameter.
+    /// </summary>
+    /// <value>The parameter identifier, or <see langword="null"/> if not provided.</value>
     public required string? Id { get; init; }
 
+    /// <summary>
+    /// Gets the default value of the parameter.
+    /// </summary>
+    /// <value>The default value, or <see langword="null"/> if not provided.</value>
     public required object? DefaultValue { get; init; }
 
+    /// <summary>
+    /// Gets the minimum value allowed for the parameter.
+    /// </summary>
+    /// <value>The minimum value, or <see langword="null"/> if not provided.</value>
     public required object? MinValue { get; init; }
 
+    /// <summary>
+    /// Gets the maximum value allowed for the parameter.
+    /// </summary>
+    /// <value>The maximum value, or <see langword="null"/> if not provided.</value>
     public required object? MaxValue { get; init; }
 
+    /// <summary>
+    /// Gets the raw type string as reported by the CCU.
+    /// </summary>
+    /// <value>The type string, or <see langword="null"/> if not provided.</value>
     public required string? Type { get; init; }
 
+    /// <summary>
+    /// Gets the strongly-typed data type of the parameter.
+    /// </summary>
+    /// <value>One of the enumeration values that specifies the data type.</value>
     public required ParameterDataType DataType { get; init; }
 
+    /// <summary>
+    /// Gets the unit string of the parameter value.
+    /// </summary>
+    /// <value>The unit string, or <see langword="null"/> if not provided.</value>
     public required string? Unit { get; init; }
 
+    /// <summary>
+    /// Gets the tab order used when displaying the parameter in the CCU UI.
+    /// </summary>
+    /// <value>The tab-order index.</value>
     public required int TabOrder { get; init; }
 
+    /// <summary>
+    /// Gets the control hint used for the parameter in the CCU UI.
+    /// </summary>
+    /// <value>The control hint string, or <see langword="null"/> if not provided.</value>
     public required string? Control { get; init; }
 
+    /// <summary>
+    /// Gets the list of allowed enum value names when the parameter is of enum type.
+    /// </summary>
+    /// <value>The enumerable of enum value names.</value>
     public required IEnumerable<string> ValuesList { get; init; } = [];
 
+    /// <summary>
+    /// Gets the special values associated with the parameter (for example, error or invalid markers).
+    /// </summary>
+    /// <value>The enumerable of special-value dictionaries keyed by name.</value>
     public required IEnumerable<Dictionary<string, object>> SpecialValues { get; init; } = [];
 }

source/CreativeCoders.HomeMatic.Core/Devices/ICcuDevice.cs

@@ -2,7 +2,14 @@
 
 namespace CreativeCoders.HomeMatic.Core.Devices;
 
+/// <summary>
+/// Represents a HomeMatic device including its channels and runtime state.
+/// </summary>
 public interface ICcuDevice : ICcuDeviceBase, ICcuDeviceData
 {
+    /// <summary>
+    /// Gets the channels that belong to this device.
+    /// </summary>
+    /// <value>The enumerable of <see cref="ICcuDeviceChannel"/> instances.</value>
     IEnumerable<ICcuDeviceChannel> Channels { get; }
 }