Skip to content

fix: a skill is delivered whole or refused; one big skill no longer hides the rest - #71

Open
alexeybe1kin wants to merge 8 commits into
mainfrom
fix/skill-size-limits
Open

alexeybe1kin wants to merge 8 commits into
mainfrom
fix/skill-size-limits

Conversation

@alexeybe1kin

Copy link
Copy Markdown
Collaborator

Hunt findings memorygate-2 F2 and F3.

  • A skill body was accepted up to 12000 characters but cut to 8192 bytes on every read, so an editor that loaded and saved it stored the cut text. Now create and edit refuse a skill whose title and body exceed 8192 bytes (422), and reads return the stored text.
  • In /context/skills, a skill that did not fit the remaining budget ended the loop (break), dropping every smaller skill after it. It is now skipped (continue).

Tests: two new tests in tests/test_skills.py, both failing on main. ruff and types ok.

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