Skip to content

fix: include long turbo modem preset#1003

Merged
danditomaso merged 1 commit intomeshtastic:mainfrom
fresh3nough:fix/longturbo-modem-preset
Feb 11, 2026
Merged

fix: include long turbo modem preset#1003
danditomaso merged 1 commit intomeshtastic:mainfrom
fresh3nough:fix/longturbo-modem-preset

Conversation

@fresh3nough
Copy link
Contributor

@fresh3nough fresh3nough commented Feb 10, 2026

Description

Add the Long Turbo modem preset to the LoRa modem preset list by updating protobufs and verifying availability in validation.

Related Issues

Fixes #999

Steps to Reproduce

  1. Connect to a node running firmware 2.7.17 or newer.
  2. Go to Radio Config > LoRa.
  3. Open the Modem Preset list.
  4. Observe that Long Turbo is missing.

Changes Made

  • bump @meshtastic/protobufs to 2.7.18 (includes LONG_TURBO)
  • add a lora config test that asserts LONG_TURBO is present
  • update pnpm lockfile

Testing Done

  • Pre-fix check (main): node -e "import('@meshtastic/protobufs').then(m=>console.log(Object.prototype.hasOwnProperty.call(m.Config.Config_LoRaConfig_ModemPreset,'LONG_TURBO'))).catch(err=>{console.error(err);process.exit(1);})" (result: false)
  • Post-fix check (branch): node -e "import('@meshtastic/protobufs').then(m=>console.log(Object.prototype.hasOwnProperty.call(m.Config.Config_LoRaConfig_ModemPreset,'LONG_TURBO'))).catch(err=>{console.error(err);process.exit(1);})" (result: true)
  • pnpm --filter @meshtastic/web test -- --run (fails: @meshtastic/transport-node-serial src/transport.test.ts | TransportNodeSerial > emits DeviceDisconnected when the underlying link drops)

Screenshots (if applicable)

N/A

Checklist

  • Code follows project style guidelines
  • Documentation has been updated or added
  • Tests have been added or updated
  • All i18n translation labels have been added (not applicable)

cc @philon-

Co-Authored-By: Warp agent@warp.dev

Co-Authored-By: Warp <agent@warp.dev>
@vercel
Copy link

vercel bot commented Feb 10, 2026

@fresh3nough is attempting to deploy a commit to the Meshtastic Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Collaborator

@danditomaso danditomaso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@danditomaso danditomaso merged commit ce06bc4 into meshtastic:main Feb 11, 2026
1 of 3 checks passed
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.

[Bug]: LongTurbo is missing from Modem Preset List

3 participants