Skip to content

Flash sizing#81

Draft
netmindz wants to merge 3 commits intomasterfrom
flash-sizing
Draft

Flash sizing#81
netmindz wants to merge 3 commits intomasterfrom
flash-sizing

Conversation

@netmindz
Copy link
Copy Markdown
Member

@netmindz netmindz commented May 4, 2026

This pull request introduces several updates to improve device compatibility and simplify the user interface for the WLED web installer. The most significant changes include adding support for multiple ESP32-S3 flash sizes, updating the installer button to use a forked version of ESP Web Tools, and streamlining the release selection dropdown for a better user experience.

Device compatibility improvements:

  • Added explicit support for ESP32-S3 devices with 4MB, 8MB, and 16MB flash sizes in releases.js, including appropriate boot partition files and firmware naming conventions for each variant. This enables automatic detection and correct firmware selection for different S3 hardware. [1] [2]
  • Modified the build generation logic in releases.js to include the flashSizeMB property for ESP32-S3 variants, ensuring the installer can select the correct build based on detected flash size.

Installer and UI updates:

  • Switched the ESP Web Tools script in index.htm to use the tasmota-esp-web-tools fork, which supports the new S3 flash size detection and selection logic.
  • Simplified the firmware version selection dropdown in index.htm to display a loading message, likely in preparation for dynamic population of releases.

Cleanup:

  • Removed the obsolete auto.htm file, which previously provided a maintenance overlay and an alternative installer link.

- Replace esp-web-tools@10.1.0 with tasmota-esp-web-tools@12.2.1 (Jason2866's
  fork), which detects device flash size at runtime and selects the best
  matching firmware build via the flashSizeMB manifest field
- Split the single ESP32-S3 CHIP_CONFIG entry into three (4MB/8MB/16MB), each
  pointing to the correct partition table binary
- Add VARIANTS entries for _ESP32-S3_4M_qspi and _ESP32-S3_16MB_opi alongside
  the existing _ESP32-S3_8MB_opi; generateManifest now emits flashSizeMB on
  every S3 build object
- Add partitions_s3_4m.bin and partitions_s3_16m.bin (compiled from WLED's
  WLED_ESP32_4MB_1MB_FS.csv and WLED_ESP32_16MB_9MB_FS.csv)
- Remove all static manifest files from bin/Release, bin/Beta and bin/nightly;
  the dropdown is now fully populated from the GitHub Releases API
@netmindz netmindz requested a review from lost-hope May 4, 2026 08:31
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 4, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 57a87454-6326-4c0c-a131-f99fb569974a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch flash-sizing

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@netmindz
Copy link
Copy Markdown
Member Author

netmindz commented May 4, 2026

I'm unsure exactly which combinations we need to test, however if what I do know is that the current installer just gives every S3 the 8MB image, which means the S3 zero will fail to boot, that needs the 4MB

I can see that if I connect my 16MB it tries to give me the 16MB image, I haven't done a full install as the corsfix thing prevents the actual install

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