gptscript-ai / desktop

MIT License
22 stars 13 forks source link

Not able to add "Hubspot" tool #434

Closed sangee2004 closed 1 month ago

sangee2004 commented 1 month ago

Electron build - 0d607f997

Steps to reproduce the problem:

  1. Add "Hubspot" tool to a thread.
  2. Add Hubspot" tool to an assistant.

In both cases , adding the tool fails and following error message is seen.

2024-09-05T00:53:02.961Z [server] [ERROR] Error: failed to load github.com/gptscript-ai/tools/apis/hubspot/crm: error loading https://raw.githubusercontent.com/gptscript-ai/tools/3857968a16ddc56b09235c2da10fd75b0f6b317b/apis/hubspot/crm/tool.gpt: error loading https://raw.githubusercontent.com/gptscript-ai/tools/3857968a16ddc56b09235c2da10fd75b0f6b317b/apis/hubspot/crm/tool.gpt: 404 Not Found
    at IncomingMessage.<anonymous> (/Users/sangeethahariharan/acorn/desktop/electron-dist/mac-arm64/Acorn.app/Contents/Resources/app.asar/.next/server/chunks/666.js:28:105593)
    at IncomingMessage.emit (node:events:531:35)
    at endReadableNT (node:internal/streams/readable:1696:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  digest: '3365377860'
}
2024-09-05T00:53:02.972Z [client] [ERROR] Error: An error occurred in the Server Components render. The specific message is omitted in production builds to avoid leaking sensitive details. A digest property is included on this error instance which may provide additional details about the nature of the error.
drpebcak commented 1 month ago

This will be resolved when https://github.com/gptscript-ai/desktop/pull/429 is merged

sangee2004 commented 1 month ago

I am testing with build -29efc787

  1. I am still able to reproduce the issue with not being able to add Hubspot tool to an assistant from assistant edit view using "Featured Tools" option to select the Hubspot entry.

  2. Able to Add "Hubspot" tool successfully to a thread. This entry shows up as "HubSpot CRM Write". Should this be "HubSpot CRM"? Image

Reopening for getting 1 addressed and clarification for 2

drpebcak commented 1 month ago

Looks like a regression was added in #406. Should be fixed now.

As far as the naming convention goes, I will leave that up to @cjellick but I wouldn't consider it a bug. It's just the name of the write tool. If we want to mask it to just be 'HubSpot' for now I can change that in the tools repo.

cjellick commented 1 month ago

rename this tool simply "Hubspot CRM" and the read tool "Hubspot CRM - Read only"

cjellick commented 1 month ago

@drpebcak move this back to testing if its ready to go

sangee2004 commented 1 month ago

Able to add "HubSpot CRM" tool from Edit assistant view. This shows up as "HubSpot CRM"