Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Normalize TaskPath -> Path in typed process outputs#7090

Merged
bentsherman merged 3 commits into
nextflow-io:masterfrom
stekaz:outputs
May 14, 2026
Merged

Normalize TaskPath -> Path in typed process outputs#7090
bentsherman merged 3 commits into
nextflow-io:masterfrom
stekaz:outputs

Conversation

@stekaz

@stekaz stekaz commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Fixes #7032

This change fixes the output-side problem where a process can emit a record containing TaskPath values. These TaskPath values are only meaningful inside the upstream task, so downstream processes can fail during hashing/staging or see invalid relative paths. The fix is to normalize the process output values before binding them to output channels.

#7083 makes nested record paths become TaskPaths when entering a task
This PR makes nested TaskPath values become durable paths when leaving a task

@netlify

netlify Bot commented Apr 30, 2026

Copy link
Copy Markdown

Deploy Preview for nextflow-docs-staging canceled.

Name Link
🔨 Latest commit ec6307b
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/6a06109214052e00087df5c9

@stekaz stekaz marked this pull request as ready for review May 1, 2026 01:23
stekaz and others added 2 commits May 14, 2026 12:29
Signed-off-by: Ben Sherman <[email protected]>
@bentsherman bentsherman changed the title Prevent TaskPath leakage from typed record outputs Prevent TaskPath leakage from typed process outputs May 14, 2026
@bentsherman bentsherman changed the title Prevent TaskPath leakage from typed process outputs Normalize TaskPath -> Path in typed process outputs May 14, 2026
Signed-off-by: Ben Sherman <[email protected]>
@bentsherman bentsherman merged commit 38c0723 into nextflow-io:master May 14, 2026
23 checks passed
@stekaz stekaz deleted the outputs branch May 15, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unused files in process inputs are of nextflow.processor.TaskPath type which makes invalid input paths for other processes

2 participants