Add ticket code support for cards with auto-incrementing numbers

server/api/cards.go

@@ -28,6 +28,7 @@ func (a *API) registerCardsRoutes(r *mux.Router) {
 	r.HandleFunc("/boards/{boardID}/cards", a.sessionRequired(a.handleGetCards)).Methods("GET")
 	r.HandleFunc("/cards/{cardID}", a.sessionRequired(a.handlePatchCard)).Methods("PATCH")
 	r.HandleFunc("/cards/{cardID}", a.sessionRequired(a.handleGetCard)).Methods("GET")
+	r.HandleFunc("/teams/{teamID}/cards/by-ticket-code/{ticketCode}", a.sessionRequired(a.handleGetCardByTicketCode)).Methods("GET")
 }
 
 func (a *API) handleCreateCard(w http.ResponseWriter, r *http.Request) {
@@ -390,3 +391,74 @@ func (a *API) handleGetCard(w http.ResponseWriter, r *http.Request) {
 
 	auditRec.Success()
 }
+
+func (a *API) handleGetCardByTicketCode(w http.ResponseWriter, r *http.Request) {
+	// swagger:operation GET /teams/{teamID}/cards/by-ticket-code/{ticketCode} getCardByTicketCode
+	//
+	// Fetches a card by its ticket code (e.g. PROJ-42).
+	//
+	// ---
+	// produces:
+	// - application/json
+	// parameters:
+	// - name: teamID
+	//   in: path
+	//   description: Team ID
+	//   required: true
+	//   type: string
+	// - name: ticketCode
+	//   in: path
+	//   description: Ticket code (e.g. PROJ-42)
+	//   required: true
+	//   type: string
+	// security:
+	// - BearerAuth: []
+	// responses:
+	//   '200':
+	//     description: success
+	//     schema:
+	//       $ref: '#/definitions/Card'
+	//   default:
+	//     description: internal error
+	//     schema:
+	//       "$ref": "#/definitions/ErrorResponse"
+
+	userID := getUserID(r)
+	teamID := mux.Vars(r)["teamID"]
+	ticketCode := mux.Vars(r)["ticketCode"]
+
+	card, err := a.app.GetCardByTicketCode(ticketCode, teamID)
+	if err != nil {
+		a.errorResponse(w, r, err)
+		return
+	}
+
+	if !a.permissions.HasPermissionToBoard(userID, card.BoardID, model.PermissionViewBoard) {
+		a.errorResponse(w, r, model.NewErrPermission("access denied to fetch card"))
+		return
+	}
+
+	auditRec := a.makeAuditRecord(r, "getCardByTicketCode", audit.Fail)
+	defer a.audit.LogRecord(audit.LevelRead, auditRec)
+	auditRec.AddMeta("teamID", teamID)
+	auditRec.AddMeta("ticketCode", ticketCode)
+	auditRec.AddMeta("cardID", card.ID)
+
+	a.logger.Debug("GetCardByTicketCode",
+		mlog.String("teamID", teamID),
+		mlog.String("ticketCode", ticketCode),
+		mlog.String("cardID", card.ID),
+		mlog.String("userID", userID),
+	)
+
+	data, err := json.Marshal(card)
+	if err != nil {
+		a.errorResponse(w, r, err)
+		return
+	}
+
+	// response
+	jsonBytesResponse(w, http.StatusOK, data)
+
+	auditRec.Success()
+}

server/app/cards.go

@@ -22,6 +22,19 @@ func (a *App) CreateCard(card *model.Card, boardID string, userID string, disabl
 	card.UpdateAt = now
 	card.DeleteAt = 0
 
+	// Auto-assign ticket number: atomically increment the board's card counter
+	cardNumber, err := a.store.IncrementBoardCardCount(boardID)
+	if err != nil {
+		return nil, fmt.Errorf("cannot assign ticket number: %w", err)
+	}
+	card.CardNumber = cardNumber
+
+	// Generate the ticket code if the board has a prefix
+	board, err := a.store.GetBoard(boardID)
+	if err == nil && board.CardPrefix != "" {
+		card.TicketCode = fmt.Sprintf("%s-%d", board.CardPrefix, cardNumber)
+	}
+
 	block := model.Card2Block(card)
 
 	newBlocks, err := a.InsertBlocksAndNotify([]*model.Block{block}, userID, disableNotify)
@@ -34,6 +47,11 @@ func (a *App) CreateCard(card *model.Card, boardID string, userID string, disabl
 		return nil, err
 	}
 
+	// Re-attach the ticket code (it's not stored in block fields, it's computed)
+	if board != nil && board.CardPrefix != "" {
+		newCard.TicketCode = fmt.Sprintf("%s-%d", board.CardPrefix, newCard.CardNumber)
+	}
+
 	return newCard, nil
 }
 
@@ -50,12 +68,18 @@ func (a *App) GetCardsForBoard(boardID string, page int, perPage int) ([]*model.
 		return nil, err
 	}
 
+	// Fetch board prefix for ticket code computation
+	board, _ := a.store.GetBoard(boardID)
+
 	cards := make([]*model.Card, 0, len(blocks))
 	for _, blk := range blocks {
 		b := blk
 		if card, err := model.Block2Card(b); err != nil {
 			return nil, fmt.Errorf("Block2Card fail: %w", err)
 		} else {
+			if board != nil && board.CardPrefix != "" && card.CardNumber > 0 {
+				card.TicketCode = fmt.Sprintf("%s-%d", board.CardPrefix, card.CardNumber)
+			}
 			cards = append(cards, card)
 		}
 	}
@@ -92,5 +116,33 @@ func (a *App) GetCardByID(cardID string) (*model.Card, error) {
 		return nil, err
 	}
 
+	// Populate ticket code from board prefix
+	if card.CardNumber > 0 {
+		board, boardErr := a.store.GetBoard(card.BoardID)
+		if boardErr == nil && board.CardPrefix != "" {
+			card.TicketCode = fmt.Sprintf("%s-%d", board.CardPrefix, card.CardNumber)
+		}
+	}
+
+	return card, nil
+}
+
+func (a *App) GetCardByTicketCode(ticketCode string, teamID string) (*model.Card, error) {
+	prefix, number, err := model.ParseTicketCode(ticketCode)
+	if err != nil {
+		return nil, err
+	}
+
+	block, err := a.store.GetCardByTicketCode(prefix, number, teamID)
+	if err != nil {
+		return nil, err
+	}
+
+	card, err := model.Block2Card(block)
+	if err != nil {
+		return nil, err
+	}
+
+	card.TicketCode = ticketCode
 	return card, nil
 }

server/model/board.go

@@ -32,6 +32,7 @@ const (
 	BoardSearchFieldNone         BoardSearchField = ""
 	BoardSearchFieldTitle        BoardSearchField = "title"
 	BoardSearchFieldPropertyName BoardSearchField = "property_name"
+	BoardSearchFieldCardPrefix   BoardSearchField = "card_prefix"
 )
 
 // Board groups a set of blocks and its layout
@@ -97,6 +98,14 @@ type Board struct {
 	// required: false
 	CardProperties []map[string]interface{} `json:"cardProperties"`
 
+	// The short prefix used for ticket codes (e.g. "PROJ", "BUG")
+	// required: false
+	CardPrefix string `json:"cardPrefix"`
+
+	// The auto-incrementing counter for card ticket numbers
+	// required: false
+	CardCount int64 `json:"cardCount"`
+
 	// The creation time in miliseconds since the current epoch
 	// required: true
 	CreateAt int64 `json:"createAt"`
@@ -156,6 +165,10 @@ type BoardPatch struct {
 	// required: false
 	ChannelID *string `json:"channelId"`
 
+	// The short prefix used for ticket codes (e.g. "PROJ", "BUG")
+	// required: false
+	CardPrefix *string `json:"cardPrefix"`
+
 	// The board updated properties
 	// required: false
 	UpdatedProperties map[string]interface{} `json:"updatedProperties"`
@@ -297,6 +310,10 @@ func (p *BoardPatch) Patch(board *Board) *Board {
 		board.ChannelID = *p.ChannelID
 	}
 
+	if p.CardPrefix != nil {
+		board.CardPrefix = *p.CardPrefix
+	}
+
 	for key, property := range p.UpdatedProperties {
 		board.Properties[key] = property
 	}
@@ -377,6 +394,10 @@ func (p *BoardPatch) IsValid() error {
 		return InvalidBoardErr{"invalid-channel-id"}
 	}
 
+	if p.CardPrefix != nil && len(*p.CardPrefix) > 10 {
+		return InvalidBoardErr{"invalid-card-prefix-too-long"}
+	}
+
 	return nil
 }
 
@@ -447,6 +468,8 @@ func BoardSearchFieldFromString(field string) (BoardSearchField, error) {
 		return BoardSearchFieldTitle, nil
 	case string(BoardSearchFieldPropertyName):
 		return BoardSearchFieldPropertyName, nil
+	case string(BoardSearchFieldCardPrefix):
+		return BoardSearchFieldCardPrefix, nil
 	}
 	return BoardSearchFieldNone, ErrInvalidBoardSearchField
 }

server/model/card.go

@@ -4,14 +4,38 @@
 package model
 
 import (
+	"encoding/json"
 	"errors"
 	"fmt"
+	"strconv"
+	"strings"
 
 	"github.com/mattermost/mattermost-plugin-boards/server/utils"
 	"github.com/rivo/uniseg"
 )
 
 var ErrBoardIDMismatch = errors.New("Board IDs do not match")
+var ErrInvalidTicketCode = errors.New("invalid ticket code format, expected PREFIX-NUMBER")
+
+// ParseTicketCode parses a ticket code like "PROJ-42" into its prefix ("PROJ") and number (42).
+func ParseTicketCode(ticketCode string) (string, int64, error) {
+	parts := strings.SplitN(ticketCode, "-", 2)
+	if len(parts) != 2 {
+		return "", 0, ErrInvalidTicketCode
+	}
+
+	prefix := strings.TrimSpace(parts[0])
+	if prefix == "" {
+		return "", 0, ErrInvalidTicketCode
+	}
+
+	number, err := strconv.ParseInt(strings.TrimSpace(parts[1]), 10, 64)
+	if err != nil || number <= 0 {
+		return "", 0, ErrInvalidTicketCode
+	}
+
+	return strings.ToUpper(prefix), number, nil
+}
 
 type ErrInvalidCard struct {
 	msg string
@@ -76,6 +100,14 @@ type Card struct {
 	// required: false
 	Properties map[string]any `json:"properties"`
 
+	// The sequential card number within the board (auto-assigned)
+	// required: false
+	CardNumber int64 `json:"cardNumber"`
+
+	// The full ticket code combining board prefix and card number (e.g. "PROJ-42")
+	// required: false
+	TicketCode string `json:"ticketCode,omitempty"`
+
 	// The creation time in milliseconds since the current epoch
 	// required: false
 	CreateAt int64 `json:"createAt"`
@@ -203,6 +235,7 @@ func Card2Block(card *Card) *Block {
 	fields["icon"] = card.Icon
 	fields["isTemplate"] = card.IsTemplate
 	fields["properties"] = card.Properties
+	fields["cardNumber"] = card.CardNumber
 
 	return &Block{
 		ID:         card.ID,
@@ -230,6 +263,7 @@ func Block2Card(block *Block) (*Card, error) {
 	icon := ""
 	isTemplate := false
 	properties := make(map[string]any)
+	var cardNumber int64
 
 	if co, ok := block.Fields["contentOrder"]; ok {
 		switch arr := co.(type) {
@@ -274,6 +308,19 @@ func Block2Card(block *Block) (*Card, error) {
 		}
 	}
 
+	if cn, ok := block.Fields["cardNumber"]; ok {
+		switch v := cn.(type) {
+		case float64:
+			cardNumber = int64(v)
+		case int64:
+			cardNumber = v
+		case json.Number:
+			if n, err := v.Int64(); err == nil {
+				cardNumber = n
+			}
+		}
+	}
+
 	card := &Card{
 		ID:           block.ID,
 		BoardID:      block.BoardID,
@@ -284,6 +331,7 @@ func Block2Card(block *Block) (*Card, error) {
 		Icon:         icon,
 		IsTemplate:   isTemplate,
 		Properties:   properties,
+		CardNumber:   cardNumber,
 		CreateAt:     block.CreateAt,
 		UpdateAt:     block.UpdateAt,
 		DeleteAt:     block.DeleteAt,