Repository navigation
Batch 19: Skills framework - #35
Merged
Merged
Conversation
…ep authored skills and base.md upgrades - Import confirm carries the previewed package digest and is refused if the package changed; the panel sends it. - Skill endpoints cap request bodies (imports 22MB, others 512KB). - The personal skill import endpoint is removed (no UI used it); users author skills, admins import. - New skill refuses a name already in use (409). - Creates and imports of one name are serialized; Save and Approve are disabled while in flight; re-import only self-heals a real not-found. - A shard can't bind two skills of one name; existing pairs serve the owner's. - Saving the system prompt unchanged doesn't create an override; both settings are written in one transaction. - Shard skill prompt blocks are capped like chat, keeping built-ins. - Weather forecasts use the location's dates; the weather cache is bounded. - Skill catalog and chat toggle fixes in the panel. Breaking: scripted imports must send the preview's digest. Tests cover each change. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UwpkZbULrgj6JLeSra78AT
sixvolts
force-pushed
the
ci/fix-18-platform
branch
from
September 28, 2026 01:47
32e85e4 to
93907a2
Compare
sixvolts
force-pushed
the
ci/fix-19-skills
branch
from
September 28, 2026 01:47
4f73730 to
48e05e8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on
ci/fix-18-platform(PR 14 of 16 precedes it). Merge with "Create a merge commit" (squash or rebase would re-mint untested commits), one PR at a time, bottom-up. GitHub retargets this PR when the one below it merges and its branch is deleted.fix(skills): admit only the previewed package, cap request bodies, keep authored skills and base.md upgrades
the package changed; the panel sends it.
author skills, admins import.
disabled while in flight; re-import only self-heals a real not-found.
owner's.
settings are written in one transaction.
bounded.
Breaking: scripted imports must send the preview's digest.
Tests cover each change.
Testing
The icecube tiers passed on this PR's tree before a migration-lock commit was added under the stack, as the first commit of #21: Go tests without and with Postgres, Playwright without a model, and Playwright against a local model. That commit changes only internal/db/migrate.go and its test, and the db package's tests pass with Postgres at every commit of the stack. The CI run on this PR tests the head as pushed.
🤖 Generated with Claude Code
https://claude.ai/code/session_01UwpkZbULrgj6JLeSra78AT