Code improvements to make Scribe ready for future features

[linreal]

Terms

• I have searched open and closed feature requests
• I agree to follow Scribe-Android's Code of Conduct

Description

After working on Add invalid state to keyboard commands and show localized Not in Wikidata and app crashing at startup on api 30 I've noticed some issues, which could lead to maintenance problems in future, and also push away potential maintainers.

Specifically, I see problems with GeneralKeyboardIME and and classes inheriting it.

Main things that, in my opinion, should be addressed:

• GeneralKeyboardIME mixes view management, database access, state handling, and business logic in ~2,100 lines. The suppression of TooManyFunctions and LargeClass hints at maintainability issues. This makes it difficult for potential contributors to understand or extend the code.
• isTablet duplicates in every subclass
• heavy use of non-null assertions (!!) in the subclasses when setting up the keyboard view
• fuzzy state management without stateholder
• base class knows about classes inheriting it (i.e baseKeyboardOfAnyLanguage function and many others)
• languages logic is based on strings comparison instead enums or sealed classes (i.e getLanguageAlias function)
• various code style issues (like using "when" in setupCommandBarTheme function), invalid log tags (like "my-tag") and so on

Contribution

I would like to discuss if necessary and fix some of listed problems in one (or multiple) separate PR's
Let me know, if you ok with it

[andrewtavis]

Amazing to have this issue, @linreal! Thank you so much for the thorough review of the application 🙏 Your points definitely make sense to me 😊

CC @angrezichatterbox and @bhanu-dev82 for the conversation here. Let's wait until #411 is in before we work on any of the above, but we could convert the above list into a checklist and then work through it in multiple PRs :)

[angrezichatterbox]

Happy to have a discussion on this. The GeneralKeyboardIME should be refactored at this point. I thought of ending the Conjugate feature and then move to refactoring. But happy to discuss on it. We should also look into standardizing the labels mode switch while this is being done.

[angrezichatterbox]

@linreal Happy to have a discussion on this on a call on elements or however you would like.
@andrewtavis I think that would be better right ?

[andrewtavis]

Happy to have a call on this!

[andrewtavis]

Hey @linreal 👋 Would you have interest in joining the Scribe developer sync this upcoming Saturday at 15:00 UTC? Announcements for that are in the General channel on Matrix, with the reminder going out tomorrow that will share the meeting link :) We could spend some time there discussing this issue. Also happy to do a call another time, or we can also make sub issues async!

[linreal]

Hey @linreal 👋 Would you have interest in joining the Scribe developer sync this upcoming Saturday at 15:00 UTC? Announcements for that are in the General channel on Matrix, with the reminder going out tomorrow that will share the meeting link :) We could spend some time there discussing this issue. Also happy to do a call another time, or we can also make sub issues async!

@andrewtavis @angrezichatterbox gentlemen, I'm very sorry for delayed replies - got buried with work tasks.

In my opinion, the main (and also, the most complicated) problem of the above list is the management of the keyboard state, which should be carried out in StateHolder. All others could be relatively easy fixed in separate issues (correct me if you not agree)

I'll try to connect to call on Saturday, but if not - then I will start working more thoughtfully on this from next week

[linreal]

Wow, wow, missclick! @andrewtavis can you reopen this, please?