{"entries":[{"lastCommit":{"author":{"email":"son@huggingface.co","name":"Xuan-Son Nguyen","when":"2026-04-13T22:23:45.000000+02:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-13T15:23:45.000000-05:00"},"hash":"243ae8b40858b67ec291e8c349e10e2ac0749423","message":"feat: show crash reason on boot (#1453)\n\n## Summary\n\nIf the system reboots from crash, display the reason and tell user to\ninclude `crash_report.txt` file.\n\nTo test this, simply add an `assert(false)` somewhere inside the code.\n\n\n![screenshot-74453.bmp](https://github.com/user-attachments/files/26160373/screenshot-74453.bmp)\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? **NO**","tree":"7c9c845a1fb0b3637d61cf9b318e0c3c8c55cd11"},"mode":"0100644","name":"CrashActivity.cpp","size":1839},{"lastCommit":{"author":{"email":"son@huggingface.co","name":"Xuan-Son Nguyen","when":"2026-04-13T22:23:45.000000+02:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-13T15:23:45.000000-05:00"},"hash":"243ae8b40858b67ec291e8c349e10e2ac0749423","message":"feat: show crash reason on boot (#1453)\n\n## Summary\n\nIf the system reboots from crash, display the reason and tell user to\ninclude `crash_report.txt` file.\n\nTo test this, simply add an `assert(false)` somewhere inside the code.\n\n\n![screenshot-74453.bmp](https://github.com/user-attachments/files/26160373/screenshot-74453.bmp)\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? **NO**","tree":"7c9c845a1fb0b3637d61cf9b318e0c3c8c55cd11"},"mode":"0100644","name":"CrashActivity.h","size":348},{"lastCommit":{"author":{"email":"zach@zdnelson.com","name":"Zach Nelson","when":"2026-04-14T16:41:05.000000-05:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-14T16:41:05.000000-05:00"},"hash":"23aad213fc254c078cc4725dca0367857367b519","message":"refactor: Removed redundant FsFile close() calls (#1434)\n\n## Summary\n\n**What is the goal of this PR?** (e.g., Implements the new feature for\nfile uploading.)\n\n`DESTRUCTOR_CLOSES_FILE=1` is set in platformio.ini, which makes SdFat's\nFsBaseFile destructor call close() automatically when a file goes out of\nscope.\n\nThree categories of file close calls remain untouched:\n1. Close before Storage.remove() on the same path: ScreenshotUtil.cpp\ncloses the file before deleting it on write error. The remove might fail\nif the file is still open.\n2. Close before reopening the same variable: Epub.cpp writes a temp\nNCX/nav file, closes it, then reopens it for reading. The\nRecentBooksStore.cpp close before saveToFile() is the same pattern, it\nrewrites the same file.\n3. Close on member variables: BookMetadataCache.cpp (bookFile,\nspineFile, tocFile), Section.cpp (file), XtcParser.cpp (m_file),\nZipFile.cpp (file). These persist beyond any single function scope, so\nthe destructor timing doesn't match the intended close point.\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? _**PARTIALLY**_","tree":"932bf4d15d1c2c9bd56d7a5c4b4670687eb2665f"},"mode":"0100644","name":"FileBrowserActivity.cpp","size":11637},{"lastCommit":{"author":{"email":"telemaniaka@gmail.com","name":"Jan Ivanov","when":"2026-04-11T20:20:46.000000+03:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-11T20:20:46.000000+03:00"},"hash":"8d6b35b8e75989506cb762c821f23efdf9e45087","message":"fix: back navigation from BMPViewer (#1597)\n\n## Summary\n\nThis fixes navigating back from the BMP Viewer to the FileBrowser which\nwas broken when moving to the new ActivityManager\n\nThis is fixed by making FileBrowserActivity able to take a full file\npath on enter and splitting the basePath and fileName from it and\nnavigating to the correct place.\n\nfixes:\nhttps://github.com/crosspoint-reader/crosspoint-reader/issues/1553\n\nduplicates:\nhttps://github.com/crosspoint-reader/crosspoint-reader/pull/910 to some\nextend but mine has the file cursor at the correct file instead of the\nfirst one in the folder\n\n## Additional Context\n\n* Add any other information that might be helpful for the reviewer\n(e.g., performance implications, potential risks,\n specific areas to focus on).\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? _**PARTIALLY**_\n\n---------\n\nCo-authored-by: Jan Ivanov ","tree":"bba162e4cc127cfe47ab6d480989108aa5c274d6"},"mode":"0100644","name":"FileBrowserActivity.h","size":932},{"lastCommit":{"author":{"email":"lisnake@gmail.com","name":"Arthur Tazhitdinov","when":"2026-04-22T02:35:41.000000+03:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-21T18:35:41.000000-05:00"},"hash":"1cf223974272c859311a2ee45f86bf46405b724b","message":"feat: Support for multiple OPDS servers (#1209)\n\n## Summary\n\n* Add support for configuring and using multiple OPDS servers, replacing\nthe previous single-server limitation. Closes\nhttps://github.com/crosspoint-reader/crosspoint-reader/issues/1178\n* New OpdsServerStore singleton (modeled after WifiCredentialStore) that\npersists up to 8 OPDS servers to /.crosspoint/opds.json with MAC-based\npassword obfuscation.\n* One-time migration from legacy single-server fields in\nCrossPointSettings to the new store on first boot.\n* New OpdsServerListActivity for the device UI — works in two modes: a\nsettings list (add/edit/delete servers) and a picker (select which\nserver to browse). When only one server is configured, the picker is\nskipped automatically.\n* Renamed CalibreSettingsActivity → OpdsSettingsActivity for clarity. It\nnow edits individual OpdsServer entries (name, URL, username, password,\ndelete).\n* OpdsBookBrowserActivity now receives an OpdsServer at construction and\nuses its credentials for all fetches/downloads, and shows the server\nname in the header.\n* HttpDownloader::fetchUrl and downloadToFile accept optional per-call\nusername/password parameters instead of reading from global settings.\n* REST API endpoints on CrossPointWebServer: GET /api/opds, POST\n/api/opds, POST /api/opds/delete — passwords are never exposed over the\nAPI (only a hasPassword flag), and omitting the password field on update\npreserves the existing one.\n* Web UI (SettingsPage.html) with dynamic OPDS server management cards —\nadd, edit, save, and delete servers from the browser.\n $\"SCR-20260416-stvu\"\nsrc=\"https://github.com/user-attachments/assets/a8f18d84-4204-46a0-bb31-b73d24b3255f\"\n/$ \n\n\n## Additional Context\n\n* The OpdsServerStore JSON format and obfuscation scheme are identical\nto WifiCredentialStore, so the same JsonSettingsIO infrastructure\nhandles both.\n* The web API uses POST /api/opds/delete instead of DELETE /api/opds\nbecause the ESP32 WebServer doesn't support the DELETE method with a\nrequest body.\n* Existing single-server configurations are migrated automatically — no\nuser action required. After migration the legacy CrossPointSettings\nfields are cleared so it only runs once.\n* The HttpDownloader changes are backward-compatible: the credential\nparameters default to empty strings, so existing callers are unaffected.\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? _**< YES >**_\n\n---------\n\nCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>","tree":"45b20858498014309362535c494e7ff0ca9a16e1"},"mode":"0100644","name":"HomeActivity.cpp","size":8423},{"lastCommit":{"author":{"email":"lisnake@gmail.com","name":"Arthur Tazhitdinov","when":"2026-04-22T02:35:41.000000+03:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-21T18:35:41.000000-05:00"},"hash":"1cf223974272c859311a2ee45f86bf46405b724b","message":"feat: Support for multiple OPDS servers (#1209)\n\n## Summary\n\n* Add support for configuring and using multiple OPDS servers, replacing\nthe previous single-server limitation. Closes\nhttps://github.com/crosspoint-reader/crosspoint-reader/issues/1178\n* New OpdsServerStore singleton (modeled after WifiCredentialStore) that\npersists up to 8 OPDS servers to /.crosspoint/opds.json with MAC-based\npassword obfuscation.\n* One-time migration from legacy single-server fields in\nCrossPointSettings to the new store on first boot.\n* New OpdsServerListActivity for the device UI — works in two modes: a\nsettings list (add/edit/delete servers) and a picker (select which\nserver to browse). When only one server is configured, the picker is\nskipped automatically.\n* Renamed CalibreSettingsActivity → OpdsSettingsActivity for clarity. It\nnow edits individual OpdsServer entries (name, URL, username, password,\ndelete).\n* OpdsBookBrowserActivity now receives an OpdsServer at construction and\nuses its credentials for all fetches/downloads, and shows the server\nname in the header.\n* HttpDownloader::fetchUrl and downloadToFile accept optional per-call\nusername/password parameters instead of reading from global settings.\n* REST API endpoints on CrossPointWebServer: GET /api/opds, POST\n/api/opds, POST /api/opds/delete — passwords are never exposed over the\nAPI (only a hasPassword flag), and omitting the password field on update\npreserves the existing one.\n* Web UI (SettingsPage.html) with dynamic OPDS server management cards —\nadd, edit, save, and delete servers from the browser.\n $\"SCR-20260416-stvu\"\nsrc=\"https://github.com/user-attachments/assets/a8f18d84-4204-46a0-bb31-b73d24b3255f\"\n/$ \n\n\n## Additional Context\n\n* The OpdsServerStore JSON format and obfuscation scheme are identical\nto WifiCredentialStore, so the same JsonSettingsIO infrastructure\nhandles both.\n* The web API uses POST /api/opds/delete instead of DELETE /api/opds\nbecause the ESP32 WebServer doesn't support the DELETE method with a\nrequest body.\n* Existing single-server configurations are migrated automatically — no\nuser action required. After migration the legacy CrossPointSettings\nfields are cleared so it only runs once.\n* The HttpDownloader changes are backward-compatible: the credential\nparameters default to empty strings, so existing callers are unaffected.\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? _**< YES >**_\n\n---------\n\nCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>","tree":"45b20858498014309362535c494e7ff0ca9a16e1"},"mode":"0100644","name":"HomeActivity.h","size":1430},{"lastCommit":{"author":{"email":"zach@zdnelson.com","name":"Zach Nelson","when":"2026-03-05T10:12:22.000000-06:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-03-05T10:12:22.000000-06:00"},"hash":"c3f1dbfa092f85f70f6e6056775e35d1fb7689ff","message":"perf: Avoid creating strings for file extension checks (#1303)\n\n## Summary\n\n**What is the goal of this PR?**\n\nThis change avoids the pattern of creating a `std::string` using\n`.substr` in order to compare against a file extension literal.\n```c++\nstd::string path;\nif (path.length() >= 4 && path.substr(path.length() - 4) == \".ext\")\n```\n\nThe `checkFileExtension` utility has moved from StringUtils to\nFsHelpers, to be available to code in lib/. The signature now accepts a\n`std::string_view` instead of `std::string`, which makes the single\nimplementation reusable for Arduino `String`.\n\nAdded utility functions for commonly repeated extensions.\n\nThese changes **save about 2 KB of flash (5,999,427 to 5,997,343)**.\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? _**NO**_","tree":"65a926d34dd21aac7d3aa4901c19cd80809aeaa2"},"mode":"0100644","name":"RecentBooksActivity.cpp","size":3539},{"lastCommit":{"author":{"email":"son@huggingface.co","name":"Xuan-Son Nguyen","when":"2026-02-27T07:32:40.000000+01:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-02-27T00:32:40.000000-06:00"},"hash":"c4fc4effbdb8912204f6580fbffbf61769785d04","message":"refactor: implement ActivityManager (#1016)\n\n## Summary\n\nRef comment:\nhttps://github.com/crosspoint-reader/crosspoint-reader/pull/1010#pullrequestreview-3828854640\n\nThis PR introduces `ActivityManager`, which mirrors the same concept of\nActivity in Android, where an activity represents a single screen of the\nUI. The manager is responsible for launching activities, and ensuring\nthat only one activity is active at a time.\n\nMain differences from Android's ActivityManager:\n- No concept of Bundle or Intent extras\n- No onPause/onResume, since we don't have a concept of background\nactivities\n- onActivityResult is implemented via a callback instead of a separate\nmethod, for simplicity\n\n## Key changes\n\n- Single `renderTask` shared across all activities\n- No more sub-activity, we manage them using a stack; Results can be\npassed via `startActivityForResult` and `setResult`\n- Activity can call `finish()` to destroy themself, but the actual\ndeletion will be handled by `ActivityManager` to avoid `delete this`\npattern\n\nAs a bonus: the manager will automatically call `requestUpdate()` when\nreturning from another activity\n\n## Example usage\n\n**BEFORE**:\n\n```cpp\n// caller\n enterNewActivity(new WifiSelectionActivity(renderer, mappedInput,\n [this](const bool connected) { onWifiSelectionComplete(connected); }));\n\n// subactivity\n onComplete(true); // will eventually call exitActivity(), which deletes the caller instance (dangerous behavior)\n``` \n\n**AFTER**: (mirrors the `startActivityForResult` and `setResult` from\nandroid)\n\n```cpp\n// caller\n startActivityForResult(new NetworkModeSelectionActivity(renderer, mappedInput),\n [this](const ActivityResult& result) { onNetworkModeSelected(result.selectedNetworkMode); });\n\n// subactivity\n ActivityResult result;\n result.isCancelled = false;\n result.selectedNetworkMode = mode;\n setResult(result);\n finish(); // signals to ActivityManager to go back to last activity AFTER this function returns\n```\n\nTODO:\n- [x] Reconsider if the `Intent` is really necessary or it should be\nremoved (note: it's inspired by\n[Intent](https://developer.android.com/guide/components/intents-common)\nfrom Android API) ==> I decided to keep this pattern fr clarity\n- [x] Verify if behavior is still correct (i.e. back from sub-activity)\n- [x] Refactor the `ActivityWithSubactivity` to just simple `Activity`\n--> We are using a stack for keeping track of sub-activity now\n- [x] Use single task for rendering --> avoid allocating 8KB stack per\nactivity\n- [x] Implement the idea of [Activity\nresult](https://developer.android.com/training/basics/intents/result)\n--> Allow sub-activity like Wifi to report back the status (connected,\nfailed, etc)\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? **PARTIALLY**, some\nrepetitive migrations are done by Claude, but I'm the one how ultimately\napprove it\n\n---------\n\nCo-authored-by: Zach Nelson ","tree":"de4457ab8f1f135075d890d1d6483edee304565b"},"mode":"0100644","name":"RecentBooksActivity.h","size":687}],"lastCommit":{"author":{"email":"lisnake@gmail.com","name":"Arthur Tazhitdinov","when":"2026-04-22T02:35:41.000000+03:00"},"committer":{"email":"noreply@github.com","name":"GitHub","when":"2026-04-21T18:35:41.000000-05:00"},"hash":"1cf223974272c859311a2ee45f86bf46405b724b","message":"feat: Support for multiple OPDS servers (#1209)\n\n## Summary\n\n* Add support for configuring and using multiple OPDS servers, replacing\nthe previous single-server limitation. Closes\nhttps://github.com/crosspoint-reader/crosspoint-reader/issues/1178\n* New OpdsServerStore singleton (modeled after WifiCredentialStore) that\npersists up to 8 OPDS servers to /.crosspoint/opds.json with MAC-based\npassword obfuscation.\n* One-time migration from legacy single-server fields in\nCrossPointSettings to the new store on first boot.\n* New OpdsServerListActivity for the device UI — works in two modes: a\nsettings list (add/edit/delete servers) and a picker (select which\nserver to browse). When only one server is configured, the picker is\nskipped automatically.\n* Renamed CalibreSettingsActivity → OpdsSettingsActivity for clarity. It\nnow edits individual OpdsServer entries (name, URL, username, password,\ndelete).\n* OpdsBookBrowserActivity now receives an OpdsServer at construction and\nuses its credentials for all fetches/downloads, and shows the server\nname in the header.\n* HttpDownloader::fetchUrl and downloadToFile accept optional per-call\nusername/password parameters instead of reading from global settings.\n* REST API endpoints on CrossPointWebServer: GET /api/opds, POST\n/api/opds, POST /api/opds/delete — passwords are never exposed over the\nAPI (only a hasPassword flag), and omitting the password field on update\npreserves the existing one.\n* Web UI (SettingsPage.html) with dynamic OPDS server management cards —\nadd, edit, save, and delete servers from the browser.\n $\"SCR-20260416-stvu\"\nsrc=\"https://github.com/user-attachments/assets/a8f18d84-4204-46a0-bb31-b73d24b3255f\"\n/$ \n\n\n## Additional Context\n\n* The OpdsServerStore JSON format and obfuscation scheme are identical\nto WifiCredentialStore, so the same JsonSettingsIO infrastructure\nhandles both.\n* The web API uses POST /api/opds/delete instead of DELETE /api/opds\nbecause the ESP32 WebServer doesn't support the DELETE method with a\nrequest body.\n* Existing single-server configurations are migrated automatically — no\nuser action required. After migration the legacy CrossPointSettings\nfields are cleared so it only runs once.\n* The HttpDownloader changes are backward-compatible: the credential\nparameters default to empty strings, so existing callers are unaffected.\n\n---\n\n### AI Usage\n\nWhile CrossPoint doesn't have restrictions on AI tools in contributing,\nplease be transparent about their usage as it\nhelps set the right context for reviewers.\n\nDid you use AI tools to help write this code? _**< YES >**_\n\n---------\n\nCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>","tree":"45b20858498014309362535c494e7ff0ca9a16e1"}}