Linkify URLs in Actions workflow logs by silverwind · Pull Request #36986 · go-gitea/gitea

silverwind · 2026-03-25T22:56:19Z

Detect URLs in Actions log output and render them as clickable links, similar to how GitHub Actions handles this. Pre-existing links from ansi_up's OSC 8 parsing are also kept intact.

Detect URLs in Actions log output and render them as clickable links, similar to how GitHub Actions handles this. URLs are detected after ANSI-to-HTML conversion and wrapped in anchor tags that open in a new tab. Link color uses a new --color-console-link CSS variable that overrides any ANSI colors and provides sufficient contrast on both light and dark themes. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Copilot

Pull request overview

This PR adds URL detection and “linkification” to the rendered GitHub Actions job log output, so plain URLs in ANSI-rendered log lines become clickable links in the UI.

Changes:

Add a linkifyURLs() utility to convert detected http(s)://... substrings into <a> tags while preserving existing ANSI-generated HTML tags.
Apply URL linkification to renderAnsi() output so Actions logs automatically render clickable URLs.
Add styling for links in Actions logs and introduce a console link theme variable, plus unit tests for the new behavior.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
web_src/js/utils/url.ts	Introduces URL linkification logic for ANSI-rendered HTML strings.
web_src/js/utils/url.test.ts	Adds unit tests covering URL linkification cases and HTML-tag-preservation.
web_src/js/render/ansi.ts	Runs `linkifyURLs()` on ANSI-to-HTML rendered output.
web_src/js/render/ansi.test.ts	Adds tests verifying URLs become clickable links after ANSI rendering.
web_src/js/components/ActionRunJobView.vue	Styles links inside job step logs using a console-themed link color.
web_src/css/themes/theme-gitea-light.css	Adds `--color-console-link` for light theme.
web_src/css/themes/theme-gitea-dark.css	Adds `--color-console-link` for dark theme.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Use simpler URL regex that also matches localhost and dotless hosts. Use case-insensitive regex test for the fast-path guard. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Not worth the extra code path now that it's a regex test instead of a simple includes() call. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

The html helper was doing no escaping since both values were htmlRaw. Use direct string concatenation instead for less overhead. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Signed-off-by: silverwind <me@silverwind.io>

Track whether we are inside an existing <a> tag and skip URL linkification there to prevent invalid nested anchor elements. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Modern browsers imply noopener for target="_blank" and the rest of the codebase does not use it. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Signed-off-by: silverwind <me@silverwind.io>

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Signed-off-by: silverwind <me@silverwind.io>

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 25, 2026

github-actions Bot added the modifies/frontend label Mar 25, 2026

lunny reviewed Mar 25, 2026

View reviewed changes

Comment thread web_src/js/utils/url.ts Outdated

silverwind requested a review from Copilot March 25, 2026 23:20

Copilot started reviewing on behalf of silverwind March 25, 2026 23:20 View session

Copilot AI reviewed Mar 25, 2026

View reviewed changes

Comment thread web_src/js/utils/url.ts Outdated

Comment thread web_src/js/utils/url.ts Outdated

silverwind and others added 3 commits March 26, 2026 00:32

Broaden URL pattern and fix case-insensitive fast-path

933220f

Use simpler URL regex that also matches localhost and dotless hosts. Use case-insensitive regex test for the fast-path guard. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Drop early-return guard in linkifyURLs

9778415

Not worth the extra code path now that it's a regex test instead of a simple includes() call. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Simplify linkifyURLs: drop html helper, use plain string concat

66f725e

The html helper was doing no escaping since both values were htmlRaw. Use direct string concatenation instead for less overhead. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

silverwind added type/feature Completely new functionality. Can only be merged if feature freeze is not active. topic/gitea-actions related to the actions of Gitea labels Mar 25, 2026

silverwind commented Mar 25, 2026

View reviewed changes

Comment thread web_src/js/utils/url.ts Outdated

silverwind and others added 4 commits March 26, 2026 00:47

Apply suggestion from @silverwind

683bb4c

Signed-off-by: silverwind <me@silverwind.io>

Avoid nested links from ansi_up OSC 8 hyperlinks

45f65e7

Track whether we are inside an existing <a> tag and skip URL linkification there to prevent invalid nested anchor elements. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Drop rel="noopener noreferrer" for consistency

fd5f243

Modern browsers imply noopener for target="_blank" and the rest of the codebase does not use it. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Rename linkifyURLs parameter to html

41ca83f

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

silverwind commented Mar 26, 2026

View reviewed changes

Comment thread web_src/js/utils/url.ts Outdated

silverwind and others added 2 commits March 26, 2026 01:19

Apply suggestion from @silverwind

2315437

Signed-off-by: silverwind <me@silverwind.io>

make linkifyURLs for general use, it doesn't depends on asni_up's output

fa63e2c

lunny approved these changes Mar 26, 2026

View reviewed changes

GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 26, 2026

silverwind and others added 3 commits March 26, 2026 07:29

Add XSS tests for linkifyURLs

2b34c3e

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

Add one more xss test

38b5923

Remove useless XSS test case

6187509

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

silverwind force-pushed the actionlinks branch from b663668 to 6187509 Compare March 26, 2026 06:55

silverwind commented Mar 26, 2026

View reviewed changes

Comment thread web_src/js/utils/url.test.ts Outdated

silverwind and others added 2 commits March 26, 2026 07:55

Apply suggestion from @silverwind

0a6ae22

Signed-off-by: silverwind <me@silverwind.io>

Fix eslint no-script-url in XSS test

fed2af6

Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>

silverwind added type/enhancement An improvement of existing functionality and removed type/feature Completely new functionality. Can only be merged if feature freeze is not active. labels Mar 26, 2026

wxiaoguang approved these changes Mar 26, 2026

View reviewed changes

GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 26, 2026

Merge branch 'main' into actionlinks

ec9071b

wxiaoguang enabled auto-merge (squash) March 26, 2026 09:47

wxiaoguang merged commit 9583e1a into go-gitea:main Mar 26, 2026
26 checks passed

wxiaoguang deleted the actionlinks branch March 26, 2026 09:48

GiteaBot added this to the 1.26.0 milestone Mar 26, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Linkify URLs in Actions workflow logs#36986

Linkify URLs in Actions workflow logs#36986
wxiaoguang merged 16 commits into
go-gitea:mainfrom
silverwind:actionlinks

silverwind commented Mar 25, 2026 •

edited

Loading

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

silverwind commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

silverwind commented Mar 25, 2026 •

edited

Loading