Revert TTC Conversational UX commits (#8149)

* Revert "Update onboarding with new conversational ttc ui (#8147)"

This reverts commit e7626e9.

* Revert "CSS tweaks follow to TTC UI (#8137)"

This reverts commit 199d5a3.

* Revert "Don't include the current file prompt if there is no current file (#8135)"

This reverts commit 5c1a417.

* Revert "Add a source range prompt containing the current file content range (#8129)"

This reverts commit 49d40f2.

* Revert "Move clearing of TTC project-specific state to SystemIO (#8130)"

This reverts commit e42e5f8.

* Revert "Rename "Text-to-CAD Create" command to "Create project using Text-to-CAD" (#8127)"

This reverts commit f7fc3b2.

* Revert "Text-to-CAD UX revamp phase 1 (#7542)"

This reverts commit c699bb0.

* Update snapshots

* Update snapshots

* Fix tsc

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: pierremtb

e2e/playwright/command-bar-tests.spec.ts

@@ -762,7 +762,7 @@ theta = 45deg
     await homePage.textToCadBtn.click()
     await cmdBar.expectState({
       stage: 'arguments',
-      commandName: 'Create Project using Text-to-CAD',
+      commandName: 'Text-to-CAD Create',
       currentArgKey: 'prompt',
       currentArgValue: '',
       headerArguments: {

e2e/playwright/fixtures/cmdBarFixture.ts

@@ -143,9 +143,15 @@ export class CmdBarFixture {
     await submitButton.click()
   }
 
-  openCmdBar = async () => {
+  openCmdBar = async (selectCmd?: 'promptToEdit') => {
     await this.cmdBarOpenBtn.click()
     await expect(this.page.getByPlaceholder('Search commands')).toBeVisible()
+    if (selectCmd === 'promptToEdit') {
+      const promptEditCommand = this.selectOption({ name: 'Text-to-CAD Edit' })
+      await expect(promptEditCommand.first()).toBeVisible()
+      await promptEditCommand.first().scrollIntoViewIfNeeded()
+      await promptEditCommand.first().click()
+    }
   }
 
   closeCmdBar = async () => {
@@ -261,18 +267,15 @@ export class CmdBarFixture {
           if (part.startsWith('--')) continue
 
           const nameMatch = part.match(/name="([^"]+)"/)
-          if (!nameMatch) {
-            console.log('No name match found in part:', part.substring(0, 100))
-            continue
-          }
+          if (!nameMatch) continue
 
           const name = nameMatch[1]
           const content = part.split(/\r?\n\r?\n/)[1]?.trim()
           if (!content) continue
 
-          if (name === 'body') {
+          if (name === 'event') {
             eventData = JSON.parse(content)
-          } else if (name === 'files') {
+          } else {
             files[name] = content
           }
         }

e2e/playwright/fixtures/editorFixture.ts

@@ -55,8 +55,8 @@ export class EditorFixture {
       await this.scrollToBottom()
       if (!shouldNormalise) {
         const result = not
-          ? await expect(this.codeContent).not.toContainText(code, { timeout })
-          : await expect(this.codeContent).toContainText(code, { timeout })
+          ? await expect(this.codeContent).not.toContainText(code)
+          : await expect(this.codeContent).toContainText(code)
 
         await resetPane()
         return result

e2e/playwright/fixtures/fixtureSetup.ts

@@ -250,7 +250,7 @@ export class ElectronZoo {
     }
 
     if (!this.firstUrl) {
-      await this.page.getByRole('tab', { name: 'Projects' }).count()
+      await this.page.getByRole('heading', { name: 'Projects' }).count()
       this.firstUrl = this.page.url()
     }
 

e2e/playwright/fixtures/homePageFixture.ts

@@ -100,7 +100,9 @@ export class HomePageFixture {
 
   expectIsCurrentPage = async () => {
     await expect(this.page).toHaveURL(/.*home/)
-    await expect(this.page.getByRole('tab', { name: 'Projects' })).toBeVisible()
+    await expect(
+      this.page.getByRole('heading', { name: 'Projects' })
+    ).toBeVisible()
   }
 
   projectsLoaded = async () => {

e2e/playwright/fixtures/toolbarFixture.ts

@@ -326,13 +326,4 @@ export class ToolbarFixture {
     await expect(goToDefinitionMenuButton).toBeVisible()
     await goToDefinitionMenuButton.click()
   }
-
-  async fireTtcPrompt(prompt: string) {
-    await this.openPane('text-to-cad')
-    await expect(
-      this.page.getByTestId('ml-ephant-conversation-input')
-    ).toBeVisible()
-    await this.page.getByTestId('ml-ephant-conversation-input').fill(prompt)
-    await this.page.getByTestId('ml-ephant-conversation-input-button').click()
-  }
 }

e2e/playwright/native-file-menu.spec.ts

@@ -246,6 +246,14 @@ test.describe(
         )
         await openSettingsExpectLocator(page, '#themeColor')
       })
+      await test.step('Modeling.Edit.Modify with Zoo Text-To-CAD', async () => {
+        await page.waitForTimeout(250)
+        await clickElectronNativeMenuById(
+          tronApp,
+          'Edit.Modify with Zoo Text-To-CAD'
+        )
+        await cmdBar.expectCommandName('Text-to-CAD Edit')
+      })
       await test.step('Modeling.Edit.Edit parameter', async () => {
         await page.waitForTimeout(250)
         await clickElectronNativeMenuById(tronApp, 'Edit.Edit parameter')
@@ -510,7 +518,17 @@ test.describe(
           'Design.Create with Zoo Text-To-CAD'
         )
         await cmdBar.toBeOpened()
-        await cmdBar.expectCommandName('Create Project using Text-to-CAD')
+        await cmdBar.expectCommandName('Text-to-CAD Create')
+      })
+
+      await test.step('Modeling.Design.Modify with Zoo Text-To-CAD', async () => {
+        await page.waitForTimeout(250)
+        await clickElectronNativeMenuById(
+          tronApp,
+          'Design.Modify with Zoo Text-To-CAD'
+        )
+        await cmdBar.toBeOpened()
+        await cmdBar.expectCommandName('Text-to-CAD Edit')
       })
 
       await test.step('Modeling.Help.KCL code samples', async () => {

e2e/playwright/projects.spec.ts

@@ -654,7 +654,7 @@ test.describe(`Project management commands`, () => {
       const projectNameOption = page.getByRole('option', { name: projectName })
       const commandWarning = page.getByText('Are you sure you want to delete?')
       const toastMessage = page.getByText(`Successfully deleted`)
-      const noProjectsMessage = page.getByText('No results found')
+      const noProjectsMessage = page.getByText('No projects found')
 
       await test.step(`Setup`, async () => {
         await page.setBodyDimensions({ width: 1200, height: 500 })
@@ -758,7 +758,7 @@ test.describe(`Project management commands`, () => {
       const projectNameOption = page.getByRole('option', { name: projectName })
       const commandWarning = page.getByText('Are you sure you want to delete?')
       const toastMessage = page.getByText(`Successfully deleted`)
-      const noProjectsMessage = page.getByText('No results found')
+      const noProjectsMessage = page.getByText('No projects found')
 
       await test.step(`Setup`, async () => {
         await page.setBodyDimensions({ width: 1200, height: 500 })
@@ -1042,7 +1042,7 @@ test(
     })
 
     await test.step('Check that the home page is empty', async () => {
-      await expect(page.getByText('No results found')).toBeVisible()
+      await expect(page.getByText('No projects found')).toBeVisible()
     })
 
     await test.step('Check we can still create a project', async () => {
@@ -1120,7 +1120,7 @@ test(
 
     await test.step('should be shorted by modified initially', async () => {
       const lastModifiedButton = page.getByRole('button', {
-        name: 'Age',
+        name: 'Last Modified',
       })
       await expect(lastModifiedButton).toBeVisible()
       await expect(lastModifiedButton.getByLabel('arrow down')).toBeVisible()
@@ -1141,7 +1141,7 @@ test(
 
     await test.step('Reverse modified order', async () => {
       const lastModifiedButton = page.getByRole('button', {
-        name: 'Age',
+        name: 'Last Modified',
       })
       await lastModifiedButton.click()
       await expect(lastModifiedButton).toBeVisible()
@@ -1217,8 +1217,8 @@ test(
     // Constants and locators
     const projectLinks = page.getByTestId('project-link')
 
-    // expect to see text "No results found"
-    await expect(page.getByText('No results found')).toBeVisible()
+    // expect to see text "No projects found"
+    await expect(page.getByText('No projects found')).toBeVisible()
 
     await createProject({ name: 'project-000', page, returnHome: true })
     await expect(projectLinks.getByText('project-000')).toBeVisible()
@@ -1365,7 +1365,7 @@ test(
 
       await homePage.projectsLoaded()
 
-      await expect(page.getByText('No results found')).toBeVisible()
+      await expect(page.getByText('No projects found')).toBeVisible()
       await createProject({ name: 'project-000', page, returnHome: true })
       await expect(
         page.getByTestId('project-link').filter({ hasText: 'project-000' })
@@ -1432,7 +1432,7 @@ test(
     page.on('console', console.log)
 
     // Our locator constants
-    const searchInput = page.getByPlaceholder('Search')
+    const searchInput = page.getByPlaceholder('Search projects')
     const projectLinks = page.getByTestId('project-link')
 
     await test.step('Search for "basi"', async () => {

e2e/playwright/prompt-to-edit-snapshot-tests.spec.ts

@@ -1,4 +1,4 @@
-import { test } from '@e2e/playwright/zoo-test'
+import { expect, test } from '@e2e/playwright/zoo-test'
 import path from 'path'
 import fsp from 'fs/promises'
 
@@ -64,7 +64,7 @@ test.describe('edit with AI example snapshots', () => {
     `change colour`,
     // TODO this is more of a snapshot, but atm it needs to be manually run locally to update the files
     { tag: ['@desktop'] },
-    async ({ context, homePage, cmdBar, editor, page, scene, toolbar }) => {
+    async ({ context, homePage, cmdBar, editor, page, scene }) => {
       const project = 'test-dir'
       await context.folderSetupFn(async (dir) => {
         const projectDir = path.join(dir, project)
@@ -85,6 +85,7 @@ test.describe('edit with AI example snapshots', () => {
         body1CapCoords.x,
         body1CapCoords.y
       )
+      const submittingToast = page.getByText('Submitting to Text-to-CAD API...')
 
       await test.step('wait for scene to load select body and check selection came through', async () => {
         await clickBody1Cap()
@@ -97,9 +98,16 @@ test.describe('edit with AI example snapshots', () => {
 
       await test.step('fire off edit prompt', async () => {
         await cmdBar.captureTextToCadRequestSnapshot(test.info())
-        await toolbar.fireTtcPrompt('make this neon green please, use #39FF14')
+        await cmdBar.openCmdBar('promptToEdit')
         await page.waitForTimeout(100)
+        await cmdBar.progressCmdBar()
+        // being specific about the color with a hex means asserting pixel color is more stable
+        await page
+          .getByTestId('cmd-bar-arg-value')
+          .fill('make this neon green please, use #39FF14')
         await page.waitForTimeout(100)
+        await cmdBar.progressCmdBar()
+        await expect(submittingToast).toBeVisible()
       })
     }
   )