feat: add inline @mention support to comment create

Author: nnemirovsky

cmd/comment/create.go

@@ -5,6 +5,7 @@ import (
 	"io"
 	"os"
 	"path/filepath"
+	"regexp"
 	"strconv"
 	"strings"
 
@@ -15,9 +16,12 @@ import (
 	"github.com/needmore/bc4/internal/markdown"
 	"github.com/needmore/bc4/internal/parser"
 	"github.com/needmore/bc4/internal/ui"
+	"github.com/needmore/bc4/internal/utils"
 	"github.com/spf13/cobra"
 )
 
+var mentionRe = regexp.MustCompile(`@[\w]+(?:\.[\w]+)*`)
+
 func newCreateCmd(f *factory.Factory) *cobra.Command {
 	var content string
 	var attachmentPath string
@@ -121,6 +125,26 @@ You can provide comment content in several ways:
 				richContent = rc
 			}
 
+			// Replace inline @Name mentions with bc-attachment tags
+			// Supports @FirstName and @First.Last for disambiguation
+			inlineMatches := mentionRe.FindAllString(richContent, -1)
+			if len(inlineMatches) > 0 {
+				resolver := utils.NewUserResolver(client.Client, projectID)
+				// Convert @First.Last to "First Last" for resolution
+				identifiers := make([]string, len(inlineMatches))
+				for i, m := range inlineMatches {
+					identifiers[i] = strings.ReplaceAll(strings.TrimPrefix(m, "@"), ".", " ")
+				}
+				people, err := resolver.ResolvePeople(f.Context(), identifiers)
+				if err != nil {
+					return fmt.Errorf("failed to resolve mentions: %w", err)
+				}
+				for i, match := range inlineMatches {
+					tag := attachments.BuildTag(people[i].AttachableSGID)
+					richContent = strings.Replace(richContent, match, tag, 1)
+				}
+			}
+
 			// Attach file if provided
 			if attachmentPath != "" {
 				fileData, err := os.ReadFile(attachmentPath)

internal/api/client.go

@@ -347,15 +347,16 @@ type Company struct {
 
 // Person represents a Basecamp user
 type Person struct {
-	ID           int64    `json:"id"`
-	Name         string   `json:"name"`
-	EmailAddress string   `json:"email_address"`
-	Title        string   `json:"title"`
-	AvatarURL    string   `json:"avatar_url"`
-	Company      *Company `json:"company,omitempty"`
-	CreatedAt    string   `json:"created_at,omitempty"`
-	Admin        bool     `json:"admin,omitempty"`
-	Owner        bool     `json:"owner,omitempty"`
+	ID             int64    `json:"id"`
+	AttachableSGID string   `json:"attachable_sgid"`
+	Name           string   `json:"name"`
+	EmailAddress   string   `json:"email_address"`
+	Title          string   `json:"title"`
+	AvatarURL      string   `json:"avatar_url"`
+	Company        *Company `json:"company,omitempty"`
+	CreatedAt      string   `json:"created_at,omitempty"`
+	Admin          bool     `json:"admin,omitempty"`
+	Owner          bool     `json:"owner,omitempty"`
 }
 
 // Todo represents a Basecamp todo item

internal/utils/users.go

@@ -77,6 +77,42 @@ func (ur *UserResolver) GetPeople(ctx context.Context) ([]api.Person, error) {
 	return ur.people, nil
 }
 
+// ResolvePeople resolves a list of user identifiers to full Person objects
+func (ur *UserResolver) ResolvePeople(ctx context.Context, identifiers []string) ([]api.Person, error) {
+	if err := ur.ensurePeopleCached(ctx); err != nil {
+		return nil, err
+	}
+
+	var people []api.Person
+	var notFound []string
+
+	for _, identifier := range identifiers {
+		identifier = strings.TrimSpace(identifier)
+		if identifier == "" {
+			continue
+		}
+
+		personID, found := ur.resolveIdentifier(identifier)
+		if !found {
+			notFound = append(notFound, identifier)
+			continue
+		}
+
+		for _, p := range ur.people {
+			if p.ID == personID {
+				people = append(people, p)
+				break
+			}
+		}
+	}
+
+	if len(notFound) > 0 {
+		return nil, fmt.Errorf("could not find users: %s", strings.Join(notFound, ", "))
+	}
+
+	return people, nil
+}
+
 // ensurePeopleCached loads the project people if not already cached
 func (ur *UserResolver) ensurePeopleCached(ctx context.Context) error {
 	if ur.cached {