Fix: console errors in unix format (#96)

* fix: console errors in unix format

* remove dupicate coments

* fix: console errors in unix format

* Fix: more error codes

* Fix: more error codes

* Fix: more error codes

* Fix: more error codes

* Fix: more error codes

* version update

Author: sojinantony01

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-cron-generator",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "A powerful React component for building cron expressions with support for both Unix (5 fields) and Quartz (7 fields) formats. Features validation, format conversion, TypeScript support, and accessibility.",
   "keywords": [
     "react",

src/lib/cron.test.tsx

@@ -559,6 +559,179 @@ describe('Cron Component - Custom Headers', () => {
       expect(screen.getByText('Monthly')).toBeInTheDocument();
       expect(screen.getByText('Custom')).toBeInTheDocument();
     });
+  });
+
+  describe('Unix Format Initialization with Values', () => {
+    it('should initialize with Unix format value without console errors', () => {
+      const onChange = vi.fn();
+      const consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
+      const consoleWarn = vi.spyOn(console, 'warn').mockImplementation(() => {});
+
+      render(
+        <Cron
+          value="*/5 * * * *"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      // Should display the Unix value
+      expect(screen.getByText('*/5 * * * *')).toBeInTheDocument();
+
+      // Should not have any console errors
+      expect(consoleError).not.toHaveBeenCalled();
+      expect(consoleWarn).not.toHaveBeenCalled();
+
+      consoleError.mockRestore();
+      consoleWarn.mockRestore();
+    });
+
+    it('should initialize with hourly Unix format value', () => {
+      const onChange = vi.fn();
+      const consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
+
+      render(
+        <Cron
+          value="0 * * * *"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      expect(screen.getByText('0 * * * *')).toBeInTheDocument();
+      expect(consoleError).not.toHaveBeenCalled();
+
+      consoleError.mockRestore();
+    });
+
+    it('should initialize with daily Unix format value', () => {
+      const onChange = vi.fn();
+      const consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
+
+      render(
+        <Cron
+          value="0 0 * * *"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      expect(screen.getByText('0 0 * * *')).toBeInTheDocument();
+      expect(consoleError).not.toHaveBeenCalled();
+
+      consoleError.mockRestore();
+    });
+
+    it('should initialize with weekly Unix format value', () => {
+      const onChange = vi.fn();
+      const consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
+
+      render(
+        <Cron
+          value="0 0 * * 1"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      expect(screen.getByText('0 0 * * 1')).toBeInTheDocument();
+      expect(consoleError).not.toHaveBeenCalled();
+
+      consoleError.mockRestore();
+    });
+
+    it('should initialize with monthly Unix format value', () => {
+      const onChange = vi.fn();
+      const consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
+
+      render(
+        <Cron
+          value="0 0 1 * *"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      expect(screen.getByText('0 0 1 * *')).toBeInTheDocument();
+      expect(consoleError).not.toHaveBeenCalled();
+
+      consoleError.mockRestore();
+    });
+
+    it('should handle complex Unix cron expressions', () => {
+      const onChange = vi.fn();
+      const consoleError = vi.spyOn(console, 'error').mockImplementation(() => {});
+
+      render(
+        <Cron
+          value="*/15 2-5 * * 1-5"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      expect(screen.getByText('*/15 2-5 * * 1-5')).toBeInTheDocument();
+      expect(consoleError).not.toHaveBeenCalled();
+
+      consoleError.mockRestore();
+    });
+
+    it('should convert Unix to Quartz internally and back to Unix for display', () => {
+      const onChange = vi.fn();
+
+      render(
+        <Cron
+          value="*/5 * * * *"
+          onChange={onChange}
+          showResultText={true}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      // Should display Unix format
+      expect(screen.getByText('*/5 * * * *')).toBeInTheDocument();
+      // Should have human-readable text
+      expect(screen.getByText(/every 5 minutes/i)).toBeInTheDocument();
+    });
+
+    it('should handle onChange callback with Unix format', async () => {
+      const onChange = vi.fn();
+      const user = userEvent.setup();
+
+      render(
+        <Cron
+          value="*/5 * * * *"
+          onChange={onChange}
+          showResultText={false}
+          showResultCron={true}
+          isUnix={true}
+        />,
+      );
+
+      // Switch to hourly tab
+      const hourlyTab = screen.getByLabelText('Select Hourly tab');
+      await user.click(hourlyTab);
+
+      await waitFor(() => {
+        expect(onChange).toHaveBeenCalled();
+        // Should call with Unix format (5 fields)
+        const lastCall = onChange.mock.calls[onChange.mock.calls.length - 1];
+        expect(lastCall[0].split(' ').length).toBe(5);
+      });
+    });
 
     it('should work with custom headers and translations', () => {
       const onChange = vi.fn();

src/lib/cron.tsx

@@ -177,16 +177,24 @@ const Cron: React.FunctionComponent<CronProp> = (props) => {
 
       // Convert Unix to Quartz if needed for internal representation
       if (props.isUnix && value) {
-        const parts = value.split(' ');
-        if (parts.length === 5) {
+        if (value.split(' ').length === 5) {
           try {
             processedValue = unixToQuartz(value);
           } catch (e) {
-            console.error('Error converting Unix to Quartz:', e);
+            console.error('Error: converting Unix to Quartz:', e);
             processedValue = defaultCron;
           }
         }
       }
+      if (!props.isUnix && value && value.split(' ').length === 5) {
+        try {
+          console.error('Warning: Unix value found,Converting to Quartz');
+          processedValue = unixToQuartz(value);
+        } catch (e) {
+          console.error('Error: converting Unix to Quartz:', e);
+          processedValue = defaultCron;
+        }
+      }
 
       let valueArray = processedValue.replace(/,/g, '!').split(' ');
 
@@ -286,7 +294,8 @@ const Cron: React.FunctionComponent<CronProp> = (props) => {
    * Sync with external value prop
    */
   useEffect(() => {
-    const compareVal = convertToOutputFormat(stateRef.current.value);
+    const hasValidState = stateRef.current.value && stateRef.current.value.length > 0;
+    const compareVal = hasValidState ? convertToOutputFormat(stateRef.current.value) : '';
 
     if (props.value !== compareVal) {
       setValue(props.value ? props.value : '');
@@ -297,9 +306,6 @@ const Cron: React.FunctionComponent<CronProp> = (props) => {
     }
   }, [props.value, props.translateFn, props.locale, setValue, convertToOutputFormat]);
 
-  /**
-   * Handle isUnix prop changes - notify parent with converted format
-   */
   /**
    * Handle isUnix prop changes - notify parent with converted format
    */
@@ -381,6 +387,10 @@ const Cron: React.FunctionComponent<CronProp> = (props) => {
    * Get display value for result cron (uses single source of truth)
    */
   const displayCron = useMemo(() => {
+    // Only convert if we have a valid state with actual values
+    if (!state.value || state.value.length === 0) {
+      return '';
+    }
     return convertToOutputFormat(state.value, props.isUnix);
   }, [state.value, props.isUnix, convertToOutputFormat]);