[CL][Swap Router]: move poolIsActive() check from gamm to swaprouter

[p0mvn]

    TODO: consider moving this to swaprotuer

Originally posted by @p0mvn in #3764 (comment)

[p0mvn]

Blocked on: #3807 . We need to make sure that API is introduced to make sure that we don't miss adding this check to it.

[mattverse]

@p0mvn What's the reason we want to move that check to the swaprouter? Seems that the caller function(SwapExactAmountIn) is getting called in other places as well within the gamm module, I'm concerned that if we move that check to the swap router layer, any other methods calling the SwapExactAmountIn would not have the isActive check

[p0mvn]

The reason is that it would be great to perform the IsActive check for CL pools as well. As a result, x/swaprouter might be a good shared location for doing so.

I think the issue you are referring to (ref: #3130) is orthogonal toIsActive`,

No client other than swaprouter should call SwapExactAmountIn in x/gamm. #3130 is to investigate how we can prevent these other clients from doing so.

The IsActive check here is to have a quick switch guard for us to turn off all pools quickly (including CL) if there is a security incident in progress, assuming that #3130 is addressed