ack PR comments

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/logs/components/log-details/components/execution-snapshot/execution-snapshot.tsx

@@ -30,10 +30,25 @@ interface BlockExecutionData {
   durationMs: number
 }
 
+/**
+ * Migrated logs have special properties to indicate they came from the old logging system
+ */
+interface MigratedWorkflowState extends WorkflowState {
+  _migrated: true
+  _note?: string
+}
+
+/**
+ * Type guard to check if a workflow state is from a migrated log
+ */
+function isMigratedWorkflowState(state: WorkflowState): state is MigratedWorkflowState {
+  return (state as MigratedWorkflowState)._migrated === true
+}
+
 interface ExecutionSnapshotData {
   executionId: string
   workflowId: string
-  workflowState: WorkflowState
+  workflowState: WorkflowState | MigratedWorkflowState
   executionMetadata: {
     trigger: string
     startedAt: string
@@ -178,8 +193,7 @@ export function ExecutionSnapshot({
       )
     }
 
-    const isMigratedLog = (data.workflowState as any)?._migrated === true
-    if (isMigratedLog) {
+    if (isMigratedWorkflowState(data.workflowState)) {
       return (
         <div
           className={cn('flex flex-col items-center justify-center gap-[16px] p-[32px]', className)}
@@ -194,7 +208,7 @@ export function ExecutionSnapshot({
             is not available.
           </div>
           <div className='text-[12px] text-[var(--text-tertiary)]'>
-            Note: {(data.workflowState as any)?._note}
+            Note: {data.workflowState._note}
           </div>
         </div>
       )

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/components/general/general.tsx

@@ -338,7 +338,7 @@ export function GeneralDeploy({
                 : 'Live Workflow'}
             </ModalHeader>
             <ModalBody className='!p-0 min-h-0 flex-1'>
-              <div className='flex h-full w-full overflow-hidden rounded-[4px] border border-[var(--border)]'>
+              <div className='flex h-full w-full overflow-hidden'>
                 <div className='h-full flex-1'>
                   <WorkflowPreview
                     workflowState={workflowToShow}

apps/sim/app/workspace/[workspaceId]/w/components/preview/components/block-details-sidebar.tsx

@@ -482,7 +482,7 @@ function BlockDetailsSidebarContent({
 
   if (!blockConfig) {
     return (
-      <div className='flex h-full w-80 flex-col border-[var(--border)] border-l bg-[var(--surface-1)]'>
+      <div className='flex h-full w-80 flex-col overflow-hidden rounded-r-[8px] border-[var(--border)] border-l bg-[var(--surface-1)]'>
         <div className='flex items-center gap-[8px] bg-[var(--surface-4)] px-[12px] py-[8px]'>
           <div className='flex h-[18px] w-[18px] items-center justify-center rounded-[4px] bg-[var(--surface-3)]' />
           <span className='font-medium text-[14px] text-[var(--text-primary)]'>
@@ -513,7 +513,7 @@ function BlockDetailsSidebarContent({
         : 'gray'
 
   return (
-    <div className='flex h-full w-80 flex-col border-[var(--border)] border-l bg-[var(--surface-1)]'>
+    <div className='flex h-full w-80 flex-col overflow-hidden rounded-r-[8px] border-[var(--border)] border-l bg-[var(--surface-1)]'>
       {/* Header - styled like editor */}
       <div className='flex flex-shrink-0 items-center gap-[8px] bg-[var(--surface-4)] px-[12px] py-[8px]'>
         <div

apps/sim/app/workspace/[workspaceId]/w/components/preview/preview.tsx

@@ -280,7 +280,10 @@ export function WorkflowPreview({
         type: nodeType,
         position: absolutePosition,
         draggable: false,
-        className: executionStatus ? `execution-${executionStatus}` : undefined,
+        className:
+          executionStatus && executionStatus !== 'not-executed'
+            ? `execution-${executionStatus}`
+            : undefined,
         data: {
           type: block.type,
           config: blockConfig,
@@ -333,7 +336,6 @@ export function WorkflowPreview({
         sourceHandle: edge.sourceHandle,
         targetHandle: edge.targetHandle,
         data: executionStatus ? { executionStatus } : undefined,
-        className: executionStatus === 'not-executed' ? 'execution-not-executed' : undefined,
       }
     })
   }, [edgesStructure, workflowState.edges, isValidWorkflowState, executedBlocks])
@@ -362,7 +364,7 @@ export function WorkflowPreview({
       >
         <style>{`
           ${cursorStyle ? `.preview-mode .react-flow__pane { cursor: ${cursorStyle} !important; }` : ''}
-          
+
           /* Execution status styling for nodes */
           .preview-mode .react-flow__node.execution-success {
             border-radius: 8px;

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/workspace-header/components/invite-modal/invite-modal.tsx

@@ -18,9 +18,10 @@ import { useSession } from '@/lib/auth/auth-client'
 import { cn } from '@/lib/core/utils/cn'
 import { quickValidateEmail } from '@/lib/messaging/email/validation'
 import { useWorkspacePermissionsContext } from '@/app/workspace/[workspaceId]/providers/workspace-permissions-provider'
+import { EmailTag } from '@/app/workspace/[workspaceId]/w/components/sidebar/components/workspace-header/components/invite-modal/components/email-tag'
+import { PermissionsTable } from '@/app/workspace/[workspaceId]/w/components/sidebar/components/workspace-header/components/invite-modal/components/permissions-table'
 import { API_ENDPOINTS } from '@/stores/constants'
-import type { PermissionType, UserPermissions } from '.'
-import { EmailTag, PermissionsTable } from '.'
+import type { PermissionType, UserPermissions } from './components/types'
 
 const logger = createLogger('InviteModal')