From 6a0e878711ef2dd3b1dffb8822fa7a5a8ed16368 Mon Sep 17 00:00:00 2001
From: Korrrba <gitcoiner@gmail.com>
Date: Wed, 24 Jan 2024 11:12:01 +0100
Subject: [PATCH] chore: limit diamond storage check API calls

Narrow down test matrix scope to changed Dollar libraries

Resolves: #876
---
 .github/workflows/diamond-storage-check.yml | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/diamond-storage-check.yml b/.github/workflows/diamond-storage-check.yml
index be62c3f16..e7a4a8466 100644
--- a/.github/workflows/diamond-storage-check.yml
+++ b/.github/workflows/diamond-storage-check.yml
@@ -24,10 +24,24 @@ jobs:
         with:
           version: nightly
 
-      - id: set-matrix
+      - name: Narrow down test matrix scope to changed Dollar libraries to limit API requests
+        id: changed-libraries
+        uses: tj-actions/changed-files@v42
+        with:
+          files_yaml: |
+            libraries:
+              - packages/contracts/src/dollar/libraries/Lib*.sol
+
+      - name: Set contracts matrix
+        id: set-matrix
         working-directory: packages/contracts
+        if: steps.changed-libraries.outputs.libraries_any_changed == 'true'
+        env:
+          CHANGED_LIBS: ${{ steps.changed-libraries.outputs.libraries_all_changed_files }}
         run: |
-          forge tree | grep -E '^src/dollar/libraries/Lib' | cut -d' ' -f1 | xargs basename -a | cut -d'.' -f1 | xargs -I{} echo src/dollar/libraries/{}.sol:{} >> contracts.txt
+          for DIAMOND_LIB in "$CHANGED_LIBS"; do
+            echo ${DIAMOND_LIB} | xargs basename -a | cut -d'.' -f1 | xargs -I{} echo src/dollar/libraries/{}.sol:{} >> contracts.txt
+          done
           echo "matrix=$(cat contracts.txt | jq -R -s -c 'split("\n")[:-1]')" >> $GITHUB_OUTPUT
 
     outputs: