Skip to content

feat(rename): expand lore name pool#26

Merged
RANGROO merged 2 commits into
unstablefrom
codex/rename-name-expansion
Jul 20, 2026
Merged

feat(rename): expand lore name pool#26
RANGROO merged 2 commits into
unstablefrom
codex/rename-name-expansion

Conversation

@RANGROO

@RANGROO RANGROO commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

What

  • add 1,000 new single-word lore names, expanding the built-in pool from 72 to 1,072
  • seed the complete pool for fresh installs
  • import only missing base names on existing installs
  • preserve existing enabled/disabled states and custom overrides
  • cover the importer with the existing rename-pool regression

Scope

This PR contains only the name data and the code required to seed it:

  • data/rename_names_seed.csv
  • data/schema.sql
  • debug/db_updates.php
  • lib/rename_name_pool_functions.php
  • tests/rename_name_pool_regression.php

No generator, promoter, blocklist, OpenRouter integration, or generation-only CI test is included.

Dataset

  • 1,072 total names
  • 1,072 case-insensitive unique names
  • 358 male / 357 female / 357 neutral
  • all names are one ASCII word matching ^[A-Z][a-z]{2,9}$

Existing installs

Patch rename_name_pool_expansion (202607190102) reads the committed CSV and inserts only case-insensitively missing base rows. Existing base rows are not updated, so disabled state is preserved. Custom rows are untouched.

Checks

  • old-install simulation: 1072:1072:1071:false after starting from 72 names with Arin disabled
  • second migration run remains at 1,072 rows
  • all 24 remaining PR regression tests pass
  • deployed locally with database result 1072|1072|1072
  • deployed CSV checksum matches this branch
  • removed generation files are absent from the local deployment
  • health route and Rename Names page return HTTP 200

@RANGROO
RANGROO marked this pull request as ready for review July 20, 2026 15:51
@RANGROO
RANGROO merged commit 249d2d4 into unstable Jul 20, 2026
1 check passed
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