fix(ci): detect breaking ci by comparing with base branch instead of main

[walldiss]

Refactor the action to:

• compare changes with target branch instead of main
• detect base branch changes to trigger ci
• don't apply kind:break! if breaking changes are detected, but there is label already
• apply kind:break! only if it is not present

TODO: tokens need permissions to write to pull requests to allow this action to set up the label

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.54%. Comparing base (2469e7a) to head (0238dac).
Report is 191 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3692      +/-   ##
==========================================
+ Coverage   44.83%   45.54%   +0.70%     
==========================================
  Files         265      281      +16     
  Lines       14620    16032    +1412     
==========================================
+ Hits         6555     7301     +746     
- Misses       7313     7889     +576     
- Partials      752      842      +90     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[walldiss]

@MSevey could you please check the GitHub token access to ensure it has write privileges for pull requests?

[MSevey]

@MSevey could you please check the GitHub token access to ensure it has write privileges for pull requests?

Why do you think you need updated permissions?

The new action ran https://github.com/celestiaorg/celestia-node/actions/runs/10790001843

[MSevey]

the majority of this condition is redundant since you are just checking that the action was trigger and the only action triggers are the ones you listed.

So you can just check for dependabot

Suggested change

	if: |
	(github.event.action == 'opened' ||
	github.event.action == 'synchronize' ||
	github.event.action == 'reopened' ||
	github.event.action == 'edited' ||
	github.event.action == 'labeled' ||
	github.event.action == 'unlabeled') &&
	github.actor != 'dependabot[bot]'
	if: github.actor != 'dependabot[bot]'

[MSevey]

This might be useful https://github.com/egmacke/action-check-label

[walldiss]

It is a bit concerning action has no stars or forks. It feels safer to avoid using it

[MSevey]

this might be useful https://github.com/marketplace/actions/actions-ecosystem-add-labels

[walldiss]

I've tried to use it, but it didn't work for some reason. Perhaps, you might want to contribute to this PR?

[MSevey]

Error in the action file preventing it from running https://github.com/celestiaorg/celestia-node/actions/runs/10813060223

You should install actionlint locally so you can catch these before pushing 👍🏻
https://github.com/rhysd/actionlint