botserver/PROMPT.md

botserver Development Prompt Guide

Version: 6.1.0
Purpose: Consolidated LLM context for botserver development

---

ZERO TOLERANCE POLICY

This project has the strictest code quality requirements possible:

[lints.clippy]
all = "warn"
pedantic = "warn"
nursery = "warn"
cargo = "warn"
unwrap_used = "warn"
expect_used = "warn"
panic = "warn"
todo = "warn"

EVERY SINGLE WARNING MUST BE FIXED. NO EXCEPTIONS.

---

ABSOLUTE PROHIBITIONS

❌ NEVER use #![allow()] or #[allow()] in source code to silence warnings
❌ NEVER use _ prefix for unused variables - USE the variable (add logging)
❌ NEVER use .unwrap() - use ? or proper error handling
❌ NEVER use .expect() - use ? or proper error handling  
❌ NEVER use panic!() or unreachable!() - handle all cases
❌ NEVER use todo!() or unimplemented!() - write real code
❌ NEVER leave unused imports - DELETE them
❌ NEVER leave dead code - USE IT (add logging, make public, add fallback methods)
❌ NEVER delete unused struct fields - USE them in logging or make them public
❌ NEVER use approximate constants (3.14159) - use std::f64::consts::PI
❌ NEVER silence clippy in code - FIX THE CODE or configure in Cargo.toml
❌ NEVER use CDN links - all assets must be local
❌ NEVER run cargo check or cargo clippy - USE ONLY the diagnostics tool
❌ NEVER add comments - code must be self-documenting via types and naming
❌ NEVER add file header comments (//! or /*!) - no module docs
❌ NEVER add function doc comments (///) - types are the documentation
❌ NEVER add ASCII art or banners in code
❌ NEVER add TODO/FIXME/HACK comments - fix it or delete it

---

CARGO.TOML LINT EXCEPTIONS

When a clippy lint has technical false positives that cannot be fixed in code, disable it in Cargo.toml with a comment explaining why:

[lints.clippy]
# Disabled: has false positives for functions with mut self, heap types (Vec, String)
missing_const_for_fn = "allow"
# Disabled: Tauri commands require owned types (Window) that cannot be passed by reference
needless_pass_by_value = "allow"
# Disabled: transitive dependencies we cannot control
multiple_crate_versions = "allow"

Approved exceptions:

• missing_const_for_fn - false positives for mut self, heap types
• needless_pass_by_value - Tauri/framework requirements
• multiple_crate_versions - transitive dependencies
• future_not_send - when async traits require non-Send futures

---

MANDATORY CODE PATTERNS

Error Handling - Use ? Operator

// ❌ WRONG
let value = something.unwrap();
let value = something.expect("msg");

// ✅ CORRECT
let value = something?;
let value = something.ok_or_else(|| Error::NotFound)?;

Option Handling - Use Combinators

// ❌ WRONG
if let Some(x) = opt {
    x
} else {
    default
}

// ✅ CORRECT
opt.unwrap_or(default)
opt.unwrap_or_else(|| compute_default())
opt.map_or(default, |x| transform(x))

Match Arms - Must Be Different

// ❌ WRONG - identical arms
match x {
    A => do_thing(),
    B => do_thing(),
    C => other(),
}

// ✅ CORRECT - combine identical arms
match x {
    A | B => do_thing(),
    C => other(),
}

Self Usage in Impl Blocks

// ❌ WRONG
impl MyStruct {
    fn new() -> MyStruct { MyStruct { } }
}

// ✅ CORRECT
impl MyStruct {
    fn new() -> Self { Self { } }
}

Format Strings - Inline Variables

// ❌ WRONG
format!("Hello {}", name)
log::info!("Processing {}", id);

// ✅ CORRECT
format!("Hello {name}")
log::info!("Processing {id}");

Display vs ToString

// ❌ WRONG
impl ToString for MyType {
    fn to_string(&self) -> String { }
}

// ✅ CORRECT
impl std::fmt::Display for MyType {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result { }
}

Derive Eq with PartialEq

// ❌ WRONG
#[derive(PartialEq)]
struct MyStruct { }

// ✅ CORRECT
#[derive(PartialEq, Eq)]
struct MyStruct { }

Must Use Attributes

// ❌ WRONG - pure function without #[must_use]
pub fn calculate() -> i32 { }

// ✅ CORRECT
#[must_use]
pub fn calculate() -> i32 { }

Zero Comments Policy

// ❌ WRONG - any comments
/// Returns the user's full name
fn get_full_name(&self) -> String { }

// Validate input before processing
fn process(data: &str) { }

//! This module handles user authentication

// ✅ CORRECT - self-documenting code, no comments
fn full_name(&self) -> String { }

fn process_validated_input(data: &str) { }

Why zero comments:

• Rust's type system documents intent (Result, Option, traits)
• Comments become stale when code changes
• LLMs can infer intent from well-structured code
• Good naming > comments
• Types are the documentation

Const Functions

// ❌ WRONG - could be const but isn't
pub fn default_value() -> i32 { 42 }

// ✅ CORRECT
pub const fn default_value() -> i32 { 42 }

Pass by Reference

// ❌ WRONG - needless pass by value
fn process(data: String) { println!("{data}"); }

// ✅ CORRECT
fn process(data: &str) { println!("{data}"); }

Clone Only When Needed

// ❌ WRONG - redundant clone
let x = value.clone();
use_value(&x);

// ✅ CORRECT
use_value(&value);

Mathematical Constants

// ❌ WRONG
let pi = 3.14159;
let e = 2.71828;

// ✅ CORRECT
use std::f64::consts::{PI, E};
let pi = PI;
let e = E;

Async Functions

// ❌ WRONG - async without await
async fn process() { sync_operation(); }

// ✅ CORRECT - remove async if no await needed
fn process() { sync_operation(); }

---

Build Rules

# Development - ALWAYS debug build
cargo build
cargo check

# NEVER use release unless deploying
# cargo build --release  # NO!

---

Version Management

Version is 6.1.0 - NEVER CHANGE without explicit approval

---

Database Standards

TABLES AND INDEXES ONLY:

✅ CREATE TABLE IF NOT EXISTS
✅ CREATE INDEX IF NOT EXISTS
✅ Inline constraints

❌ CREATE VIEW
❌ CREATE TRIGGER  
❌ CREATE FUNCTION
❌ Stored Procedures

JSON Columns: Use TEXT with _json suffix, not JSONB

---

Code Generation Rules

- KISS, NO TALK, SECURED ENTERPRISE GRADE THREAD SAFE CODE ONLY
- Use rustc 1.90.0+
- No placeholders, no explanations, no comments
- All code must be complete, professional, production-ready
- REMOVE ALL COMMENTS FROM GENERATED CODE
- Always include full updated code files - never partial
- Only return files that have actual changes
- Return 0 warnings - FIX ALL CLIPPY WARNINGS
- NO DEAD CODE - implement real functionality

---

Documentation Rules

- Rust code examples ONLY in docs/reference/architecture.md
- All other docs: BASIC, bash, JSON, SQL, YAML only
- Keep only README.md and PROMPT.md at project root level

---

Frontend Rules

- Use HTMX to minimize JavaScript - delegate logic to Rust server
- NO external CDN - all JS/CSS must be local in vendor/ folders
- Server-side rendering with Askama templates returning HTML
- Endpoints return HTML fragments, not JSON (for HTMX)

---

Rust Patterns

// Random number generation
let mut rng = rand::rng();

// Database - ONLY diesel, never sqlx
use diesel::prelude::*;

// Config from AppConfig - no hardcoded values
let url = config.drive.endpoint.clone();

// Logging - all-in-one-line, unique messages, inline vars
info!("Processing request id={id} user={user_id}");

---

Dependencies

- Use diesel - remove any sqlx references
- After adding to Cargo.toml: cargo audit must show 0 vulnerabilities
- If audit fails, find alternative library
- Minimize redundancy - check existing libs before adding

---

Key Files

src/main.rs              # Entry point
src/lib.rs               # Module exports
src/basic/               # BASIC language keywords
src/core/                # Core functionality
src/shared/state.rs      # AppState definition
src/shared/utils.rs      # Utility functions
src/shared/models.rs     # Database models

---

Dependencies (Key Libraries)

Library	Version	Purpose
axum	0.7.5	Web framework
diesel	2.1	PostgreSQL ORM
tokio	1.41	Async runtime
rhai	git	BASIC scripting
reqwest	0.12	HTTP client
serde	1.0	Serialization
askama	0.12	HTML Templates

---

Efficient Warning Fix Strategy

IDE DIAGNOSTICS ARE THE SOURCE OF TRUTH - Never run cargo clippy manually.

When fixing clippy warnings in files:

1. TRUST DIAGNOSTICS - Use diagnostics() tool, not cargo commands
2. READ FULL FILE - Use read_file with line ranges to get complete file content
3. FIX ALL WARNINGS - Apply all fixes in memory before writing
4. OVERWRITE FILE - Use edit_file with mode: "overwrite" to replace entire file
5. BATCH FILES - Get diagnostics for multiple files, fix in parallel
6. RE-CHECK - Call diagnostics(path) after edits to verify fixes

This is FASTER than incremental edits. Never make single-warning fixes.

// Workflow:
1. diagnostics() - get project overview (files with warning counts)
2. diagnostics(path) - get specific warnings with line numbers
3. read_file(path, start, end) - read full file in chunks
4. edit_file(path, mode="overwrite") - write fixed version
5. diagnostics(path) - verify warnings are fixed
6. Repeat for next file

IMPORTANT: Diagnostics may be stale after edits. Re-read the file or call diagnostics again to refresh.

---

Current Warning Status (Session 10)

Completed Files (0 warnings):

• main.rs - Fixed unused imports, io_other_error, redundant field names
• score_lead.rs - Fixed redundant clones, map_or→is_some_and, manual_clamp
• auto_task.rs - Fixed derive Eq, use_self in impl blocks
• hear_talk.rs - Renamed from_str→parse_type, removed redundant clones

High-Priority Files Remaining:

File	Warnings	Main Issues
crm/attendance.rs	27	manual_let_else, redundant_clone, comparison_chain
attendance/llm_assist.rs	25	trim_split_whitespace, format_push_string, match_same_arms
core/bootstrap/mod.rs	24	unused_self, if_not_else, unnecessary_debug_formatting
drive/vectordb.rs	24	dead_code, significant_drop_tightening, unused_async
http_operations.rs	22	TBD
add_bot.rs	22	TBD
api_tool_generator.rs	22	TBD
document_processor.rs	22	TBD
llm/observability.rs	21	TBD
mcp_client.rs	21	TBD
llm/local.rs	21	TBD
console/mod.rs	20	TBD

Common Fix Patterns for Remaining Files:

• manual_let_else: let x = match opt { Some(v) => v, None => return } → let Some(x) = opt else { return }
• redundant_clone: Remove .clone() on last usage of variable
• format_push_string: s.push_str(&format!(...)) → use std::fmt::Write; let _ = write!(s, ...)
• unnecessary_debug_formatting: {:?} on PathBuf → {path.display()}
• if_not_else: if !x { a } else { b } → if x { b } else { a }
• match_same_arms: Combine identical arms with |
• or_fun_call: .unwrap_or(fn()) → .unwrap_or_else(fn)
• unused_self: Convert to associated function with Self::method() calls
• significant_drop_tightening: Wrap lock in block { let guard = lock.await; use(guard); }

---

Remember

• ZERO WARNINGS - Every clippy warning must be fixed
• ZERO COMMENTS - No comments, no doc comments, no file headers, no ASCII art
• NO ALLOW IN CODE - Never use #[allow()] in source files
• CARGO.TOML EXCEPTIONS OK - Disable lints with false positives in Cargo.toml with comment
• NO DEAD CODE - Delete unused code, never prefix with _
• NO UNWRAP/EXPECT - Use ? operator or proper error handling
• NO APPROXIMATE CONSTANTS - Use std::f64::consts
• 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
• USE DIAGNOSTICS - Use IDE diagnostics tool, never call cargo clippy directly
• PASS BY REF - Don't clone unnecessarily
• CONST FN - Make functions const when possible
• MUST USE - Add #[must_use] to pure functions
• diesel: No sqlx references
• Sessions: Always retrieve by ID when present
• Config: Never hardcode values, use AppConfig
• Bootstrap: Never suggest manual installation
• Version: Always 6.1.0 - do not change
• Migrations: TABLES AND INDEXES ONLY
• JSON: Use TEXT columns with _json suffix
• Session Continuation: When running out of context, create detailed summary: (1) what was done, (2) what remains, (3) specific files and line numbers, (4) exact next steps.

---

Monitor Keywords (ON EMAIL, ON CHANGE)

ON EMAIL

ON EMAIL "support@company.com"
    email = GET LAST "email_received_events"
    TALK "New email from " + email.from_address
END ON

ON CHANGE

ON CHANGE "gdrive://myaccount/folder"
    files = GET LAST "folder_change_events"
    FOR EACH file IN files
        TALK "File changed: " + file.name
    NEXT
END ON

TriggerKind Enum:

• Scheduled = 0
• TableUpdate = 1
• TableInsert = 2
• TableDelete = 3
• Webhook = 4
• EmailReceived = 5
• FolderChange = 6