Skip to content

GrblHAL compatibility#116

Open
bullestock wants to merge 2 commits into
LaserWeb:masterfrom
Hal9k-dk:grblhal-compat
Open

GrblHAL compatibility#116
bullestock wants to merge 2 commits into
LaserWeb:masterfrom
Hal9k-dk:grblhal-compat

Conversation

@bullestock

Copy link
Copy Markdown

grblHAL prints "grblHAL 1.1" by default, whereas standard grbl prints "grbl 1.1".

This PR adds compatibility with grblHAL by locating the space before parsing the version number.

colintrachte added a commit to colintrachte/lw.comm-server that referenced this pull request Apr 1, 2026
colintrachte added a commit to colintrachte/lw.comm-server that referenced this pull request Jun 20, 2026
…atibility

GrblHAL version detection (PR LaserWeb#116):
- Fix Grbl version parsing in Telnet and ESP paths to use dynamic
  space-finding instead of hardcoded substr(5,4), matching the already-
  fixed USB path. GrblHAL sends "GrblHAL 1.1" where the version starts
  at a different offset than plain "Grbl 1.1h".

GRBL 0.x framing format (PR LaserWeb#118):
- Strip {count/total} framing prefix that GRBL 0.8 prepends to responses
  before further parsing, preventing those lines from being mishandled.

GRBL 0.x feature flags (PR LaserWeb#118):
- firmwareFeatures.get() now accepts an optional version parameter.
  GRBL 0.x versions return feedOverride:false and spindleOverride:false
  since those real-time override commands did not exist in that era.
- Pass fVersion through the getFeatureList socket handler.

Node 18 / Docker (PR LaserWeb#117):
- Replace end-of-life resin/raspberrypi3-node:10 base image with node:18.
- Tighten package.json engines field to >=18 (minimum with long-term support).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant