Skip to content

feat(wealthbox): added wealthbox crm - #669

Merged
icecrasher321 merged 11 commits into
stagingfrom
feat/wealthbx
Jul 14, 2025
Merged

feat(wealthbox): added wealthbox crm#669
icecrasher321 merged 11 commits into
stagingfrom
feat/wealthbx

Conversation

@aadamgough

@aadamgoughaadamgough commented Jul 11, 2025

Copy link
Copy Markdown
Contributor
wbshortdemo.mov

Description

Added wealthbox as a tool

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested each operation and confirmed that they were working properly.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally and in CI (bun run test)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have updated version numbers as needed (if needed)
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • My changes do not introduce any new security vulnerabilities
  • I have considered the security implications of my changes

@vercel

vercelBot commented Jul 11, 2025

Copy link
Copy Markdown

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

NameStatusPreviewCommentsUpdated (UTC)
sim✅ Ready (Inspect)Visit Preview💬 Add feedbackJul 14, 2025 8:09pm
1 Skipped Deployment
NameStatusPreviewCommentsUpdated (UTC)
docs⬜️ Skipped (Inspect)Jul 14, 2025 8:09pm

@delve-auditor

delve-auditorBot commented Jul 11, 2025

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to a936c7e.

Security Overview
  • 🔎 Scanned files: 23 changed file(s)
Detected Code Changes
Change TypeRelevant files
Enhancement► route.ts
Add params visibility to Wealthbox API routes
► meta.json
Add wealthbox to tools list
► wealthbox.mdx
Add wealthbox documentation
► registry.ts
Register wealthbox tools
► wealthbox/*.ts
Implement wealthbox API integration

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@aadamgough
aadamgough marked this pull request as ready for review July 11, 2025 19:21

@greptile-appsgreptile-appsBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR Summary

Comprehensive integration of Wealthbox CRM functionality with OAuth authentication and CRUD operations for contacts, notes, and tasks.

  • Extensive API integration in apps/sim/tools/wealthbox/*.ts with six new tools for reading/writing CRM data, though has code duplication between directExecution and transformResponse methods
  • OAuth implementation in apps/sim/lib/oauth/oauth.ts uses synthetic user profiles due to missing user info endpoint
  • Added contact selector UI component in apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/workflow-block/components/sub-block/components/file-selector/components/wealthbox-file-selector.tsx with debounced search
  • Introduced model capabilities system in apps/sim/providers/model-capabilities.ts for temperature ranges and tool usage control
  • Multiple type safety concerns across new files using 'any' type and potential undefined object access

25 files reviewed, 52 comments
Edit PR Review Bot Settings | Greptile

Comment threadapps/sim/lib/env.ts
Comment threadapps/sim/tools/wealthbox/index.ts
Comment threadapps/sim/providers/model-capabilities.test.ts Outdated
Comment threadapps/sim/lib/auth.ts
Comment threadapps/sim/tools/wealthbox/types.ts
Comment threadapps/sim/app/api/tools/wealthbox/items/route.ts
Comment threadapps/sim/app/api/tools/wealthbox/item/route.ts
Comment threadapps/sim/tools/wealthbox/types.ts
Comment threadapps/sim/app/api/tools/wealthbox/items/route.ts
@vercel
vercelBottemporarily deployed to Preview – docs July 14, 2025 20:02 Inactive
@icecrasher321icecrasher321 self-assigned this Jul 14, 2025
@icecrasher321icecrasher321 changed the title feat(wealthbx): added wealthbx crmfeat(wealthbox): added wealthbox crmJul 14, 2025
@icecrasher321
icecrasher321 merged commit d65bdaf into stagingJul 14, 2025
5 of 6 checks passed
waleedlatif1 pushed a commit that referenced this pull request Jul 14, 2025
* feat: wealthbox
* feat: added tools
* feat: tested and finished tools
* feat: tested and finished tools
* feat: added refresh token
* fix: added docs
* bun lint
* feat: removed files #669
* fix: greptile comments
* fix: stringified messages #669
* add visibilty to params
---------
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
waleedlatif1 added a commit that referenced this pull request Jul 14, 2025
…dration issues by consolidating environment checking function (#678)
* fix(permissions): make permissions check case insensitive
* consolidated use of helpers to fetch environment
* use import aliases
* fix(voice): added voice functionality back to chat client (#676)
* fix(voice): added voice functioanlity back to chat clinet
* add logic to support deployed chat in staging
* fix(api-timeout): increase timeout for API block to 2 min (#677)
* feat(wealthbox): added wealthbox crm (#669)
* feat: wealthbox
* feat: added tools
* feat: tested and finished tools
* feat: tested and finished tools
* feat: added refresh token
* fix: added docs
* bun lint
* feat: removed files #669
* fix: greptile comments
* fix: stringified messages #669
* add visibilty to params
---------
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
---------
Co-authored-by: Vikhyath Mondreti <vikhyathvikku@gmail.com>
Co-authored-by: Adam Gough <77861281+aadamgough@users.noreply.github.com>
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
@waleedlatif1
waleedlatif1 deleted the feat/wealthbx branch July 14, 2025 21:21
arenadeveloper02 referenced this pull request in arenadeveloper02/p2-sim Sep 19, 2025
* feat: wealthbox
* feat: added tools
* feat: tested and finished tools
* feat: tested and finished tools
* feat: added refresh token
* fix: added docs
* bun lint
* feat: removed files #669
* fix: greptile comments
* fix: stringified messages #669
* add visibilty to params
---------
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
arenadeveloper02 referenced this pull request in arenadeveloper02/p2-sim Sep 19, 2025
…dration issues by consolidating environment checking function (#678)
* fix(permissions): make permissions check case insensitive
* consolidated use of helpers to fetch environment
* use import aliases
* fix(voice): added voice functionality back to chat client (#676)
* fix(voice): added voice functioanlity back to chat clinet
* add logic to support deployed chat in staging
* fix(api-timeout): increase timeout for API block to 2 min (#677)
* feat(wealthbox): added wealthbox crm (#669)
* feat: wealthbox
* feat: added tools
* feat: tested and finished tools
* feat: tested and finished tools
* feat: added refresh token
* fix: added docs
* bun lint
* feat: removed files #669
* fix: greptile comments
* fix: stringified messages #669
* add visibilty to params
---------
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
---------
Co-authored-by: Vikhyath Mondreti <vikhyathvikku@gmail.com>
Co-authored-by: Adam Gough <77861281+aadamgough@users.noreply.github.com>
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Vikhyath Mondreti <vikhyath@simstudio.ai>
Sign up for freeto 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

@aadamgough@icecrasher321