From 2765fa2ebadc91435e8d90f068b4c96dbb77329b Mon Sep 17 00:00:00 2001 From: "Rodrigo Rodriguez (Pragmatismo)" Date: Wed, 4 Feb 2026 13:54:26 -0300 Subject: [PATCH] Update: delete PROMPT.md and update files --- PROMPT.md | 114 --------------------------------- README.md | 171 ++++++++++++++++++++++++++++++++++++++++++++++++- src/logging.rs | 12 ++-- 3 files changed, 175 insertions(+), 122 deletions(-) delete mode 100644 PROMPT.md diff --git a/PROMPT.md b/PROMPT.md deleted file mode 100644 index 44b16e0..0000000 --- a/PROMPT.md +++ /dev/null @@ -1,114 +0,0 @@ -# BotLib Development Guide - -**Version:** 6.2.0 -**Purpose:** Shared library for General Bots workspace - ---- - -## ZERO TOLERANCE POLICY - -**EVERY SINGLE WARNING MUST BE FIXED. NO EXCEPTIONS.** - ---- - -## ❌ ABSOLUTE PROHIBITIONS - -``` -❌ NEVER use #![allow()] or #[allow()] in source code -❌ NEVER use _ prefix for unused variables - DELETE or USE them -❌ NEVER use .unwrap() - use ? or proper error handling -❌ NEVER use .expect() - use ? or proper error handling -❌ NEVER use panic!() or unreachable!() -❌ NEVER use todo!() or unimplemented!() -❌ NEVER leave unused imports or dead code -❌ NEVER add comments - code must be self-documenting -``` - ---- - -## 🏗️ MODULE STRUCTURE - -``` -src/ -├── lib.rs # Public exports, feature gates -├── error.rs # Error types (thiserror) -├── models.rs # Shared data models -├── message_types.rs # Message type definitions -├── http_client.rs # HTTP client wrapper (feature-gated) -├── branding.rs # Version, branding constants -└── version.rs # Version information -``` - ---- - -## ✅ MANDATORY CODE PATTERNS - -### Error Handling - -```rust -// ❌ WRONG -let value = something.unwrap(); - -// ✅ CORRECT -let value = something?; -let value = something.ok_or_else(|| Error::NotFound)?; -``` - -### Self Usage - -```rust -impl MyStruct { - fn new() -> Self { Self { } } // ✅ Not MyStruct -} -``` - -### Format Strings - -```rust -format!("Hello {name}") // ✅ Not format!("{}", name) -``` - -### Display vs ToString - -```rust -// ❌ WRONG -impl ToString for MyType { } - -// ✅ CORRECT -impl std::fmt::Display for MyType { } -``` - -### Derive Eq with PartialEq - -```rust -#[derive(PartialEq, Eq)] // ✅ Always both -struct MyStruct { } -``` - ---- - -## 📦 KEY DEPENDENCIES - -| Library | Version | Purpose | -|---------|---------|---------| -| anyhow | 1.0 | Error handling | -| thiserror | 2.0 | Error derive | -| chrono | 0.4 | Date/time | -| serde | 1.0 | Serialization | -| uuid | 1.11 | UUIDs | -| diesel | 2.1 | Database ORM | -| reqwest | 0.12 | HTTP client | - ---- - -## 🔑 REMEMBER - -- **ZERO WARNINGS** - Every clippy warning must be fixed -- **NO ALLOW IN CODE** - Never use #[allow()] in source files -- **NO DEAD CODE** - Delete unused code -- **NO UNWRAP/EXPECT** - Use ? operator -- **INLINE FORMAT ARGS** - `format!("{name}")` not `format!("{}", name)` -- **USE SELF** - In impl blocks, use Self not the type name -- **DERIVE EQ** - Always derive Eq with PartialEq -- **DISPLAY NOT TOSTRING** - Implement Display, not ToString -- **Version 6.2.0** - do not change without approval \ No newline at end of file diff --git a/README.md b/README.md index 8620c74..4d4ca59 100644 --- a/README.md +++ b/README.md @@ -1,3 +1,170 @@ -General Bots® base library for building Node.js TypeScript Apps packages (.gbapp). +# BotLib - General Bots Shared Library -See: https://github.com/pragmatismo-io/botserver for main documentation. \ No newline at end of file +**Version:** 6.2.0 +**Purpose:** Shared library for General Bots workspace + +--- + +## Overview + +BotLib is the foundational shared library for the General Bots workspace, providing common types, error handling, HTTP client functionality, and utilities used across all projects. It serves as the core dependency for botserver, botui, botapp, and other workspace members, ensuring consistency and reducing code duplication. + +For comprehensive documentation, see **[docs.pragmatismo.com.br](https://docs.pragmatismo.com.br)** or the **[BotBook](../botbook)** for detailed guides and API references. + +--- + +## 🏗️ Module Structure + +``` +src/ +├── lib.rs # Public exports, feature gates +├── error.rs # Error types (thiserror) +├── models.rs # Shared data models +├── message_types.rs # Message type definitions +├── http_client.rs # HTTP client wrapper (feature-gated) +├── branding.rs # Version, branding constants +└── version.rs # Version information +``` + +--- + +## ✅ ZERO TOLERANCE POLICY + +**EVERY SINGLE WARNING MUST BE FIXED. NO EXCEPTIONS.** + +### Absolute Prohibitions + +``` +❌ NEVER use #![allow()] or #[allow()] in source code +❌ NEVER use _ prefix for unused variables - DELETE or USE them +❌ NEVER use .unwrap() - use ? or proper error handling +❌ NEVER use .expect() - use ? or proper error handling +❌ NEVER use panic!() or unreachable!() +❌ NEVER use todo!() or unimplemented!() +❌ NEVER leave unused imports or dead code +❌ NEVER add comments - code must be self-documenting +``` + +--- + +## 📦 Key Dependencies + +| Library | Version | Purpose | +|---------|---------|---------| +| anyhow | 1.0 | Error handling | +| thiserror | 2.0 | Error derive | +| chrono | 0.4 | Date/time | +| serde | 1.0 | Serialization | +| uuid | 1.11 | UUIDs | +| diesel | 2.1 | Database ORM | +| reqwest | 0.12 | HTTP client | + +--- + +## 🔧 Features + +### Feature Gates + +BotLib uses Cargo features to enable optional functionality: + +```toml +[features] +default = [] +http-client = ["reqwest"] # Enable HTTP client +# Add more features as needed +``` + +### Using Features + +```toml +# In dependent crate's Cargo.toml +[dependencies.botlib] +workspace = true +features = ["http-client"] # Enable HTTP client +``` + +--- + +## ✅ Mandatory Code Patterns + +### Error Handling + +```rust +// ❌ WRONG +let value = something.unwrap(); + +// ✅ CORRECT +let value = something?; +let value = something.ok_or_else(|| Error::NotFound)?; +``` + +### Self Usage + +```rust +impl MyStruct { + fn new() -> Self { Self { } } // ✅ Not MyStruct +} +``` + +### Format Strings + +```rust +format!("Hello {name}") // ✅ Not format!("{}", name) +``` + +### Display vs ToString + +```rust +// ❌ WRONG +impl ToString for MyType { } + +// ✅ CORRECT +impl std::fmt::Display for MyType { } +``` + +### Derive Eq with PartialEq + +```rust +#[derive(PartialEq, Eq)] // ✅ Always both +struct MyStruct { } +``` + +--- + +## 📚 Documentation + +For complete documentation, guides, and API references: + +- **[docs.pragmatismo.com.br](https://docs.pragmatismo.com.br)** - Full online documentation +- **[BotBook](../botbook)** - Local comprehensive guide with tutorials and examples +- **[General Bots Repository](https://github.com/GeneralBots/BotServer)** - Main project repository + +--- + +## 🔗 Related Projects + +- **[botserver](https://github.com/GeneralBots/botserver)** - Main API server +- **[botui](https://github.com/GeneralBots/botui)** - Web UI interface +- **[botapp](https://github.com/GeneralBots/botapp)** - Desktop application +- **[botbook](https://github.com/GeneralBots/botbook)** - Documentation + +--- + +## 🔑 Remember + +- **ZERO WARNINGS** - Every clippy warning must be fixed +- **NO ALLOW IN CODE** - Never use #[allow()] in source files +- **NO DEAD CODE** - Delete unused code +- **NO UNWRAP/EXPECT** - Use ? operator +- **INLINE FORMAT ARGS** - `format!("{name}")` not `format!("{}", name)` +- **USE SELF** - In impl blocks, use Self not the type name +- **DERIVE EQ** - Always derive Eq with PartialEq +- **DISPLAY NOT TOSTRING** - Implement Display, not ToString +- **Version 6.2.0** - Do not change without approval +- **GIT WORKFLOW** - ALWAYS push to ALL repositories (github, pragmatismo) + +--- + +## 📄 License + +AGPL-3.0 - See [LICENSE](LICENSE) for details. \ No newline at end of file diff --git a/src/logging.rs b/src/logging.rs index 4870ff0..a3ee7a8 100644 --- a/src/logging.rs +++ b/src/logging.rs @@ -25,7 +25,7 @@ pub fn compact_format(buf: &mut Formatter, record: &Record) -> std::io::Result<( // Length: 18 + 1 + 1 + 1 + module.len() + 1 = 22 + module.len() let prefix = format!("{} {} {}:", timestamp, level_char, module); - // Max width 80 + // Max width 100 // If prefix + message fits, print it. // Else, wrap. // Indent for wrapping is 21 spaces (18 timestamp + 1 space + 1 level + 1 space) @@ -38,15 +38,15 @@ pub fn compact_format(buf: &mut Formatter, record: &Record) -> std::io::Result<( let message = record.args().to_string(); let indent = " "; // 21 spaces - if prefix.len() + message.len() <= 80 { + if prefix.len() + message.len() <= 100 { writeln!(buf, "{}{}", prefix, message) } else { - let available_first_line = if prefix.len() < 80 { - 80 - prefix.len() + let available_first_line = if prefix.len() < 100 { + 100 - prefix.len() } else { 0 }; - let available_other_lines = 80 - 21; // 59 chars + let available_other_lines = 100 - 21; // 99 chars let mut current_pos = 0; let chars: Vec = message.chars().collect(); @@ -55,7 +55,7 @@ pub fn compact_format(buf: &mut Formatter, record: &Record) -> std::io::Result<( // First line write!(buf, "{}", prefix)?; - // If prefix is already >= 80, we force a newline immediately? + // If prefix is already >= 100, we force a newline immediately? // Or we just print a bit and wrap? // Let's assume typical usage where module name isn't huge.