add file_count for InstructCompletion

[yileicn]

PR Type

Enhancement

---

Description

• Add file_count state tracking for InstructCompletion requests

• Set file_count state when Files collection is not empty

---

Diagram Walkthrough

flowchart LR
  input["InstructMessageModel input"] -- "check Files collection" --> check{"Files not empty?"}
  check -- "yes" --> setState["SetState file_count"]
  check -- "no" --> skip["Skip state setting"]
  setState --> state["IConversationStateService"]
  skip --> state

Loading

File Walkthrough

	Relevant files
Enhancement	InstructModeController.cs Add file_count state tracking logic                                            src/Infrastructure/BotSharp.OpenAPI/Controllers/InstructModeController.cs Added conditional check for non-empty Files collection in input Set file_count state with the count of files when available Inserted blank line for code readability +2/-0

---

[qodo-merge-pro]

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 <!-- /create_ticket --create_ticket=true --> </details></td></tr>
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
⚪	No custom compliance provided Follow the guide to enable custom compliance check.
Update

Compliance status legend

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

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	✅ Add missing state source parameter Suggestion Impact: The commit updated the SetState call for "file_count" to include source: StateSource.External exactly as suggested. code diff: - if (!input.Files.IsNullOrEmpty()) state.SetState("file_count", input.Files.Count); + if (!input.Files.IsNullOrEmpty()) state.SetState("file_count", input.Files.Count, source: StateSource.External); Add the missing source: StateSource.External parameter to the state.SetState call to maintain consistency with other state modifications in the method. src/Infrastructure/BotSharp.OpenAPI/Controllers/InstructModeController.cs [39] -if (!input.Files.IsNullOrEmpty()) state.SetState("file_count", input.Files.Count); +if (!input.Files.IsNullOrEmpty()) state.SetState("file_count", input.Files.Count, source: StateSource.External); [Suggestion processed] Suggestion importance[1-10]: 6 __ Why: The suggestion correctly points out an inconsistency with surrounding code, where all other SetState calls include the source parameter. Applying this change improves code consistency and robustness.	Low
Update