fix remaining blocks who had files visibility set to hidden

waleedlatif1 · waleedlatif1 · commit 030c61bd4700 · 2026-02-12T15:06:50.000-08:00
diff --git a/apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/components/tools/credential-selector.tsx b/apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/components/tools/credential-selector.tsx
@@ -4,6 +4,7 @@ import { Button, Combobox } from '@/components/emcn/components'
 import {
   getCanonicalScopesForProvider,
   getProviderIdFromServiceId,
+  getServiceConfigByProviderId,
   OAUTH_PROVIDERS,
   type OAuthProvider,
   type OAuthService,
@@ -26,6 +27,11 @@ const getProviderIcon = (providerName: OAuthProvider) => {
 }
 
 const getProviderName = (providerName: OAuthProvider) => {
+  const serviceConfig = getServiceConfigByProviderId(providerName)
+  if (serviceConfig) {
+    return serviceConfig.name
+  }
+
   const { baseProvider } = parseProvider(providerName)
   const baseProviderConfig = OAUTH_PROVIDERS[baseProvider]
 
@@ -54,7 +60,7 @@ export function ToolCredentialSelector({
   onChange,
   provider,
   requiredScopes = [],
-  label = 'Select account',
+  label,
   serviceId,
   disabled = false,
 }: ToolCredentialSelectorProps) {
@@ -64,6 +70,7 @@ export function ToolCredentialSelector({
   const { activeWorkflowId } = useWorkflowRegistry()
 
   const selectedId = value || ''
+  const effectiveLabel = label || `Select ${getProviderName(provider)} account`
 
   const effectiveProviderId = useMemo(() => getProviderIdFromServiceId(serviceId), [serviceId])
 
@@ -203,7 +210,7 @@ export function ToolCredentialSelector({
         selectedValue={selectedId}
         onChange={handleComboboxChange}
         onOpenChange={handleOpenChange}
-        placeholder={label}
+        placeholder={effectiveLabel}
         disabled={disabled}
         editable={true}
         filterOptions={!isForeign}
diff --git a/apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/tool-input.tsx b/apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/tool-input.tsx
@@ -1761,7 +1761,6 @@ export const ToolInput = memo(function ToolInput({
                               ?.requiredScopes ||
                             getCanonicalScopesForProvider(oauthConfig.provider)
                           }
-                          label={`Select ${oauthConfig.provider} account`}
                           serviceId={oauthConfig.provider}
                           disabled={disabled}
                         />
diff --git a/apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx b/apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx
@@ -227,6 +227,8 @@ interface BlockData {
  * Renders the ReactFlow canvas with blocks, edges, and all interactive features.
  */
 const WorkflowContent = React.memo(() => {
+  const workflowRootRef = useRef<HTMLDivElement | null>(null)
+  const viewportMoveLogCountRef = useRef(0)
   const [isCanvasReady, setIsCanvasReady] = useState(false)
   const [potentialParentId, setPotentialParentId] = useState<string | null>(null)
   const [selectedEdges, setSelectedEdges] = useState<SelectedEdgesMap>(new Map())
@@ -255,6 +257,52 @@ const WorkflowContent = React.memo(() => {
 
   const addNotification = useNotificationStore((state) => state.addNotification)
 
+  const logWorkflowLayoutSnapshot = useCallback(
+    (
+      hypothesisId: 'H8' | 'H9' | 'H10' | 'H11',
+      message: string,
+      extraData: Record<string, unknown>
+    ) => {
+      const workflowRoot = workflowRootRef.current
+      const flowContainer = document.querySelector('.workflow-container') as HTMLElement | null
+      const openDialogs = document.querySelectorAll('[role="dialog"][data-state="open"]').length
+      const bodyStyle = document.body.style
+
+      // #region agent log
+      fetch('http://127.0.0.1:7243/ingest/77a2b2bc-808d-4bfd-a366-739b0b04635d', {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({
+          runId: 'initial',
+          hypothesisId,
+          location: 'workflow.tsx:WorkflowContent',
+          message,
+          data: {
+            openDialogs,
+            windowInnerHeight: window.innerHeight,
+            windowInnerWidth: window.innerWidth,
+            pageScrollY: window.scrollY,
+            visualViewportHeight: window.visualViewport?.height ?? null,
+            visualViewportOffsetTop: window.visualViewport?.offsetTop ?? null,
+            bodyOverflow: bodyStyle.overflow || null,
+            bodyPaddingRight: bodyStyle.paddingRight || null,
+            bodyPosition: bodyStyle.position || null,
+            bodyTop: bodyStyle.top || null,
+            workflowRootTop: workflowRoot?.getBoundingClientRect().top ?? null,
+            workflowRootHeight: workflowRoot?.getBoundingClientRect().height ?? null,
+            flowContainerTop: flowContainer?.getBoundingClientRect().top ?? null,
+            flowContainerHeight: flowContainer?.getBoundingClientRect().height ?? null,
+            reactFlowViewport: reactFlowInstance.getViewport(),
+            ...extraData,
+          },
+          timestamp: Date.now(),
+        }),
+      }).catch(() => {})
+      // #endregion
+    },
+    [reactFlowInstance]
+  )
+
   const {
     workflows,
     activeWorkflowId,
@@ -1068,6 +1116,43 @@ const WorkflowContent = React.memo(() => {
     isPointInLoopNode,
   ])
 
+  useEffect(() => {
+    const observer = new MutationObserver(() => {
+      const openDialogs = document.querySelectorAll('[role="dialog"][data-state="open"]').length
+      logWorkflowLayoutSnapshot('H8', 'Dialog/open-state mutation observed in workflow page', {
+        openDialogs,
+      })
+
+      requestAnimationFrame(() => {
+        logWorkflowLayoutSnapshot('H11', 'Next-frame workflow layout after dialog mutation', {
+          openDialogs,
+        })
+      })
+    })
+
+    observer.observe(document.body, {
+      subtree: true,
+      attributes: true,
+      childList: true,
+      attributeFilter: ['data-state', 'style'],
+    })
+
+    return () => observer.disconnect()
+  }, [logWorkflowLayoutSnapshot])
+
+  useEffect(() => {
+    const viewport = window.visualViewport
+    if (!viewport) return
+
+    const onViewportResize = () => {
+      if (document.querySelectorAll('[role="dialog"][data-state="open"]').length === 0) return
+      logWorkflowLayoutSnapshot('H10', 'visualViewport resize while modal open', {})
+    }
+
+    viewport.addEventListener('resize', onViewportResize)
+    return () => viewport.removeEventListener('resize', onViewportResize)
+  }, [logWorkflowLayoutSnapshot])
+
   const handleContextDuplicate = useCallback(() => {
     copyBlocks(contextMenuBlocks.map((b) => b.id))
     executePasteOperation('duplicate', DEFAULT_PASTE_OFFSET)
@@ -3485,7 +3570,7 @@ const WorkflowContent = React.memo(() => {
   ])
 
   return (
-    <div className='flex h-full w-full flex-col overflow-hidden'>
+    <div ref={workflowRootRef} className='flex h-full w-full flex-col overflow-hidden'>
       <div className='relative h-full w-full flex-1'>
         {/* Loading spinner - always mounted, animation paused when hidden to avoid overhead */}
         <div
@@ -3569,6 +3654,18 @@ const WorkflowContent = React.memo(() => {
               elevateNodesOnSelect={true}
               autoPanOnConnect={effectivePermissions.canEdit}
               autoPanOnNodeDrag={effectivePermissions.canEdit}
+              onMoveEnd={(_, viewport) => {
+                if (viewportMoveLogCountRef.current >= 8) return
+                viewportMoveLogCountRef.current += 1
+                const openDialogs = document.querySelectorAll(
+                  '[role="dialog"][data-state="open"]'
+                ).length
+                logWorkflowLayoutSnapshot('H9', 'Main workflow ReactFlow viewport moved', {
+                  viewport,
+                  moveLogCount: viewportMoveLogCountRef.current,
+                  openDialogs,
+                })
+              }}
             />
 
             <Cursors />
diff --git a/apps/sim/tools/jira/add_attachment.ts b/apps/sim/tools/jira/add_attachment.ts
@@ -36,7 +36,7 @@ export const jiraAddAttachmentTool: ToolConfig<JiraAddAttachmentParams, JiraAddA
       files: {
         type: 'file[]',
         required: true,
-        visibility: 'hidden',
+        visibility: 'user-only',
         description: 'Files to attach to the Jira issue',
       },
       cloudId: {
diff --git a/apps/sim/tools/linear/create_attachment.ts b/apps/sim/tools/linear/create_attachment.ts
@@ -35,7 +35,7 @@ export const linearCreateAttachmentTool: ToolConfig<
     file: {
       type: 'file',
       required: false,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'File to attach',
     },
     title: {
diff --git a/apps/sim/tools/pulse/parser.ts b/apps/sim/tools/pulse/parser.ts
@@ -18,7 +18,7 @@ export const pulseParserTool: ToolConfig<PulseParserInput, PulseParserOutput> =
     file: {
       type: 'file',
       required: false,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'Document file to be processed',
     },
     fileUpload: {
@@ -268,7 +268,7 @@ export const pulseParserV2Tool: ToolConfig<PulseParserV2Input, PulseParserOutput
     file: {
       type: 'file',
       required: true,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'Document to be processed',
     },
     pages: pulseParserTool.params.pages,
diff --git a/apps/sim/tools/reducto/parser.ts b/apps/sim/tools/reducto/parser.ts
@@ -22,7 +22,7 @@ export const reductoParserTool: ToolConfig<ReductoParserInput, ReductoParserOutp
     file: {
       type: 'file',
       required: false,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'Document file to be processed',
     },
     fileUpload: {
@@ -196,7 +196,7 @@ export const reductoParserV2Tool: ToolConfig<ReductoParserV2Input, ReductoParser
     file: {
       type: 'file',
       required: true,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'PDF document to be processed',
     },
     pages: reductoParserTool.params.pages,
diff --git a/apps/sim/tools/sftp/upload.ts b/apps/sim/tools/sftp/upload.ts
@@ -53,7 +53,7 @@ export const sftpUploadTool: ToolConfig<SftpUploadParams, SftpUploadResult> = {
     files: {
       type: 'file[]',
       required: false,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'Files to upload',
     },
     fileContent: {
diff --git a/apps/sim/tools/vision/tool.ts b/apps/sim/tools/vision/tool.ts
@@ -106,7 +106,7 @@ export const visionToolV2: ToolConfig<VisionV2Params, VisionResponse> = {
     imageFile: {
       type: 'file',
       required: true,
-      visibility: 'hidden',
+      visibility: 'user-only',
       description: 'Image file to analyze',
     },
     model: visionTool.params.model,
diff --git a/apps/sim/tools/wordpress/upload_media.ts b/apps/sim/tools/wordpress/upload_media.ts
@@ -27,7 +27,7 @@ export const uploadMediaTool: ToolConfig<WordPressUploadMediaParams, WordPressUp
       file: {
         type: 'file',
         required: false,
-        visibility: 'hidden',
+        visibility: 'user-only',
         description: 'File to upload (UserFile object)',
       },
       filename: {

Original file line number	Diff line number	Diff line change
`@@ -1761,7 +1761,6 @@ export const ToolInput = memo(function ToolInput({`
`1761`	`1761`	`?.requiredScopes \|\|`
`1762`	`1762`	`getCanonicalScopesForProvider(oauthConfig.provider)`
`1763`	`1763`	`}`
`1764`		- label={`Select ${oauthConfig.provider} account`}
`1765`	`1764`	`serviceId={oauthConfig.provider}`
`1766`	`1765`	`disabled={disabled}`
`1767`	`1766`	`/>`