{Interactive} Fix Scenario Idx out of bound when selecting recommende…

…d scenario (#6535) * Fix Scenario Idx out of bound when selecting recommended scenario * Add message in `HISTORY.rst`
Azure · Aug 1, 2023 · 8e16b69 · 8e16b69
1 parent 16b8e3b
commit 8e16b69
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/interactive/HISTORY.rst b/src/interactive/HISTORY.rst
@@ -5,6 +5,7 @@ Release History
 
 upcoming
 +++++
+* Fix Scenario Idx out of bound when selecting recommended scenario
 * Add command skipped message in Scenario Execution Mode
 
 0.5.1

diff --git a/src/interactive/azext_interactive/azclishell/app.py b/src/interactive/azext_interactive/azclishell/app.py
@@ -536,11 +536,11 @@ def handle_scenario(self, text):
         # e.g. :: 1  => `selected_option='1'`
         selected_option = text.partition(SELECT_SYMBOL['example'])[2].strip()
         try:
-            selected_option = int(selected_option)
+            selected_option = int(selected_option) - 1
         except ValueError:
             print("An Integer should follow the colon", file=self.output)
             return
-        if 0 <= selected_option <= len(self.recommender.get_scenarios() or []):
+        if 0 <= selected_option < len(self.recommender.get_scenarios() or []):
             scenario = self.recommender.get_scenarios()[selected_option]
             self.recommender.feedback_scenario(selected_option, scenario)
         else: