fix(files): name folders in a workflow diff, and bound depth at the block

Two more from cubic, both correct.

The workflow diff rendered every folder change as the same generic label. Adding
sim-folder-tree-selector to SELECTOR_TYPES_HYDRATION_REQUIRED without a resolver
makes each consumer of that list fall back to a semantic placeholder — the
canvas card showed "-" for the same reason, which I fixed there and not here. The
comparison path now calls the same resolveFolderPathLabel, so a diff between two
folders shows which two.

Max Depth and Limit were bounded only at the contract, so an out-of-range value
was accepted while configuring and failed on the first run. They now carry the
same ceilings in the block, where the mistake is made.

Author: mzxchandra

apps/sim/blocks/blocks/file.ts

@@ -1,6 +1,11 @@
 import { createLogger } from '@sim/logger'
 import { DocumentIcon } from '@/components/icons'
-import { encodeFolderPathSegment, ROOT_FOLDER_PATH } from '@/lib/folders/paths'
+import { MAX_FILE_LIST_LIMIT } from '@/lib/api/contracts/tools/file'
+import {
+  encodeFolderPathSegment,
+  MAX_FOLDER_PATH_SEGMENTS,
+  ROOT_FOLDER_PATH,
+} from '@/lib/folders/paths'
 import { inferContextFromKey } from '@/lib/uploads/utils/file-utils'
 import { readFolderPath } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/sim-folder-tree-selector/selection'
 import type { BlockConfig, SubBlockType } from '@/blocks/types'
@@ -1656,14 +1661,18 @@ export const FileV5Block: BlockConfig<FileParserV3Output> = {
           return {
             path: optionalText(params.folderRef),
             recursive: switchValue(params.folderRecursive),
+            /* Bounded here as well as at the contract, so a typo fails while
+               configuring rather than on the first run. */
             depth: parseOptionalNumberInput(params.folderDepth, 'Max Depth', {
               integer: true,
               min: 1,
+              max: MAX_FOLDER_PATH_SEGMENTS,
             }),
             search: optionalText(params.folderSearch),
             limit: parseOptionalNumberInput(params.folderLimit, 'Limit', {
               integer: true,
               min: 1,
+              max: MAX_FILE_LIST_LIMIT,
             }),
             workspaceId: params._context?.workspaceId,
           }

apps/sim/lib/workflows/comparison/resolve-values.ts

@@ -8,6 +8,7 @@ import { buildSelectorRawContext, projectSelectorContext } from '@/lib/selectors
 import { getSelectorManifestEntry, type SelectorKey } from '@/lib/selectors/manifest'
 import type { SelectorContext, SelectorScope } from '@/lib/selectors/types'
 import { getDependsOnFields } from '@/lib/workflows/subblocks/dependencies'
+import { resolveFolderPathLabel } from '@/lib/workflows/subblocks/display'
 import { getBlock } from '@/blocks/registry'
 import { SELECTOR_TYPES_HYDRATION_REQUIRED, type SubBlockConfig } from '@/blocks/types'
 import { isUuid } from '@/executor/constants'
@@ -263,6 +264,17 @@ export async function resolveValueForDisplay(
     }
   }
 
+  /*
+   * A folder picker is in the hydration list but has no selector manifest entry,
+   * so without this it falls through to the generic semantic fallback and a diff
+   * renders both the old and the new path as the same word — hiding the change
+   * it exists to show. Same resolver the canvas card uses, so the two agree.
+   */
+  const folderPathLabel = resolveFolderPathLabel(subBlockConfig, value)
+  if (folderPathLabel) {
+    return { original: value, displayLabel: folderPathLabel, resolved: true }
+  }
+
   if (SELECTOR_TYPES_HYDRATION_REQUIRED.includes(subBlockConfig.type)) {
     const selectorKey = subBlockConfig.selectorKey
     const scope: SelectorScope | undefined = context.workflowId