🐛 Address review comments: restore expect.txt, fix stored fields, fix ID collision

- Restore KSK and kustomize to spell check expect.txt (both still used in
  checked .lr files)
- Change notebookinstancedetails ipAddressType, platformIdentifier,
  volumeSizeInGB, failureReason from stored properties to computed methods
  (they are populated via Internal struct cache, not CreateResource)
- Fix clusterInstanceGroup ID collision by including cluster name in the
  composite ID (region/clusterName/instanceGroupName)

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

Author: tas50

.github/actions/spelling/expect.txt

@@ -145,7 +145,9 @@ kerberoastable
 keyspace
 kqueue
 krbtgt
+KSK
 kustomization
+kustomize
 kubenet
 labelmatchstatement
 launchconfiguration

providers/aws/resources/aws.lr

@@ -1702,13 +1702,13 @@ private aws.sagemaker.notebookinstancedetails @defaults("arn") {
   // VPC subnet the notebook instance is deployed in
   subnet() aws.vpc.subnet
   // IP address type: ipv4 or dualstack
-  ipAddressType string
+  ipAddressType() string
   // Platform identifier of the notebook instance runtime environment
-  platformIdentifier string
+  platformIdentifier() string
   // Size (in GB) of the ML storage volume attached to the instance
-  volumeSizeInGB int
+  volumeSizeInGB() int
   // Failure reason if the notebook instance is in a failed state
-  failureReason string
+  failureReason() string
   // IAM role associated with the instance
   iamRole() aws.iam.role
   // VPC security groups associated with the instance

providers/aws/resources/aws.lr.go

@@ -1979,6 +1979,7 @@ func (a *mqlAwsSagemakerCluster) instanceGroups() ([]any, error) {
 			return nil, err
 		}
 		igRes := mqlIG.(*mqlAwsSagemakerClusterInstanceGroup)
+		igRes.cacheClusterName = a.Name.Data
 		igRes.cacheExecutionRole = ig.ExecutionRole
 		igRes.cacheLifecycleConfig = ig.LifeCycleConfig
 		res = append(res, mqlIG)
@@ -2073,12 +2074,13 @@ func (a *mqlAwsSagemakerCluster) nodes() ([]any, error) {
 }
 
 type mqlAwsSagemakerClusterInstanceGroupInternal struct {
+	cacheClusterName     string
 	cacheExecutionRole   *string
 	cacheLifecycleConfig any
 }
 
 func (a *mqlAwsSagemakerClusterInstanceGroup) id() (string, error) {
-	return a.Region.Data + "/" + a.InstanceGroupName.Data, nil
+	return a.Region.Data + "/" + a.cacheClusterName + "/" + a.InstanceGroupName.Data, nil
 }
 
 func (a *mqlAwsSagemakerClusterInstanceGroup) iamRole() (*mqlAwsIamRole, error) {