Skip to content

fix(catalog): raise soul and show brief caps to 2000 - #124

Merged
perminder-klair merged 1 commit into
mainfrom
fix/soul-and-show-caps-2000
Oct 30, 2026
Merged

perminder-klair merged 1 commit into
mainfrom
fix/soul-and-show-caps-2000

Conversation

@perminder-klair

Copy link
Copy Markdown
Contributor

Summary

Raises the soul and show brief limits from 1000 to 2000 characters, matching what stations already accept. Unblocks #123, whose 1495-character soul fits a station's editor (1495 / 2000) but was refused by the submission bot.

Cap Raised in the station Builder Workflow Issue form Docs
Persona soul v0.45.0 (perminder-klair/subwave#1113) build-catalog.mjs persona-submission.yml add-persona.yml CONTRIBUTING.md
Show topic v1.3.0 (perminder-klair/subwave#1269) build-catalog.mjs show-submission.yml add-show.yml CONTRIBUTING.md

Older stations stay safe. Before v0.45.0, a persona over the old cap is left out of the listing, so it can't fail on install. Before v1.3.0, a long show brief is cut to that station's cap.

Evidence

  • node scripts/build-catalog.mjs --check passes on this branch, and catalog.json is unchanged.
  • With temporary entries in a scratch copy, 2000 characters passes and 2001 fails, for both personas and shows.
  • With the limit at 2000, the soul from Add persona: Clubbing Chloe #123 builds into the catalog with no errors or warnings of its own.

After merge

#123 needs an edit, or the persona-submission label re-applied, for the bot to open its PR.

Stations have accepted a 2000-character soul since v0.45.0 and a
2000-character show topic since v1.3.0, but the catalog builder, the
submission workflows, the issue forms and CONTRIBUTING all still said 1000.
A soul written in a station's own editor could pass there and then be
refused here (#123).
@perminder-klair perminder-klair mentioned this pull request Oct 30, 2026
3 tasks done
@perminder-klair
perminder-klair merged commit 39aced8 into main Oct 30, 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