fix plaid-mapper

[openint-bot]

Important

Fix delimiter in bank_category field in transaction mapper in mapper.ts to use '/'.

• Behavior:
  • Change delimiter in bank_category field in transaction mapper from empty string to '/'.
  • Affects how personal_finance_category fields are joined when not null.

^{This description was created by for a2c6434. It will automatically update as commits are pushed.}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
openint	🔄 Building (Inspect)	Visit Preview	💬 Add feedback	Mar 7, 2025 6:43pm

[recurseml]

✨ No issues found! Your code is sparkling clean! ✨

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to a2c6434 in 32 seconds

More details

• Looked at 13 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. unified/unified-accounting/adapters/plaid-adapter/mapper.ts:55

• Draft comment:
  Using join('/') improves readability compared to join(''), but consider trimming category values in case of extra whitespace.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50%
  None

2. unified/unified-accounting/adapters/plaid-adapter/mapper.ts:56

• Draft comment:
  Using '/' as the join delimiter in bank_category now clearly separates the primary and detailed category values. This fix improves readability and seems to be the intended behavior.
• Reason this comment was not posted:
  Comment did not seem useful. Confidence is useful = 0% <= threshold 50%
  This comment is purely informative, praising the use of '/' as a delimiter. It doesn't provide any actionable feedback or suggestions for improvement.

Workflow ID: wflow_E4K89vmhw5L1vXcu

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.