Skip to content

Hide add-matcher and remove-matcher from actions job logs#36520

Merged
wxiaoguang merged 31 commits intomainfrom
copilot/hide-workflow-command-lines
Feb 5, 2026
Merged

Hide add-matcher and remove-matcher from actions job logs#36520
wxiaoguang merged 31 commits intomainfrom
copilot/hide-workflow-command-lines

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 2, 2026 •
edited by silverwind
Loading

Hides ::add-matcher::, ##[add-matcher] and ::remove-matcher in job step logs. These are used to configure regex matchers to detect lines that should trigger annotation comments on the UI, currently unsupported by Gitea and these have no relevance to the user. Example before this change:

image

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 2, 2026
Copilot AI and others added 3 commits February 2, 2026 18:54
@silverwind
Add filtering for workflow command lines and tests
88619be 
Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com>
@silverwind
Fix linting issues in test file
91d2ff8 
Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com>
@silverwind
Simplify test to validate log line filtering behavior
552833a 
Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com>
Copilot AI changed the title [WIP] Hide irrelevant workflow command lines in TypeScript Hide GitHub Actions workflow commands from job logs Feb 2, 2026
Copilot AI requested a review from silverwind February 2, 2026 18:59
@silverwind silverwind changed the title Hide GitHub Actions workflow commands from job logs Hide Actions workflow commands from job logs Feb 2, 2026
@silverwind
Copy link
Copy Markdown
Member

silverwind commented Feb 2, 2026

@copilot

  1. Move all functions related to log line processing to a new file web_src/js/render/log.ts. Remove RepoActionView.test.ts
  2. Move log line processing tests to web_src/js/render/log.test.ts
  3. Make it show ::workflow-command again.
  4. Remove obvious added comments.

@silverwind
Refactor log line processing to separate module
b79f034 
- Move LogLine processing functions (parseLineCommand, shouldHideLine) to web_src/js/render/log.ts
- Move log-related types (LogLine, LogLineCommand) and constants to log.ts
- Move tests to web_src/js/render/log.test.ts
- Remove RepoActionView.test.ts
- Remove ::workflow-command from hidden list (now visible)
- Remove obvious comments

Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 2, 2026

@copilot

  1. Move all functions related to log line processing to a new file web_src/js/render/log.ts. Remove RepoActionView.test.ts
    ...

Done in commit b79f034:

  1. ✅ Moved all log line processing functions to web_src/js/render/log.ts
  2. ✅ Moved tests to web_src/js/render/log.test.ts and removed RepoActionView.test.ts
  3. ✅ Removed ::workflow-command from hidden list (now visible)
  4. ✅ Removed obvious comments

The log.ts module now contains:

  • Types: LogLine, LogLineCommand
  • Constants: LogLinePrefixesGroup, LogLinePrefixesEndGroup, LogLinePrefixesHidden
  • Functions: parseLineCommand(), shouldHideLine()

silverwind
silverwind reviewed Feb 2, 2026
View reviewed changes
@silverwind
Apply suggestion from @silverwind
03e12ea 
Signed-off-by: silverwind <me@silverwind.io>
@silverwind silverwind changed the title Hide Actions workflow commands from job logs Hide actions workflow commands from job logs Feb 2, 2026
@silverwind silverwind changed the title Hide actions workflow commands from job logs Hide actions matcher commands from job logs Feb 2, 2026
@silverwind
Copy link
Copy Markdown
Member

silverwind commented Feb 2, 2026

Tested, works as expected.

@silverwind silverwind marked this pull request as ready for review February 2, 2026 19:49
@GiteaBot GiteaBot removed the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 2, 2026
@silverwind
Copy link
Copy Markdown
Member

silverwind commented Feb 5, 2026

tests added

wxiaoguang
wxiaoguang reviewed Feb 5, 2026
View reviewed changes
wxiaoguang
wxiaoguang reviewed Feb 5, 2026
View reviewed changes
@silverwind @claude
use realistic workflow command examples in test fixtures
36b9d1d 
Update test fixtures to use actual formats from actions/setup-go:
- Fix ::remove-matcher:: format (space before owner, trailing ::)
- Use realistic Go toolchain paths and version

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
wxiaoguang
wxiaoguang reviewed Feb 5, 2026
View reviewed changes
wxiaoguang
wxiaoguang reviewed Feb 5, 2026
View reviewed changes
@wxiaoguang
Update web_src/js/components/RepoActionView.vue
8036b7b 
Signed-off-by: wxiaoguang <wxiaoguang@gmail.com>
wxiaoguang
wxiaoguang reviewed Feb 5, 2026
View reviewed changes
wxiaoguang
wxiaoguang reviewed Feb 5, 2026
View reviewed changes
@GiteaBot 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 Feb 5, 2026
@wxiaoguang wxiaoguang merged commit fca94bc into main Feb 5, 2026
28 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Feb 5, 2026
@wxiaoguang wxiaoguang deleted the copilot/hide-workflow-command-lines branch February 5, 2026 11:11
zjjhot added a commit to zjjhot/gitea that referenced this pull request Feb 9, 2026
@zjjhot
Merge remote-tracking branch 'giteaofficial/main'
f55a8b3 
* giteaofficial/main:
  Refactor merge conan and container auth preserve actions taskID (go-gitea#36560)
  Fix assignee sidebar links and empty placeholder after go-gitea#32465 refactor (go-gitea#36559)
  Fix various version parsing problems (go-gitea#36553)
  Fix highlight diff result (go-gitea#36539)
  Refactor Nuget Auth to reuse Basic Auth Token Validation (go-gitea#36558)
  Update go dependencies (go-gitea#36548)
  Prevent navigation keys from triggering actions during IME composition (go-gitea#36540)
  Fix various mermaid bugs (go-gitea#36547)
  Add `elk` layout support to mermaid (go-gitea#36486)
  Allow configuring default PR base branch (fixes go-gitea#36412) (go-gitea#36425)
  [skip ci] Updated translations via Crowdin
  Color command/error logs in Actions log (go-gitea#36538)
  Add paging headers (go-gitea#36521)
  Fix issues filter dropdown showing empty label scope section (go-gitea#36535)
  [SECURITY] fix: Adjust the toolchain version (go-gitea#36537)
  Hide `add-matcher` and `remove-matcher` from actions job logs (go-gitea#36520)
  Improve timeline entries for WIP prefix changes in pull requests (go-gitea#36518)
Sirherobrine23 pushed a commit to Sirherobrine23/gitea that referenced this pull request Mar 4, 2026
@Sirherobrine23
Hide add-matcher and remove-matcher from actions job logs (go-git…
069a0fb 
…ea#36520)

Hides `::add-matcher::`, `##[add-matcher]` and `::remove-matcher` in job
step logs. These are used to configure regex matchers to detect lines
that should trigger annotation comments on the UI, currently unsupported
by Gitea and these have no relevance to the user.

---------

Signed-off-by: silverwind <me@silverwind.io>
Signed-off-by: wxiaoguang <wxiaoguang@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silverwind silverwind silverwind approved these changes

@wxiaoguang wxiaoguang wxiaoguang approved these changes

Assignees

@silverwind silverwind

Copilot code review Copilot

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend

Projects

None yet

Milestone

1.26.0

Development

Successfully merging this pull request may close these issues.

4 participants

@silverwind @wxiaoguang @GiteaBot