gem.sh icon indicating copy to clipboard operation
gem.sh copied to clipboard

Extract a `Gems::BaseController`

Open kaspth opened this issue 2 months ago • 0 comments

We've got a subsection of the app that expects and works off of a @gem variable. I think it'd be clearer if we made a dedicated hosting place for that than putting it in a concern.

Additionally the other controller concerns seem like premature extractions, some only being used in lone controller show actions. I think it'd be clearer if we inlined them for now.

The more complex finder is the set_target one that's used in both the gem class/instance methods controllers. It seems like they both share the use of params[:class_id] and params[:module_id] with a fallback to the general analyzer. I don't quite understand the general fallback to the general analyzer, but I kept it for now.

kaspth avatar Oct 20 '25 12:10 kaspth