Fixed type from bool to uint8

https://github.com/open-x4-epaper/community-sdk/pull/6 fixes a power down issue with the display which was causing streaks to appear on the sleep screen

.github/FUNDING.yml

@@ -0,0 +1,2 @@
+github: [daveallie]
+ko_fi: daveallie

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -0,0 +1,54 @@
+name: Bug Report
+description: Report an issue or unexpected behavior
+title: "Short, descriptive title of the issue"
+labels: ["bug", "triage"]
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thanks for taking the time to report this bug! Please fill out the details below.
+
+  - type: input
+    id: version
+    attributes:
+      label: Affected Version
+      description: What version of the project/library are you using? (e.g., v1.2.3, master branch commit SHA)
+      placeholder: Ex. v1.2.3
+    validations:
+      required: true
+
+  - type: textarea
+    id: bug-description
+    attributes:
+      label: Describe the Bug
+      description: A clear and concise description of what the bug is.
+      placeholder:
+    validations:
+      required: true
+
+  - type: textarea
+    id: steps-to-reproduce
+    attributes:
+      label: Steps to Reproduce
+      description: Clearly list the steps necessary to reproduce the unexpected behavior.
+      placeholder: |
+        1. Go to '...'
+        2. Select '...'
+        3. Crash
+    validations:
+      required: true
+
+  - type: textarea
+    id: expected-behavior
+    attributes:
+      label: Expected Behavior
+      description: A clear and concise description of what you expected to happen.
+    validations:
+      required: true
+
+  - type: textarea
+    id: logs
+    attributes:
+      label: Relevant Log Output/Screenshots
+      description: If applicable, error messages, or log output to help explain your problem. You can drag and drop images here.
+      render: shell

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,9 @@
+## Summary
+
+* **What is the goal of this PR?** (e.g., Fixes a bug in the user authentication module, Implements the new feature for 
+  file uploading.)
+* **What changes are included?**
+
+## Additional Context
+
+* Add any other information that might be helpful for the reviewer (e.g., performance implications, potential risks, specific areas to focus on).

.github/workflows/ci.yml

@@ -0,0 +1,40 @@
+name: CI
+on: push
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          submodules: recursive
+      - uses: actions/cache@v5
+        with:
+          path: |
+            ~/.cache/pip
+            ~/.platformio/.cache
+          key: ${{ runner.os }}-pio
+      - uses: actions/setup-python@v6
+        with:
+          python-version: '3.14'
+
+      - name: Install PlatformIO Core
+        run: pip install --upgrade platformio
+
+      - name: Install clang-format-21
+        run: |
+          wget https://apt.llvm.org/llvm.sh
+          chmod +x llvm.sh
+          sudo ./llvm.sh 21
+          sudo apt-get update
+          sudo apt-get install -y clang-format-21
+
+      - name: Run cppcheck
+        run: pio check --fail-on-defect medium --fail-on-defect high
+
+      - name: Run clang-format
+        run: PATH="/usr/lib/llvm-21/bin:$PATH" ./bin/clang-format-fix && git diff --exit-code || (echo "Please run 'bin/clang-format-fix' to fix formatting issues" && exit 1)
+
+      - name: Build CrossPoint
+        run: pio run

.github/workflows/release.yml

@@ -0,0 +1,40 @@
+name: Compile Release
+on:
+  push:
+    tags:
+      - '*'
+
+jobs:
+  build-release:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          submodules: recursive
+      - uses: actions/cache@v5
+        with:
+          path: |
+            ~/.cache/pip
+            ~/.platformio/.cache
+          key: ${{ runner.os }}-pio
+      - uses: actions/setup-python@v6
+        with:
+          python-version: '3.14'
+
+      - name: Install PlatformIO Core
+        run: pip install --upgrade platformio
+
+      - name: Build CrossPoint
+        run: pio run -e gh_release
+
+      - name: Upload Artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: CrossPoint-${{ github.ref_name }}
+          path: |
+            .pio/build/gh_release/bootloader.bin
+            .pio/build/gh_release/firmware.bin
+            .pio/build/gh_release/firmware.elf
+            .pio/build/gh_release/firmware.map
+            .pio/build/gh_release/partitions.bin

lib/Epub/Epub/Section.cpp

@@ -9,7 +9,7 @@
 #include "Page.h"
 #include "parsers/ChapterHtmlSlimParser.h"
 
-constexpr uint8_t SECTION_FILE_VERSION = 4;
+constexpr uint8_t SECTION_FILE_VERSION = 5;
 
 void Section::onPageComplete(std::unique_ptr<Page> page) {
   const auto filePath = cachePath + "/page_" + std::to_string(pageCount) + ".bin";

open-x4-sdk

@@ -1 +1 @@
-Subproject commit 7e0dce916706da7d80ec225fade191aea6b87fb6
+Subproject commit 4d0dcd5ff87fcd86eb2966a123e85b03284a03db

platformio.ini

@@ -8,6 +8,9 @@ board = esp32-c3-devkitm-1
 framework = arduino
 monitor_speed = 115200
 upload_speed = 921600
+check_tool = cppcheck
+check_skip_packages = yes
+check_severity = medium, high
 
 board_upload.flash_size = 16MB
 board_upload.maximum_size = 16777216

src/CrossPointSettings.cpp

@@ -4,24 +4,27 @@
 #include <SD.h>
 #include <Serialization.h>
 
+#include <cstdint>
 #include <fstream>
 
 // Initialize the static instance
 CrossPointSettings CrossPointSettings::instance;
 
 constexpr uint8_t SETTINGS_FILE_VERSION = 1;
+constexpr uint8_t SETTINGS_COUNT = 2;
 constexpr char SETTINGS_FILE[] = "/sd/.crosspoint/settings.bin";
 
 bool CrossPointSettings::saveToFile() const {
   // Make sure the directory exists
   SD.mkdir("/.crosspoint");
-  
+
   std::ofstream outputFile(SETTINGS_FILE);
   serialization::writePod(outputFile, SETTINGS_FILE_VERSION);
+  serialization::writePod(outputFile, SETTINGS_COUNT);
   serialization::writePod(outputFile, whiteSleepScreen);
   serialization::writePod(outputFile, extraParagraphSpacing);
   outputFile.close();
-  
+
   Serial.printf("[%lu] [CPS] Settings saved to file\n", millis());
   return true;
 }
@@ -42,10 +45,21 @@ bool CrossPointSettings::loadFromFile() {
     return false;
   }
 
-  serialization::readPod(inputFile, whiteSleepScreen);
-  serialization::readPod(inputFile, extraParagraphSpacing);
+  uint8_t fileSettingsCount = 0;
+  serialization::readPod(inputFile, fileSettingsCount);
+
+  // load settings that exist
+  switch (fileSettingsCount) {
+      case 1:
+          serialization::readPod(inputFile, whiteSleepScreen);
+          break;
+      case 2:
+          serialization::readPod(inputFile, whiteSleepScreen);
+          serialization::readPod(inputFile, extraParagraphSpacing);
+          break;
+  }
 
   inputFile.close();
   Serial.printf("[%lu] [CPS] Settings loaded from file\n", millis());
   return true;
-}
+}

src/CrossPointSettings.h

@@ -1,25 +1,26 @@
 #pragma once
+#include <cstdint>
 #include <iosfwd>
 
 class CrossPointSettings {
  private:
   // Private constructor for singleton
   CrossPointSettings() = default;
-  
+
   // Static instance
   static CrossPointSettings instance;
-  
+
  public:
   // Delete copy constructor and assignment
   CrossPointSettings(const CrossPointSettings&) = delete;
   CrossPointSettings& operator=(const CrossPointSettings&) = delete;
-  
+
   // Sleep screen settings
-  bool whiteSleepScreen = false;
-  
+  uint8_t whiteSleepScreen = 0;
+
   // Text rendering settings
-  bool extraParagraphSpacing = false;
-  
+  uint8_t extraParagraphSpacing = 1;
+
   ~CrossPointSettings() = default;
 
   // Get singleton instance
@@ -32,4 +33,4 @@ class CrossPointSettings {
 };
 
 // Helper macro to access settings
-#define SETTINGS CrossPointSettings::getInstance()
+#define SETTINGS CrossPointSettings::getInstance()

src/main.cpp

@@ -133,7 +133,6 @@ void waitForPowerRelease() {
 // Enter deep sleep mode
 void enterDeepSleep() {
   exitScreen();
-  SETTINGS.saveToFile();
   enterNewScreen(new SleepScreen(renderer, inputManager));
 
   Serial.printf("[%lu] [   ] Power button released after a long press. Entering deep sleep.\n", millis());

src/screens/SettingsScreen.cpp

@@ -6,6 +6,7 @@
 #include "config.h"
 
 // Define the static settings list
+
 const SettingInfo SettingsScreen::settingsList[SettingsScreen::settingsCount] = {
   {"White Sleep Screen", &CrossPointSettings::whiteSleepScreen},
   {"Extra Paragraph Spacing", &CrossPointSettings::extraParagraphSpacing}
@@ -134,7 +135,7 @@ void SettingsScreen::render() const {
   }
 
   // Draw help text
-  renderer.drawText(SMALL_FONT_ID, 20, pageHeight - 40,
+  renderer.drawText(SMALL_FONT_ID, 20, pageHeight - 30,
                    "Press OK to toggle, BACK to save & exit");
 
   // Always use standard refresh for settings screen

src/screens/SettingsScreen.h

@@ -1,4 +1,5 @@
 #pragma once
+#include <cstdint>
 #include <freertos/FreeRTOS.h>
 #include <freertos/semphr.h>
 #include <freertos/task.h>
@@ -13,7 +14,7 @@ class CrossPointSettings;
 // Structure to hold setting information
 struct SettingInfo {
   const char* name;       // Display name of the setting
-  bool CrossPointSettings::* valuePtr;  // Pointer to member in CrossPointSettings
+  uint8_t CrossPointSettings::* valuePtr;  // Pointer to member in CrossPointSettings
 };
 
 class SettingsScreen final : public Screen {
@@ -22,7 +23,7 @@ class SettingsScreen final : public Screen {
   bool updateRequired = false;
   int selectedSettingIndex = 0;  // Currently selected setting
   const std::function<void()> onGoHome;
-  
+
   // Static settings list
   static constexpr int settingsCount = 2;  // Number of settings
   static const SettingInfo settingsList[settingsCount];

src/screens/SleepScreen.h

@@ -1,11 +1,8 @@
 #pragma once
 #include "Screen.h"
 
-class CrossPointSettings;
-
 class SleepScreen final : public Screen {
  public:
-  explicit SleepScreen(GfxRenderer& renderer, InputManager& inputManager)
-      : Screen(renderer, inputManager) {}
+  explicit SleepScreen(GfxRenderer& renderer, InputManager& inputManager) : Screen(renderer, inputManager) {}
   void onEnter() override;
 };