Skip to content

Chore: Optimizations - #4

Merged
dayvster merged 9 commits into
mainfrom
dev
Feb 14, 2026
Merged

dayvster merged 9 commits into
mainfrom
dev

Conversation

@dayvster

Copy link
Copy Markdown
Owner

This pull request introduces several improvements across the build system, commit workflow, and application logic. The most significant changes are the addition of a commit message linter workflow, enhancements to the CMake build configuration (including better test integration and stricter warnings), and a substantial refactor of the AppReader logic for reading desktop entries. The changes also improve code organization, add new command-line options, and update the commit conventions documentation.

Build System and Workflow Improvements

  • Added a GitHub Actions workflow (.github/workflows/commitlint.yml) that checks commit messages for compliance with the Conventional Commits standard, including support for the new bugfix and type types. This ensures consistent commit history.
  • Enhanced CMakeLists.txt to export compile commands for tooling, introduced options for building tests and enabling strict compiler warnings, and improved test target integration for easier development and CI. [1] [2]

AppReader and Desktop Entry Handling

  • Refactored and relocated AppReader logic from src/apps/readApps.cpp to src/core/apps/readApps.cpp and updated its interface. The new implementation supports expanded and deduplicated search paths, improved cache handling, and new options for including hidden/system apps. [1] [2] [3]
  • Updated main.cpp to use the refactored AppReader, added command-line flags (--show-system, --dump), and improved frequency tracking with a new FrequencyStore abstraction. [1] F14305d6L31R31, [2] [3]

Code Organization and Usability

  • Updated include paths throughout the codebase for consistency and modularity, especially after moving AppReader and related headers. [1] [2]
  • Improved application startup and search logic for better performance, correctness, and user experience, including lazy row creation and robust fallback behaviors. [1] [2] [3] [4] [5]

Documentation

  • Updated CONTRIBUTING.md to clarify and promote the use of the new bugfix and type commit types, aligning documentation with the new commit linting workflow.

@dayvster
dayvster merged commit 48c1123 into main Feb 14, 2026
1 check failed
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