fix: Lint errors for chatmail integration

This commit is contained in:
Ivan Agosto 2026-03-31 21:36:04 -06:00
parent 9f59c2e0ef
commit 4b00f3d904
2 changed files with 14 additions and 14 deletions

View file

@ -54,7 +54,7 @@ func (c *ChatmailChannel) Start(ctx context.Context) error {
accountPath = filepath.Join(homeDir, ".accounts", "chatmail") accountPath = filepath.Join(homeDir, ".accounts", "chatmail")
} }
if err := os.MkdirAll(accountPath, 0700); err != nil { if err := os.MkdirAll(accountPath, 0o700); err != nil {
return fmt.Errorf("failed to create account directory: %w", err) return fmt.Errorf("failed to create account directory: %w", err)
} }
@ -96,9 +96,9 @@ func (c *ChatmailChannel) Start(ctx context.Context) error {
if !isConfigured { if !isConfigured {
botFlag := "1" botFlag := "1"
if err := c.rpc.SetConfig(accId, "bot", &botFlag); err != nil { if cfgErr := c.rpc.SetConfig(accId, "bot", &botFlag); cfgErr != nil {
transport.Close() transport.Close()
return fmt.Errorf("failed to set bot flag: %w", err) return fmt.Errorf("failed to set bot flag: %w", cfgErr)
} }
logger.InfoC("chatmail", "Account configured as bot") logger.InfoC("chatmail", "Account configured as bot")
@ -107,9 +107,9 @@ func (c *ChatmailChannel) Start(ctx context.Context) error {
inviteQR = "dcaccount:https://nine.testrun.org/new" inviteQR = "dcaccount:https://nine.testrun.org/new"
} }
if err := c.rpc.AddTransportFromQr(accId, inviteQR); err != nil { if qrErr := c.rpc.AddTransportFromQr(accId, inviteQR); qrErr != nil {
transport.Close() transport.Close()
return fmt.Errorf("failed to add transport from QR: %w", err) return fmt.Errorf("failed to add transport from QR: %w", qrErr)
} }
logger.InfoCF("chatmail", "Account configured from invite", map[string]any{"qr": inviteQR}) logger.InfoCF("chatmail", "Account configured from invite", map[string]any{"qr": inviteQR})
} }
@ -232,13 +232,13 @@ func (c *ChatmailChannel) ReactToMessage(ctx context.Context, chatID, messageID
msgId, err := c.parseMsgID(messageID) msgId, err := c.parseMsgID(messageID)
if err != nil { if err != nil {
return func() {}, nil return func() {}, err
} }
reactions := []string{"\U0001F440"} reactions := []string{"\U0001F440"}
if _, err := c.rpc.SendReaction(c.accId, msgId, reactions); err != nil { if _, reactErr := c.rpc.SendReaction(c.accId, msgId, reactions); reactErr != nil {
logger.DebugCF("chatmail", "Failed to add reaction", map[string]any{"error": err.Error()}) logger.DebugCF("chatmail", "Failed to add reaction", map[string]any{"error": reactErr.Error()})
return func() {}, nil return func() {}, reactErr
} }
// Keep the reaction permanently - return no-op undo function // Keep the reaction permanently - return no-op undo function

View file

@ -630,12 +630,12 @@ type IRCConfig struct {
} }
type ChatmailConfig struct { type ChatmailConfig struct {
Enabled bool `json:"enabled" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_ENABLED"` Enabled bool `json:"enabled" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_ENABLED"`
AccountPath string `json:"account_path" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_ACCOUNT_PATH"` AccountPath string `json:"account_path" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_ACCOUNT_PATH"`
AllowFrom FlexibleStringSlice `json:"allow_from" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_ALLOW_FROM"` AllowFrom FlexibleStringSlice `json:"allow_from" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_ALLOW_FROM"`
GroupTrigger GroupTriggerConfig `json:"group_trigger,omitempty" yaml:"-"` GroupTrigger GroupTriggerConfig `json:"group_trigger,omitempty" yaml:"-"`
ReasoningChannelID string `json:"reasoning_channel_id" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_REASONING_CHANNEL_ID"` ReasoningChannelID string `json:"reasoning_channel_id" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_REASONING_CHANNEL_ID"`
InviteQR string `json:"invite_qr" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_INVITE_QR"` InviteQR string `json:"invite_qr" yaml:"-" env:"PICOCLAW_CHANNELS_CHATMAIL_INVITE_QR"`
} }
type HeartbeatConfig struct { type HeartbeatConfig struct {