Poll files node: Add deleted files output and configurable 'all results' output #27
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
Add a new field to the "All results" output payload that reports the poll interval duration in seconds, alongside the existing nextPoll timestamp. This makes it easier for downstream nodes to understand the polling schedule. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Make it a config option to include it again. Hoping to avoid people connecting it to stuff without reading the docs.
🐳 Docker PR Build
One or more Docker image builds failed. Check the workflow logs for details. |
Tests were written for upstream's 2-output structure (All, New) but the current implementation uses outputAllPolls to control outputs: - Default: 2 outputs (New, Deleted) - outputAllPolls=true: 3 outputs (All, New, Deleted) Updated 7 tests to use outputAllPolls: true and 3-output wire config. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Enhances the Poll files node with improved output configuration and deleted file detection:
pollIntervalSecondsfield to output payload alongsidenextPollfor easier interval trackingDetails
New
pollIntervalSecondsfieldThe "All results" output now includes
msg.payload.pollIntervalSeconds(in seconds) alongside the existingmsg.payload.nextPolltimestamp, making it easier for downstream nodes to understand the polling schedule.Configurable "All results" output
The output that fires on every poll is now disabled by default. Most users only need the "New results" output, so this reduces clutter. Enable via the new "Output all results on every poll" checkbox.
Deleted files detection
New output that fires when files are removed from the Data Link between polls. This enables workflows that need to react to file deletions (e.g., cleanup tasks, audit logging).
Output configuration:
Test plan
pollIntervalSecondsis present in the "All results" payload when enabled🤖 Generated with Claude Code