pre-commit: add more rules

.github/workflows/go.yml

@@ -23,13 +23,11 @@ jobs:
       - name: Install libolm
         run: sudo apt-get install libolm-dev libolm3
 
-      - name: Install goimports
+      - name: Install dependencies
         run: |
           go install golang.org/x/tools/cmd/goimports@latest
+          go install honnef.co/go/tools/cmd/staticcheck@latest
           export PATH="$HOME/go/bin:$PATH"
 
-      - name: Install pre-commit
-        run: pip install pre-commit
-
-      - name: Lint
-        run: pre-commit run -a
+      - name: Run pre-commit
+        uses: pre-commit/[email protected]

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v4.6.0
     hooks:
       - id: trailing-whitespace
         exclude_types: [markdown]
@@ -12,4 +12,16 @@ repos:
     rev: v1.0.0-rc.1
     hooks:
       - id: go-imports-repo
+        args:
+          - "-local"
+          - "go.mau.fi/mautrix-signal"
+          - "-w"
       - id: go-vet-repo-mod
+      - id: go-staticcheck-repo-mod
+      - id: go-mod-tidy
+
+  - repo: https://github.com/beeper/pre-commit-go
+    rev: v0.3.1
+    hooks:
+      - id: zerolog-ban-msgf
+      - id: zerolog-use-stringer

pkg/connector/handleslack.go

@@ -84,7 +84,7 @@ func (s *SlackClient) HandleSlackEvent(rawEvt any) {
 			s.UserLogin.Bridge.QueueRemoteEvent(s.UserLogin, wrapped)
 		}
 	case *slack.EmojiChangedEvent:
-		//go ut.handleEmojiChange(ctx, evt)
+		go s.handleEmojiChange(ctx, evt)
 	case *slack.FileSharedEvent, *slack.FilePublicEvent, *slack.FilePrivateEvent,
 		*slack.FileCreatedEvent, *slack.FileChangeEvent, *slack.FileDeletedEvent,
 		*slack.DesktopNotificationEvent, *slack.ReconnectUrlEvent, *slack.LatencyReport:
@@ -452,6 +452,12 @@ func (s *SlackMessage) GetTargetMessage() networkid.MessageID {
 }
 
 func (s *SlackMessage) GetChatInfoChange(ctx context.Context) (*bridgev2.ChatInfoChange, error) {
-	// TODO implement
-	return &bridgev2.ChatInfoChange{}, nil
+	fullChatInfo, err := s.Client.fetchChatInfo(ctx, s.Data.Channel, false)
+	if err != nil {
+		return nil, fmt.Errorf("failed to get chat info: %w", err)
+	}
+	// TODO implement deltas instead of full resync
+	return &bridgev2.ChatInfoChange{
+		ChatInfo: fullChatInfo,
+	}, nil
 }

pkg/msgconv/msgconv.go

@@ -59,9 +59,9 @@ func (mc *MessageConverter) GetMentionedUserInfo(ctx context.Context, userID str
 	if err != nil {
 		zerolog.Ctx(ctx).Err(err).Msg("Failed to get mentioned ghost")
 	} else if ghost != nil {
-		if ghost.Name == "" {
+		/*if ghost.Name == "" {
 			// TODO update ghost info
-		}
+		}*/
 		name = ghost.Name
 		mxid = ghost.Intent.GetMXID()
 	}