refactor(utils): add safe JSON parsing util

- Added `safeParseJSON` utility function.
- Improves error handling for JSON parsing.
- Prevents crashes from invalid JSON input.
- Handles undefined or null input gracefully.
- Added JSON parsing error logging.

Author: taj54

src/index.ts

@@ -1,5 +1,5 @@
 import { UpdaterService, GitService, ChangelogService } from './services';
-import { FileHandler } from './utils';
+import { FileHandler, safeParseJSON } from './utils';
 import { UpdaterRegistry } from './registry';
 import {
   PlatformDetectionError,
@@ -8,6 +8,7 @@ import {
   InvalidManifestError,
 } from './errors';
 import { RELEASE_TYPE, TARGET_PLATFORM, GIT_TAG, TARGET_PATH, BUMP_TARGETS } from './config';
+
 import * as core from '@actions/core';
 
 async function initializeServices() {
@@ -37,7 +38,7 @@ async function run() {
     const platform = updaterService.getPlatform(targetPlatform);
     core.info(`Detected platform: ${platform}`);
 
-    const bumpTargets = JSON.parse(BUMP_TARGETS);
+    const bumpTargets = safeParseJSON<{ path: string; variable: string }[]>(BUMP_TARGETS);
     const version = updaterService.updateVersion(platform, releaseType, bumpTargets);
     core.setOutput('new_version', version);
 

src/utils/index.ts

@@ -1,3 +1,4 @@
 export * from './versionUtil';
 export * from './fileHandler';
 export * from './manifestParser';
+export * from './jsonUtils';

src/utils/jsonUtils.ts

@@ -0,0 +1,16 @@
+/**
+ * Safely parses a JSON string.
+ * Returns null (or a fallback) if parsing fails or input is invalid.
+ */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+export function safeParseJSON<T>(str: string | undefined | null): T | undefined {
+  try {
+    if (!str || typeof str !== 'string' || str.trim() === '') {
+      return undefined;
+    }
+    return JSON.parse(str) as T;
+  } catch (e) {
+    console.error('Invalid JSON:', e);
+    return undefined;
+  }
+}