99 lines
3.6 KiB
Markdown
99 lines
3.6 KiB
Markdown
# ToDo 8 — Bekannte Bugs
|
||
|
||
## Ziel
|
||
|
||
Konkrete, im Code identifizierte Fehler beheben — unabhängig von den Architektur-Refactorings in den anderen ToDo-Dateien.
|
||
|
||
---
|
||
|
||
## Bug 1: `TelnetSenderGRBL` — `close`-Event verliert `this`-Kontext ✅ ERLEDIGT
|
||
|
||
> Behoben im ToDo-2-Refactoring: Der `close`-Handler nutzt jetzt eine Arrow-Function
|
||
> (`robot/TelnetSenderGRBL.js`), `this` zeigt korrekt auf die Sender-Instanz.
|
||
|
||
**Datei:** `robot/TelnetSenderGRBL.js`, Zeile 54–57
|
||
|
||
**Problem:** Das `close`-Event verwendet eine reguläre `function()` statt einer Arrow Function. Dadurch zeigt `this` innerhalb des Handlers auf das EventEmitter-Objekt, nicht auf die `TelnetSenderGRBL`-Instanz. `this.tSocket = null` hat keinen Effekt — nach einer Verbindungstrennung bleibt `tSocket` auf dem alten, ungültigen Objekt.
|
||
|
||
```js
|
||
// Falsch:
|
||
this.tSocket.on("close", function () {
|
||
this.tSocket = null; // 'this' ist hier NICHT TelnetSenderGRBL
|
||
});
|
||
|
||
// Richtig:
|
||
this.tSocket.on("close", () => {
|
||
this.tSocket = null;
|
||
});
|
||
```
|
||
|
||
---
|
||
|
||
## Bug 2: `FFirst` und `FLast` sind nicht implementiert
|
||
|
||
**Datei:** `robot/GCode.js`
|
||
|
||
**Problem:** `ContainsFilesCommand()` erkennt `FFirst` und `FLast` und leitet sie an `receiveFC()` weiter. `receiveFC()` behandelt sie aber nicht — die Befehle werden stillschweigend ignoriert und es wird nur `getM114` zurückgegeben.
|
||
|
||
**Erwartetes Verhalten:**
|
||
- `FFirst` — Cursor auf den ersten Eintrag der Log-Datei setzen und die Position anfahren
|
||
- `FLast` — Cursor auf den letzten Eintrag setzen und die Position anfahren
|
||
|
||
---
|
||
|
||
## Bug 3: G92/M92-Mismatch
|
||
|
||
**Datei:** `robot/GCode.js`
|
||
|
||
**Problem:** `containsCommand()` erkennt `G92`, aber `receiveGCode()` prüft auf `g[0] == "M92"`. Ein eingehender Befehl `G92 X10` wird als G-Code erkannt, fällt dann aber durch alle Bedingungen in `receiveGCode()`, und löst unbeabsichtigt `calculateAngles3D()` + `sendCommand()` aus, ohne die Position zu setzen.
|
||
|
||
**Klärungsbedarf:** Ist G92 oder M92 der korrekte Eingabe-Befehl? Beides konsistent machen.
|
||
|
||
---
|
||
|
||
## Bug 4: `logs/`-Verzeichnis wird nicht sichergestellt ✅ ERLEDIGT
|
||
|
||
> Behoben: `initInputWS()` ruft `ensureLogDir()` (`fs.mkdirSync('./logs', { recursive: true })`)
|
||
> beim Start auf. `ensureLogDir` ist exportiert und idempotent.
|
||
> Test: `test/InputWS.logDir.test.js`.
|
||
|
||
**Datei:** `server/InputWS.js`, Zeilen 66–67 und 77–78
|
||
|
||
**Problem:** `fs.appendFileSync('./logs/gcode_commands.log', ...)` und `fs.appendFileSync('./logs/pings.log', ...)` crashen beim ersten Aufruf, wenn das `logs/`-Verzeichnis nicht existiert.
|
||
|
||
**Fix:** Beim Start `fs.mkdirSync('./logs', { recursive: true })` aufrufen, z. B. in `startRobot.js` oder am Anfang von `initInputWS`.
|
||
|
||
---
|
||
|
||
## Bug 5: Falscher Finitude-Check in `TelnetSenderGRBL.execCommand`
|
||
|
||
**Datei:** `robot/TelnetSenderGRBL.js`, Zeile 161
|
||
|
||
**Problem:**
|
||
```js
|
||
if(this.aAxisGrbl == "x" && mNew.xMotorChanged && Number.isFinite(mNew.y)){
|
||
```
|
||
Der Check prüft `mNew.y` statt `mNew.x`. Wenn `mNew.x` `NaN` oder `Infinity` wäre, würde das trotzdem durchgehen.
|
||
|
||
---
|
||
|
||
## Bug 6: `containsMCode` matcht zu breit ✅ ERLEDIGT
|
||
|
||
> Behoben: `containsMCode` nutzt jetzt `s === 'M1' || s.startsWith('M1 ')`.
|
||
> Test: `test/GCode.containsMCode.test.js`.
|
||
> (Hinweis bleibt: Methode wird im Produktivcode noch nicht aufgerufen.)
|
||
|
||
**Datei:** `robot/GCode.js`, Zeile 12
|
||
|
||
**Problem:** `s.indexOf('M1') == 0` trifft auch auf `M10`, `M11`, `M12` usw. zu.
|
||
|
||
```js
|
||
// Aktuell:
|
||
static containsMCode(s){ return s.indexOf('M1') == 0 }
|
||
|
||
// Präziser:
|
||
static containsMCode(s){ return s === 'M1' || s.startsWith('M1 ') }
|
||
```
|
||
|
||
Hinweis: Diese Methode wird im aktuellen Code nicht aufgerufen — sie hat keine Wirkung, ist aber irreführend.
|