Merge pull request #48 from DevopsArtFactory/feat/verbose-logging

feat: add debug logging and --verbose flag (M4.2)

Author: YoungJinJung

README.md

@@ -31,6 +31,10 @@ unic
 unic --profile my-profile
 unic --region ap-northeast-2
 
+# Enable verbose debug logging (writes to ~/.config/unic/logs/unic.log)
+unic --verbose
+unic -v
+
 # Initialize config file
 unic init                      # Create default config
 unic init --force              # Overwrite existing config
@@ -40,32 +44,100 @@ unic init --force              # Overwrite existing config
 
 `~/.config/unic/config.yaml` (created via `unic init` or auto-generated on first run)
 
+### Legacy Format (Flat)
+
 ```yaml
-# Simple format
 default_profile: my-profile
 default_region: ap-northeast-2
 ```
 
+### Context-Based Format
+
 ```yaml
-# Context-based format
 current: dev-sso
 
+defaults:
+  region: us-east-1
+
 contexts:
+  # SSO authentication
   - name: dev-sso
-    profile: dev-sso
     region: ap-northeast-2
+    auth_type: sso
+    sso_start_url: https://my-sso-portal.awsapps.com/start
+    sso_account_id: "123456789012"
+    sso_role_name: DeveloperRole
 
-  - name: prod-admin
-    profile: prod-admin
+  # Assume Role (cross-account)
+  - name: prod-assume
     region: us-east-1
+    auth_type: assume_role
+    profile: base-profile
+    role_arn: arn:aws:iam::987654321098:role/CrossAccountRole
+    external_id: optional-external-id
+
+  # Credential profile
+  - name: staging-creds
+    region: eu-west-1
+    auth_type: credential
+    profile: staging
 ```
 
+### Auth Types
+
+| Auth Type | Required Fields | Description |
+|-----------|----------------|-------------|
+| `sso` | `sso_start_url`, `sso_account_id`, `sso_role_name` | AWS SSO portal login with token caching |
+| `credential` | `profile` | Uses `~/.aws/credentials` profile directly |
+| `assume_role` | `profile`, `role_arn` | Assumes a cross-account role from a base profile |
+
 **Priority**: CLI flags (`--profile`, `--region`) > context settings > config defaults > hardcoded defaults (`us-east-1`)
 
 ## Currently Implemented Features
 
 | Service | Feature | Status |
 |---------|---------|--------|
+| EC2 | SSM Session Manager (connect to running, SSM-managed instances) | ✅ Implemented |
+| VPC | VPC Browser (VPCs → Subnets → Available IPs with reserved-IP exclusion) | ✅ Implemented |
+| RDS | RDS Browser (list, start/stop, failover, Aurora cluster support, auto-polling) | ✅ Implemented |
+| Route53 | DNS Browser (Hosted Zones → Records → Record Detail, public/private zones) | ✅ Implemented |
+| Secrets Manager | Secrets Browser (list secrets, view key-value pairs or raw values) | ✅ Implemented |
+
+## TUI Key Bindings
+
+### Global Navigation
+
+| Key | Action |
+|-----|--------|
+| `j`/`k` or `↑`/`↓` | Navigate list |
+| `Enter` | Select item |
+| `Esc` | Go back one screen |
+| `q` | Quit (on service list) |
+| `H` | Jump to home (service list) |
+| `C` | Open context switcher |
+| `/` | Toggle filter mode |
+| `Ctrl+C` | Force quit |
+
+### RDS Detail Actions
+
+| Key | Action | Condition |
+|-----|--------|-----------|
+| `s` | Start database | Instance/cluster is stopped |
+| `x` | Stop database | Instance/cluster is available |
+| `f` | Failover database | Multi-AZ standalone or Aurora cluster |
+| `r` | Refresh status | Always |
+
+### Context Switcher
+
+| Key | Action |
+|-----|--------|
+| `Enter` | Switch to selected context |
+| `a` | Add new context (wizard) |
+| `Esc` | Back |
+
+### Filtering
+
+Available on: EC2 instances, VPC/Subnets, RDS instances, Route53 zones/records, Secrets Manager. Press `/` to enter filter mode, type to search, `Esc` or `Enter` to exit filter mode.
 | EC2 | SSM Session Manager (connect to EC2 instances) | ✅ Implemented |
 | EC2 | Security Group Browser (list/filter SGs, view inbound/outbound rules) | ✅ Implemented |
 | VPC | VPC Browser (VPCs → subnets → available IPs) | ✅ Implemented |

cmd/unic/main.go

@@ -10,12 +10,18 @@ import (
 	"unic/internal/app"
 	"unic/internal/cli"
 	"unic/internal/config"
+	uniclog "unic/internal/log"
 )
 
 func main() {
 	rootCmd := cli.NewRootCmd()
 
 	rootCmd.RunE = func(cmd *cobra.Command, args []string) error {
+		if err := uniclog.Init(cli.Verbose()); err != nil {
+			return fmt.Errorf("logger init error: %w", err)
+		}
+		defer uniclog.Close()
+
 		configPath, err := config.DefaultPath()
 		if err != nil {
 			return err
@@ -30,6 +36,13 @@ func main() {
 			return fmt.Errorf("config load error: %w", err)
 		}
 
+		uniclog.Info("config", "config loaded",
+			"profile", cfg.Profile,
+			"region", cfg.Region,
+			"context", cfg.ContextName,
+			"auth_type", string(cfg.AuthType),
+		)
+
 		p := tea.NewProgram(app.New(cfg, configPath), tea.WithAltScreen())
 		if _, err := p.Run(); err != nil {
 			return fmt.Errorf("TUI error: %w", err)

go.mod

@@ -9,6 +9,7 @@ require (
 	github.com/aws/aws-sdk-go-v2/service/ec2 v1.296.0
 	github.com/aws/aws-sdk-go-v2/service/rds v1.116.3
 	github.com/aws/aws-sdk-go-v2/service/route53 v1.62.4
+	github.com/aws/aws-sdk-go-v2/service/secretsmanager v1.41.4
 	github.com/aws/aws-sdk-go-v2/service/ssm v1.68.3
 	github.com/aws/aws-sdk-go-v2/service/sso v1.30.13
 	github.com/aws/aws-sdk-go-v2/service/sts v1.41.9
@@ -25,7 +26,6 @@ require (
 	github.com/aws/aws-sdk-go-v2/internal/ini v1.8.6 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.7 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.13.20 // indirect
-	github.com/aws/aws-sdk-go-v2/service/secretsmanager v1.41.4 // indirect
 	github.com/aws/aws-sdk-go-v2/service/signin v1.0.8 // indirect
 	github.com/aws/aws-sdk-go-v2/service/ssooidc v1.35.17 // indirect
 	github.com/aws/smithy-go v1.24.2 // indirect

internal/auth/auth.go

@@ -13,12 +13,15 @@ import (
 	"github.com/aws/aws-sdk-go-v2/service/sts"
 
 	"unic/internal/config"
+	uniclog "unic/internal/log"
 	awsservice "unic/internal/services/aws"
 )
 
 // PostSwitch performs the auth action after switching to a context.
 // Returns a human-readable status message.
 func PostSwitch(cfg *config.Config) (string, error) {
+	uniclog.Info("auth", "post-switch started", "context", cfg.ContextName, "auth_type", string(cfg.AuthType))
+
 	var msg string
 	var err error
 
@@ -33,6 +36,7 @@ func PostSwitch(cfg *config.Config) (string, error) {
 		msg = fmt.Sprintf("Context %q activated (profile: %s, region: %s)", cfg.ContextName, cfg.Profile, cfg.Region)
 	}
 	if err != nil {
+		uniclog.Error("auth", "post-switch failed", "error", err.Error())
 		return "", err
 	}
 
@@ -46,6 +50,7 @@ func PostSwitch(cfg *config.Config) (string, error) {
 }
 
 func postSwitchSSO(cfg *config.Config) (string, error) {
+	uniclog.Debug("auth", "starting SSO login", "sso_start_url", cfg.SSOStartURL)
 	if err := awsservice.RunSSOLogin(cfg); err != nil {
 		return "", err
 	}
@@ -73,6 +78,7 @@ func postSwitchCredential(cfg *config.Config) (string, error) {
 }
 
 func postSwitchAssumeRole(cfg *config.Config) (string, error) {
+	uniclog.Debug("auth", "assuming role", "role_arn", cfg.RoleArn)
 	ctx := context.Background()
 
 	opts := []func(*awsconfig.LoadOptions) error{

internal/cli/root.go

@@ -10,6 +10,7 @@ var (
 
 	profile string
 	region  string
+	verbose bool
 )
 
 func NewRootCmd() *cobra.Command {
@@ -22,6 +23,7 @@ func NewRootCmd() *cobra.Command {
 
 	cmd.PersistentFlags().StringVarP(&profile, "profile", "p", "", "AWS profile to use")
 	cmd.PersistentFlags().StringVar(&region, "region", "", "AWS region to use")
+	cmd.PersistentFlags().BoolVarP(&verbose, "verbose", "v", false, "Enable verbose debug logging")
 
 	cmd.AddCommand(newInitCmd())
 
@@ -43,3 +45,8 @@ func Region() *string {
 	}
 	return &region
 }
+
+// Verbose returns true if --verbose was passed.
+func Verbose() bool {
+	return verbose
+}

internal/cli/root_test.go

@@ -24,4 +24,11 @@ func TestRootCmdHasFlags(t *testing.T) {
 	if rf == nil {
 		t.Error("expected --region flag")
 	}
+	vf := cmd.PersistentFlags().Lookup("verbose")
+	if vf == nil {
+		t.Error("expected --verbose flag")
+	}
+	if vf != nil && vf.Shorthand != "v" {
+		t.Errorf("expected --verbose shorthand 'v', got '%s'", vf.Shorthand)
+	}
 }

internal/config/config.go

@@ -6,6 +6,8 @@ import (
 	"path/filepath"
 
 	"gopkg.in/yaml.v3"
+
+	uniclog "unic/internal/log"
 )
 
 const (
@@ -139,6 +141,14 @@ func Load(cliProfile, cliRegion *string, configPath string) (*Config, error) {
 		region = *cliRegion
 	}
 
+	uniclog.Debug("config", "config resolved",
+		"path", configPath,
+		"profile", profile,
+		"region", region,
+		"context", contextName,
+		"auth_type", string(authType),
+	)
+
 	return &Config{
 		Profile:      profile,
 		Region:       region,