Fix SDK

[KenVanHoeylandt]

Summary by CodeRabbit

• New Features

  • Added a full SDK integration test suite with a minimal sample app and automated multi-platform build/run flows.
  • Added a new CLI build tool to orchestrate SDK download, build, packaging, and device install/run.

• Documentation

  • Added typography design token suggestion to ideas.

• Chores

  • Updated CI to run SDK integration prep and test steps.
  • Adjusted build artifact paths, license mappings, and repository ignore rules (including .tactility).

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds SDK integration testing: GitHub Actions workflow gains two post-release steps to prepare and run a test SDK with ESP-IDF v5.5. Updates build scripts to point LVGL binary and licence mappings to new paths. Adds Tests/SdkIntegration with CMake files, a minimal LVGL test app in C, manifest.properties, and a large tactility.py CLI tool implementing SDK download, build, packaging, and HTTP-based run/install/uninstall flows. Updates .gitignore to add .tactility and adjust build-related patterns. Documentation adds a font design token item.

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Fix SDK' is vague and generic, providing no meaningful information about the specific changes made.	Replace with a more descriptive title that conveys the main change, such as 'Add SDK integration testing framework' or 'Refactor SDK build and release process'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-sdk

---

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

[coderabbitai]

Actionable comments posted: 6

[coderabbitai]

Actionable comments posted: 3

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

Tests/HelloWorld/tactility.py (1)

504-529: Make skip_build explicit to avoid hidden global coupling.

build_action() reads skip_build at lines 525 and 527, but that variable is only defined in __main__. Importing and calling build_action() directly from another module will raise NameError. Pass skip_build explicitly as a parameter with a default value.

♻️ Suggested refactor

-def build_action(manifest, platform_arg):
+def build_action(manifest, platform_arg, skip_build=False):
     # Environment validation
     validate_environment()
     platforms_to_build = get_manifest_target_platforms(manifest, platform_arg)
@@
-        if not build_action(manifest, platform):
+        if not build_action(manifest, platform, skip_build):
             sys.exit(1)
@@
-        if build_action(manifest, platform):
+        if build_action(manifest, platform, skip_build):
             if install_action(sys.argv[2], platforms_to_install):
                 run_action(manifest, sys.argv[2])

[coderabbitai]

Actionable comments posted: 1