Refactoring

[idle-river]

This pull request refactors the CLI and TUI code to consistently use the Silicate struct for all password manager operations, improving code organization and encapsulation. Instead of calling free functions from silicate_core, the CLI and TUI now create a Silicate instance and use its methods, which simplifies dependency management and makes the codebase more modular and maintainable.

Refactoring to use the Silicate struct for all operations:

• Replaced all direct calls to silicate_core free functions in main.rs with corresponding methods on a Silicate instance, including password encryption/decryption, key management, password listing, tag management, and stats retrieval. A single Silicate instance is now created at the start of main() and passed where needed. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21] [22]

• Updated helper functions such as get_key to accept a &Silicate parameter and use its methods for key retrieval and derivation, further centralizing logic within the struct.

TUI integration improvements:

• Modified the TUI App struct to store a Silicate instance, and updated its constructor and usage to accept and use this instance, ensuring all TUI operations also go through the unified interface. [1] [2]

Code style and minor cleanups:

• Applied minor formatting and style improvements in the TUI code for better readability, such as expanding certain statements for clarity. [1] [2]

Dependency and import adjustments:

• Updated imports in both CLI and TUI modules to import Silicate and its error types directly, removing unused or redundant imports. [1] [2]

Initialization consistency:

• Ensured that Silicate is always initialized with the configuration directory, and that all file and password operations are routed through the struct for consistency and future extensibility. [1] [2]

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.