fix: escape quotes in DOT renderer node/edge strings (closes #212) #213

Closed
Rockachopa wants to merge 2 commits from fix/212-dependency-graph-quoting into main
Owner

Problem

Lines 152-153 in scripts/dependency_graph.py used unescaped double quotes inside Python double-quoted string literals, causing a SyntaxError:

# BROKEN — quotes close the Python string prematurely
lines.append("  node [shape=box, style=filled, fillcolor="#1a1a2e", fontcolor="#e6edf3"];")
lines.append("  edge [color="#4a4a6a"];")

Fix

Switch to single-quoted Python strings with escaped inner double quotes:

lines.append('  node [shape=box, style=filled, fillcolor="#1a1a2e", fontcolor="#e6edf3"];')
lines.append('  edge [color="#4a4a6a"];')

Verification

python3 -m py_compile scripts/dependency_graph.py — passes cleanly.

Closes #212

## Problem Lines 152-153 in `scripts/dependency_graph.py` used unescaped double quotes inside Python double-quoted string literals, causing a SyntaxError: ```python # BROKEN — quotes close the Python string prematurely lines.append(" node [shape=box, style=filled, fillcolor="#1a1a2e", fontcolor="#e6edf3"];") lines.append(" edge [color="#4a4a6a"];") ``` ## Fix Switch to single-quoted Python strings with escaped inner double quotes: ```python lines.append(' node [shape=box, style=filled, fillcolor="#1a1a2e", fontcolor="#e6edf3"];') lines.append(' edge [color="#4a4a6a"];') ``` ## Verification `python3 -m py_compile scripts/dependency_graph.py` — passes cleanly. Closes #212
Rockachopa added 1 commit 2026-04-21 11:15:01 +00:00
fix: escape quotes in DOT renderer strings (#212)
Some checks failed
Test / pytest (pull_request) Failing after 25s
39905d92aa
Rockachopa added 1 commit 2026-04-21 11:15:02 +00:00
fix: escape quotes in DOT renderer strings (#212)
Some checks failed
Test / pytest (pull_request) Failing after 25s
39905d92aa
Rockachopa added 1 commit 2026-04-21 11:18:16 +00:00
fix: use single quotes in DOT renderer (#212)
Some checks failed
Test / pytest (pull_request) Failing after 52s
c28999f270
Author
Owner

Superseded by burn/212-fix-dot-quoting.

Superseded by burn/212-fix-dot-quoting.
Rockachopa closed this pull request 2026-04-21 11:20:31 +00:00
Some checks failed
Test / pytest (pull_request) Failing after 52s

Pull request closed

Sign in to join this conversation.