RESP-4311 Mostly update forked slack go library

.github/workflows/stale.yaml

@@ -0,0 +1,25 @@
+name: 'Close stale issues and PRs'
+on:
+  schedule:
+    - cron: '0 0 * * 1' # every Monday 0:00 UTC
+
+  workflow_dispatch:
+
+permissions:
+  issues: write
+  pull-requests: write
+
+jobs:
+  stale:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/stale@28ca1036281a5e5922ead5184a1bbf96e5fc984e # v9.0.0
+        with:
+          any-of-labels: 'feedback given'
+          days-before-stale: 45
+          days-before-pr-close: 10
+          stale-issue-message: 'This issue is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.'
+          stale-pr-message: 'This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.'
+          close-issue-message: 'This issue was closed because it has been stalled for 10 days with no activity.'
+          close-pr-message: 'This PR was closed because it has been stalled for 10 days with no activity.'
+          operations-per-run: 120

.github/workflows/test.yml

@@ -12,13 +12,16 @@ jobs:
     strategy:
       matrix:
         go:
-          - '1.17'
           - '1.18'
           - '1.19'
+          - '1.20'
+          - '1.21'
+          - '1.22'
+          - '1.23'
     name: test go-${{ matrix.go }}
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-go@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
         with:
           go-version: ${{ matrix.go }}
       - name: run test
@@ -29,13 +32,17 @@ jobs:
     runs-on: ubuntu-22.04
     name: lint
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/setup-go@v5
+        with:
+          go-version: '1.20'
+          cache: false
+      - uses: actions/checkout@v4
         with:
           # NOTE: Because we are a fork,
           # we must fetch all history for all branches
           # and tags.
-          fetch-depth: '0'
+          fetch-depth: 0
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@537aa1903e5d359d0b27dbc19ddd22c5087f3fbc # v3.2.0
+        uses: golangci/golangci-lint-action@aaa42aa0628b4ae2578232a66b541047968fac86 # v6.1.0
         with:
-          version: v1.49.0
+          version: v1.52.2

.gitignore

@@ -1,3 +1,4 @@
 *.test
 *~
 .idea/
+/vendor/

apps.go

@@ -19,11 +19,15 @@ type EventAuthorization struct {
 	IsEnterpriseInstall bool   `json:"is_enterprise_install"`
 }
 
+// ListEventAuthorizations lists authed users and teams for the given event_context.
+// You must provide an app-level token to the client using OptionAppLevelToken.
+// For more details, see ListEventAuthorizationsContext documentation.
 func (api *Client) ListEventAuthorizations(eventContext string) ([]EventAuthorization, error) {
 	return api.ListEventAuthorizationsContext(context.Background(), eventContext)
 }
 
-// ListEventAuthorizationsContext lists authed users and teams for the given event_context. You must provide an app-level token to the client using OptionAppLevelToken. More info: https://api.slack.com/methods/apps.event.authorizations.list
+// ListEventAuthorizationsContext lists authed users and teams for the given event_context with a custom context.
+// Slack API docs: https://api.slack.com/methods/apps.event.authorizations.list
 func (api *Client) ListEventAuthorizationsContext(ctx context.Context, eventContext string) ([]EventAuthorization, error) {
 	resp := &listEventAuthorizationsResponse{}
 
@@ -43,10 +47,14 @@ func (api *Client) ListEventAuthorizationsContext(ctx context.Context, eventCont
 	return resp.Authorizations, nil
 }
 
+// UninstallApp uninstalls your app from a workspace.
+// For more details, see UninstallAppContext documentation.
 func (api *Client) UninstallApp(clientID, clientSecret string) error {
 	return api.UninstallAppContext(context.Background(), clientID, clientSecret)
 }
 
+// UninstallAppContext uninstalls your app from a workspace with a custom context.
+// Slack API docs: https://api.slack.com/methods/apps.uninstall
 func (api *Client) UninstallAppContext(ctx context.Context, clientID, clientSecret string) error {
 	values := url.Values{
 		"client_id":     {clientID},

auth.go

@@ -23,12 +23,14 @@ func (api *Client) authRequest(ctx context.Context, path string, values url.Valu
 	return response, response.Err()
 }
 
-// SendAuthRevoke will send a revocation for our token
+// SendAuthRevoke will send a revocation for our token.
+// For more details, see SendAuthRevokeContext documentation.
 func (api *Client) SendAuthRevoke(token string) (*AuthRevokeResponse, error) {
 	return api.SendAuthRevokeContext(context.Background(), token)
 }
 
-// SendAuthRevokeContext will send a revocation request for our token to api.revoke with context
+// SendAuthRevokeContext will send a revocation request for our token to api.revoke with a custom context.
+// Slack API docs: https://api.slack.com/methods/auth.revoke
 func (api *Client) SendAuthRevokeContext(ctx context.Context, token string) (*AuthRevokeResponse, error) {
 	if token == "" {
 		token = api.token
@@ -52,12 +54,13 @@ type ListTeamsParameters struct {
 }
 
 // ListTeams returns all workspaces a token can access.
-// More info: https://api.slack.com/methods/admin.teams.list
+// For more details, see ListTeamsContext documentation.
 func (api *Client) ListTeams(params ListTeamsParameters) ([]Team, string, error) {
 	return api.ListTeamsContext(context.Background(), params)
 }
 
-// ListTeams returns all workspaces a token can access with a custom context.
+// ListTeamsContext returns all workspaces a token can access with a custom context.
+// Slack API docs: https://api.slack.com/methods/auth.teams.list
 func (api *Client) ListTeamsContext(ctx context.Context, params ListTeamsParameters) ([]Team, string, error) {
 	values := url.Values{
 		"token": {api.token},

block.go

@@ -19,6 +19,7 @@ const (
 	MBTHeader   MessageBlockType = "header"
 	MBTRichText MessageBlockType = "rich_text"
 	MBTCall     MessageBlockType = "call"
+	MBTVideo    MessageBlockType = "video"
 )
 
 // Block defines an interface all block types should implement
@@ -41,6 +42,7 @@ type BlockAction struct {
 	Text                  TextBlockObject     `json:"text"`
 	Value                 string              `json:"value"`
 	RichTextValue         RichTextBlock       `json:"rich_text_value"`
+	Files                 []File              `json:"files"`
 	ActionTs              string              `json:"action_ts"`
 	SelectedOption        OptionBlockObject   `json:"selected_option"`
 	SelectedOptions       []OptionBlockObject `json:"selected_options"`

block_call.go

@@ -1,53 +1,24 @@
 package slack
 
-// CallBlock defines information
+// CallBlock defines data that is used to display a call in slack.
+//
+// More Information: https://api.slack.com/apis/calls#post_to_channel
 type CallBlock struct {
-	Type                   MessageBlockType `json:"type"`
-	CallID                 string           `json:"call_id"`
-	BlockID                string           `json:"block_id"`
-	APIDecorationAvailable bool             `json:"api_decoration_available"`
-	Call                   Call             `json:"call"`
-}
-
-type AppIconUrls struct {
-	Image32       string `json:"image_32"`
-	Image36       string `json:"image_36"`
-	Image48       string `json:"image_48"`
-	Image64       string `json:"image_64"`
-	Image72       string `json:"image_72"`
-	Image96       string `json:"image_96"`
-	Image128      string `json:"image_128"`
-	Image192      string `json:"image_192"`
-	Image512      string `json:"image_512"`
-	Image1024     string `json:"image_1024"`
-	ImageOriginal string `json:"image_original"`
-}
-type CallInfo struct {
-	ID                 string        `json:"id"`
-	AppID              string        `json:"app_id"`
-	AppIconUrls        AppIconUrls   `json:"app_icon_urls"`
-	DateStart          int           `json:"date_start"`
-	ActiveParticipants []interface{} `json:"active_participants"`
-	AllParticipants    []interface{} `json:"all_participants"`
-	DisplayID          string        `json:"display_id"`
-	JoinURL            string        `json:"join_url"`
-	DesktopAppJoinURL  string        `json:"desktop_app_join_url"`
-	Name               string        `json:"name"`
-	CreatedBy          string        `json:"created_by"`
-	DateEnd            int           `json:"date_end"`
-	Channels           []string      `json:"channels"`
-	IsDmCall           bool          `json:"is_dm_call"`
-	WasRejected        bool          `json:"was_rejected"`
-	WasMissed          bool          `json:"was_missed"`
-	WasAccepted        bool          `json:"was_accepted"`
-	HasEnded           bool          `json:"has_ended"`
-}
-type Call struct {
-	CallInfo         CallInfo `json:"v1"`
-	MediaBackendType string   `json:"media_backend_type"`
+	Type    MessageBlockType `json:"type"`
+	BlockID string           `json:"block_id,omitempty"`
+	CallID  string           `json:"call_id"`
+	Call    Call             `json:"call"`
 }
 
 // BlockType returns the type of the block
 func (s CallBlock) BlockType() MessageBlockType {
 	return s.Type
 }
+
+// NewCallBlock returns a new instance of a file block
+func NewCallBlock(callID string) *CallBlock {
+	return &CallBlock{
+		Type:   MBTCall,
+		CallID: callID,
+	}
+}

block_call_test.go

@@ -0,0 +1,13 @@
+package slack
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestNewCallBlock(t *testing.T) {
+	callBlock := NewCallBlock("ACallID")
+	assert.Equal(t, string(callBlock.Type), "call")
+	assert.Equal(t, callBlock.CallID, "ACallID")
+}

block_conv.go

@@ -2,7 +2,6 @@ package slack
 
 import (
 	"encoding/json"
-	"errors"
 	"fmt"
 )
 
@@ -72,6 +71,8 @@ func (b *Blocks) UnmarshalJSON(data []byte) error {
 			block = &SectionBlock{}
 		case "call":
 			block = &CallBlock{}
+		case "video":
+			block = &VideoBlock{}
 		default:
 			block = &UnknownBlock{}
 		}
@@ -136,8 +137,10 @@ func (b *InputBlock) UnmarshalJSON(data []byte) error {
 		e = &RadioButtonsBlockElement{}
 	case "number_input":
 		e = &NumberInputBlockElement{}
+	case "file_input":
+		e = &FileInputBlockElement{}
 	default:
-		return errors.New("unsupported block element type")
+		return fmt.Errorf("unsupported block element type %v", s.TypeVal)
 	}
 
 	if err := json.Unmarshal(a.Element, e); err != nil {
@@ -443,7 +446,7 @@ func (e *ContextElements) UnmarshalJSON(data []byte) error {
 
 			e.Elements = append(e.Elements, elem.(*ImageBlockElement))
 		default:
-			return errors.New("unsupported context element type")
+			return fmt.Errorf("unsupported context element type %v", contextElementType)
 		}
 	}