Fix Ci (#2565)

* fix: Validate not running cron

* fix: always upload artifact

* fix: generate fail on first comment

.github/workflows/benchmark.yml

@@ -42,7 +42,6 @@ jobs:
           output-file-path: output.txt
           external-data-json-path: benchmarksResult/data.json
       - uses: actions/upload-artifact@v3
-        if: ${{ github.ref != 'refs/heads/main' }}
         with:
           name: benchmarkResults
           path: benchmarksResult/data.json

.github/workflows/validateCache.yml

@@ -1,12 +1,27 @@
 name: Validate Cache
 
 on:
+  schedule:
+    - cron: "0 0 * * *"
   workflow_dispatch:
 
-# This is just a dummy action for real one in benchies branch to run
 jobs:
   validate-cache:
     name: Validate Cache
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
+        with:
+          ref: "benchies"
+      - uses: denoland/setup-deno@main
+        with:
+          deno-version: "v1.x"
+      - name: Check And Update Objects
+        run: deno run -A cache/checkAndUpdateObjects.ts
+        env:
+          DISCORD_TOKEN: ${{ secrets.DISCORD_TOKEN }}
+      - name: Commit and push
+        uses: EndBug/add-and-commit@v9
+        with:
+          add: cache/cachedObject
+          message: "[Scheduled] Auto patch objects"

performance/generateMessage.ts

@@ -36,8 +36,8 @@ interface CompareTable {
 }
 
 const benchmarks = results.entries.Benchmark.slice(-2) as BenchmarksData[];
-const latestHeadBenchmarks = benchmarks[1];
-const lastHeadBenchmarks = benchmarks[0];
+const latestHeadBenchmarks = benchmarks.length === 2 ? benchmarks[1] : benchmarks[0];
+const lastHeadBenchmarks = benchmarks.length === 2 ? benchmarks[0] : undefined;
 //@ts-ignore
 const latestBaseBenchmarks = JSON.parse(JSON.stringify(window.BENCHMARK_DATA.entries.Benchmark)).slice(
   -1,
@@ -46,11 +46,13 @@ const latestBaseBenchmarks = JSON.parse(JSON.stringify(window.BENCHMARK_DATA.ent
 const compareWithHead: CompareTable = {};
 const compareWithBase: CompareTable = {};
 
-for (const benchmark of lastHeadBenchmarks.benches) {
-  compareWithHead[benchmark.name] = {
-    previous: benchmark,
-    current: {},
-  };
+if (lastHeadBenchmarks) {
+  for (const benchmark of lastHeadBenchmarks.benches) {
+    compareWithHead[benchmark.name] = {
+      previous: benchmark,
+      current: {},
+    };
+  }
 }
 for (const benchmark of latestBaseBenchmarks.benches) {
   compareWithBase[benchmark.name] = {
@@ -75,7 +77,7 @@ for (const benchmark of latestHeadBenchmarks.benches) {
 
 let message = "";
 
-const compareTableInfo = [{ name: "last head", commit: lastHeadBenchmarks.commit.id }, {
+const compareTableInfo = [{ name: "last head", commit: lastHeadBenchmarks ? lastHeadBenchmarks.commit.id : "" }, {
   name: "base",
   commit: latestBaseBenchmarks.commit.id,
 }];