Skip to content

updates#4

Open
lincoltd7 wants to merge 52 commits into
TeamRobotmad:mainfrom
Robotmad:main
Open

updates#4
lincoltd7 wants to merge 52 commits into
TeamRobotmad:mainfrom
Robotmad:main

Conversation

@lincoltd7

Copy link
Copy Markdown

No description provided.

Robotmad and others added 30 commits May 3, 2026 16:17
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Enhance submodule management with branch verification.
Removed checkout command for HexManager submodule.
Added a step to checkout the badge repository based on the PR branch.
Added a step to checkout the badge repository based on the PR branch.
Updated Python setup step to use a fixed version 3.11.
Agent-Logs-Url: https://github.com/Robotmad/HexManager/sessions/8324d722-1083-4ef3-820b-ab85cad7843c

Co-authored-by: Robotmad <3315650+Robotmad@users.noreply.github.com>
Robotmad and others added 22 commits May 13, 2026 18:25
Pin CI pytest major version to restore HexManager test compatibility
Fix CI: disable pytest cacheprovider to avoid sim shim incompatibility
…on; update HexCurrent submodule to latest commit
Copilot AI review requested due to automatic review settings June 23, 2026 00:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors HexManager to use vendored (submodule) hexpansion apps instead of in-repo EEPROM sources, adds support for a separate friendly_name stored in the EEPROM header, and improves UX/error reporting (startup warnings + paginated messages). It also adds EEPROM geometry probing for blank chips and updates dev/release tooling to build external app artifacts.

Changes:

  • Move HexDrive/GPS/etc app sources to vendor/ submodules and update imports, build scripts, and JSON metadata accordingly.
  • Add friendly_name to hexpansion type definitions and propagate it into EEPROM header writing + UI rendering.
  • Add blank-EEPROM geometry detection + “Scan” UI flow, plus improved startup warning reporting and paginated message display.

Reviewed changes

Copilot reviewed 19 out of 34 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
hexpansion_mgr.py Adds EEPROM geometry probing/caching, blank-port scan flow, and friendly_name header usage.
app.py Adds import-failure reason tracking, startup warning formatting, and message wrapping/pagination with navigation.
hexpansions.json Updates schema/help text, adds friendly_name, updates/extends hexpansion entries and app metadata.
README.md Documents friendly_name and new minification pipeline.
dev/minify.py New minify+compile pipeline for vendor modules.
dev/download_to_device.py Updates module list to compile vendor modules and optionally invoke minification.
dev/build_release.py Builds release artifacts while excluding vendored sources from the final bundle.
dev/dev_requirements.txt Adds dev dependencies for minification and related tooling.
tests/test_smoke.py Updates vendored imports and adds tests for startup warning pagination + version extraction from sources.
tests/test_serialise.py Adds Fake I2C probing and tests for geometry detection, scan UI flow, and friendly-name behavior.
tests/conftest.py Updates HexDrive runtime import to vendored path.
settings_mgr.py Makes setting persistence safer/cleaner by caching key/index once and early-returning on missing index.
pyproject.toml Disables pytest cache provider.
.gitmodules Adds vendor submodules for external apps.
.gitignore Ignores minifier intermediate artifacts under vendor/.
.gitattributes Excludes vendor/ from export bundles.
.github/workflows/tests.yml Updates CI checkout/submodule logic and Python version.
.github/workflows/release.yml Ensures release workflow checks out submodules recursively.
EEPROM/hexdrive.py Removed (HexDrive app now vendored).
EEPROM/gps.py Removed (GPS app now vendored).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread hexpansion_mgr.py
Comment thread dev/build_release.py
Comment thread dev/download_to_device.py
Comment thread dev/minify.py
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.

4 participants