Skip to content

chore: prevent node and pipe name clashing for vulnerability endpoints#3940

Open
epipav wants to merge 1 commit intomainfrom
fix/vulnerability-tb-name-clash
Open

chore: prevent node and pipe name clashing for vulnerability endpoints#3940
epipav wants to merge 1 commit intomainfrom
fix/vulnerability-tb-name-clash

Conversation

@epipav
Copy link
Collaborator

@epipav epipav commented Mar 20, 2026

Note

Low Risk
Low risk rename-only change to Tinybird pipe node identifiers; main risk is breaking any callers that still reference the old node names.

Overview
Renames Tinybird vulnerability pipe NODE identifiers to use a *_result suffix (e.g., vulnerabilities_by_ecosystem_result, vulnerabilities_by_severity_result, vulnerabilities_list_result, vulnerabilities_summary_result) to prevent node/pipe naming collisions.

Query logic and filters remain unchanged; only the output node names are updated.

Written by Cursor Bugbot for commit e66ef40. This will update automatically on new commits. Configure here.

Copilot AI review requested due to automatic review settings March 20, 2026 10:32
@epipav epipav self-assigned this Mar 20, 2026
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

2 similar comments
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Renames Tinybird pipe NODEs for vulnerability endpoints to avoid name collisions between the pipe (file) name and the primary endpoint node name.

Changes:

  • Rename NODE vulnerabilities_summaryNODE vulnerabilities_summary_result
  • Rename NODE vulnerabilities_listNODE vulnerabilities_list_result
  • Rename NODE vulnerabilities_by_severityNODE vulnerabilities_by_severity_result
  • Rename NODE vulnerabilities_by_ecosystemNODE vulnerabilities_by_ecosystem_result

Reviewed changes

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

File Description
services/libs/tinybird/pipes/vulnerabilities_summary.pipe Renames the final endpoint node to *_result to avoid clashing with the pipe name.
services/libs/tinybird/pipes/vulnerabilities_list.pipe Renames the endpoint node to *_result to avoid clashing with the pipe name.
services/libs/tinybird/pipes/vulnerabilities_by_severity.pipe Renames the endpoint node to *_result to avoid clashing with the pipe name.
services/libs/tinybird/pipes/vulnerabilities_by_ecosystem.pipe Renames the endpoint node to *_result to avoid clashing with the pipe name.

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

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants