Skip to content

feat(ai-proxy): add kolar tools#1537

Merged
nbouliol merged 8 commits intomainfrom
feat/add-kolar-tools
Apr 21, 2026
Merged

feat(ai-proxy): add kolar tools#1537
nbouliol merged 8 commits intomainfrom
feat/add-kolar-tools

Conversation

@nbouliol
Copy link
Copy Markdown
Member

@nbouliol nbouliol commented Apr 10, 2026

Fixes PRD-245

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made

Note

Add Kolar KYB and transaction screening tools to ai-proxy

  • Adds a new Kolar integration to ForestIntegrationClient alongside the existing Zendesk integration, extending ForestIntegrationName and CustomConfig union types accordingly.
  • Implements four DynamicStructuredTool factories in packages/ai-proxy/src/integrations/kolar/: kolar_create_merchant_application, kolar_get_merchant_application_result, kolar_screen_transaction, and kolar_get_screening_result.
  • Each tool makes sequential HTTP calls to the Kolar API (create + trigger job, or fetch result) and returns JSON strings; errors surface with action, HTTP status, and parsed message via a shared assertResponseOk helper.
  • Credential validation is handled by validateKolarConfig, which hits GET /auth/verify and throws McpConnectionError on failure.

Macroscope summarized 282d76f.

@linear
Copy link
Copy Markdown

linear Bot commented Apr 10, 2026

@qltysh
Copy link
Copy Markdown

qltysh Bot commented Apr 10, 2026

5 new issues

Tool Category Rule Count
qlty Duplication Found 19 lines of similar code in 2 locations (mass = 83) 5

@qltysh
Copy link
Copy Markdown

qltysh Bot commented Apr 10, 2026

Qlty


Coverage Impact

⬆️ Merging this pull request will increase total coverage on main by 0.02%.

Modified Files with Diff Coverage (7)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
packages/ai-proxy/src/forest-integration-client.ts100.0%
New Coverage rating: A
...rc/integrations/kolar/tools/get-merchant-application-result.ts100.0%
New Coverage rating: A
...xy/src/integrations/kolar/tools/create-merchant-application.ts100.0%
New Coverage rating: A
...es/ai-proxy/src/integrations/kolar/tools/screen-transaction.ts100.0%
New Coverage rating: A
packages/ai-proxy/src/integrations/kolar/tools.ts100.0%
New Coverage rating: A
.../ai-proxy/src/integrations/kolar/tools/get-screening-result.ts100.0%
New Coverage rating: A
packages/ai-proxy/src/integrations/kolar/utils.ts100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@nbouliol nbouliol force-pushed the feat/add-kolar-tools branch from 793b999 to 282d76f Compare April 20, 2026 14:25
Copy link
Copy Markdown
Member

@christophebrun-forest christophebrun-forest left a comment

Choose a reason for hiding this comment

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

Jest Tests : OK
Tested on my env :OK with one workflow ( kolar_screen_transaction -> kolar_get_screening_result)
Ok for me

@nbouliol nbouliol merged commit a966400 into main Apr 21, 2026
29 checks passed
@nbouliol nbouliol deleted the feat/add-kolar-tools branch April 21, 2026 07:28
forest-bot added a commit that referenced this pull request Apr 21, 2026
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