Add Aliases using 1080p width in cloudfront, including TAGs in describe

[paulofloresjunior]

Add Aliases using 1080p width in cloudfront, including TAGs in describe

[paulofloresjunior]

Code review

Found 1 issue:

1. Tags are rendered manually in the detail view instead of using the existing DetailBuilder.Tags() method, which already handles sorting and rendering. This duplicates logic and uses d.Field() instead of d.Tag(), resulting in a different visual style than other resources (e.g., cloudformation/stacks uses d.Tags(appaws.TagsToMap(...))).

claws/custom/cloudfront/distributions/render.go

Lines 191 to 204 in 73631ff

	// Tags (only available after d:describe / Enter)
	if tags := dist.GetTags(); len(tags) > 0 {
	d.Section("Tags")
	keys := make([]string, 0, len(tags))
	for k := range tags {
	keys = append(keys, k)
	}
	sort.Strings(keys)
	for _, k := range keys {
	d.Field(k, tags[k])
	}
	}

The DetailBuilder.Tags() method already exists in internal/render/detail.go#L135-L152. The block could be replaced with a single call:

d.Tags(dist.GetTags())

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

[yimsk]

Sorry for the slow reply, and thanks for the update!
The CloudFront changes look good to me now.

Could you also revert the unrelated changes in:

internal/render/detail.go
.github/workflows/claude-code-review.yml

I’d prefer to keep this PR scoped to the CloudFront improvements only. Thanks again!

[yimsk]

Thanks again for the updates, and sorry for the slow follow-up. Merged now! really nice improvement. Appreciate the contribution!