Adding schema diagram api.

[aasimkhan30]

Sends a json representation of tables and foreign key relationships.

[Copilot]

Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.

Files not reviewed (2)

• src/Microsoft.SqlTools.ServiceLayer/HostLoader.cs: Evaluated as low risk
• src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/Contracts/GetSchemaModelRequest.cs: Evaluated as low risk

Comments suppressed due to low confidence (1)

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/SchemaDesignerService.cs:300

• [nitpick] Returning null in the catch block without proper handling can lead to null reference exceptions. Consider throwing a specific exception or handling the error more explicitly.

return null;

Copilot reviewed 5 out of 7 changed files in this pull request and generated no comments.

Files not reviewed (2)

• src/Microsoft.SqlTools.ServiceLayer/HostLoader.cs: Evaluated as low risk
• src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/Contracts/GetSchemaModelRequest.cs: Evaluated as low risk

Comments suppressed due to low confidence (3)

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/SchemaDesignerService.cs:185

• [nitpick] The variable name 'randomUri' is not descriptive. Consider renaming it to 'newConnectionUri'.

string randomUri = Guid.NewGuid().ToString();

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/SchemaDesignerService.cs:100

• [nitpick] The error message 'Failed to get schema information' is too generic. Consider providing more specific details about the failure.

var columnResult = await RunSimpleQuery(connectionCompleteParams, requestParams.DatabaseName, columnQuery) ?? throw new Exception("Failed to get schema information");

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/SchemaDesignerService.cs:304

• Ensure that the behavior for handling the 'MapOnAction' method is covered by tests.

internal static OnAction MapOnAction(string action)