🐛 Fix ID collisions, access-denied handling, and spell check

Fix featureDefinition and clusterNode cache key collisions by including
parent resource identifiers in IDs. Add Is400AccessDeniedError handling
to ListClusterNodes. Add Slurm to spell-check dictionary and document
modelPackages listing limitation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: tas50

.github/actions/spelling/expect.txt

@@ -272,6 +272,7 @@ Sflags
 signin
 singlequeryargument
 sizeconstraintstatement
+Slurm
 smbv
 Snat
 SNYK

providers/aws/resources/aws.lr

@@ -1645,7 +1645,7 @@ aws.sagemaker {
   clusters() []aws.sagemaker.cluster
   // List of SageMaker feature groups
   featureGroups() []aws.sagemaker.featureGroup
-  // List of SageMaker model packages
+  // List of SageMaker model packages (excludes packages within model package groups)
   modelPackages() []aws.sagemaker.modelPackage
   // List of SageMaker model package groups
   modelPackageGroups() []aws.sagemaker.modelPackageGroup

providers/aws/resources/aws.lr.go

@@ -2044,6 +2044,10 @@ func (a *mqlAwsSagemakerCluster) nodes() ([]any, error) {
 	for paginator.HasMorePages() {
 		page, err := paginator.NextPage(ctx)
 		if err != nil {
+			if Is400AccessDeniedError(err) {
+				log.Warn().Str("cluster", clusterName).Msg("error accessing AWS SageMaker cluster nodes")
+				return res, nil
+			}
 			return nil, err
 		}
 		for _, node := range page.ClusterNodeSummaries {
@@ -2067,6 +2071,8 @@ func (a *mqlAwsSagemakerCluster) nodes() ([]any, error) {
 			if err != nil {
 				return nil, err
 			}
+			nodeRes := mqlNode.(*mqlAwsSagemakerClusterNode)
+			nodeRes.cacheClusterName = clusterName
 			res = append(res, mqlNode)
 		}
 	}
@@ -2103,8 +2109,12 @@ func (a *mqlAwsSagemakerClusterInstanceGroup) lifecycleConfig() (map[string]any,
 	return convert.JsonToDict(a.cacheLifecycleConfig)
 }
 
+type mqlAwsSagemakerClusterNodeInternal struct {
+	cacheClusterName string
+}
+
 func (a *mqlAwsSagemakerClusterNode) id() (string, error) {
-	return a.Region.Data + "/" + a.InstanceGroupName.Data + "/" + a.InstanceId.Data, nil
+	return a.Region.Data + "/" + a.cacheClusterName + "/" + a.InstanceGroupName.Data + "/" + a.InstanceId.Data, nil
 }
 
 // ---- Feature Groups ----
@@ -2273,13 +2283,19 @@ func (a *mqlAwsSagemakerFeatureGroup) featureDefinitions() ([]any, error) {
 		if err != nil {
 			return nil, err
 		}
+		fdRes := mqlFD.(*mqlAwsSagemakerFeatureDefinition)
+		fdRes.cacheFeatureGroupArn = a.Arn.Data
 		res = append(res, mqlFD)
 	}
 	return res, nil
 }
 
+type mqlAwsSagemakerFeatureDefinitionInternal struct {
+	cacheFeatureGroupArn string
+}
+
 func (a *mqlAwsSagemakerFeatureDefinition) id() (string, error) {
-	return a.FeatureName.Data + "/" + a.FeatureType.Data, nil
+	return a.cacheFeatureGroupArn + "/" + a.FeatureName.Data + "/" + a.FeatureType.Data, nil
 }
 
 func (a *mqlAwsSagemakerFeatureGroup) eventTimeFeatureName() (string, error) {