Skip to content

fix(app): pass AppImage catalog validation - #296

Merged
av merged 4 commits into
mainfrom
fix/appimage-catalog
Oct 24, 2026
Merged

av merged 4 commits into
mainfrom
fix/appimage-catalog

Conversation

@av

@av av commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Harbor’s AppImage had a broken .DirIcon pointing at GitHub’s build directory. Update Tauri CLI and both lockfiles to 2.11.4 (upstream fix). The full test exposed two more blockers: a launcher ordinary users cannot execute, fixed by a Linux pre-bundle hook, and a blank window from circular React/Markdown loading, fixed by keeping shared JavaScript helpers with React.

Add # Harbor: the catalog’s README heuristic was reading a shell comment. Add PR-only build/catalog validation to the release workflow; publishing remains excluded on PRs. No new project dependencies or modified catalog tests.

Local Ubuntu 22.04 VM, unchanged catalog worker from the failed run:

v0.5.16: FATAL: .DirIcon is missing in /tmp/.mount_runtimFdpBhm
         exit 1
branch:  Lint found no fatal issues
         Screenshot: 87% one color, 16 words of text
         We will assume the test is OK (a pull request event was triggered and the required files exist).
         exit 0

Desktop, icon, launch, screenshot and catalog generation pass. Remaining warnings: no AppStream metadata, bundled libwayland-client, no update information. CI passed; full logs and screenshot are attached to that run. Publishing jobs were skipped.

After Ivan approves publication: merge; choose the next version in .scripts/seed.ts, run harbor dev seed and (cd app/src-tauri && cargo update --workspace), then commit/push. Wait for Harbor App Release, publish its completed draft with the rebuilt amd64 AppImage, and verify the published asset with the catalog test. Post exactly this on the catalog PR:

/retest

@av av changed the title fix(app): make AppImage icon links portable fix(app): pass AppImage catalog validation Oct 24, 2026
@av
av marked this pull request as ready for review October 24, 2026 10:33
@av
av merged commit 5eb0123 into main Oct 24, 2026
3 checks passed
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