Skip to content

feat(integrations): add AgentLine provider#6700

Open
Sameersribot wants to merge 5 commits into
NangoHQ:masterfrom
AgentLineHQ:feat/integrations-agentline
Open

feat(integrations): add AgentLine provider#6700
Sameersribot wants to merge 5 commits into
NangoHQ:masterfrom
AgentLineHQ:feat/integrations-agentline

Conversation

@Sameersribot

@Sameersribot Sameersribot commented Jul 6, 2026

Copy link
Copy Markdown

What

Adds AgentLine as a new integration provider.

AgentLine is the voice infrastructure for AI agents — phone numbers, calls, and SMS through a single API.

Provider config

Changes

  • Added agentline entry to packages/providers/providers.yaml (alphabetically before agiloft)
  • Created docs/api-integrations/agentline/connect.mdx with setup guide
  • Added AgentLine logo (docs/api-integrations/agentline/account.png)
  • Entry in docs/docs.json alphabetically sorted

Review in cubic

Add AgentLine as a new API_KEY provider:
- providers.yaml: agentline entry with communication category
- docs: connect.mdx setup guide and account.png logo
- docs.json: alphabetical entry in api-integrations
Copilot AI review requested due to automatic review settings July 6, 2026 17:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds AgentLine as a new API integration provider, including provider configuration and initial documentation so users can connect via an API key.

Changes:

  • Added an agentline provider entry to packages/providers/providers.yaml (API key auth + proxy verification).
  • Added a connection setup guide at docs/api-integrations/agentline/connect.mdx (with screenshot).
  • Added api-integrations/agentline to docs/docs.json for docs routing/navigation.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 5 comments.

File Description
packages/providers/providers.yaml Introduces the AgentLine provider config (auth, proxy base URL, verification endpoint, docs links, credential schema).
docs/docs.json Registers the AgentLine docs route in the docs index/navigation list.
docs/api-integrations/agentline/connect.mdx Adds a “Connect” setup guide describing how to obtain and enter an AgentLine API key.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/providers/providers.yaml Outdated
proxy:
base_url: https://api.agentline.cloud
headers:
authorization: Bearer ***
Comment on lines +779 to +780
docs: https://nango.dev/docs/api-integrations/agentline
docs_connect: https://nango.dev/docs/api-integrations/agentline/connect
Comment thread docs/docs.json
Comment on lines 936 to 939
"api-integrations/agiloft",
"api-integrations/agiloft-cc",
"api-integrations/agentline",
"integrations/all/aimfox",
Comment thread docs/docs.json Outdated
"integrations/all/affinity-v2",
"api-integrations/agiloft",
"api-integrations/agiloft-cc",
"api-integrations/agentline",
2. Go to your dashboard and copy your API key (starts with `sk_live_`)

<Frame>
<img src="/api-integrations/agentline/account.png" />

@cubic-dev-ai cubic-dev-ai Bot 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.

1 issue found across 4 files

Confidence score: 5/5

  • In docs/docs.json, the api-integrations/agentline item being out of alphabetical order is a minor docs-maintainability issue; merging as-is is unlikely to affect product behavior but can make future list scans and automated ordering checks noisier — reorder the entry before or in a quick follow-up.

Not reviewed (too large): packages/providers/providers.yaml (~24 lines) - if these are generated or fixture files, add them to ignored paths to exclude them from future reviews.

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread docs/docs.json Outdated
- providers.yaml: fix Bearer *** -> Bearer \ (auth was broken)

- providers.yaml: add offset paginate config for list endpoints

- providers.yaml: remove invalid prefix field under credentials (schema)

- docs.json: place agentline before agiloft (alphabetical)

- docs: add api-integrations/agentline.mdx (required by provider validator)

@hassan254-prog hassan254-prog 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.

Could you run npm run docs to ensure all automatically generated docs is generated correctly?

@hassan254-prog hassan254-prog Jul 7, 2026

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.

For this page, I would recommend using the same style as the other docs pages, including the correct sections, structure and Nango form screenshot for consistency.

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.

Similar to this, we should keep the same style and structure to ensure consistency with the rest of our Nango docs.

Comment thread packages/providers/providers.yaml Outdated
Comment on lines +775 to +778
paginate:
type: offset
offset_name_in_request: offset
limit_name_in_request: limit

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.

Suggested change
paginate:
type: offset
offset_name_in_request: offset
limit_name_in_request: limit

We recommend defining this within the functions themselves rather than in the provider configuration.

type: string
title: API Key
description: Your AgentLine API key.
pattern: '^sk_live_[a-zA-Z0-9_]+$'

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.

Is it possible for the API key to have a separate format for other environments, such as staging, since this appears to be for live accounts only?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Api is for live accounts only, not possible to have other formats.

- providers.yaml: remove paginate config (define in functions per review)

- connect.mdx: align with standard Nango docs structure

- agentline.mdx: align with standard Nango docs structure

- regenerate auto-generated docs (api-catalog, snippets)
Previous commit introduced CRLF (edit tool under autocrlf=false). Restore LF to match repo convention and keep the PR diff clean.

@hassan254-prog hassan254-prog 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.

Could you also run npm run docs to update the automatically generated doc files?

Comment on lines +26 to +35
#### Step 2: Enter your API key in Nango

Once you have your **API key**:
1. In Nango, go to [Connections](https://app.nango.dev/dev/connections) -> _Add Test Connection_ -> select _AgentLine_.
2. Enter your **API key** in the designated field.
3. Submit the form, and you should be successfully authenticated.

You are now connected to AgentLine.

<Note>Contribute improvements to the setup guide by [editing this page](https://github.com/nangohq/nango/tree/master/docs/api-integrations/agentline/connect.mdx)</Note>

@hassan254-prog hassan254-prog Jul 9, 2026

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.

Please use the same format for this section as compared to the other connect ui docs, and also add the form screenshot, to match the other docs.

Comment on lines +64 to +71
## 📚 AgentLine Integration Guides

- [Connect to AgentLine](/api-integrations/agentline/connect)
Obtain your AgentLine API key and connect it to Nango.

Official docs: [AgentLine API](https://docs.agentline.cloud)

---

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.

Same with this, this should contain the ## 🧩 Pre-built syncs section.

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.

This should be a valid svg file.

…pre-built syncs)

- agentline.mdx: add Pre-built syncs section; fix docs URL (agentline.cloud/docs)

- connect.mdx: canonical Step 2 format + add form.png screenshot

- llms-full.txt: add agentline row (auto-generated)

- add agentline.svg logo + form.png
- communication
auth_mode: API_KEY
proxy:
base_url: https://api.agentline.cloud

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1: Unverified provider may proxy user API keys to an externally-controlled domain with no independent presence

This PR adds a new "AgentLine" provider whose proxy base_url (https://api.agentline.cloud) receives every configured user's API key as a Bearer token. Independent web searches for the company return no results outside its own recently-launched site, which lists no verifiable company history, team, or physical address, and uses an auto-generated-looking social handle. The PR originates from a freshly created fork (AgentLineHQ/nango) by a contributor with no prior presence in the repo, and the API key pattern ^sk_live_[a-zA-Z0-9_]+$ mirrors Stripe's well-known live-key format. If merged and later adopted, Nango would relay users' credentials to a domain whose operator is not independently established.

Require independent verification of AgentLine's legitimacy (business registration, team identity, industry presence, and a live, documented API at api.agentline.cloud) before merging this provider. Treat the provider config and any logo/branding as untrusted until vetted, and confirm the sk_live_ key format is genuinely AgentLine's and not borrowed from another service.

AI prompt
Check if this security scanner issue is valid. If so, understand the root cause and fix it. If appropriate, update or add tests. Keep the change focused and preserve intended behavior.

<file name="packages/providers/providers.yaml">
<violation number="1" location="packages/providers/providers.yaml:772">
<priority>P1</priority>
<title>Unverified provider may proxy user API keys to an externally-controlled domain with no independent presence</title>
<evidence>This PR adds a new "AgentLine" provider whose proxy base_url (https://api.agentline.cloud) receives every configured user's API key as a Bearer token (authorization: Bearer ${apiKey}). Independent web searches for the company return no results outside its own recently-launched site, which lists no verifiable company history, team, or physical address and uses an auto-generated-looking social handle. The PR originates from a freshly created fork (AgentLineHQ/nango) by a contributor with no prior presence in the repo, and the API key pattern '^sk_live_[a-zA-Z0-9_]+$' mirrors Stripe's well-known live-key format. If merged and later adopted, Nango would relay users' credentials to a domain whose operator is not independently established.</evidence>
<recommendation>Require independent verification of AgentLine's legitimacy (business registration, team identity, industry presence, and a live documented API at api.agentline.cloud) before merging this provider. Treat the provider config and any logo/branding as untrusted until vetted, and confirm the sk_live_ key format is genuinely AgentLine's and not borrowed from another service.</recommendation>
</violation>
</file>

@superagent-security superagent-security Bot added the pr:flagged PR flagged for review by security analysis. label Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:flagged PR flagged for review by security analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants