- name: Install linux dependencies (GTK4 default)
        uses: awalsh128/cache-apt-pkgs-action@latest
        if: matrix.os == 'ubuntu-latest'
        with:
          packages: libgtk-3-dev libwebkit2gtk-4.1-dev libwayland-dev build-essential pkg-config xvfb x11-xserver-utils at-spi2-core xdg-desktop-portal-gtk
          version: 1.0

      - name: Install linux dependencies (GTK3 legacy)
        uses: awalsh128/cache-apt-pkgs-action@latest
        if: matrix.os == 'ubuntu-latest'
        with:
          packages: libgtk-4-dev libwebkitgtk-6.0-dev
          version: 1.0

      - name: Install linux dependencies (GTK4 default)
        uses: awalsh128/cache-apt-pkgs-action@latest
        if: matrix.os == 'ubuntu-latest'
        with:
          packages: libgtk-3-dev libwebkit2gtk-4.1-dev libwayland-dev build-essential pkg-config
          version: 1.0

      - name: Install linux dependencies (GTK3 legacy)
        uses: awalsh128/cache-apt-pkgs-action@latest
        if: matrix.os == 'ubuntu-latest'
        with:
          packages: libgtk-4-dev libwebkitgtk-6.0-dev
          version: 1.0

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@IMPLEMENTATION.md`:
- Around line 7-10: The IMPLEMENTATION.md doc is partially updated after the GTK
flip and now contains conflicting defaults and missing metadata; perform a full
consistency pass: update all occurrences that state the default stack (e.g., the
section around Line 118 and the file-reference blocks previously at Lines 374
and 392) to say "GTK4 + WebKitGTK 6.0 is the default for v3.0.0 GA" and that
"GTK3 + WebKit2GTK 4.1 is legacy opt-in via -tags gtk3 for one v3 cycle"; amend
the build constraint examples (the blocks near Lines 39–40) to include the
`!server` tag to match current server-exclusion behavior; add a dated
changelog/files-updated entry for 2026-05-16 listing files created/modified and
the flip decision, and ensure any other references to the old default are
updated to the new post-flip wording so the tracker is authoritative.

In `@v3/examples/badge-custom/build/linux/nfpm/nfpm.yaml`:
- Around line 27-29: The depends list contains incorrect Debian/Ubuntu package
names ("gtk4" and "libwebkitgtk-6.0"); update nfpm.yaml to add
distribution-specific overrides so .deb packages depend on the correct names
(use libgtk-4-1 and libwebkitgtk-6.0-4 for Debian/Ubuntu) while leaving other
distro defaults intact — modify the depends section and add a deb override
(following the pattern used in the other examples) so the deb package uses the
corrected dependency names.

In `@v3/examples/badge/build/linux/nfpm/nfpm.yaml`:
- Around line 27-29: Update the nfpm manifest so the top-level depends list does
not use non-Debian package names directly; instead follow the
distribution-specific override pattern used in the android example by replacing
or moving the generic depends entries (gtk4, libwebkitgtk-6.0) into an
overrides: deb: depends: [...] mapping with Debian/Ubuntu package names and
leaving a safe default depends array (or empty) at top-level; edit the depends
key and add an overrides -> deb block so Debian/Ubuntu packages resolve
correctly (use the android example as a template for structure and naming).

In `@v3/examples/custom-protocol-example/build/linux/nfpm/nfpm.yaml`:
- Around line 27-29: The current nfpm.yaml uses distro-incorrect generic
dependencies under the depends key (gtk4, libwebkitgtk-6.0); replace this with a
neutral/common depends list and add distribution-specific overrides like the
pattern used in v3/examples/android/build/linux/nfpm/nfpm.yaml so Debian/Ubuntu
get correct package names—update the depends section and add an overrides block
(referencing the depends key) that maps Debian/Ubuntu to their package names
(e.g., libgtk-4-1 or libwebkit2gtk-6.0-0 equivalents) so package resolution
works on Debian/Ubuntu.

In `@v3/examples/notifications/build/linux/nfpm/nfpm.yaml`:
- Around line 27-29: The depends list uses incorrect package names: replace the
generic entries under depends (gtk4, libwebkitgtk-6.0) with
distribution-appropriate package names; for Debian/Ubuntu use libgtk-4-1 and
libwebkitgtk-6.0-4, for Fedora/RHEL use gtk4 and webkitgtk6.0, and for Arch use
gtk4 and webkitgtk-6.0 (or implement a mapping strategy in nfpm.yaml per
distro). Update the depends entries accordingly so dependency resolution
succeeds.

In `@v3/pkg/application/application_linux_gtk3.go`:
- Around line 156-168: The early return after emitting the
ApplicationOpenedWithFile event causes startup handlers to be skipped; remove
the `return nil` so that after calling newApplicationEventContext(),
setOpenedWithFile(arg1) and sending to `applicationEvents` the code continues to
register the ApplicationStartup handler and invoke `setupCommonEvents()`,
`monitorThemeChanges()`, and `monitorPowerEvents()` (or, if the early-exit was
intentional, add a clear comment explaining why); locate the block that checks
`a.parent.options.FileAssociations` and modify it so the event is emitted but
normal startup flow (handlers and monitors) still runs.
- Around line 249-260: In getTheme, there's an out-of-bounds risk because
body[2] is accessed without ensuring the slice has at least 3 elements; update
the guard (and related checks) to require len(body) >= 3 before reading body[2],
and also safely type-assert body[2] to dbus.Variant and its Value() to string
(returning "", false on any failed assertion) so getTheme never panics when
given a 2-element body or unexpected types.

In `@v3/pkg/application/linux_cgo_gtk3.go`:
- Around line 569-573: The function appName calls C.g_get_application_name()
which returns GLib-owned memory that must not be freed; remove the defer
C.free(unsafe.Pointer(name)) line in appName so we don't free internal GLib
storage (C.GoString already copies the C string into Go memory), leaving appName
to simply call C.g_get_application_name(), convert with C.GoString, and return.
- Around line 697-701: clipboardGet leaks the gchar* returned by
C.gtk_clipboard_wait_for_text; after calling C.GoString(text) you must free the
C string with C.g_free to avoid the leak. Update the clipboardGet function to
check for nil (text == nil) before converting, call C.g_free(text) after
converting (or when nil not returned), and ensure no use-after-free occurs by
converting to Go string before freeing.
- Around line 460-472: The Calloc methods use value receivers so appends and
resets operate on a copy and the original pool is never updated; change the
receiver for both Calloc.String and Calloc.Free to pointer receivers (use
*Calloc) so append(c.pool, ...) and c.pool = ... modify the original struct, and
ensure Free clears the pool (e.g., set to nil or empty slice) after calling
C.free on each pointer.
- Around line 1838-1841: The zoomOut implementation uses a negative multiplier
(ZoomInFactor := -1.10) which yields a negative zoom that setZoom clamps to 1.0;
change zoomOut to use the reciprocal of the zoom-in factor (e.g., multiply
current zoom by 1.0/1.10 or compute zoomOutFactor := 1.0/ZoomInFactor) so
getZoom() * zoomOutFactor reduces zoom correctly; update the zoomOut function
(referencing zoomOut, ZoomInFactor, getZoom, setZoom) to apply a positive
reciprocal multiplier instead of a negative value.
- Around line 1095-1108: In execJS, the temporary C string created by
C.CString("") passed to webkit_web_view_evaluate_javascript is never freed,
causing a leak; replace that allocation by reusing a single preallocated empty C
string (if one exists) or explicitly free the allocation after the call—locate
the C.CString("") in execJS and either swap it for the shared empty C string
symbol or call C.free(unsafe.Pointer(...)) for that allocated pointer after
webkit_web_view_evaluate_javascript returns.

In `@v3/pkg/application/menuitem_linux_gtk3.go`:
- Around line 79-85: The setHidden implementation in linuxMenuItem incorrectly
passes the hidden flag directly to widgetSetVisible, inverting behavior; inside
linuxMenuItem.setHidden (the InvokeSync closure that calls l.blockSignal()/defer
l.unBlockSignal()), negate the boolean when calling widgetSetVisible so it
receives visibility (e.g., pass !hidden for l.native) to ensure hidden=true
makes the widget invisible.
- Around line 87-99: Remove the development stdout print from the setAccelerator
method: delete the fmt.Println("setAccelerator", accelerator) call in func (l
linuxMenuItem) setAccelerator(accelerator *accelerator) and, if runtime debug
output is desired, replace it with a proper logger call (e.g., use the project's
logging facility) guarded by a debug flag; ensure no stray fmt imports remain
after removal.

---

Nitpick comments:
In `@v3/pkg/application/application_linux.go`:
- Around line 308-324: The three blank identifier assignments for dbus,
filepath, and operatingsystem at the bottom of application_linux.go are
redundant because those packages are already referenced by functions like
isDarkMode, listenForSystemThemeChanges, getIconForFile, init, and
logPlatformInfo; remove the lines "var _ = dbus.SessionBus", "var _ =
filepath.Ext", and "var _ = operatingsystem.Info" so imports are no longer
shadowed by useless assignments and let the compiler recognize the real usages
in functions such as isVisible, getNativeApplication, and the listed
theme/file/icon helpers.