chore: fix review comment

[turip]

late fix

Summary by CodeRabbit

• Refactor
  • Simplified credit priority handling in credit purchase flows to use explicit priority values without fallback defaults.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8ff7dfcc-4696-4005-934a-638f6dc8699c

📥 Commits

Reviewing files that changed from the base of the PR and between 9f763d9 and 9cc9d21.

📒 Files selected for processing (1)

• openmeter/ledger/chargeadapter/creditpurchase.go

---

📝 Walkthrough

Walkthrough

This PR simplifies credit priority assignment logic by removing the creditPurchasePriority helper function and its fallback mechanism, instead passing charge.Intent.Priority directly to both promotional and initiated credit purchase flows.

Changes

Cohort / File(s)	Summary
Credit Purchase Logic Simplification openmeter/ledger/chargeadapter/creditpurchase.go	Removed creditPurchasePriority helper function and stopped applying fallback to ledger.DefaultCustomerFBOPriority; CreditPriority is now directly assigned from charge.Intent.Priority without dereferencing or defaults.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• feat: credit priority and effective at #4066: Introduced the creditPurchasePriority helper function that this PR removes, making these PRs inverse operations in the credit priority assignment flow.

Suggested labels

area/billing

Suggested reviewers

• GAlexIHU

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title is vague and generic, using the term 'fix review comment' without clearly specifying what was changed or what the primary purpose of the changeset is.	Consider using a more descriptive title that explains the actual change, such as 'Remove creditPurchasePriority helper and simplify priority handling' to clearly convey the main modification.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/fix-review-comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[GAlexIHU]

approve after the fact