Skip to content

Report Skill upload and update field errors - #860

Merged
SaladDay merged 3 commits into
mainfrom
fix/skill-upload-field-errors
Oct 25, 2026
Merged

SaladDay merged 3 commits into
mainfrom
fix/skill-upload-field-errors

Conversation

@SaladDay

@SaladDay SaladDay commented Oct 24, 2026 •

Copy link
Copy Markdown
Collaborator

Official API alignment (FS-01/02/03, FS-04/05, FS-12, FS-13/14).

  • Skill uploads report field errors: an invalid bundle is invalid_value with param: "files"; a missing files part, default on create and a bad default name their param. default accepts true/false in any case.
  • The public upload rejects a name with _ and a description over 1024 characters; installed and plugin Skills are unchanged.
  • skills.update body errors name their param; a missing default_version returns 404 with param: "default_version".
  • Skill version downloads are application/zip named <name>-<version>.zip.
  • Docs (source-files.md + zh) updated.

Checks: go test ./internal/api -run 'Skill|ServeStored|SourceFiles', ./internal/skills/... ./internal/persistence/postgres/skillpg and ./tests/integration -run Skill against a test database, go vet ./..., gofmt.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@blacksmith-sh

This comment has been minimized.

@SaladDay
SaladDay force-pushed the fix/skill-upload-field-errors branch from 82ff4af to 13cb8f5 Compare October 25, 2026 02:43
@SaladDay
SaladDay merged commit 98a63c8 into main Oct 25, 2026
1 check passed
@SaladDay
SaladDay deleted the fix/skill-upload-field-errors branch October 25, 2026 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant