fixup: update comments

Author: Anemy

packages/compass-components/src/components/workspace-tabs/tab.tsx

@@ -149,6 +149,7 @@ const draggingTabStyles = css({
   cursor: 'grabbing !important',
 });
 
+// TODO: Should this have a dark mode version?
 const nonExistentStyles = css({
   color: palette.gray.base,
 });

packages/compass-components/src/components/workspace-tabs/workspace-tabs.tsx

@@ -302,29 +302,16 @@ const SortableItem = ({
   const tab = useMemo(
     () =>
       renderTab({
-        // ...tabProps,
-        // TODO: id here?
         isSelected,
         isDragging,
         tabContentId: tabId,
         onSelect: onTabSelected,
         onClose: onTabClosed,
       }),
     [renderTab]
-  ); // TODO: Add the rest of the tab props. Maybe no useMemo.
+  );
 
   return tab;
-
-  // return tab;(
-  //   <Tab
-  //     {...tabProps}
-  //     isSelected={isSelected}
-  //     isDragging={isDragging}
-  //     tabContentId={tabId}
-  //     onSelect={onTabSelected}
-  //     onClose={onTabClosed}
-  //   />
-  // );
 };
 
 function WorkspaceTabs({

packages/compass-data-modeling/src/index.ts

@@ -11,13 +11,11 @@ import { dataModelStorageServiceLocator } from './provider';
 import { activateDataModelingStore } from './store';
 import { PluginTabTitle, WorkspaceName } from './plugin-tab-title';
 
-// TODO: Maybe rename to DataModelingPlugin
 export const WorkspaceTab: WorkspacePlugin<typeof WorkspaceName> = {
   name: WorkspaceName,
   provider: registerHadronPlugin(
     {
       name: 'DataModeling',
-      // component: DataModelingComponent,
       component: function DataModelingProvider({ children }) {
         return React.createElement(React.Fragment, null, children);
       },
@@ -34,34 +32,4 @@ export const WorkspaceTab: WorkspacePlugin<typeof WorkspaceName> = {
   ),
   content: DataModelingComponent,
   header: PluginTabTitle,
-  // ({
-  //   // todo: type
-  //   id,
-  // }: {
-  //   id: string;
-  // }) => ({
-  //   id,
-  //   type: WorkspaceName,
-  //   title: WorkspaceName,
-  //   iconGlyph: 'Diagram' as const,
-  // }),
 };
-
-// export const WorkspaceTab: WorkspaceComponent<typeof WorkspaceName> = {
-//   name: WorkspaceName,
-//   provider: DataModelingPlugin,
-//   content: DataModelingComponent,
-//   // component: DataModelingPlugin,
-//   header: ({
-//     id
-//   }: {
-//     id: string;
-//   }) => ({
-//     id,
-//     type: WorkspaceName,
-//     title: WorkspaceName,
-//     iconGlyph: 'Diagram' as const,
-//   })
-// };
-
-// export default DataModelingPlugin;

packages/compass-workspaces/src/types.ts

@@ -82,7 +82,6 @@ export type WorkspaceTab = {
   id: string;
 } & WorkspaceTabProps;
 
-// TODO: Can we remove this type?
 export type AnyWorkspace =
   | WelcomeWorkspace
   | MyQueriesWorkspace
@@ -93,21 +92,19 @@ export type AnyWorkspace =
   | CollectionsWorkspace
   | CollectionWorkspace;
 
-// TODO: can we remove this type?
 export type Workspace<T extends AnyWorkspace['type']> = Extract<
   AnyWorkspace,
   { type: T }
 >;
 
-// TODO: can we remove this type?
 export type WorkspacePluginProps<T extends AnyWorkspace['type']> = Omit<
   Workspace<T>,
   'type' | 'connectionId'
 >;
 
 export type WorkspacePlugin<T extends AnyWorkspace['type']> = {
   name: T;
-  // TODO: How to type the services and activate.
+  // TODO: Type the services and activate if we can.
   // provider: HadronPluginComponent<T, Record<string, () => unknown>, any>;
   provider: HadronPluginComponent<WorkspacePluginProps<T>, any, any>;
   content: