Add initial Claude Skills, and apply them#84
Merged
Merged
Conversation
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.
Purpose
Content
settings.json- it encourages users to use skills and subagents. It prevents Claude using non-read-type git commands.skill-creator- to build new skills.base-showadds show and summary methods to structsdocstringsmanages the docstring formatting and managesdocs/src/APIerror-message-managermanages the error messaging format, to ensure actionable, context-laden messagesbase-showwas used to create show and summary methods for all structs. Includes both a compact and full show-type method (e.g. so that in arrays the compact 1-line show() method is used). unit tests are also written for this.docstringsmethods was used to overhaul the docstrings.error-message-managerwas used to remove@assertand update all error messages, it also addstest_throwerror messages tested in the unit tests. Any longer>3 linemessage is now put into a helper function_throw_xyzthat are listed at the bottom of files.skill-creatorensures that skills have a self-improvement request after every application. Skills were performed a couple of times, using this self-improvement to improve their performance. Also I had it add examples from the repo into skills to help with consistency.Examples:
Base-show
full show methods, compact methods for vectors, and summaries in
show.jldocstrings
Now structs contain typedef, and recommended constructors/factories (even if names differ from struct name).
Also the docs API has been added to
error-message-manager
Constructs error-messages with context
Also i like this neat
@test_throwstype error block to ensure (the main expected/got are retained)