Outbound links with target="_self" not navigating

[adriaandotcom]

Summary

Closes #89

Bug

With auto-events.js active, tracked links that have target="_self" fired the Simple Analytics event but never navigated, so the link looked dead. Blocking the script made them work again. Found on YourHosting, button "Ik wil zelf alles regelen".

Cause

collectLink rewrites tracked links with an inline onclick and appends return false when the link has no target, or target="_self". That cancels the browser's native navigation, so saAutomatedLink has to navigate itself. Its callback only did that when the link had no target at all:

if (!sent && !element.hasAttribute("target"))
  document.location = element.getAttribute("href");

For target="_self" the navigation was cancelled but never triggered again, so the link did nothing.

Fix

Navigate under the same condition collectLink uses to add return false:

if (
  !sent &&
  (!element.hasAttribute("target") ||
    element.getAttribute("target") === "_self")
)
  document.location = element.getAttribute("href");

Links with no target and _self links now navigate in the same tab after the event, while _blank and _parent keep their native behaviour. Verified against all four target values in a jsdom harness.

Also in this PR

The Prettier 3 action was reformatting source with trailingComma: "all", which adds ES2017 commas the ES5 only UglifyJS build cannot parse, breaking npm run build. Fixed by adding .prettierrc.json with trailingComma: "es5", reformatting the two build source files, and changing the Prettier workflow step to report only, so it no longer writes or commits.

Security implications

No security impact

Testing

With browserstack.

Checklist

• Linked to an issue
• Tested
• Asked for a review

[github-actions]

Claude review checkpoint

Reviewed commit 57c99dd0e524e1e95770a099c11d9376fb94d9bd. This is used to keep later automated reviews focused on changes Claude has not checked yet.

[adriaandotcom]

@Maimunar when you review, the most important change is in src/auto-events.js.