fix typo

[iceljc]

PR Type

Bug fix

---

Description

• Fix typo in property name InnderIndex to InnerIndex

• Update all references across codebase to use corrected property name

---

Diagram Walkthrough

flowchart LR
  A["UpdateMessageRequest<br/>InnderIndex typo"] -- "Fix property name" --> B["UpdateMessageRequest<br/>InnerIndex"]
  B -- "Update references" --> C["FileRepository"]
  B -- "Update references" --> D["ConversationController"]
  B -- "Update references" --> E["MongoRepository"]

Loading

File Walkthrough

Relevant files

Bug fix

UpdateMessageRequest.cs Fix property name typo                                                                      src/Infrastructure/BotSharp.Abstraction/Conversations/Models/UpdateMessageRequest.cs Corrected property name from InnderIndex to InnerIndex +1/-1      FileRepository.Conversation.cs Update property reference in FileRepository                            src/Infrastructure/BotSharp.Core/Repository/FileRepository/FileRepository.Conversation.cs Updated reference to use corrected InnerIndex property name in UpdateConversationMessage method +1/-1      ConversationController.cs Update property assignment in controller                                  src/Infrastructure/BotSharp.OpenAPI/Controllers/Conversation/ConversationController.cs Updated property assignment to use corrected InnerIndex property name +1/-1      MongoRepository.Conversation.cs Update property reference in MongoRepository                          src/Plugins/BotSharp.Plugin.MongoStorage/Repository/MongoRepository.Conversation.cs Updated reference to use corrected InnerIndex property name in UpdateConversationMessage method +1/-1

---

[qodo-code-review]

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢	No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪	🎫 No ticket provided Create ticket/issue
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
🟢	Generic: Comprehensive Audit Trails Objective: To create a detailed and reliable record of critical system actions for security analysis and compliance. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Meaningful Naming and Self-Documenting Code Objective: Ensure all identifiers clearly express their purpose and intent, making code self-documenting Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Robust Error Handling and Edge Case Management Objective: Ensure comprehensive error handling that provides meaningful context and graceful degradation Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Secure Error Handling Objective: To prevent the leakage of sensitive system information through error messages while providing sufficient detail for internal debugging. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Secure Logging Practices Objective: To ensure logs are useful for debugging and auditing without exposing sensitive information like PII, PHI, or cardholder data. Status: Passed Learn more about managing compliance generic rules or creating your own custom rules
	Generic: Security-First Input Validation and Data Handling Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent vulnerabilities Status: Passed Learn more about managing compliance generic rules or creating your own custom rules

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

[qodo-code-review]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Use more efficient index-based access Replace the use of Where with a lambda to find an element by index with the more performant and readable ElementAtOrDefault() method. src/Infrastructure/BotSharp.Core/Repository/FileRepository/FileRepository.Conversation.cs [240] -var found = candidates.Where((_, idx) => idx == request.InnerIndex).FirstOrDefault(); +var found = candidates.ElementAtOrDefault(request.InnerIndex); Apply / Chat Suggestion importance[1-10]: 5 __ Why: The suggestion correctly proposes replacing Where().FirstOrDefault() with ElementAtOrDefault() for index-based access, which improves both code readability and performance.	Low
More