refactor: remove deprecated features and streamline question generation process

- Deleted the feature file for assimilating with ignored points as it is no longer needed.
- Removed related step definitions and methods for handling ignored checklist topics in the question generation service.
- Updated the assimilation page object to eliminate unused methods, enhancing clarity and maintainability.

Co-authored-by: terry <terry@odd-e.com>

Author: terryyin

e2e_test/features/assimilation/assimilating_with_ignored_points.feature

@@ -14,9 +14,6 @@ Feature: Remove points from note when assimilating
       | English dominates the internet and digital world                                         |
       | Proficiency in English enhances career opportunities and simplifies international travel |
 
-    When I start assimilating "English"
-    And I mark the point "English is the world's most widespread lingua franca"
-
     Given OpenAI will rephrase the note as "English plays a crucial role in global communication, enabling people from different cultures and nations to understand one another. It is the primary language used in international business, science, technology, and diplomacy, which significantly broadens professional and academic opportunities. The language is deeply embedded in the digital world, with a large majority of online content created in English. Global media—such as films, music, and entertainment—relies heavily on English, offering access to a wide range of cultures and ideas. Knowledge of English also makes international travel easier, helping people navigate foreign environments and communicate with locals more effectively. As an official language in over 60 countries and a widely taught foreign language elsewhere, English is essential for global participation and education."
     And OpenAI generates understanding checklist with points:
       | English enables effective communication between people from different cultures and countries |

e2e_test/features/assimilation/remove_points.feature

@@ -169,23 +169,6 @@ export const assumeAssimilationPage = () => ({
     cy.contains('Understanding Checklist:').should('not.exist')
     return this
   },
-  selectChecklistTopicToIgnore(topicText: string) {
-    cy.pageIsNotLoading()
-    // Find the understanding checklist container
-    cy.contains('Understanding Checklist:')
-      .closest('.daisy-bg-accent')
-      .should('be.visible')
-      .within(() => {
-        // Find the list item containing the topic text
-        cy.contains('li', topicText)
-          .should('be.visible')
-          .within(() => {
-            // Find and check the checkbox within the list item
-            cy.get('input[type="checkbox"]').check()
-          })
-      })
-    return this
-  },
   rephraseNoteWithIgnoredTopic() {
     cy.pageIsNotLoading()
     cy.contains('Understanding Checklist:')

e2e_test/start/pageObjects/assimilationPage.ts

@@ -88,71 +88,4 @@ export const questionGenerationService = () => ({
       }
     })
   },
-
-  stubQuestionWhenPromptDoesNotIncludeIgnoreText: (
-    ignoreText: string,
-    record: Record<string, string>
-  ) => {
-    const mcqWithAnswer = createMcqWithAnswer(
-      record['Question Stem']!,
-      record['Correct Choice']!,
-      record['Incorrect Choice 1']!,
-      record['Incorrect Choice 2']!
-    )
-    const reply = JSON.stringify(mcqWithAnswer)
-    cy.then(async () => {
-      // Set up the less specific stub (doesn't include) - this should be registered after the "includes" stub
-      await mock_services
-        .openAi()
-        .chatCompletion()
-        .requestMatchesWithExclusion(
-          {
-            messages: [
-              {
-                role: 'user',
-                content: '.*',
-              },
-            ],
-          },
-          {
-            messages: [
-              {
-                role: 'user',
-                content: `.*${ignoreText}.*`,
-              },
-            ],
-          }
-        )
-        .stubQuestionGenerationWithBodyMatch(reply)
-    })
-  },
-
-  stubQuestionWhenPromptIncludesIgnoreText: (
-    ignoreText: string,
-    record: Record<string, string>
-  ) => {
-    const mcqWithAnswer = createMcqWithAnswer(
-      record['Question Stem']!,
-      record['Correct Choice']!,
-      record['Incorrect Choice 1']!,
-      record['Incorrect Choice 2']!
-    )
-    const reply = JSON.stringify(mcqWithAnswer)
-    cy.then(async () => {
-      // Set up the more specific stub first (includes ignore text)
-      // The ignoreText should match literally - single quotes are not special regex chars
-      await mock_services
-        .openAi()
-        .chatCompletion()
-        .requestMatches({
-          messages: [
-            {
-              role: 'user',
-              content: `.*${ignoreText}.*`,
-            },
-          ],
-        })
-        .stubQuestionGenerationWithBodyMatch(reply)
-    })
-  },
 })

e2e_test/start/questionGenerationService.ts

@@ -201,43 +201,6 @@ Given(
   }
 )
 
-Given(
-  "AI will generate a question when prompt doesn't include {string}:",
-  (ignoreText: string, questionTable: DataTable) => {
-    const hashes = questionTable.hashes()
-    if (hashes.length !== 1 || !hashes[0]) {
-      throw new Error(
-        `Expected exactly one row in the data table, but got ${hashes.length}`
-      )
-    }
-    // Extract the actual ignore text from the quoted string (remove quotes)
-    const actualIgnoreText = ignoreText.replace(/^"|"$/g, '')
-    start
-      .questionGenerationService()
-      .stubQuestionWhenPromptDoesNotIncludeIgnoreText(
-        actualIgnoreText,
-        hashes[0]
-      )
-  }
-)
-
-Given(
-  'AI will generate a question when prompt include {string}:',
-  (ignoreText: string, questionTable: DataTable) => {
-    const hashes = questionTable.hashes()
-    if (hashes.length !== 1 || !hashes[0]) {
-      throw new Error(
-        `Expected exactly one row in the data table, but got ${hashes.length}`
-      )
-    }
-    // Extract the actual ignore text from the quoted string (remove quotes)
-    const actualIgnoreText = ignoreText.replace(/^"|"$/g, '')
-    start
-      .questionGenerationService()
-      .stubQuestionWhenPromptIncludesIgnoreText(actualIgnoreText, hashes[0])
-  }
-)
-
 Given(
   'OpenAI will rephrase the note as {string}',
   (rephrasedContent: string) => {

e2e_test/step_definitions/ai.ts

@@ -110,16 +110,6 @@ When('I am assimilating the note {string}', (noteTitle: string) => {
   start.jumpToNotePage(noteTitle).moreOptions().openAssimilationPage()
 })
 
-When(
-  'one of the checklist topic is selected to ignore {string} and assimilate the note',
-  (topicText: string) => {
-    start
-      .assumeAssimilationPage()
-      .selectChecklistTopicToIgnore(topicText)
-      .assimilateCurrentNote()
-  }
-)
-
 Then('I assimilate the note {string}', (noteTitle: string) => {
   start.jumpToNotePage(noteTitle).moreOptions().assimilateNote()
 })
@@ -320,10 +310,6 @@ Then('I can continue with the assimilation', () => {
   })
 })
 
-When('I mark the point {string}', (point: string) => {
-  start.assumeAssimilationPage().selectChecklistTopicToIgnore(point)
-})
-
 Then('I should see the note rephrased as {string}', (rephrasedNote: string) => {
   start.assumeAssimilationPage().expectNoteRephrased(rephrasedNote)
 })