danger fix

Author: coado

.github/actions/diff-js-api-breaking-changes/action.yml

@@ -17,6 +17,9 @@ runs:
       env:
         SCRATCH_DIR: ${{ runner.temp }}/diff-js-api-breaking-changes
       run: |
+        # Check if the file was changed in this PR, if not, exit early without saving to the output.json
+        git diff --name-status | awk '{print $2}' | grep 'packages/react-native/ReactNativeApi.d.ts' || exit 0
+
         node ./scripts/js-api/diff-api-snapshot \
         ${{ github.workspace }}/packages/react-native/ReactNativeApi.d.ts  \
         $SCRATCH_DIR/ReactNativeApi-after.d.ts \

private/react-native-bots/dangerfile.js

@@ -30,26 +30,22 @@ const isFromPhabricator = body_contains('differential revision:');
 // Provides advice if a summary section is missing, or body is too short
 const includesSummary = body_contains('## summary', 'summary:');
 
-const snapshot_output = JSON.parse(
-  fs.readFileSync(
-    path.join(
-      process.env.RUNNER_TEMP,
-      'diff-js-api-breaking-changes/output.json',
-    ),
-    'utf8',
-  ),
+const snapshot_path = path.join(
+  process.env.RUNNER_TEMP,
+  'diff-js-api-breaking-changes/output.json',
 );
 
-console.log('Snapshot output:', snapshot_output);
-
-if (snapshot_output && snapshot_output.result !== 'NON_BREAKING') {
-  const title = ':exclamation: JavaScript API change detected';
-  const idea =
-    'This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native's public JavaScript API. ' +
-    'Please include a clear changelog message. ' +
-    'This change will be subject to extra review.\n\n' +
-    `This change was flagged as: <code>${snapshot_output.result}</code>`;
-  warn(`${title} - <i>${idea}</i>`);
+if (fs.existsSync(snapshot_path)) {
+  const snapshot_output = JSON.parse(fs.readFileSync(snapshot_path, 'utf8'));
+  if (snapshot_output && snapshot_output.result !== 'NON_BREAKING') {
+    const title = ':exclamation: JavaScript API change detected';
+    const idea =
+      'This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native&#39;s public JavaScript API. ' +
+      'Please include a clear changelog message. ' +
+      'This change will be subject to extra review.\n\n' +
+      `This change was flagged as: <code>${snapshot_output.result}</code>`;
+    warn(`${title} - <i>${idea}</i>`);
+  }
 }
 
 const hasNoUsefulBody =