ignore outdated actions on generating popular actions data set

scripts/generate-popular-actions/main.go

@@ -13,6 +13,7 @@ import (
 	"net/http"
 	"os"
 	"sort"
+	"strconv"
 	"strings"
 
 	"github.com/rhysd/actionlint"
@@ -61,6 +62,16 @@ func (r *registry) spec(tag string) string {
 //go:embed popular_actions.json
 var defaultPopularActionsJSON []byte
 
+const minNodeRunnerVersion = 16
+
+func isOutdatedRunner(r string) bool {
+	if !strings.HasPrefix(r, "node") {
+		return false
+	}
+	v, err := strconv.ParseUint(r[len("node"):], 10, 8)
+	return err == nil && v < minNodeRunnerVersion
+}
+
 type gen struct {
 	stdout      io.Writer
 	stderr      io.Writer
@@ -169,6 +180,10 @@ func (g *gen) fetchRemote() (map[string]*actionlint.ActionMetadata, error) {
 			close(done)
 			return nil, f.err
 		}
+		if isOutdatedRunner(f.meta.Runs.Using) {
+			g.log.Printf("Ignore outdated action %q since runner is %q", f.spec, f.meta.Runs.Using)
+			continue
+		}
 		ret[f.spec] = f.meta
 	}
 

scripts/generate-popular-actions/main_test.go

@@ -193,23 +193,35 @@ func TestWriteGoFile(t *testing.T) {
 }
 
 func TestFetchRemoteYAML(t *testing.T) {
-	f := filepath.Join("testdata", "registry", "fetch.json")
-	stdout := &bytes.Buffer{}
-	stderr := io.Discard
-	status := newGen(stdout, stderr, io.Discard).run([]string{"test", "-r", f})
-	if status != 0 {
-		t.Fatal("exit status is non-zero:", status)
+	tests := []struct {
+		registry string
+		want     string
+	}{
+		{"fetch.json", "fetched.go"},
+		{"outdated.json", "outdated.go"},
 	}
 
-	b, err := os.ReadFile(filepath.Join("testdata", "go", "fetched.go"))
-	if err != nil {
-		panic(err)
-	}
-	want := string(b)
-	have := stdout.String()
+	for _, tc := range tests {
+		t.Run(tc.registry, func(t *testing.T) {
+			f := filepath.Join("testdata", "registry", tc.registry)
+			stdout := &bytes.Buffer{}
+			stderr := io.Discard
+			status := newGen(stdout, stderr, io.Discard).run([]string{"test", "-r", f})
+			if status != 0 {
+				t.Fatal("exit status is non-zero:", status)
+			}
+
+			b, err := os.ReadFile(filepath.Join("testdata", "go", tc.want))
+			if err != nil {
+				panic(err)
+			}
+			want := string(b)
+			have := stdout.String()
 
-	if !cmp.Equal(want, have) {
-		t.Fatalf("fetched JSONL data does not match: %s", cmp.Diff(want, have))
+			if !cmp.Equal(want, have) {
+				t.Fatalf("fetched JSONL data does not match: %s", cmp.Diff(want, have))
+			}
+		})
 	}
 }
 

scripts/generate-popular-actions/testdata/registry/fetch.json

@@ -1,7 +1,7 @@
 [
     {
       "slug": "rhysd/action-setup-vim",
-      "tags": ["v1.2.7"],
+      "tags": ["v1.3.2"],
       "next": "v2"
     },
     {

scripts/generate-popular-actions/testdata/registry/outdated.json

@@ -0,0 +1,6 @@
+[
+    {
+      "slug": "rhysd/action-setup-vim",
+      "tags": ["v1.0.0"]
+    }
+]