Skip to content

KAN-197/add-card-identifier-search-prefix-number#164

Merged
fulsomenko merged 3 commits intodevelopfrom
KAN-197/add-card-identifier-search-prefix-number
Feb 1, 2026
Merged

KAN-197/add-card-identifier-search-prefix-number#164
fulsomenko merged 3 commits intodevelopfrom
KAN-197/add-card-identifier-search-prefix-number

Conversation

@fulsomenko
Copy link
Owner

@fulsomenko fulsomenko commented Feb 1, 2026

Add card identifier search to the existing composite search:

  • Add CardIdentifierSearcher that matches cards by {prefix}-{number} (e.g. KAN-164, task-2)
  • Include it in CompositeSearcher::all() alongside title and branch name searchers
  • Supports searching by full identifier (KAN-164), prefix (KAN), or just number (164)
  • Case-insensitive substring matching, consistent with existing searchers

What: New search strategy that matches cards by their assigned prefix and card number.

Why: Allows quickly finding cards by their identifier (e.g. KAN-164) using the existing / search in the TUI.

How: Added CardIdentifierSearcher implementing CardSearcher trait. It builds "{prefix}-{number}" from card.assigned_prefix (falling back to board.card_prefix, then "task") and does case-insensitive contains matching. Added as a SearchBy::CardIdentifier variant in the composite searcher.

Testing: 3 new unit tests covering prefix+number matching, number-only matching, and composite integration. All 9 search tests pass.

@fulsomenko fulsomenko changed the title feat: add card identifier search (KAN-197) KAN-197/add-card-identifier-search-prefix-number Feb 1, 2026
@fulsomenko fulsomenko force-pushed the KAN-197/add-card-identifier-search-prefix-number branch from c4ee153 to 4a08261 Compare February 1, 2026 19:37
@fulsomenko fulsomenko merged commit 5daffa5 into develop Feb 1, 2026
7 checks passed
@fulsomenko fulsomenko deleted the KAN-197/add-card-identifier-search-prefix-number branch February 1, 2026 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant