Merge resposnes from NLU and Domain when there're no retrieval intents

changelog/7390.bugfix.md

@@ -0,0 +1 @@
+Make sure the `responses` are synced between NLU training data and the Domain even if there're no retrieval intents in the NLU training data.

data/test_nlg/test_responses.yml

@@ -0,0 +1,3 @@
+responses:
+  utter_rasa:
+    - text: this is utter_rasa!

rasa/shared/importers/importer.py

@@ -314,19 +314,18 @@ async def get_domain(self) -> Domain:
         existing_domain = await self._importer.get_domain()
         existing_nlu_data = await self._importer.get_nlu_data()
 
-        # Check if NLU data has any retrieval intents, if yes
-        # add corresponding retrieval actions with `utter_` prefix automatically
-        # to an empty domain, update the properties of existing retrieval intents
-        # and merge response templates
-        if existing_nlu_data.retrieval_intents:
-
-            domain_with_retrieval_intents = self._get_domain_with_retrieval_intents(
-                existing_nlu_data.retrieval_intents,
-                existing_nlu_data.responses,
-                existing_domain,
-            )
+        # Merge responses from NLU data with responses in the domain.
+        # If NLU data has any retrieval intents, then add corresponding
+        # retrieval actions with `utter_` prefix automatically to the
+        # final domain, update the properties of existing retrieval intents.
+        domain_with_retrieval_intents = self._get_domain_with_retrieval_intents(
+            existing_nlu_data.retrieval_intents,
+            existing_nlu_data.responses,
+            existing_domain,
+        )
 
-            existing_domain = existing_domain.merge(domain_with_retrieval_intents)
+        existing_domain = existing_domain.merge(domain_with_retrieval_intents)
+        existing_domain.check_missing_templates()
 
         return existing_domain
 
@@ -351,7 +350,10 @@ def _get_domain_with_retrieval_intents(
         response_templates: Dict[Text, List[Dict[Text, Any]]],
         existing_domain: Domain,
     ) -> Domain:
-        """Construct a domain consisting of retrieval intents listed in the NLU training data.
+        """Construct a domain consisting of retrieval intents.
+
+         The result domain will have retrieval intents that are listed
+         in the NLU training data.
 
         Args:
             retrieval_intents: Set of retrieval intents defined in NLU training data.
@@ -360,7 +362,6 @@ def _get_domain_with_retrieval_intents(
         Returns: Domain with retrieval actions added to action names and properties
         for retrieval intents updated.
         """
-
         # Get all the properties already defined
         # for each retrieval intent in other domains
         # and add the retrieval intent property to them

rasa/shared/importers/rasa.py

@@ -65,7 +65,6 @@ async def get_domain(self) -> Domain:
             return domain
         try:
             domain = Domain.load(self._domain_path)
-            domain.check_missing_templates()
         except InvalidDomain as e:
             rasa.shared.utils.io.raise_warning(
                 f"Loading domain from '{self._domain_path}' failed. Using "

tests/shared/importers/test_importer.py

@@ -361,3 +361,25 @@ async def test_nlu_data_domain_sync_with_retrieval_intents(project: Text):
     assert domain.retrieval_intent_templates == nlu_data.responses
     assert domain.templates != nlu_data.responses
     assert "utter_chitchat" in domain.action_names
+
+
+async def test_nlu_data_domain_sync_responses(project: Text):
+    config_path = os.path.join(project, DEFAULT_CONFIG_PATH)
+    domain_path = "data/test_domains/default.yml"
+    data_paths = ["data/test_nlg/test_responses.yml"]
+
+    base_data_importer = TrainingDataImporter.load_from_dict(
+        {}, config_path, domain_path, data_paths
+    )
+
+    nlu_importer = NluDataImporter(base_data_importer)
+    core_importer = CoreDataImporter(base_data_importer)
+
+    importer = RetrievalModelsDataImporter(
+        CombinedDataImporter([nlu_importer, core_importer])
+    )
+    with pytest.warns(None):
+        domain = await importer.get_domain()
+
+    # Responses were sync between "test_responses.yml" and the "domain.yml"
+    assert "utter_rasa" in domain.templates.keys()