Skip to content

chore: remove Golden Honey Badger schema, locales, and asset (PR2/2)#792

Merged
grunch merged 1 commit intoissue790-pr1-disable-ghbfrom
issue790-pr2-cleanup
Apr 28, 2026
Merged

chore: remove Golden Honey Badger schema, locales, and asset (PR2/2)#792
grunch merged 1 commit intoissue790-pr1-disable-ghbfrom
issue790-pr2-cleanup

Conversation

@grunch
Copy link
Copy Markdown
Member

@grunch grunch commented Apr 27, 2026

Summary

  • Drop is_golden_honey_badger from IOrder interface and Mongoose schema.
  • Remove golden_honey_badger i18n key from all 10 locale files.
  • Delete images/Honeybadger.png.
  • Drop the residual is_golden_honey_badger writes left over after PR1.

Stacked on top of #791 — review/merge that one first. Existing documents in MongoDB that still hold is_golden_honey_badger will simply have the field ignored once Mongoose stops casting it; no migration script needed.

Closes #790

Test plan

  • npx tsc clean
  • npm test 120 passing
  • npm run lint clean
  • Verify no is_golden_honey_badger / golden_honey_badger / Honeybadger references remain (grep -rn ...)

🤖 Generated with Claude Code

Final cleanup. Existing documents in MongoDB that still hold
is_golden_honey_badger will simply have the field ignored — no
migration script needed since Mongoose stops casting the field once
it's removed from the schema.

- models/order: drop is_golden_honey_badger from IOrder and schema
- ordersActions / takeOrder: drop is_golden_honey_badger writes
- locales: drop golden_honey_badger key from all 10 languages
- images: delete Honeybadger.png

Closes #790

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 27, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 6290da71-0ba7-4539-9964-e46e34cdc9ed

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue790-pr2-cleanup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Collaborator

@Luquitasjeffrey Luquitasjeffrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

@grunch grunch merged commit 7464651 into issue790-pr1-disable-ghb Apr 28, 2026
5 checks passed
@grunch grunch deleted the issue790-pr2-cleanup branch April 28, 2026 17:08
grunch added a commit that referenced this pull request Apr 28, 2026
* chore: disable Golden Honey Badger feature (PR1/2)

Stop assigning the Golden Honey Badger image and zero-fee bypass to new
orders, drop the user-facing congratulatory messages, and remove the
type definitions and signatures the feature relied on. Existing orders
in the database keep working since their fee was already calculated at
creation time.

- imageCache: remove probability check, honeybadger cache slot,
  honeybadgerCached stat, and isGoldenHoneyBadger from
  generateRandomImage return
- getFee: drop isGoldenHoneyBadger parameter and conditional logic
- ordersActions: hard-code is_golden_honey_badger to false and drop
  isGoldenHoneyBadger from BuildDescriptionArguments
- takeOrder: hard-code is_golden_honey_badger to false; drop the
  unnecessary await on synchronous generateRandomImage
- messages: drop congratulatory blocks and isGoldenHoneyBadger param
  from showHoldInvoiceMessage
- showHoldInvoice: always charge full fee
- .env-sample: drop GOLDEN_HONEY_BADGER_PROBABILITY

Refs #790

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* chore: remove Golden Honey Badger schema, locales, and asset (PR2/2) (#792)

Final cleanup. Existing documents in MongoDB that still hold
is_golden_honey_badger will simply have the field ignored — no
migration script needed since Mongoose stops casting the field once
it's removed from the schema.

- models/order: drop is_golden_honey_badger from IOrder and schema
- ordersActions / takeOrder: drop is_golden_honey_badger writes
- locales: drop golden_honey_badger key from all 10 languages
- images: delete Honeybadger.png

Closes #790

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

2 participants