Merge pull request #1 from YumaFuu/feature/golangci-lint

add: golangci-lint

.github/workflows/golangci.yaml

@@ -0,0 +1,36 @@
+name: golangci-lint
+
+permissions:
+  # Required: allow read access to the content for analysis.
+  contents: read
+  # Optional: allow read access to pull request. Use with `only-new-issues` option.
+  pull-requests: read
+  # Optional: allow write access to checks to allow the action to annotate code in the PR.
+  checks: write
+
+on:
+  pull_request:
+    branches:
+      - main
+  push:
+    branches:
+      - main
+
+jobs:
+  golangci-lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - uses: actions/setup-go@v5
+        with:
+          go-version-file: go.mod
+          cache: true
+
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.59
+

README.md

@@ -7,6 +7,7 @@ S1M is AWS Parameter Store TUI tool.
 
 It allows you to manage your parameters in AWS Parameter Store.
 
+[![GitHub Lint](https://img.shields.io/github/actions/workflow/status/YumaFuu/s1m/golangci.yml)]()
 [![GitHub release](https://img.shields.io/github/v/release/yumafuu/s1m)]()
 
 </div>

aqua.yaml

@@ -11,3 +11,4 @@ registries:
   ref: v4.208.0 # renovate: depName=aquaproj/aqua-registry
 packages:
 - name: goreleaser/[email protected]
+- name: golangci/[email protected]

tui/cmd/cmd.go

@@ -68,7 +68,6 @@ func (v *CmdBox) NewParameterType(dir string, param ssm.Parameter) {
 			v.NewParameterValue(dir, param)
 		}
 	})
-	return
 }
 
 // TODO: Dependency Injection

tui/func.go

@@ -83,7 +83,9 @@ func (a *Tui) WaitTopic() {
 				func() {
 					a.cmdbox.SetLabel("")
 					a.cmdbox.SetText("")
-					a.ptree.Refresh()
+					if err := a.ptree.Refresh(); err != nil {
+						a.infbox.SetText(err.Error())
+					}
 					a.app.SetFocus(a.ptree)
 				},
 			)
@@ -147,7 +149,9 @@ func (a *Tui) WaitTopic() {
 				func() {
 					a.cmdbox.SetLabel("")
 					a.cmdbox.SetText("")
-					a.ptree.Refresh()
+					if err := a.ptree.Refresh(); err != nil {
+						a.infbox.SetText(err.Error())
+					}
 					a.app.SetFocus(a.ptree)
 				},
 			)
@@ -174,7 +178,9 @@ func (a *Tui) WaitTopic() {
 				func() {
 					a.cmdbox.SetLabel("")
 					a.cmdbox.SetText("")
-					a.ptree.Refresh()
+					if err := a.ptree.Refresh(); err != nil {
+						a.infbox.SetText(err.Error())
+					}
 					a.app.SetFocus(a.ptree)
 				},
 			)

tui/infbox/infbox.go

@@ -44,11 +44,9 @@ func (v InfoBox) WaitTopic() {
 	chUpdate := v.pubsub.Sub(pubsub.TopicUpdateInfoBox)
 
 	for {
-		select {
-		case msg := <-chUpdate:
-			if s, ok := msg.(string); ok {
-				v.SetText(s)
-			}
+		msg := <-chUpdate
+		if s, ok := msg.(string); ok {
+			v.SetText(s)
 		}
 	}
 }

tui/ptree/capture.go

@@ -78,7 +78,9 @@ func (pt *ParameterTree) InputCapture(event *tcell.EventKey) *tcell.EventKey {
 			pt.pubsub.Pub(param, pubsub.TopicDeleteParam)
 		}
 	case 'r':
-		pt.Refresh()
+		if err := pt.Refresh(); err != nil {
+			pt.pubsub.Pub(err, pubsub.TopicUpdateInfoBox)
+		}
 	}
 
 	switch event.Key() {

tui/ptree/ptree.go

@@ -41,7 +41,9 @@ func NewParameterTree(
 		root,
 		INIT_POSITION,
 	}
-	pt.Refresh()
+	if err := pt.Refresh(); err != nil {
+		return nil, err
+	}
 
 	return pt, nil
 }