Skip to content

TKW: character alias search, and fill missing aliases from AWOIAF#46

Merged
codebend3r merged 2 commits into
mainfrom
nickname-search
Jul 15, 2026
Merged

TKW: character alias search, and fill missing aliases from AWOIAF#46
codebend3r merged 2 commits into
mainfrom
nickname-search

Conversation

@codebend3r

Copy link
Copy Markdown
Owner

Summary

  • Character search now matches on aliases/nicknames, not just names
  • Audited all 892 sourced characters against their cited AWOIAF page and filled in missing aliases

Search

  • filterByName in lib/search.ts ranks against an optional aliases array per item, with name matches always outranking alias-only matches
  • CharacterItem and CharacterSuggestion carry the full aliases array; app/characters/page.tsx and app/characters/[slug]/page.tsx wire it through
  • display is unchanged — still just the first alias as a badge
  • houses/weapons/dragons search unaffected (they never pass aliases)

Content

  • 141 characters gained new aliases: entries (bynames/epithets/alternate names) sourced from their AWOIAF infobox or lead prose
  • existing aliases preserved and unioned, never replaced; formal titles kept out of aliases
  • 28 characters with no citable AWOIAF source were skipped, untouched

Design

  • Spec at docs/superpowers/specs/2026-07-13-character-alias-search-design.md

- `filterByName` in `lib/search.ts` accepts an optional `aliases` array per item, ranked below name matches so a nickname like "Kingslayer" surfaces the right character without name matches losing priority
- `CharacterItem` and `CharacterSuggestion` carry the full `aliases` array; `app/characters/page.tsx` and `app/characters/[slug]/page.tsx` pass it through
- display unchanged — still shows only the first alias as a badge
- adds alias-matching coverage to `lib/search.test.ts`, `CharacterSearchInput.test.tsx`, `FilteredCharacterList.test.tsx`
- design spec at `docs/superpowers/specs/2026-07-13-character-alias-search-design.md`
- audited all 892 characters citing a `type: awoiaf` source against that page's infobox and lead prose
- 141 characters gained new `aliases:` entries (bynames, epithets, alternate names) not previously recorded
- existing aliases preserved and unioned, not replaced; formal titles/styles kept out of `aliases`
- remaining characters either already had a complete alias list or have no recorded byname on their cited source
@netlify

netlify Bot commented Jul 14, 2026

Copy link
Copy Markdown

Deploy Preview for theknownworld ready!

Name Link
🔨 Latest commit 34f6ed2
🔍 Latest deploy log https://app.netlify.com/projects/theknownworld/deploys/6a565d895807280008f8d86a
😎 Deploy Preview https://deploy-preview-46--theknownworld.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codebend3r
codebend3r merged commit 6071b10 into main Jul 15, 2026
5 checks passed
@codebend3r
codebend3r deleted the nickname-search branch July 15, 2026 05:37
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