[PR #45] [MERGED] Fix NameError in dataflow validation error logging #49

New issue

Closed

opened 2026-03-02 04:08:01 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 04:08:01 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/gadievron/raptor/pull/45
Author: @gadievron
Created: 12/22/2025
Status: ✅ Merged
Merged: 12/26/2025
Merged by: @danielcuthbert

Base: main ← Head: fix/bug-39-dataflow-nameerror

📝 Commits (1)

c18bbe7 Fix NameError in dataflow validation error logging

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 packages/codeql/dataflow_validator.py (+1 -1)

📄 Description

Fixes #39

Summary

Fixes undefined variable reference in dataflow validator error handling.

Changes

File: packages/codeql/dataflow_validator.py:213
Fix: sink.label → dataflow.sink.label

Impact

Prevents secondary NameError when logging validation failures
Low risk, single line change

Note

Fixes a crash when building the dataflow path summary by referencing the sink label correctly.

In packages/codeql/dataflow_validator.py, changes SINK: summary line from sink.label to dataflow.sink.label to prevent a NameError during validation logging

^{Written by Cursor Bugbot for commit c18bbe7184. This will update automatically on new commits. Configure here.}

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/gadievron/raptor/pull/45 **Author:** [@gadievron](https://github.com/gadievron) **Created:** 12/22/2025 **Status:** ✅ Merged **Merged:** 12/26/2025 **Merged by:** [@danielcuthbert](https://github.com/danielcuthbert) **Base:** `main` ← **Head:** `fix/bug-39-dataflow-nameerror` --- ### 📝 Commits (1) - [`c18bbe7`](https://github.com/gadievron/raptor/commit/c18bbe718415632bfa9d8e9004c8fcd9d92e959b) Fix NameError in dataflow validation error logging ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/codeql/dataflow_validator.py` (+1 -1) </details> ### 📄 Description Fixes #39 ## Summary Fixes undefined variable reference in dataflow validator error handling. ## Changes - **File:** packages/codeql/dataflow_validator.py:213 - **Fix:** `sink.label` → `dataflow.sink.label` ## Impact - Prevents secondary NameError when logging validation failures - Low risk, single line change  --- > [!NOTE] > Fixes a crash when building the dataflow path summary by referencing the sink label correctly. > > - In `packages/codeql/dataflow_validator.py`, changes `SINK:` summary line from `sink.label` to `dataflow.sink.label` to prevent a `NameError` during validation logging > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit c18bbe718415632bfa9d8e9004c8fcd9d92e959b. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup>  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>