feat(sandbox): remove empty string check and mark todo in a duplicate code

Author: BibiSebi

packages/field-plugin/helpers/vite/src/manifest.ts

@@ -1,6 +1,6 @@
 import { existsSync, readFileSync } from 'fs'
 import { resolve } from 'path'
-import { isEmpty, isOptionObject, isString } from './validation/manifest.ts'
+import { isOptionObject, isString } from './validation/manifest.ts'
 
 export const MANIFEST_FILE_NAME = 'field-plugin.config.json'
 export const MANIFEST_FULL_PATH = resolve(process.cwd(), MANIFEST_FILE_NAME)
@@ -70,13 +70,6 @@ const validateOptions = (options: unknown[]): void => {
       )
       continue
     }
-
-    if (isEmpty(option.value)) {
-      incorrectValues.push(
-        `${JSON.stringify(option)} --> Incorrect value. Must not be empty.`,
-      )
-      continue
-    }
   }
 
   if (incorrectValues.length > 0) {

packages/field-plugin/helpers/vite/src/validation/manifest.ts

@@ -2,8 +2,6 @@ import { ManifestOption } from '../manifest.ts'
 
 export const isString = (value: unknown) => typeof value === 'string'
 
-export const isEmpty = (value: string) => value === ''
-
 export const isOptionObject = (option: unknown): option is ManifestOption =>
   typeof option === 'object' &&
   option !== null &&

packages/manifest-helper/src/manifest.ts

@@ -52,6 +52,7 @@ const validateSchema = (manifest: Manifest): void => {
     throw new Error(`The 'options' property should be an array`)
   }
 
+  //TODO: update this to use the new validation functions
   manifest.options?.forEach((option) => {
     if (!('name' in option && 'value' in option)) {
       throw new Error(