Implement double-encoding escaping mechanism for column names with escape sequence prefix

[Copilot]

Why make this change?

Addresses review feedback from #3051 requesting a more robust escaping mechanism to handle edge cases where column names already contain the escape sequence "DAB_ESCAPE$".

What is this change?

Double-Encoding Strategy

Enhanced the serialization escaping logic with two-step processing:

1. Escape columns starting with DAB_ESCAPE$ → DAB_ESCAPE$DAB_ESCAPE$...
2. Escape columns starting with $ → DAB_ESCAPE$...

Deserialization reverses this in opposite order to restore original names.

Prevent Double-Escaping

Added HashSet tracking in Write() method to handle cases where SourceDefinition and TableDefinition properties reference the same object instance.

Example Transformations

• $FirstName → DAB_ESCAPE$FirstName (existing behavior, unchanged)
• DAB_ESCAPE$FirstName → DAB_ESCAPE$DAB_ESCAPE$FirstName (new edge case)
• RegularColumn → RegularColumn (unchanged)

How was this tested?

• Unit Tests

Added test cases covering:

• Column names with escape sequence prefix
• Mixed column name patterns in single table
• Exact escape sequence as column name

Note: Existing serialization tests pass, confirming backward compatibility. New edge case tests demonstrate the double-encoding logic but require investigation - the in-place dictionary modifications are not reflecting in serialized JSON output despite working correctly in isolation.

Sample Request(s)

N/A - Internal serialization mechanism change with no API surface modifications.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[JerryNixon]

One

private static void EscapeDollaredColumns(SourceDefinition sourceDef)
{
    // Mutates sourceDef.Columns dictionary directly
    sourceDef.Columns.Remove(key);
    sourceDef.Columns[newKey] = col;
}

This modifies the original object's Columns dictionary during serialization. After serialization, the in-memory object will have escaped column names instead of the original names.

Please consider creating a new dictionary with escaped keys rather than modifying in-place, or ensure the unescaping happens reliably after serialization.

Two

ColumnDefinition col = sourceDef.Columns[key];  // Line 108, 118

Consider using TryGetValue for defensive programming.