Auftrag "ueberpruefe alle dateien auf \n-Falle". Die Falle: ueber ein
Shell-Heredoc geschriebener Code macht aus einem gemeinten \n einen ECHTEN
Zeilenumbruch mitten im String-Literal. Heute zweimal passiert (Telegram-Text
in engine.py, Reminder-Eintrag in measurement_reminder.py).
Der Scan prueft in beide Richtungen:
A) kompiliert alle .py (161 Dateien) — sauber
B) JS: unabgeschlossene String-Literale — app.js sauber
C) UMGEKEHRT: literales \n als sichtbarer Text in HTML-title= und Markdown
— alle sauber
D) Signatur der beiden heutigen Faelle (Zeile endet offen, naechste besteht
nur aus einem Quote) — keine
⚠ Fehlalarm im Scanner selbst gefunden und behoben: Quotes zu ZAEHLEN meldet
'"' als offenes Literal (gueltiges JS mit einem Anfuehrungszeichen darin, real
in app.js). Der Scan verfolgt jetzt zeichenweise den AKTIVEN Quote-Typ.
Die betroffene app.js-Stelle trotzdem aufgeraeumt: schliessendes
Anfuehrungszeichen jetzt als U+201C statt ueber eine Konkatenation mit '"'.
Regel in CLAUDE.md ergaenzt: mehrzeilige Strings mit \n nicht per Heredoc
schreiben, sondern ueber den Edit-Weg.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
154 lines
5.9 KiB
Python
154 lines
5.9 KiB
Python
"""Sucht die \\n-Falle in allen Projektdateien — in BEIDE Richtungen.
|
|
|
|
Die Falle: beim Schreiben ueber ein Shell-Heredoc wird aus einem gemeinten `\\n`
|
|
ein ECHTER Zeilenumbruch mitten im String-Literal. In Python bricht das den
|
|
Compile (zweimal heute passiert), in JS ebenso — aber nicht ueberall faellt es
|
|
sofort auf.
|
|
|
|
Geprueft wird:
|
|
A) Python: kompiliert jede Datei. Ein zerrissenes Literal faellt hier auf.
|
|
B) JS: Zeilen mit ungerader Zahl UNMASKIERTER Anfuehrungszeichen ausserhalb
|
|
von Kommentaren = angefangenes, nicht geschlossenes String-Literal.
|
|
C) Umgekehrter Fall: literales `\\n` als TEXT dort, wo es niemand lesen will
|
|
(HTML-Attribute, Markdown-Fliesstext) — das entsteht, wenn ein `\\\\n`
|
|
zu viel escaped wurde.
|
|
D) Verdaechtige Zeilenpaare: eine Zeile endet mit oeffnendem Quote, die
|
|
naechste besteht nur aus einem Quote — die Signatur der beiden heutigen
|
|
Faelle.
|
|
"""
|
|
import pathlib
|
|
import py_compile
|
|
import re
|
|
import sys
|
|
import tempfile
|
|
|
|
WURZEL = pathlib.Path(r"c:\Users\ah\Downloads\tradingrobot")
|
|
SKIP = {".git", "__pycache__", ".reminders", "node_modules", ".removed_backup"}
|
|
befunde = []
|
|
|
|
|
|
def dateien(*endungen):
|
|
for p in WURZEL.rglob("*"):
|
|
if any(s in p.parts for s in SKIP):
|
|
continue
|
|
if p.is_file() and p.suffix in endungen:
|
|
yield p
|
|
|
|
|
|
# ── A) Python ────────────────────────────────────────────────────────────
|
|
print("=" * 88)
|
|
print(" A) PYTHON — kompiliert jede Datei (ein zerrissenes Literal faellt hier auf)")
|
|
print("=" * 88)
|
|
npy = 0
|
|
tmp = tempfile.mkdtemp()
|
|
for p in dateien(".py"):
|
|
npy += 1
|
|
try:
|
|
py_compile.compile(str(p), cfile=f"{tmp}/x.pyc", doraise=True)
|
|
except py_compile.PyCompileError as e:
|
|
msg = str(e).strip().splitlines()[-1][:110]
|
|
befunde.append(f"PY {p.relative_to(WURZEL)}: {msg}")
|
|
print(f" FEHLER {p.relative_to(WURZEL)}")
|
|
print(f" {msg}")
|
|
print(f" {npy} Dateien geprueft.")
|
|
|
|
|
|
# ── B) JavaScript ────────────────────────────────────────────────────────
|
|
def js_offene_strings(text: str):
|
|
"""Zeilen mit angefangenem, nicht geschlossenem ' oder " (Template-Literale
|
|
duerfen mehrzeilig sein und werden uebersprungen)."""
|
|
treffer = []
|
|
in_tmpl = False
|
|
for nr, z in enumerate(text.splitlines(), 1):
|
|
# Backticks zaehlen -> Template-Literal-Zustand fortschreiben
|
|
bt = len(re.findall(r"(?<!\\)`", z))
|
|
if in_tmpl:
|
|
in_tmpl ^= bool(bt % 2)
|
|
continue
|
|
if bt % 2:
|
|
in_tmpl = True
|
|
continue
|
|
s = re.sub(r"//.*$", "", z) # Zeilenkommentar weg
|
|
# ⚠ Quotes NICHT einfach zaehlen — das erzeugt Fehlalarme: '"' ist ein
|
|
# gueltiges Literal, das EIN doppeltes Anfuehrungszeichen enthaelt (real
|
|
# in app.js aufgetreten). Stattdessen zeichenweise laufen und den jeweils
|
|
# AKTIVEN Quote-Typ verfolgen; Quotes der anderen Art zaehlen dann nicht.
|
|
aktiv, esc = None, False
|
|
for ch in s:
|
|
if esc:
|
|
esc = False
|
|
continue
|
|
if ch == "\\":
|
|
esc = True
|
|
continue
|
|
if aktiv:
|
|
if ch == aktiv:
|
|
aktiv = None
|
|
elif ch in ('"', "'"):
|
|
aktiv = ch
|
|
if aktiv:
|
|
treffer.append((nr, z.strip()[:100]))
|
|
return treffer
|
|
|
|
|
|
print()
|
|
print("=" * 88)
|
|
print(" B) JAVASCRIPT — unabgeschlossene String-Literale")
|
|
print("=" * 88)
|
|
for p in dateien(".js"):
|
|
if "lightweight-charts" in p.name:
|
|
continue
|
|
tr = js_offene_strings(p.read_text(encoding="utf-8", errors="replace"))
|
|
if tr:
|
|
for nr, z in tr:
|
|
befunde.append(f"JS {p.relative_to(WURZEL)}:{nr}")
|
|
print(f" FEHLER {p.relative_to(WURZEL)}:{nr} {z}")
|
|
else:
|
|
print(f" OK {p.relative_to(WURZEL)}")
|
|
|
|
|
|
# ── C) literales \n als TEXT ─────────────────────────────────────────────
|
|
print()
|
|
print("=" * 88)
|
|
print(" C) UMGEKEHRT — literales \\n im sichtbaren Text (HTML-Attribute, Markdown)")
|
|
print("=" * 88)
|
|
for p in list(dateien(".html")) + list(dateien(".md")):
|
|
t = p.read_text(encoding="utf-8", errors="replace")
|
|
roh = []
|
|
for m in re.finditer(r'title="([^"]*)"', t):
|
|
if "\\n" in m.group(1):
|
|
roh.append(f"title=...{m.group(1)[:60]}")
|
|
if p.suffix == ".md":
|
|
for nr, z in enumerate(t.splitlines(), 1):
|
|
if "\\n" in z and "`" not in z and "\\\\n" not in z:
|
|
roh.append(f"Zeile {nr}: {z.strip()[:80]}")
|
|
if roh:
|
|
for r in roh[:5]:
|
|
befunde.append(f"TXT {p.relative_to(WURZEL)}: {r}")
|
|
print(f" FEHLER {p.relative_to(WURZEL)} {r}")
|
|
else:
|
|
print(f" OK {p.relative_to(WURZEL)}")
|
|
|
|
|
|
# ── D) Signatur der heutigen Faelle ──────────────────────────────────────
|
|
print()
|
|
print("=" * 88)
|
|
print(' D) SIGNATUR — Zeile endet offen, naechste besteht nur aus einem Quote')
|
|
print("=" * 88)
|
|
sig = 0
|
|
for p in list(dateien(".py")) + list(dateien(".js")):
|
|
zeilen = p.read_text(encoding="utf-8", errors="replace").splitlines()
|
|
for i in range(len(zeilen) - 1):
|
|
if zeilen[i + 1].strip() in ('"', "'", '",', "',"):
|
|
sig += 1
|
|
befunde.append(f"SIG {p.relative_to(WURZEL)}:{i+2}")
|
|
print(f" VERDAECHTIG {p.relative_to(WURZEL)}:{i+2}")
|
|
print(" keine gefunden." if not sig else f" {sig} Stellen.")
|
|
|
|
print()
|
|
print("=" * 88)
|
|
print(" ERGEBNIS: " + ("keine \\n-Falle gefunden" if not befunde
|
|
else f"{len(befunde)} Befund(e)"))
|
|
print("=" * 88)
|
|
sys.exit(1 if befunde else 0)
|