style: fix formatting issues detected by golangci-lint

- Remove trailing whitespace
- Fix spacing around comments and assignments
- Add final newlines to files

Made-with: Cursor

Author: lukegalbraithrussell

cmd/docs/docs.go

@@ -77,7 +77,7 @@ type DocsOutput struct {
 // ProgrammaticSearchOutput represents the output from local docs search
 type ProgrammaticSearchOutput = search.SearchResponse
 
-// findDocsRepo tries to locate the docs repository  
+// findDocsRepo tries to locate the docs repository
 func findDocsRepo() string {
 	return search.FindDocsRepo()
 }
@@ -109,12 +109,12 @@ func runDocsCommand(clients *shared.ClientFactory, cmd *cobra.Command, args []st
 	if cmd.Flags().Changed("search") {
 		if len(args) > 0 {
 			query = strings.Join(args, " ")
-			
+
 			// Check output format
 			if outputFormat == "json" {
 				return runProgrammaticSearchCommand(clients, ctx, query)
 			}
-			
+
 			// Default browser search
 			encodedQuery := url.QueryEscape(query)
 			docsURL = fmt.Sprintf("https://docs.slack.dev/search/?q=%s", encodedQuery)
@@ -140,22 +140,22 @@ func runDocsCommand(clients *shared.ClientFactory, cmd *cobra.Command, args []st
 			Query: query,
 			Type:  docType,
 		}
-		
+
 		jsonBytes, err := json.MarshalIndent(output, "", "  ")
 		if err != nil {
 			return slackerror.New(slackerror.ErrDocsJSONEncodeFailed)
 		}
-		
+
 		fmt.Println(string(jsonBytes))
-		
+
 		// Still print trace for analytics
 		if cmd.Flags().Changed("search") {
 			traceValue := query
 			clients.IO.PrintTrace(ctx, slacktrace.DocsSearchSuccess, traceValue)
 		} else {
 			clients.IO.PrintTrace(ctx, slacktrace.DocsSuccess)
 		}
-		
+
 		return nil
 	}
 
@@ -210,4 +210,4 @@ func runProgrammaticSearchCommand(clients *shared.ClientFactory, ctx context.Con
 	}
 	fmt.Println(string(jsonBytes))
 	return nil
-}
+}

internal/search/search.go

@@ -104,7 +104,7 @@ func stripHTML(text string) string {
 	// Remove HTML tags
 	re := regexp.MustCompile(`<[^>]*>`)
 	text = re.ReplaceAllString(text, "")
-	
+
 	// Replace HTML entities
 	replacements := map[string]string{
 		"&nbsp;": " ",
@@ -114,27 +114,27 @@ func stripHTML(text string) string {
 		"&quot;": "\"",
 		"&#39;":  "'",
 	}
-	
+
 	for entity, replacement := range replacements {
 		text = strings.ReplaceAll(text, entity, replacement)
 	}
-	
+
 	// Remove markdown links [text](url)
 	re = regexp.MustCompile(`\[([^\]]+)\]\([^)]+\)`)
 	text = re.ReplaceAllString(text, "$1")
-	
+
 	// Remove inline code `code`
 	re = regexp.MustCompile("`([^`]+)`")
 	text = re.ReplaceAllString(text, "$1")
-	
+
 	// Remove bold/italic *text* and **text**
 	re = regexp.MustCompile(`\*{1,2}([^*]+)\*{1,2}`)
 	text = re.ReplaceAllString(text, "$1")
-	
+
 	// Normalize whitespace
 	re = regexp.MustCompile(`\s+`)
 	text = re.ReplaceAllString(text, " ")
-	
+
 	return strings.TrimSpace(text)
 }
 
@@ -143,7 +143,7 @@ func extractSnippet(content, query string, maxLength int) string {
 	cleanContent := stripHTML(content)
 	queryLower := strings.ToLower(query)
 	contentLower := strings.ToLower(cleanContent)
-	
+
 	queryIndex := strings.Index(contentLower, queryLower)
 	if queryIndex == -1 {
 		// No match, return beginning
@@ -152,27 +152,27 @@ func extractSnippet(content, query string, maxLength int) string {
 		}
 		return cleanContent
 	}
-	
+
 	// Extract context around the match
 	start := queryIndex - 100
 	if start < 0 {
 		start = 0
 	}
-	
+
 	end := queryIndex + len(query) + 150
 	if end > len(cleanContent) {
 		end = len(cleanContent)
 	}
-	
+
 	snippet := cleanContent[start:end]
-	
+
 	if start > 0 {
 		snippet = "..." + snippet
 	}
 	if end < len(cleanContent) {
 		snippet = snippet + "..."
 	}
-	
+
 	return strings.TrimSpace(snippet)
 }
 
@@ -181,27 +181,27 @@ func calculateRelevance(content, title, query string) int {
 	queryLower := strings.ToLower(query)
 	titleLower := strings.ToLower(title)
 	contentLower := strings.ToLower(content)
-	
+
 	score := 0
-	
+
 	// Title matches are highly relevant
 	if strings.Contains(titleLower, queryLower) {
 		score += 100
 		if titleLower == queryLower {
 			score += 50 // Exact title match
 		}
 	}
-	
+
 	// Count occurrences in content
 	matches := strings.Count(contentLower, queryLower)
 	score += matches * 10
-	
+
 	// Boost for early occurrence
 	firstIndex := strings.Index(contentLower, queryLower)
 	if firstIndex != -1 && firstIndex < 500 {
 		score += 20
 	}
-	
+
 	return score
 }
 
@@ -211,17 +211,17 @@ func filePathToURL(filePath, contentDir string) string {
 	if err != nil {
 		return "/"
 	}
-	
+
 	// Remove .md extension
 	relPath = strings.TrimSuffix(relPath, ".md")
-	
+
 	// Handle index files
 	if strings.HasSuffix(relPath, "/index") {
 		relPath = strings.TrimSuffix(relPath, "/index")
 	} else if relPath == "index" {
 		return "/"
 	}
-	
+
 	// Convert to URL path
 	urlPath := "/" + strings.ReplaceAll(relPath, "\\", "/")
 	return urlPath
@@ -233,7 +233,7 @@ func determineType(filePath string) string {
 		return "reference"
 	}
 	if strings.Contains(filePath, "/changelog/") {
-		return "changelog"  
+		return "changelog"
 	}
 	if strings.Contains(filePath, "/tools/") {
 		return "tools"
@@ -249,9 +249,9 @@ func matchesFilter(filePath, filter string) bool {
 	if filter == "" || filter == "all" {
 		return true
 	}
-	
+
 	contentType := determineType(filePath)
-	
+
 	switch filter {
 	case "reference":
 		return contentType == "reference"
@@ -271,19 +271,19 @@ func matchesFilter(filePath, filter string) bool {
 // findMarkdownFiles recursively finds all .md files in a directory
 func findMarkdownFiles(dir string) ([]string, error) {
 	var files []string
-	
+
 	err := filepath.WalkDir(dir, func(path string, d fs.DirEntry, err error) error {
 		if err != nil {
 			return nil // Skip files we can't access
 		}
-		
+
 		if !d.IsDir() && strings.HasSuffix(d.Name(), ".md") {
 			files = append(files, path)
 		}
-		
+
 		return nil
 	})
-	
+
 	return files, err
 }
 
@@ -292,7 +292,7 @@ func SearchDocs(query, filter string, limit int, contentDir string) (*SearchResp
 	if contentDir == "" {
 		return nil, fmt.Errorf("content directory not specified")
 	}
-	
+
 	// Check if content directory exists
 	if _, err := os.Stat(contentDir); os.IsNotExist(err) {
 		return &SearchResponse{
@@ -303,44 +303,44 @@ func SearchDocs(query, filter string, limit int, contentDir string) (*SearchResp
 			Showing: 0,
 		}, fmt.Errorf("content directory not found: %s", contentDir)
 	}
-	
+
 	// Find all markdown files
 	markdownFiles, err := findMarkdownFiles(contentDir)
 	if err != nil {
 		return nil, fmt.Errorf("failed to find markdown files: %w", err)
 	}
-	
+
 	var results []SearchResult
 	queryLower := strings.ToLower(query)
-	
+
 	// Search through files
 	for _, filePath := range markdownFiles {
 		// Apply filter
 		if !matchesFilter(filePath, filter) {
 			continue
 		}
-		
+
 		// Read file
 		content, err := os.ReadFile(filePath)
 		if err != nil {
 			continue // Skip files we can't read
 		}
-		
+
 		contentStr := string(content)
-		
+
 		// Parse frontmatter
 		frontmatter, bodyContent := parseFrontMatter(contentStr)
-		
+
 		// Skip unlisted pages
 		if frontmatter.Unlisted {
 			continue
 		}
-		
+
 		// Check if query matches (case insensitive)
 		if !strings.Contains(strings.ToLower(contentStr), queryLower) {
 			continue
 		}
-		
+
 		// Extract metadata
 		title := frontmatter.Title
 		if title == "" {
@@ -349,68 +349,68 @@ func SearchDocs(query, filter string, limit int, contentDir string) (*SearchResp
 		if title == "" {
 			title = "Untitled"
 		}
-		
+
 		url := SiteURL + filePathToURL(filePath, contentDir)
 		contentType := determineType(filePath)
 		snippet := extractSnippet(bodyContent, query, 250)
 		score := calculateRelevance(contentStr, title, query)
-		
+
 		result := SearchResult{
 			Title:   title,
 			URL:     url,
 			Snippet: snippet,
 			Type:    contentType,
 			Score:   score,
 		}
-		
+
 		results = append(results, result)
 	}
-	
+
 	// Sort by relevance score (highest first)
 	sort.Slice(results, func(i, j int) bool {
 		return results[i].Score > results[j].Score
 	})
-	
+
 	// Limit results
 	total := len(results)
 	if limit > 0 && limit < len(results) {
 		results = results[:limit]
 	}
-	
+
 	if filter == "" {
 		filter = "all"
 	}
-	
+
 	response := &SearchResponse{
 		Query:   query,
 		Filter:  filter,
 		Results: results,
 		Total:   total,
 		Showing: len(results),
 	}
-	
+
 	return response, nil
 }
 
 // FindDocsRepo attempts to locate the docs repository
 func FindDocsRepo() string {
 	candidates := []string{
 		"../docs",
-		"../../docs", 
+		"../../docs",
 		"./docs",
 	}
-	
+
 	for _, candidate := range candidates {
 		absPath, err := filepath.Abs(candidate)
 		if err != nil {
 			continue
 		}
-		
+
 		contentDir := filepath.Join(absPath, "content")
 		if _, err := os.Stat(contentDir); err == nil {
 			return absPath
 		}
 	}
-	
+
 	return ""
-}
+}