Skip to content

Conversation

@idirouhab
Copy link
Contributor

Summary

This PR adds a new List Length (LLEN) operation to the Redis node, allowing users to retrieve the length of a Redis list directly from n8n workflows.

image
  • New "List Length" operation in the Redis node operations dropdown
  • Parameter field to specify the Redis list name
  • Execution logic that calls Redis LLEN command
  • Comprehensive test coverage (3 test cases covering success and error scenarios)

How to test:

  1. Add a Redis node to a workflow
  2. Select "List Length" from the operations dropdown
  3. Enter a list name (e.g., bull:main:q)
  4. Execute the node
  5. Verify the output contains the list length: { "bull:main:q": 42 }

Example use case:

Monitor queue sizes in Bull/BullMQ workflows by checking Redis list lengths (e.g., LLEN bull:main:q).

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team node/improvement New feature or request labels Oct 31, 2025
@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 2 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="packages/nodes-base/nodes/Redis/Redis.node.ts">

<violation number="1" location="packages/nodes-base/nodes/Redis/Redis.node.ts:614">
The new LLEN branch returns a fresh object without preserving `pairedItem`, so the output item is no longer linked to its input and breaks downstream item pairing. Please include the paired reference when pushing the result.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

@idirouhab idirouhab requested review from a team and mike12345567 and removed request for a team October 31, 2025 16:23
Copy link
Contributor

@mike12345567 mike12345567 left a comment

Choose a reason for hiding this comment

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

LGTM!

@currents-bot
Copy link

currents-bot bot commented Nov 3, 2025

E2E Tests: n8n tests failed after 8m 37.8s

🟢 583 · 🔴 1 · ⚪️ 12 · 🟣 2

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Failed

  • Commit: cb8d886

  • Spec files: 96

  • Overall tests: 596

  • Duration: 8m 37.8s

  • Parallelization: 8

Failed Spec Files

Spec File Failures
tests/ui/inject-previous-execution.spec.ts 1

Groups

GroupId Results Spec Files Progress
ui 🟢 528 · 🔴 1 · ⚪️ 12 · 🟣 2 89 / 89
ui:isolated 🟢 55 · 🔴 0 · ⚪️ 0 7 / 7


This message was posted automatically by currents.dev | Integration Settings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

n8n team Authored by the n8n team node/improvement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants