manager: optimized import

[YumeYuka]

#2731

[Copilot]

Pull Request Overview

This PR optimizes imports across the codebase by consolidating multiple specific imports into wildcard imports and removing unused imports. The changes also include some minor improvements like using more efficient comparison operators and string templates.

• Replaced multiple specific imports with wildcard imports for cleaner code organization
• Removed unused imports and variables
• Applied minor code optimizations and style improvements

Reviewed Changes

Copilot reviewed 35 out of 35 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
WebViewInterface.kt	Added unused suppression and optimized string interpolation
SuFilePathHandler.java	Removed unnecessary blank lines between imports
TemplateViewModel.kt	Converted specific locale import to wildcard
SuperUserViewModel.kt	Converted specific locale import to wildcard
ModuleViewModel.kt	Consolidated compose runtime imports to wildcard
SELinuxChecker.kt	Simplified SELinux status checking logic
KsuCli.kt	Major refactoring of shell handling and removed unused imports
HanziToPinyin.java	Converted to record class and optimized Unicode character handling
Multiple screen files	Consolidated layout and compose imports to wildcards
Multiple component files	Consolidated foundation layout and runtime imports

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Invalid string interpolation syntax. The double dollar signs ($$) are incorrect. Should use single dollar signs ($) for Kotlin string templates.

Suggested change

	$$"javascript: (function() { try { $${callbackFunc}.emit('exit', $${result.code}); } catch(e) { console.error(`emitExit error: ${e}`); } })();"
	"javascript: (function() { try { ${callbackFunc}.emit('exit', ${result.code}); } catch(e) { console.error(`emitExit error: ${'$'}{e}`); } })();"

[Copilot]

Converting from a class to a record is a breaking change that removes the constructor and makes the field immutable. The getInstance() method still tries to call 'new HanziToPinyin(true/false)' which will fail with a record.

[Copilot]

[nitpick] The removed variable 'userId' was storing the result of 'userProfile.hashCode()' before adding to list. While functionally equivalent, the variable made the code more readable and potentially easier to debug.

Suggested change

	result.add(userProfile.hashCode());
	int userId = userProfile.hashCode();
	result.add(userId);