[C#] Module bindings for Typed Query Builder (#4159)

# Description of Changes
This is the implementation of the Typed Query Builder for C# modules
outlined in #3750.
This requires the changes from #4146 to be in place before it can
properly function.

This aligns the C# typed query builder SQL formatter with the Rust
implementation:
* Supports `And/Or` with same grouping styles as Rust.
* All comparison operators (`Eq/Neq/Lt/…`) wrap expressions in
parentheses so chained predicates produce byte-for-byte identical SQL.

# API and ABI breaking changes
Not breaking. Existing modules keep producing the same SQL semantics;
only redundant parentheses were added to match the Rust formatter.

# Expected complexity level and risk
2 — localized string-format updates plus parity harness tweaks. Low
functional risk; largest concern is ensuring every comparison path
gained parentheses.

# Testing
- [X] Tested against a local test harness validating output from these
changes match current Rust behavior.

---------

Signed-off-by: Ryan <r.ekhoff@clockworklabs.io>
Co-authored-by: joshua-spacetime <josh@clockworklabs.io>

Author: rekhoff

crates/bindings-csharp/BSATN.Runtime/QueryBuilder.cs

@@ -26,6 +26,9 @@ public static async Task<Fixture> Compile(string name)
             return new(projectDir, (CSharpCompilation)compilation!);
         }
 
+        public Task Verify(string fileName, object target) =>
+            Verifier.Verify(target).UseDirectory($"{projectDir}/snapshots").UseFileName(fileName);
+
         private static CSharpGeneratorDriver CreateDriver(
             IIncrementalGenerator generator,
             LanguageVersion languageVersion
@@ -42,9 +45,6 @@ LanguageVersion languageVersion
             );
         }
 
-        public Task Verify(string fileName, object target) =>
-            Verifier.Verify(target).UseDirectory($"{projectDir}/snapshots").UseFileName(fileName);
-
         private async Task<IEnumerable<SyntaxTree>> RunAndCheckGenerator(
             IIncrementalGenerator generator
         )

crates/bindings-csharp/Codegen.Tests/Tests.cs

@@ -1,5 +1,6 @@
-using System;
+using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Runtime.InteropServices;
 using SpacetimeDB;
 
@@ -39,6 +40,9 @@ public enum CustomEnum
 namespace System.Runtime.CompilerServices
 {
     internal static class IsExternalInit { } // https://stackoverflow.com/a/64749403/1484415
+
+    [AttributeUsage(AttributeTargets.Method, Inherited = false, AllowMultiple = false)]
+    internal sealed class ModuleInitializerAttribute : Attribute { }
 }
 
 [SpacetimeDB.Type]
@@ -48,6 +52,7 @@ public partial record CustomTaggedEnum
 [SpacetimeDB.Type]
 public partial struct PublicTable
 {
+    public int Id;
     public byte ByteField;
     public ushort UshortField;
     public uint UintField;
@@ -74,3 +79,98 @@ public partial struct PublicTable
     public int? NullableValueField;
     public string? NullableReferenceField;
 }
+
+internal static class PublicTableViewRegressions
+{
+    [global::System.Runtime.CompilerServices.ModuleInitializer]
+    internal static void Initialize()
+    {
+        ValidatePublicTableQuerySql();
+        ValidatePublicTableViewSql();
+        ValidateFindPublicTableByIdentitySql();
+    }
+
+    private sealed class PublicTableCols
+    {
+        public Col<PublicTable, int> Id { get; }
+
+        public PublicTableCols(string tableName)
+        {
+            Id = new Col<PublicTable, int>(tableName, "Id");
+        }
+    }
+
+    private sealed class PublicTableIxCols
+    {
+        public IxCol<PublicTable, int> Id { get; }
+
+        public PublicTableIxCols(string tableName)
+        {
+            Id = new IxCol<PublicTable, int>(tableName, "Id");
+        }
+    }
+
+    private static Table<PublicTable, PublicTableCols, PublicTableIxCols> MakeTable()
+    {
+        const string tableName = "PublicTable";
+        return new Table<PublicTable, PublicTableCols, PublicTableIxCols>(
+            tableName,
+            new PublicTableCols(tableName),
+            new PublicTableIxCols(tableName)
+        );
+    }
+
+    private static string BuildPublicTableQuerySql() =>
+        MakeTable().Where(cols => cols.Id.Eq(0)).Build().Sql;
+
+    private static string BuildPublicTableViewSql()
+    {
+        var cols = new PublicTableCols("PublicTable");
+        return MakeTable().Where(_ => cols.Id.Eq(0)).Build().Sql;
+    }
+
+    private static string BuildFindPublicTableByIdentitySql()
+    {
+        var table = MakeTable();
+        return table.Where(cols => cols.Id.Eq(0)).Build().Sql;
+    }
+
+    /// <summary>
+    /// Mirrors Module.PublicTableQuery server view to ensure the generated SQL stays in sync.
+    /// </summary>
+    [Conditional("DEBUG")]
+    public static void ValidatePublicTableQuerySql()
+    {
+        var sql = BuildPublicTableQuerySql();
+        Debug.Assert(
+            sql == "SELECT * FROM \"PublicTable\" WHERE (\"PublicTable\".\"Id\" = 0)",
+            $"Unexpected SQL produced for public_table_query: {sql}"
+        );
+    }
+
+    /// <summary>
+    /// Mirrors Module.PublicTableByIdentity (public_table_view) returning Option<PublicTable>.
+    /// </summary>
+    [Conditional("DEBUG")]
+    public static void ValidatePublicTableViewSql()
+    {
+        var sql = BuildPublicTableViewSql();
+        Debug.Assert(
+            sql == "SELECT * FROM \"PublicTable\" WHERE (\"PublicTable\".\"Id\" = 0)",
+            $"Unexpected SQL produced for public_table_view: {sql}"
+        );
+    }
+
+    /// <summary>
+    /// Mirrors Module.FindPublicTableByIdentity anonymous view.
+    /// </summary>
+    [Conditional("DEBUG")]
+    public static void ValidateFindPublicTableByIdentitySql()
+    {
+        var sql = BuildFindPublicTableByIdentitySql();
+        Debug.Assert(
+            sql == "SELECT * FROM \"PublicTable\" WHERE (\"PublicTable\".\"Id\" = 0)",
+            $"Unexpected SQL produced for find_public_table__by_identity: {sql}"
+        );
+    }
+}

crates/bindings-csharp/Codegen.Tests/fixtures/client/Lib.cs

@@ -6,6 +6,7 @@ partial struct PublicTable : System.IEquatable<PublicTable>, SpacetimeDB.BSATN.I
 {
     public void ReadFields(System.IO.BinaryReader reader)
     {
+        Id = BSATN.IdRW.Read(reader);
         ByteField = BSATN.ByteFieldRW.Read(reader);
         UshortField = BSATN.UshortFieldRW.Read(reader);
         UintField = BSATN.UintFieldRW.Read(reader);
@@ -35,6 +36,7 @@ public void ReadFields(System.IO.BinaryReader reader)
 
     public void WriteFields(System.IO.BinaryWriter writer)
     {
+        BSATN.IdRW.Write(writer, Id);
         BSATN.ByteFieldRW.Write(writer, ByteField);
         BSATN.UshortFieldRW.Write(writer, UshortField);
         BSATN.UintFieldRW.Write(writer, UintField);
@@ -68,10 +70,11 @@ object SpacetimeDB.BSATN.IStructuralReadWrite.GetSerializer()
     }
 
     public override string ToString() =>
-        $"PublicTable {{ ByteField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ByteField)}, UshortField = {SpacetimeDB.BSATN.StringUtil.GenericToString(UshortField)}, UintField = {SpacetimeDB.BSATN.StringUtil.GenericToString(UintField)}, UlongField = {SpacetimeDB.BSATN.StringUtil.GenericToString(UlongField)}, U128Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(U128Field)}, U256Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(U256Field)}, SbyteField = {SpacetimeDB.BSATN.StringUtil.GenericToString(SbyteField)}, ShortField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ShortField)}, IntField = {SpacetimeDB.BSATN.StringUtil.GenericToString(IntField)}, LongField = {SpacetimeDB.BSATN.StringUtil.GenericToString(LongField)}, I128Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(I128Field)}, I256Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(I256Field)}, BoolField = {SpacetimeDB.BSATN.StringUtil.GenericToString(BoolField)}, FloatField = {SpacetimeDB.BSATN.StringUtil.GenericToString(FloatField)}, DoubleField = {SpacetimeDB.BSATN.StringUtil.GenericToString(DoubleField)}, StringField = {SpacetimeDB.BSATN.StringUtil.GenericToString(StringField)}, IdentityField = {SpacetimeDB.BSATN.StringUtil.GenericToString(IdentityField)}, ConnectionIdField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ConnectionIdField)}, CustomStructField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomStructField)}, CustomClassField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomClassField)}, CustomEnumField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomEnumField)}, CustomTaggedEnumField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomTaggedEnumField)}, ListField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ListField)}, NullableValueField = {SpacetimeDB.BSATN.StringUtil.GenericToString(NullableValueField)}, NullableReferenceField = {SpacetimeDB.BSATN.StringUtil.GenericToString(NullableReferenceField)} }}";
+        $"PublicTable {{ Id = {SpacetimeDB.BSATN.StringUtil.GenericToString(Id)}, ByteField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ByteField)}, UshortField = {SpacetimeDB.BSATN.StringUtil.GenericToString(UshortField)}, UintField = {SpacetimeDB.BSATN.StringUtil.GenericToString(UintField)}, UlongField = {SpacetimeDB.BSATN.StringUtil.GenericToString(UlongField)}, U128Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(U128Field)}, U256Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(U256Field)}, SbyteField = {SpacetimeDB.BSATN.StringUtil.GenericToString(SbyteField)}, ShortField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ShortField)}, IntField = {SpacetimeDB.BSATN.StringUtil.GenericToString(IntField)}, LongField = {SpacetimeDB.BSATN.StringUtil.GenericToString(LongField)}, I128Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(I128Field)}, I256Field = {SpacetimeDB.BSATN.StringUtil.GenericToString(I256Field)}, BoolField = {SpacetimeDB.BSATN.StringUtil.GenericToString(BoolField)}, FloatField = {SpacetimeDB.BSATN.StringUtil.GenericToString(FloatField)}, DoubleField = {SpacetimeDB.BSATN.StringUtil.GenericToString(DoubleField)}, StringField = {SpacetimeDB.BSATN.StringUtil.GenericToString(StringField)}, IdentityField = {SpacetimeDB.BSATN.StringUtil.GenericToString(IdentityField)}, ConnectionIdField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ConnectionIdField)}, CustomStructField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomStructField)}, CustomClassField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomClassField)}, CustomEnumField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomEnumField)}, CustomTaggedEnumField = {SpacetimeDB.BSATN.StringUtil.GenericToString(CustomTaggedEnumField)}, ListField = {SpacetimeDB.BSATN.StringUtil.GenericToString(ListField)}, NullableValueField = {SpacetimeDB.BSATN.StringUtil.GenericToString(NullableValueField)}, NullableReferenceField = {SpacetimeDB.BSATN.StringUtil.GenericToString(NullableReferenceField)} }}";
 
     public readonly partial struct BSATN : SpacetimeDB.BSATN.IReadWrite<PublicTable>
     {
+        internal static readonly SpacetimeDB.BSATN.I32 IdRW = new();
         internal static readonly SpacetimeDB.BSATN.U8 ByteFieldRW = new();
         internal static readonly SpacetimeDB.BSATN.U16 UshortFieldRW = new();
         internal static readonly SpacetimeDB.BSATN.U32 UintFieldRW = new();
@@ -123,6 +126,7 @@ SpacetimeDB.BSATN.ITypeRegistrar registrar
             registrar.RegisterType<PublicTable>(_ => new SpacetimeDB.BSATN.AlgebraicType.Product(
                 new SpacetimeDB.BSATN.AggregateElement[]
                 {
+                    new("Id", IdRW.GetAlgebraicType(registrar)),
                     new("ByteField", ByteFieldRW.GetAlgebraicType(registrar)),
                     new("UshortField", UshortFieldRW.GetAlgebraicType(registrar)),
                     new("UintField", UintFieldRW.GetAlgebraicType(registrar)),
@@ -164,6 +168,7 @@ SpacetimeDB.BSATN.ITypeRegistrar registrar
 
     public override int GetHashCode()
     {
+        var ___hashId = Id.GetHashCode();
         var ___hashByteField = ByteField.GetHashCode();
         var ___hashUshortField = UshortField.GetHashCode();
         var ___hashUintField = UintField.GetHashCode();
@@ -202,7 +207,8 @@ public override int GetHashCode()
         var ___hashNullableValueField = NullableValueField.GetHashCode();
         var ___hashNullableReferenceField =
             NullableReferenceField == null ? 0 : NullableReferenceField.GetHashCode();
-        return ___hashByteField
+        return ___hashId
+            ^ ___hashByteField
             ^ ___hashUshortField
             ^ ___hashUintField
             ^ ___hashUlongField
@@ -232,6 +238,7 @@ public override int GetHashCode()
 #nullable enable
     public bool Equals(PublicTable that)
     {
+        var ___eqId = this.Id.Equals(that.Id);
         var ___eqByteField = this.ByteField.Equals(that.ByteField);
         var ___eqUshortField = this.UshortField.Equals(that.UshortField);
         var ___eqUintField = this.UintField.Equals(that.UintField);
@@ -291,7 +298,8 @@ public bool Equals(PublicTable that)
             this.NullableReferenceField == null
                 ? that.NullableReferenceField == null
                 : this.NullableReferenceField.Equals(that.NullableReferenceField);
-        return ___eqByteField
+        return ___eqId
+            && ___eqByteField
             && ___eqUshortField
             && ___eqUintField
             && ___eqUlongField