Thanks @tbulva for all the work that you put in work on this MR and addressing my concerns. Code looks good from backend perspective. I've approved it.

What does this MR do and why?

Add creditsAvailable to fetchWorkflowEvents GraphQL response

Fetch credit availability from the server when loading existing workflows, preventing stale HAML prop from overriding runtime credit exhaustion state on component remount.

References

• https://gitlab.com/gitlab-org/gitlab/-/issues/583968+

Screenshots or screen recordings

How to set up and validate locally

1. Get the credits to run out.
2. You can simulate it using this patch

      # to simulate credits-out mid conversation
      touch tmp/simulate_no_credits
      
      # to simulate credits-in 
      rm tmp/simulate_no_credits

3. Test the no credits stickiness

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

@mksionek could you please give this MR a light review? Don't look at tests (they are automatically generated with Duo, I did't verify them yet). I'm interested to hear if you agree with this solution and if I understand the task correctly.

Tetiana Chupryna (c46a0314) at 19 Mar 14:37

... and 484 more commits

@tbulva the tests look good but I think another thing with rebase happened in this commit !223639 (d8c9351e)

the whole point it so delegate to TanukiBot in case if namespace is empty. there should be in the resolver:

namespace = if namespace_id
                      authorized_find!(id: namespace_id)
                    else
                      nil
                    end
                    ```

that was in the previous version of the diff.

Tetiana Chupryna (2594b6ca) at 18 Mar 01:53

... and 1 more commit

Tetiana Chupryna (5dd9fb7a) at 18 Mar 01:53

What does this MR do and why?

TanukiBot and PageLayoutHelper are not properly considering the governing_namespace for the current action. With the codebase consolidated, these changes fixes the resolution.

References

Screenshots or screen recordings

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #585827

thanks @Kaveh_Nejad_Gitlab for the review and testing!

Thanks @eduardobonet and @lulalala for the work on this MR. The refactoring looks very nice! Let's merge it.

What does this MR do and why?

TanukiBot and PageLayoutHelper are not properly considering the governing_namespace for the current action. With the codebase consolidated, these changes fixes the resolution.

References

Screenshots or screen recordings

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #585827

Tetiana Chupryna (0a598410) at 17 Mar 16:10

Tetiana Chupryna (2b027906) at 17 Mar 16:04

... and 60 more commits