Skip to content

Add total letter count to share text#1

Open
nori-sessions[bot] wants to merge 1 commit into
mainfrom
feat/share-letter-count
Open

Add total letter count to share text#1
nori-sessions[bot] wants to merge 1 commit into
mainfrom
feat/share-letter-count

Conversation

@nori-sessions
Copy link
Copy Markdown

@nori-sessions nori-sessions Bot commented May 26, 2026

Summary

🤖 Generated with Nori

  • Include total letter count in share text stats line (e.g., 10/10 | 40 letters | 1:00)
  • Uses singular "letter" when count is exactly 1, plural "letters" otherwise
  • Updated docs to reflect new share text format

Test Plan

  • All 313 existing tests pass (0 regressions)
  • 1 new test: singular "letter" when count is 1
  • 1 new test: varying answer lengths sum correctly
  • 6 existing generateShareText tests updated with letter count assertions
  • Manual verification: play a game and use the share button

Upstream target: theahura/reword (open cross-fork PR manually if needed)

Share Nori with your team: https://www.npmjs.com/package/nori-skillsets

Include the sum of answer lengths in the share text stats line so
players can compare letter efficiency. Format: "N letters" (or
"1 letter" singular) pipe-delimited between solve count and time.
🤖 Generated with [Nori](https://noriagentic.com)

Co-Authored-By: Nori <contact@tilework.tech>
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