Show resource path params as custom properties in the flow diagram

[LakshanWeerasinghe]

Purpose

$subject

Fixes: issue

Screen.Recording.2025-01-06.at.14.34.50.mov

[Copilot]

Copilot reviewed 21 out of 36 changed files in this pull request and generated 1 comment.

Files not reviewed (15)

• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/diagram_generator/config/resource_action_call_github.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/diagram_generator/source/github_resource_call.bal: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/get_connectors/covid.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/get_connectors/default.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/get_connectors/http.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/get_connectors/sql.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/get_functions/config/custom_sum.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/get_functions/config/log.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/node_template/config/new_connection-covid.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/node_template/config/new_connection-docusign.dsadmin.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/node_template/config/new_connection-http.json: Language not supported
• flow-model-generator/modules/flow-model-generator-ls-extension/src/test/resources/node_template/config/new_connection-redis.json: Language not supported
• flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/db/model/Parameter.java: Evaluated as low risk
• flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/db/model/ParameterResult.java: Evaluated as low risk
• flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/CommonUtils.java: Evaluated as low risk

Comments suppressed due to low confidence (3)

flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/ParamUtils.java:79

• [nitpick] The variable name 'type' is ambiguous. It should be renamed to 'typeSignature' for clarity.

String type = TypeUtils.getTypeSignature(pathParameterSymbol.typeDescriptor(), errorTypeSymbol, true);

flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/ParamUtils.java:78

• [nitpick] The variable name 'defaultValue' is ambiguous. It should be renamed to 'defaultValueForType' for clarity.

String defaultValue = DefaultValueGeneratorUtil.getDefaultValueForType(pathParameterSymbol.typeDescriptor());

flow-model-generator/modules/flow-model-generator-core/src/main/java/io/ballerina/flowmodelgenerator/core/utils/ParamUtils.java:100

• The new behavior introduced in the 'buildResourcePathTemplate' method should be covered by tests to ensure it works correctly in all scenarios.

public record ResourcePathTemplate(String resourcePathTemplate, List<ParameterResult> pathParams) {