Check if descriptor is nil when filtering unstable packages

internal/buf/bufcheck/bufbreaking/bufbreaking_test.go

@@ -309,6 +309,15 @@ func TestRunBreakingFileNoDelete(t *testing.T) {
 	)
 }
 
+func TestRunBreakingFileNoDeleteUnstable(t *testing.T) {
+	// https://github.com/bufbuild/buf/issues/211
+	testBreaking(
+		t,
+		"breaking_file_no_delete_unstable",
+		bufanalysistesting.NewFileAnnotationNoLocationOrPath(t, "FILE_NO_DELETE"),
+	)
+}
+
 func TestRunBreakingFileSamePackage(t *testing.T) {
 	testBreaking(
 		t,

internal/buf/bufcheck/bufbreaking/testdata/breaking_file_no_delete_unstable/a/v1beta1/a.proto

@@ -0,0 +1,3 @@
+syntax = "proto3";
+
+package a.v1beta1;

internal/buf/bufcheck/bufbreaking/testdata/breaking_file_no_delete_unstable/buf.yaml

@@ -0,0 +1,3 @@
+version: v1beta1
+breaking:
+  ignore_unstable_packages: true

internal/buf/bufcheck/bufbreaking/testdata_previous/breaking_file_no_delete_unstable/a/v1beta1/a.proto

@@ -0,0 +1,3 @@
+syntax = "proto3";
+
+package a.v1beta1;

internal/buf/bufcheck/bufbreaking/testdata_previous/breaking_file_no_delete_unstable/a/v1beta1/b.proto

@@ -0,0 +1,3 @@
+syntax = "proto3";
+
+package a.v1beta1;

internal/buf/bufcheck/internal/runner.go

@@ -93,6 +93,9 @@ func (r *Runner) newIgnoreFunc(config *Config) IgnoreFunc {
 			return true
 		}
 		if config.IgnoreUnstablePackages {
+			if descriptor == nil {
+				return false
+			}
 			packageVersion, ok := protoversion.NewPackageVersionForPackage(descriptor.File().Package())
 			if !ok {
 				return false