Skip to content

fix(release-notes): expand Downloads section with AppImage, .deb, and Tauri mention - #1454

Closed
TimeToBuildBob wants to merge 1 commit into
ActivityWatch:masterfrom
TimeToBuildBob:fix/release-notes-downloads-section
Closed

TimeToBuildBob wants to merge 1 commit into
ActivityWatch:masterfrom
TimeToBuildBob:fix/release-notes-downloads-section

Conversation

@TimeToBuildBob

Copy link
Copy Markdown
Contributor

Summary

The auto-generated release notes' Downloads section (scripts/build_changelog.py) only listed the plain Windows .exe, macOS .dmg, and Linux .zip assets, even though releases now also ship a .deb, an AppImage, an Apple Silicon .dmg, and a full Tauri-based alternate build for all three platforms (see the assets on e.g. v0.14.0b7).

This adds:

  • .deb and AppImage links for Linux
  • an Apple Silicon .dmg link for macOS
  • a note pointing to the Tauri-based alternate build, which additionally supports Wayland on Linux

Scope

This fixes the release-notes part of #1453. The issue also mentions that https://activitywatch.net/downloads/ is missing the Tauri mention — I didn't find the source repo for that page from this checkout, so that part is left for a follow-up (or a pointer to the right repo).

I deliberately didn't hardcode exact Tauri asset filenames/links: the naming isn't fully consistent across platforms/formats in the current release assets (some use a v prefix, some don't), so a prose pointer to the activitywatch-tauri-* assets is safer than a link that could go stale or 404.

Test plan

  • Verified the new f-string renders correctly with a sample tag (see PR discussion / local check)
  • python3 -m py_compile scripts/build_changelog.py
  • Real verification happens on the next generated release notes

… Tauri mention

Add .deb and AppImage links for Linux, an Apple Silicon .dmg link for macOS,
and a note pointing to the Tauri-based alternate build (which additionally
supports Wayland on Linux). Addresses the release-notes part of
ActivityWatch#1453 (the activitywatch.net/downloads page still
needs the same Tauri mention, tracked separately in that issue).

Git-Session-Id: 332a6b70-940a-5f28-b303-f608651f1313
@greptile-apps

greptile-apps Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the added download links match the release artifact naming.

Summary

Expands the generated release notes’ Downloads section with Apple Silicon macOS, Linux .deb, and AppImage links, plus guidance for finding alternate Tauri builds and their Linux Wayland support.

  • Added download filenames match the packaging and release workflow.
  • Existing research-edition processing removes the standard platform download links.
  • No actionable issues identified.

Reviews (1) · Last reviewed commit: "fix(release-notes): expand Downloads sec..."

@TimeToBuildBob

Copy link
Copy Markdown
Contributor Author

CI failure investigated (Build Qt artifacts (macos-latest, ...), run 35334249691). This PR only touches scripts/build_changelog.py (release-notes text), so the failure is unrelated to the diff.

Root cause is a macOS runner environment issue inside poetry install for aw-watcher-afk: pip uninstall pyinstaller-hooks-contrib crashes with FileNotFoundError: ... aw_core-0.5.18.dist-info while pip's pkg_resources scans the venv — a stale/corrupted venv cache entry, not a code problem. Other Qt builds (ubuntu-22.04, manylinux_2_28) passed cleanly on the same commit.

I don't have admin rights on this repo to trigger gh run rerun, so I can't retry the job myself. No code action available — please rerun the failed job, or it may resolve on a subsequent push.

@TimeToBuildBob

TimeToBuildBob commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI code review

The PR updates the Downloads section of the auto-generated release notes in scripts/build_changelog.py. It adds an Apple Silicon macOS .dmg link, adds .deb and AppImage links for Linux, and appends a note about an alternate Tauri-based build with a pointer to the aw-tauri directory and a mention of Wayland support on Linux.

Not safe to merge — 1 P1 open

Confidence 3/5

1 finding · ❌ 1 P1

❌ P1 highscripts/build_changelog.py:421

The new AppImage link uses the filename pattern activitywatch-linux-x86_64.AppImage, but the actual release assets for ActivityWatch use a versioned pattern like activitywatch-{tag}-linux-x86_64.AppImage (consistent with the other links in the same section, e.g. the .deb and .zip). The PR description itself notes that Tauri asset naming is inconsistent, but the AppImage link is for the non-Tauri build, which follows the versioned convention. As written, the generated release notes will point to a URL that does not exist for any release, producing a 404 for users clicking the AppImage link. The fix is to include the {tag} segment in the filename, matching the .deb and .zip links.

- [**Linux**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.zip) (.zip) / [.deb](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.deb) / [AppImage](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.AppImage)

How this was verified: Compared the AppImage URL pattern against the .deb and .zip URLs in the same line and against the macOS/Windows patterns; all other non-Tauri assets include the {tag} segment. Checked the PR description which acknowledges naming inconsistency only for Tauri assets, not for the standard AppImage.

1 advisory finding (summary-only, not scored)

These P2 guard, heuristic, trade-off, or documentation claims are retained for judgment without opening review threads.

⚠️ P2 mediumscripts/build_changelog.py

This is a fix(...) PR but no test files are included in the diff. Erik's feedback: 'where is the repro & fixes they are supposed to catch' (gptme#3441), 'that measurement should come with a regression test' (gptme#3446). Add a test that would have caught this bug. (Advisory: Erik merged all such PRs but consistently requested tests.)

Add a test file that reproduces the bug before the fix and passes after it.

How this was verified: static preflight: fix-commit + touched-files scan (rule 7)

Files changed (1) — the diff as I read it
  • scripts/build_changelog.py — Expands the Downloads section with additional macOS, Linux, and Tauri build links and a note.

Reviewed b898224dfe52 · openrouter/deepseek/deepseek-v4-flash-0731 · llm engine · 27s · about this reviewer

Maintainer commands

@TimeToBuildBob review (own line) — fresh review · @TimeToBuildBob fix — a worker acts on the findings. Once per comment; 👀 = received.

- [**macOS**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-macos-x86_64.dmg) (.dmg)
- [**Linux**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.zip) (.zip)
- [**macOS**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-macos-x86_64.dmg) (.dmg, Intel) / [Apple Silicon](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-macos-arm64.dmg)
- [**Linux**](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.zip) (.zip) / [.deb](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-{tag}-linux-x86_64.deb) / [AppImage](https://github.com/ActivityWatch/activitywatch/releases/download/{tag}/activitywatch-linux-x86_64.AppImage)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

P1 — The new AppImage link uses the filename pattern activitywatch-linux-x86_64.AppImage, but the actual release assets for ActivityWatch use a versioned pattern like activitywatch-{tag}-linux-x86_64.AppImage (consistent with the other links in the same section, e.g. the .deb and .zip). The PR description itself notes that Tauri asset naming is inconsistent, but the AppImage link is for the non-Tauri build, which follows the versioned convention. As written, the generated release notes will point to a URL that does not exist for any release, producing a 404 for users clicking the AppImage link. The fix is to include the {tag} segment in the filename, matching the .deb and .zip links.

@TimeToBuildBob

Copy link
Copy Markdown
Contributor Author

Superseded by #1455 which has a more complete implementation (handles Tauri AppImage naming, extracts base URL, adds proper sections). Closing this in favor of #1455.

Note: both PRs have a pre-existing CI failure in 'Build Qt artifacts (macos-latest)' — the error is a poetry/pip venv conflict (aw_core dist-info missing during aw-watcher-afk downgrade) that also appears on master. It is unrelated to this PR's changes (which only touch scripts/build_changelog.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.

1 participant