Conversation
* Add EKS support: 5 resources (clusters, node-groups, fargate-profiles, addons, access-entries)
- clusters: standalone, 11 detail sections
- node-groups: sub-resource, scaling/health info
- fargate-profiles: sub-resource, selectors/subnets
- addons: sub-resource, versions/config
- access-entries: sub-resource, IAM→K8s mapping
- registry: add EKS to Compute category
- SDK: aws-sdk-go-v2/service/eks v1.76.3
Total: 20 files, ~1807 lines
* Fix EKS code review issues
- node-groups: Return all instance types (not just first)
- registry: Add EKS default resource (clusters)
* Add EKS navigation + test fixtures + fix global key conflicts
- EKS: Full navigation for all 5 resources (clusters, node-groups, fargate-profiles, addons, access-entries)
- Parent cluster (p), IAM roles (r), CloudWatch logs (l), VPC/SG (v/s), ASG/LaunchTemplate (g/t)
- Add EKS test fixtures: CloudFormation template + deploy/cleanup scripts for integration testing
- Add LocalStack tasks for EKS demo data
- Fix global key conflicts: c→p (Clear filter), a→o/g/t (Actions menu), m→s (Mark resource)
- Affected: ECS, RDS, autoscaling, stepfunctions, transit-gateways
* Fix IAM role/user navigation filtering
Add RoleName/UserName filter support to enable navigation from other resources (EKS, EC2, Lambda, ECS, etc).
Changes:
- roles: Check RoleName filter in ListPage(), direct Get() lookup
- users: Check UserName filter in List(), direct Get() lookup
- Return empty list if not found (not error, for filter behavior)
Fixes: Navigation failure when pressing 'r' on EKS clusters/nodes/fargate/addons
* Improve EKS test stack cleanup automation
Enhance cleanup.sh reliability w/ auto-retry & auth fix:
- Auto-retry on DELETE_FAILED (max 2 attempts)
- Fix auth mode to API before deletion (resolves deletion blocks)
- CI_MODE env var for non-interactive cleanup
- Update README: document retry behavior, env vars (CI_MODE, MAX_RETRY)
Production-grade improvements for CI/CD integration.
* Fix navigation filters: EKS parent nav + filter clear reload
- EKS clusters: Add ClusterName filter for child->parent nav
- IAM roles: Move filter check List()->consistent w/ users/clusters
- Resource browser: Reload on 'c' (fix filtered DAO cache)
* Fix deprecated strings.Title in EKS access-entries
* Fix navigation: unwrap resource for type assertion
Navigation broken - Navigations() returned empty array due to failed type assertion.
Root cause: handleNavigation() used wrapped resource from contextForResource(),
but Navigations() methods expect unwrapped concrete types (e.g. *ClusterResource).
Fix: Use dao.UnwrapResource() like getNavigationShortcuts() does.
Test: All resource navigation keys (n/f/a/e/r/l/v/s/g/t/o/p) work.
* Add filter support for ASG/LaunchTemplate/KeyPair navigation
Enables filtered navigation from EKS node-groups. DAOs now check GetFilterFromContext for AutoScalingGroupName, LaunchTemplateId, KeyName filters, returning single resource instead of full list.
* Fix IsNotFound(): match lowercase 'not found' from DAO errors
79 DAOs use fmt.Errorf('resource not found: %s', id) (lowercase).
IsNotFound() only checked AWS SDK codes + 'NotFound' (capital N).
Filter navigation (ASG/EC2/IAM/EKS) relies on IsNotFound() -> failed on non-existent resources.
Add 'not found' to hasErrorCode() -> 1 line fix, no user-facing change.
* Fix ResourceBrowser: unwrap resources for ActionMenu/DiffView
* Update docs: reflect 169 resources (EKS support)
- Update README.md: 163→169 resources, add EKS
- Regenerate imports_custom.go via task gen-imports
* Update services.md: add EKS and ECS task-definitions
- Update resource count: 163→169
- Add EKS to Containers & ML section (5 resources)
- Add Task Definitions to ECS
- Add eks alias
* Fix DiffView AI chat: preserve region/profile metadata
Problem: commit 2780b20 broke AI chat in diff mode by unwrapping
resources before passing to DiffView, losing region/profile info.
Solution: DiffView now stores both wrapped (for metadata) and
unwrapped (for rendering) resources. buildAIContext() gets correct
region/profile from wrapped resources via buildResourceRef().
Files: internal/view/{diff_view,resource_browser_input}.go
This comment was marked as resolved.
This comment was marked as resolved.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
v0.11.0 Release
New Features
EKS Support (5 resources)
Navigation Enhancements
Test Infrastructure
Bug Fixes
Stats
Notes
Merge Strategy: MERGE COMMIT (required for develop↔main)
Related