fix: replace any[] with FieldValues[] and remove stale eslint-disable in CreateFieldsImportExport

[Copilot]

Two minor code quality issues in CreateFieldsImportExport.tsx: imprecise any[] type on exported state, and a misleading eslint-disable comment on a let binding that is intentionally reassigned.

Changes

• Type precision: useState<any[]> → useState<FieldValues[]> — FieldValues was already imported; the state holds the same type as the rows prop.
• Remove stale lint suppression: Dropped // eslint-disable-next-line prefer-const before let { data, errors }. The let is correct since data is reassigned during key normalization; the comment was incorrectly suggesting otherwise.

Original prompt

Please apply the following diffs and create a pull request.
Once the PR is ready, give it a title based on the messages of the fixes being applied.

[{"message":"The type 'any[]' should be replaced with a more specific type. Consider using 'FieldValues[]' or the appropriate type that matches the structure returned by 'getRowsForExport'.","fixFiles":[{"filePath":"libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx","diff":"diff --git a/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx b/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx\n--- a/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx\n+++ b/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx\n@@ -39,7 +39,7 @@\n   const { google_apiKey, google_appId, google_clientId } = useAtomValue(applicationCookieState);\n   const { hasGoogleDriveAccess, googleShowUpgradeToPro } = useAtomValue(googleDriveAccessState);\n   const popoverRef = useRef<PopoverRef>(null);\n-  const [exportData, setExportData] = useState<any[]>(rows);\n+  const [exportData, setExportData] = useState<FieldValues[]>(rows);\n   const [exportModalOpen, setExportModalOpen] = useState(false);\n \n   function handleCloseModal() {\n"}]},{"message":"The eslint-disable comment suggests 'data' should be const, but it's reassigned on line 85. Either use const and create a new variable for the transformed data, or remove the comment if the mutation is intentional.","fixFiles":[{"filePath":"libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx","diff":"diff --git a/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx b/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx\n--- a/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx\n+++ b/libs/features/create-object-and-fields/src/CreateFieldsImportExport.tsx\n@@ -53,7 +53,6 @@\n   }\n \n   async function handleImport({ content }: InputReadFileContent) {\n-    // eslint-disable-next-line prefer-const\n     let { data, errors } = await parseFile(content, { onParsedMultipleWorkbooks });\n     trackEvent(ANALYTICS_KEYS.sobj_create_field_import_fields, {\n       numFields: data.length,\n"}]}]

[Copilot]

Pull request overview

This PR targets code quality in CreateFieldsImportExport.tsx by tightening a state type and removing a stale ESLint suppression around an intentionally reassigned let binding.

Changes:

• Replaced useState<any[]> with useState<FieldValues[]> for exportData
• Removed // eslint-disable-next-line prefer-const above let { data, errors } = ... in handleImport

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.