Merge pull request #139 from AbelOsaretin/fix/issues-92-93-95-98

fix: resolve issues #92, #93, #95, #98 - duplicate Template removal, wallet export/import, and completions sync

Author: Nanle-code

API_REFERENCE.md

@@ -222,6 +222,50 @@ starforge wallet rename alice alice-testnet
 
 ---
 
+### `starforge wallet export`
+
+Export a wallet to a JSON backup file.
+
+**Usage:**
+```bash
+starforge wallet export <NAME> --output <FILE>
+```
+
+**Arguments:**
+- `<NAME>` - Wallet name to export
+
+**Options:**
+- `--output <FILE>` - Output file path for the backup JSON
+
+**Example:**
+```bash
+starforge wallet export alice --output ./wallet-backup.json
+```
+
+**Notes:**
+- Secrets are written only to the backup file and are never printed to stdout.
+
+---
+
+### `starforge wallet import`
+
+Import wallets from a JSON backup file.
+
+**Usage:**
+```bash
+starforge wallet import --file <FILE>
+```
+
+**Options:**
+- `--file <FILE>` - Path to a wallet backup JSON file
+
+**Example:**
+```bash
+starforge wallet import --file ./wallet-backup.json
+```
+
+---
+
 ### `starforge wallet sign`
 
 Sign an arbitrary message using a wallet.

Cargo.lock

@@ -55,12 +55,14 @@ RUST_LOG=debug cargo run -- wallet list
 #### VS Code
 
 Recommended extensions:
+
 - `rust-analyzer` - Rust language support
 - `crates` - Cargo.toml dependency management
 - `Better TOML` - TOML syntax highlighting
 - `Error Lens` - Inline error display
 
 `.vscode/settings.json`:
+
 ```json
 {
   "rust-analyzer.checkOnSave.command": "clippy",
@@ -72,6 +74,7 @@ Recommended extensions:
 #### IntelliJ IDEA / CLion
 
 Install the Rust plugin and configure:
+
 - Enable Clippy for code analysis
 - Set rustfmt as formatter
 - Enable external linter
@@ -175,7 +178,7 @@ fn private_helper() { /* ... */ }
 #[cfg(test)]
 mod tests {
     use super::*;
-    
+
     #[test]
     fn test_something() { /* ... */ }
 }
@@ -207,13 +210,13 @@ pub fn CreateWallet(Name: String, Encrypt: bool) -> Result<()> {
 
 ### Naming Conventions
 
-| Type | Convention | Example |
-|------|------------|---------|
-| Functions | `snake_case` | `fetch_account()` |
-| Types | `PascalCase` | `WalletEntry` |
-| Constants | `SCREAMING_SNAKE_CASE` | `MAX_RETRIES` |
-| Modules | `snake_case` | `hardware_wallet` |
-| Lifetimes | `'lowercase` | `'a`, `'static` |
+| Type      | Convention             | Example           |
+| --------- | ---------------------- | ----------------- |
+| Functions | `snake_case`           | `fetch_account()` |
+| Types     | `PascalCase`           | `WalletEntry`     |
+| Constants | `SCREAMING_SNAKE_CASE` | `MAX_RETRIES`     |
+| Modules   | `snake_case`           | `hardware_wallet` |
+| Lifetimes | `'lowercase`           | `'a`, `'static`   |
 
 ### Error Handling
 
@@ -239,7 +242,7 @@ let data = load_data()
 
 ### Documentation
 
-```rust
+````rust
 /// Fetches account information from Horizon API.
 ///
 /// # Arguments
@@ -267,7 +270,7 @@ let data = load_data()
 pub fn fetch_account(public_key: &str, network: &str) -> Result<AccountResponse> {
     // Implementation
 }
-```
+````
 
 ### Comments
 
@@ -326,17 +329,17 @@ pub fn handle(cmd: MyCommands) -> Result<()> {
 fn action(input: String) -> Result<()> {
     p::header("My Command");
     p::kv("Input", &input);
-    
+
     // Your logic here
-    
+
     p::success("Done!");
     Ok(())
 }
 
 #[cfg(test)]
 mod tests {
     use super::*;
-    
+
     #[test]
     fn test_action() {
         // Test your command
@@ -358,7 +361,7 @@ pub mod mycommand;
 #[derive(Subcommand)]
 enum Commands {
     // ... existing commands
-    
+
     /// My new command
     #[command(subcommand)]
     MyCommand(commands::mycommand::MyCommands),
@@ -402,7 +405,7 @@ pub fn do_something(input: &str) -> Result<String> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    
+
     #[test]
     fn test_do_something() {
         let result = do_something("test").unwrap();
@@ -502,20 +505,20 @@ impl {{PROJECT_NAME_PASCAL}} {
 #[cfg(test)]
 mod tests {
     use super::*;
-    
+
     #[test]
     fn test_basic_functionality() {
         let result = my_function("input");
         assert!(result.is_ok());
         assert_eq!(result.unwrap(), "expected");
     }
-    
+
     #[test]
     fn test_error_case() {
         let result = my_function("");
         assert!(result.is_err());
     }
-    
+
     #[test]
     #[should_panic(expected = "Invalid input")]
     fn test_panic() {
@@ -535,11 +538,11 @@ use tempfile::TempDir;
 fn test_config_lifecycle() {
     let temp = TempDir::new().unwrap();
     let config_path = temp.path().join("config.toml");
-    
+
     // Test save
     let config = config::Config::default();
     config::save(&config).unwrap();
-    
+
     // Test load
     let loaded = config::load().unwrap();
     assert_eq!(loaded.version, config.version);
@@ -686,6 +689,15 @@ cargo fmt --check
 rustfmt src/main.rs
 ```
 
+### Regenerating Shell Completions
+
+Shell completion scripts are generated by `build.rs` into the `completions/` directory.
+
+```bash
+# Regenerate completions (bash/zsh/fish)
+cargo build
+```
+
 ### Building Documentation
 
 ```bash
@@ -778,19 +790,22 @@ rust-gdb target/debug/starforge
 ### Common Issues
 
 **Issue**: Compilation errors after updating dependencies
+
 ```bash
 # Solution: Clean and rebuild
 cargo clean
 cargo build
 ```
 
 **Issue**: Tests failing intermittently
+
 ```bash
 # Solution: Run tests serially
 cargo test -- --test-threads=1
 ```
 
 **Issue**: Slow compilation
+
 ```bash
 # Solution: Use sccache
 cargo install sccache
@@ -948,6 +963,7 @@ fn creates_wallet_with_encrypted_key_when_encrypt_flag_is_true() {
 ```
 
 Types:
+
 - `feat`: New feature
 - `fix`: Bug fix
 - `docs`: Documentation
@@ -957,6 +973,7 @@ Types:
 - `chore`: Maintenance
 
 Examples:
+
 ```
 feat(wallet): add hardware wallet support
 
@@ -970,18 +987,21 @@ Closes #123
 ## Resources
 
 ### Documentation
+
 - [Rust Book](https://doc.rust-lang.org/book/)
 - [Rust API Guidelines](https://rust-lang.github.io/api-guidelines/)
 - [Stellar Documentation](https://developers.stellar.org/)
 - [Soroban Documentation](https://soroban.stellar.org/)
 
 ### Tools
+
 - [rust-analyzer](https://rust-analyzer.github.io/) - IDE support
 - [clippy](https://github.com/rust-lang/rust-clippy) - Linter
 - [rustfmt](https://github.com/rust-lang/rustfmt) - Formatter
 - [cargo-edit](https://github.com/killercup/cargo-edit) - Dependency management
 
 ### Community
+
 - [Stellar Discord](https://discord.gg/stellar)
 - [Rust Users Forum](https://users.rust-lang.org/)
 - [GitHub Discussions](https://github.com/YOUR_USERNAME/starforge/discussions)

DEVELOPER_GUIDE.md

@@ -16,11 +16,23 @@ enum Commands {
     Wallet,
     New,
     Contract,
+    Inspect,
     Deploy,
     Info,
     Tx,
     Network,
     Completions,
+    Shell,
+    Monitor,
+    Tutorial,
+    Benchmark,
+    Test,
+    Gas,
+    Plugin,
+    Template,
+    Upgrade,
+    #[command(external_subcommand)]
+    External(Vec<String>),
 }
 
 fn main() {
@@ -30,7 +42,7 @@ fn main() {
     };
 
     let mut cmd = Cli::command();
-    
+
     // Create a directory for completions in the project root for easier access
     // or just leave them in OUT_DIR as per standard practice.
     // The issue says "Install completions to appropriate system directories".
@@ -40,7 +52,8 @@ fn main() {
     fs::create_dir_all(&completions_dir).unwrap();
 
     for &shell in &[Shell::Bash, Shell::Zsh, Shell::Fish] {
-        generate_to(shell, &mut cmd, "starforge", &completions_dir).expect("Failed to generate completions");
+        generate_to(shell, &mut cmd, "starforge", &completions_dir)
+            .expect("Failed to generate completions");
     }
 
     let rustc = env::var_os("RUSTC").unwrap_or_else(|| "rustc".into());

build.rs

@@ -1,3 +1,4 @@
+pub mod benchmark;
 pub mod completions;
 pub mod contract;
 pub mod deploy;
@@ -9,12 +10,10 @@ pub mod monitor;
 pub mod network;
 pub mod new;
 pub mod plugin;
-pub mod template;
 pub mod shell;
 pub mod template;
 pub mod test;
-pub mod tx;
-pub mod wallet;
 pub mod tutorial;
-pub mod benchmark;
+pub mod tx;
 pub mod upgrade;
+pub mod wallet;