adding release notes change and some comments addressed

Signed-off-by: Amit Galitzky <amgalitz@amazon.com>

Author: amitgalitz

release-notes/opensearch-flow-framework.release-notes-2.13.0.0.md

@@ -17,4 +17,7 @@ Compatible with OpenSearch 2.13.0
 
 ### Refactoring
 * Moved workflow-steps.json to Enum ([#523](https://github.com/opensearch-project/flow-framework/pull/523))
-* Refactored logging for consistency ([#524](https://github.com/opensearch-project/flow-framework/pull/524))
+* Refactored logging for consistency ([#524](https://github.com/opensearch-project/flow-framework/pull/524))
+
+### Bug Fixes
+* Fixing create index and use case input parsing bugs ([#600](https://github.com/opensearch-project/flow-framework/pull/600))

src/main/java/org/opensearch/flowframework/rest/RestCreateWorkflowAction.java

@@ -143,7 +143,7 @@ protected RestChannelConsumer prepareRequest(RestRequest request, NodeClient cli
                     } catch (Exception ex) {
                         RestStatus status = ex instanceof IOException ? RestStatus.BAD_REQUEST : ExceptionsHelper.status(ex);
                         String errorMessage =
-                            "failure parsing request body when a use case is given, make sure to provide a map with values that are either strings, arrays, or map of strings to strings";
+                            "failure parsing request body when a use case is given, make sure to provide a map with values that are either Strings, Arrays, or Map of Strings to Strings";
                         logger.error(errorMessage, ex);
                         throw new FlowFrameworkException(errorMessage, status);
                     }

src/main/java/org/opensearch/flowframework/workflow/CreateIndexStep.java

@@ -101,9 +101,7 @@ public PlainActionFuture<WorkflowData> execute(
                     createIndexRequest.source(sourceAsMap, LoggingDeprecationHandler.INSTANCE);
                 }
             } catch (Exception ex) {
-                String errorMessage = "Failed to create the index"
-                    + indexName
-                    + " due to incorrect mapping, make sure you aren't providing _doc in mapping";
+                String errorMessage = "Failed to create the index" + indexName + ", _doc is not permitted in mapping";
                 logger.error(errorMessage, ex);
                 createIndexFuture.onFailure(new WorkflowStepException(errorMessage, RestStatus.BAD_REQUEST));
             }
@@ -153,6 +151,7 @@ public PlainActionFuture<WorkflowData> execute(
     // This method to check if the mapping contains a type `_doc` and if yes we fail the request
     // is to duplicate the behavior we have today through create index rest API, we want users
     // to encounter the same behavior and not suddenly have to add `_doc` while using our create_index step
+    // related bug: https://github.com/opensearch-project/OpenSearch/issues/12775
     private static Map<String, Object> prepareMappings(Map<String, Object> source) {
         if (source.containsKey("mappings") == false || (source.get("mappings") instanceof Map) == false) {
             return source;

src/main/resources/defaults/conversational-search-defaults.json

@@ -14,7 +14,7 @@
     "create_search_pipeline.pipeline_id": "rag-pipeline",
     "create_search_pipeline.retrieval_augmented_generation.tag": "openai_pipeline_demo",
     "create_search_pipeline.retrieval_augmented_generation.description": "Demo pipeline Using cohere Connector",
-    "create_search_pipeline.retrieval_augmented_generation.context_field_list": "[\"text\", \"hello\"]",
+    "create_search_pipeline.retrieval_augmented_generation.context_field_list": "[\"text\"]",
     "create_search_pipeline.retrieval_augmented_generation.system_prompt": "You are a helpful assistant",
     "create_search_pipeline.retrieval_augmented_generation.user_instructions": "Generate a concise and informative answer in less than 100 words for the given question"
 }