 Mukund Raghav Sharma (Moko)
|
c92faa7800
|
Add minimal-api-file-upload skill (#264)
* Move minimal-api-file-upload skill to aspnetcore plugin with 3 eval scenarios
Per repo restructuring feedback, ASP.NET Core specific skills should
be under the aspnetcore plugin rather than the dotnet plugin.
* Increase eval timeouts to 180s to avoid timeout-related scoring penalties
* Generalized and addressed feedback
* Improve eval scenarios: remove noisy tests, add dual-limit and security review scenarios
* Add CODEOWNERS for aspnetcore plugin
* Fix eval: enrich prompts for skill activation, bump security review timeout to 240s
* Drop size-limit scenario (baseline already 5.0), keep 3 high-signal scenarios
* Reframe security review as file upload fix to improve skill activation
* Review
* Contribution guide followed
* Updated name
* Apply suggestion from @Copilot
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update plugins/dotnet-aspnet/skills/minimal-api-file-upload/SKILL.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Address PR #264 review feedback from Brennan and Copilot
- Rename plugin from aspnetcore to dotnet-aspnet per repo conventions
- Fix streaming filename: remove user-controlled extension, use bare GUID
- Remove explicit sanitization code (Option B) per Brennan's feedback
- Add case-insensitive MIME type comparison
- Validate ContentType matches magic bytes (reject mismatches)
- Fix stream rewind + CopyToAsync: copy from already-opened stream
- Reword streaming section to accurately describe buffering vs chunks
- Update marketplace.json, README.md, CODEOWNERS for dotnet-aspnet rename
---------
Co-authored-by: Viktor Hofer <7412651+ViktorHofer@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
2026-03-31 10:41:25 +02:00 |
|