Feature/quote message n8n #2238
Open
+18
−18
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR ensures that when handling WhatsApp message replies, the original quoted message context is propagated through the N8N integration by extracting and including the quotedMessage field in the webhook payload, while also standardizing code style in the proxy agent utility. Sequence diagram for WhatsApp reply message payload to N8N webhooksequenceDiagram
participant User as actor User
participant WhatsApp
participant "Chatbot Service"
participant N8N
User->>WhatsApp: Reply to a message
WhatsApp->>"Chatbot Service": Send message with contextInfo.quotedMessage
"Chatbot Service"->>N8N: Send webhook payload (includes quotedMessage)
N8N->>"Chatbot Service": Process payload (now with quotedMessage)
Class diagram for updated N8nService payload structureclassDiagram
class N8nService {
+sendWebhookPayload(msg, instance)
}
class Message {
+key
+contextInfo
}
class ContextInfo {
+quotedMessage
}
N8nService --> Message : uses
Message --> ContextInfo : has
ContextInfo : quotedMessage
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Hey there - I've reviewed your changes and they look great!
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location> `src/utils/makeProxyAgent.ts:63-64` </location>
<code_context>
- const auth = username && password ? `${username}:${password}@` : ''
- proxyUrl = `${protocol}://${auth}${host}:${port}`
+ const auth = username && password ? `${username}:${password}@` : '';
+ proxyUrl = `${protocol}://${auth}${host}:${port}`;
}
</code_context>
<issue_to_address>
**suggestion:** Authentication string construction may not handle special characters in username or password.
Encoding the username and password will prevent malformed URLs when special characters are present.
```suggestion
const encodedUsername = username ? encodeURIComponent(username) : '';
const encodedPassword = password ? encodeURIComponent(password) : '';
const auth = encodedUsername && encodedPassword ? `${encodedUsername}:${encodedPassword}@` : '';
proxyUrl = `${protocol}://${auth}${host}:${port}`;
```
</issue_to_address>
### Comment 2
<location> `src/utils/makeProxyAgent.ts:79-80` </location>
<code_context>
default:
- throw new Error(`Unsupported proxy protocol: ${protocol}`)
+ throw new Error(`Unsupported proxy protocol: ${protocol}`);
}
}
</code_context>
<issue_to_address>
**suggestion:** Error message could include the original input for better debugging.
Adding the full proxy input to the error message will make it easier to diagnose issues with malformed or unexpected input.
```suggestion
default:
throw new Error(`Unsupported proxy protocol: ${protocol}. Full proxy input: ${typeof proxy === 'string' ? proxy : JSON.stringify(proxy)}`);
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Comment on lines
+63
to
+64
| const auth = username && password ? `${username}:${password}@` : ''; | ||
| proxyUrl = `${protocol}://${auth}${host}:${port}`; |
Contributor
There was a problem hiding this comment.
suggestion: Authentication string construction may not handle special characters in username or password.
Encoding the username and password will prevent malformed URLs when special characters are present.
Suggested change
| const auth = username && password ? `${username}:${password}@` : ''; | |
| proxyUrl = `${protocol}://${auth}${host}:${port}`; | |
| const encodedUsername = username ? encodeURIComponent(username) : ''; | |
| const encodedPassword = password ? encodeURIComponent(password) : ''; | |
| const auth = encodedUsername && encodedPassword ? `${encodedUsername}:${encodedPassword}@` : ''; | |
| proxyUrl = `${protocol}://${auth}${host}:${port}`; |
|
|
||
| default: | ||
| throw new Error(`Unsupported proxy protocol: ${protocol}`) | ||
| throw new Error(`Unsupported proxy protocol: ${protocol}`); |
Contributor
There was a problem hiding this comment.
suggestion: Error message could include the original input for better debugging.
Adding the full proxy input to the error message will make it easier to diagnose issues with malformed or unexpected input.
Suggested change
| default: | |
| throw new Error(`Unsupported proxy protocol: ${protocol}`) | |
| throw new Error(`Unsupported proxy protocol: ${protocol}`); | |
| default: | |
| throw new Error(`Unsupported proxy protocol: ${protocol}. Full proxy input: ${typeof proxy === 'string' ? proxy : JSON.stringify(proxy)}`); |
Copilot finished reviewing on behalf of
jamesjhonatan123
November 19, 2025 03:07
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for quoted message context in N8N webhook payloads. When a user replies to a message in WhatsApp, the N8N workflow now receives the quotedMessage field containing the original message content, aligning the N8N integration with standard webhook behavior.
- Adds
quotedMessagefield to N8N webhook payload extracted frommsg.contextInfo.quotedMessage - Includes code formatting improvements (semicolon additions) in makeProxyAgent.ts
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/utils/makeProxyAgent.ts | Code formatting improvements - added missing semicolons throughout the file for consistency |
| src/api/integrations/chatbot/n8n/services/n8n.service.ts | Added quotedMessage field to the N8N webhook payload to provide reply context |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| } else { | ||
| const { host, password, port, protocol: proto, username } = proxy | ||
| protocol = (proto || 'http').replace(':', '') | ||
| const { host, password, port, protocol: proto, username } = proxy; |
Check failure
Code scanning / CodeQL
Insecure randomness High
| } else { | ||
| const { host, password, port, protocol: proto, username } = proxy | ||
| protocol = (proto || 'http').replace(':', '') | ||
| const { host, password, port, protocol: proto, username } = proxy; |
Check failure
Code scanning / CodeQL
Insecure randomness High
| } else { | ||
| const { host, password, port, protocol: proto, username } = proxy | ||
| protocol = (proto || 'http').replace(':', '') | ||
| const { host, password, port, protocol: proto, username } = proxy; |
Check failure
Code scanning / CodeQL
Insecure randomness High
| } else { | ||
| const { host, password, port, protocol: proto, username } = proxy | ||
| protocol = (proto || 'http').replace(':', '') | ||
| const { host, password, port, protocol: proto, username } = proxy; |
Check failure
Code scanning / CodeQL
Insecure randomness High
| } else { | ||
| const { host, password, port, protocol: proto, username } = proxy | ||
| protocol = (proto || 'http').replace(':', '') | ||
| const { host, password, port, protocol: proto, username } = proxy; |
Check failure
Code scanning / CodeQL
Insecure randomness High
Collaborator
|
fix the lint please, use |
Author
|
I ran npm run lint and fixed all lint issues — that’s why the changes above appeared. In that specific section: 
The project’s ESLint config has "semi": true, so semicolons are required. 
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📋 Description
This PR addresses an issue where the quotedMessage information was missing from the payload sent to N8N webhooks. When a user replied to a message, the N8N workflow received the new message content but lacked the context of the quoted message, unlike the standard webhook behavior.
This change adds the quotedMessage field to the N8N payload, extracting it from msg.contextInfo.quotedMessage, ensuring that workflows can properly handle replies and context.
🧪 Type of Change
✨ New feature (non-breaking change which adds functionality)
Manual testing completed
Functionality verified in development environment and production environment
No breaking changes introduced
Tested with different connection types (if applicable)
Testing Details:
Verified that when a message is replied to in WhatsApp, the N8N webhook now receives the quotedMessage object containing the original message content.
📸 Screenshots (if applicable)
✅ Checklist
My code follows the project's style guidelines
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have manually tested my changes thoroughly
I have verified the changes work with different scenarios
Any dependent changes have been merged and published
📝 Additional Notes
This aligns the N8N integration payload more closely with the standard webhook payload regarding quoted messages.
Summary by Sourcery
Enable N8N workflows to receive quoted message context when relaying WhatsApp replies and clean up code formatting in the proxy agent utility.
New Features:
Enhancements: