C# convention (and the rest of this codebase) uses PascalCase for named constants. Two local constants in src/Infrastructure/Plugins/FileSystemPlugin.cs use camelCase instead:
| Line |
Current |
Correct |
| 592 |
const int maxFiles = 500 |
const int MaxFiles = 500 |
| 874 |
const int maxEntries = 500 |
const int MaxEntries = 500 |
Future consideration: FileSystemPlugin already accepts readFileSizeLimit and readBudgetPerTurn as constructor params plumbed from SecurityConfig. These list-truncation limits follow the same pattern and could eventually be promoted to SecurityConfig as something like MaxListEntriesPerCall. That is out of scope for this issue — rename first, then consider promotion separately.
Files to touch: src/Infrastructure/Plugins/FileSystemPlugin.cs
C# convention (and the rest of this codebase) uses
PascalCasefor named constants. Two local constants insrc/Infrastructure/Plugins/FileSystemPlugin.csuse camelCase instead:const int maxFiles = 500const int MaxFiles = 500const int maxEntries = 500const int MaxEntries = 500Future consideration:
FileSystemPluginalready acceptsreadFileSizeLimitandreadBudgetPerTurnas constructor params plumbed fromSecurityConfig. These list-truncation limits follow the same pattern and could eventually be promoted toSecurityConfigas something likeMaxListEntriesPerCall. That is out of scope for this issue — rename first, then consider promotion separately.Files to touch:
src/Infrastructure/Plugins/FileSystemPlugin.cs