Skip to content

Fix Cua Driver release discovery across feed pages - #347

Draft
f-trycua wants to merge 1 commit into
omacom:add-cua-driver-binfrom
f-trycua:fix/cua-driver-release-pagination
Draft

Fix Cua Driver release discovery across feed pages#347
f-trycua wants to merge 1 commit into
omacom:add-cua-driver-binfrom
f-trycua:fix/cua-driver-release-pagination

Conversation

@f-trycua

@f-trycua f-trycua commented Sep 7, 2026

Copy link
Copy Markdown

Scope

Follow-up to #306, targeting its add-cua-driver-bin branch so Spencer's original package and authorship remain intact. This PR contains only the release-discovery fix and its regression tests.

The Cua monorepo publishes several components in one GitHub release feed. Looking at only the first 100 releases can stop finding Driver releases as other components publish. This change exhausts the paginated feed before choosing the highest eligible Driver version, with a 100-page bound and fail-closed handling of incomplete scans or malformed responses.

Unchanged

Validation

  • Bash syntax and whitespace checks pass.
  • Added 37 deterministic fixture assertions to the existing bin/sync-upstream self-test entry point: later-page discovery, version ordering, stable-shaped prereleases, drafts/nightlies, quarantine boundaries, dates, checksums, and API failures.
  • The local fixture run used GNU date and a mocked dotted-version comparator; it is not native pacman evidence. The repository's Arch CI must run the fixtures with real vercmp before this draft is ready.

GitHub has paused the upstream CI run for maintainer approval of this fork contribution: https://github.com/omacom/omarchy-pkgs/actions/runs/34168429328.

If #306 lands first, retarget this follow-up to master; it is not intended to replace the original package PR.

Build on omacom#306 without replacing its package or updater workaround. Exhaust the component feed before selecting a release and cover quarantine, version ordering, invalid input, and transport failures with offline fixtures.
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