Update error handle in case there's panic error or issue in resource schema

Author: QixiaLu

.github/workflows/document-lint.yaml

@@ -37,11 +37,24 @@ jobs:
       - name: run document lint checker
         if: steps.changed-files.outputs.has-files == 'true'
         run: |
-          if make document-lint; then
+          set +e
+          make document-lint
+          exit_code=$?
+          set -e
+
+          if [ $exit_code -eq 0 ]; then
             echo "::notice::Document lint success."
-          else
+            exit 0
+          elif [ $exit_code -eq 1 ]; then
             echo "::warning::Document lint failed. Please fix the issues."
             exit 1
+          elif [ $exit_code -eq 2 ]; then
+            echo "::warning::Document lint encountered a panic. This may indicate an issue with the linter itself or resource schema."
+            echo "::notice::Allowing workflow to continue despite panic."
+            exit 0
+          else
+            echo "::error::Document lint exited with unexpected code: $exit_code"
+            exit 1
           fi
 
       - name: skip document lint checker

internal/services/resource/resource_provider_registration_resource.go

@@ -88,7 +88,7 @@ func (r ResourceProviderRegistrationResource) Attributes() map[string]*pluginsdk
 	return map[string]*pluginsdk.Schema{
 		"test_missing_attr_in_typed_resource": {
 			Type:     pluginsdk.TypeString,
-			Required: true,
+			Computed: true,
 		},
 	}
 }

internal/tools/document-lint/check/check_resource_all.go

@@ -124,6 +124,10 @@ func DiffAll(regs Resources, dryRun bool) *DiffResult {
 		var catName string
 
 		sch := schema.NewResource(res.schema, res.name)
+		if sch == nil {
+			// failed to create resource schema, skipping
+			continue
+		}
 		rd := NewResourceDiff(sch)
 		if !dryRun {
 			md.FixFileNormalize(rd.MDFile)

internal/tools/document-lint/schema/resource_schema.go

@@ -4,6 +4,7 @@
 package schema
 
 import (
+	"fmt"
 	"strings"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -32,18 +33,25 @@ type Resource struct {
 	PossibleValues map[string][]string // possible values for key(property path)
 }
 
-func ResourceForSDKType(res sdk.Resource) *schema.Resource {
+func ResourceForSDKType(res sdk.Resource) (*schema.Resource, error) {
 	r := sdk.NewResourceWrapper(res)
-	ins, _ := r.Resource()
-	return ins
+	ins, err := r.Resource()
+	if err != nil {
+		return nil, fmt.Errorf("failed to get resource schema for %s: %v", res.ResourceType(), err)
+	}
+	return ins, nil
 }
 
 // NewResourceByTyped NewResource ...
 // r is Schema.Resource or Typed SDK Resource
 func NewResourceByTyped(r sdk.Resource) *Resource {
 	s := &Resource{}
 	s.SDKResource = r
-	s.Schema = ResourceForSDKType(r)
+	schema, err := ResourceForSDKType(r)
+	if err != nil {
+		return nil
+	}
+	s.Schema = schema
 	s.ResourceType = r.ResourceType()
 	s.Init()
 	return s
@@ -68,6 +76,10 @@ func NewResource(r interface{}, rType string) *Resource {
 }
 
 func (r *Resource) Init() {
+	if r.Schema == nil {
+		return
+	}
+	
 	if r.SDKResource != nil {
 		// SDKResource is a type of interface, have to get the real
 		// vd := reflect.ValueOf(r.SDKResource).Interface()