nit

kpamaran · kpamaran · commit 3ce421acd75b · 2025-09-09T16:23:50.000-04:00
diff --git a/packages/compass-collection/src/components/collection-header-actions/collection-header-actions.tsx b/packages/compass-collection/src/components/collection-header-actions/collection-header-actions.tsx
@@ -145,6 +145,8 @@ const CollectionHeaderActions: React.FunctionComponent<
             </div>
           }
         >
+          // todo within (CLOUDP-333853): update disabled open-modal button
+          tooltip to communicate if schema analysis is incomplete
           {exceedsMaxNestingDepth &&
             'At this time we are unable to generate mock data for collections that have deeply nested documents'}
           {isCollectionEmpty &&
diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/raw-schema-confirmation-screen.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/raw-schema-confirmation-screen.tsx
@@ -61,12 +61,6 @@ const RawSchemaConfirmationScreen = ({
     ? 'A sample of document values from your collection will be sent to an LLM for processing.'
     : 'We have identified the following schema from your documents. This schema will be sent to an LLM for processing.';
 
-  // (CLOUDP-333853) todo: disable button that opens modal and update tooltip while schema analysis is underway.
-  // after the todo this state should not be reachable
-  const schemaAnalysisIncompletePlaceholder = (
-    <Body>We are analyzing your collection.</Body>
-  );
-
   return (
     <div data-testid="raw-schema-confirmation">
       {schemaAnalysis.status === 'complete' ? (
@@ -97,7 +91,8 @@ const RawSchemaConfirmationScreen = ({
           )}
         </>
       ) : (
-        schemaAnalysisIncompletePlaceholder
+        // Not reachable since schema analysis must be finished before the modal can be opened
+        <Body>We are analyzing your collection.</Body>
       )}
     </div>
   );

Original file line number	Diff line number	Diff line change
`@@ -145,6 +145,8 @@ const CollectionHeaderActions: React.FunctionComponent<`
`145`	`145`	`</div>`
`146`	`146`	`}`
`147`	`147`	`>`
	`148`	`+ // todo within (CLOUDP-333853): update disabled open-modal button`
	`149`	`+ tooltip to communicate if schema analysis is incomplete`
`148`	`150`	`{exceedsMaxNestingDepth &&`
`149`	`151`	`'At this time we are unable to generate mock data for collections that have deeply nested documents'}`
`150`	`152`	`{isCollectionEmpty &&`