docs: add SKILL_STYLE.md and align contribution guidelines - #14
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.
Closes #3
What changed
Added
SKILL_STYLE.mdat the repo root defining the OxCode skill philosophy, the six core disciplinary rules with real worked before/after examples, frontmatter reference, guidelines for choosingtoolsandworkspace, and testing standards.Updated
CONTRIBUTING.mdto linkSKILL_STYLE.md, state that PRs ignoring it will be sent back, and require before/after proof on real tasks.Updated
README.mdto linkSKILL_STYLE.md, fix plugin paths in the table, and listtriage.Updated
template/SKILL.mdso new skills scaffold directly from the style guide (defaulting totools: chatfor judgement skills and outlining disciplined rule sections).Before and After Comparison on a Real Task
Task: Reviewing a diff where a method signature changed in
user_service.pywithout all callers included in the changeset.
Before (generic/unconstrained prompt):
"Ensure all caller functions across the codebase handle null values returned by
this updated method." (guesses past missing context and makes claims without checking).
After (following
SKILL_STYLE.mdrules):"Cannot verify caller handling from this diff because
user_service.pyis outsidethe changeset." (refuses to claim what wasn't checked; treats missing context as a finding).
What was deliberately left out and why
Model-specific prompt tuning or mentioning LLM providers: OxCode routes dynamically via
capability, so naming models or prompt tricks tied to a specific provider version was excluded to avoid obsolescence.Prescriptive domain templates: The guide defines the discipline and standard of
truth rather than attempting to catalog every possible software domain.
Tool Configuration Verification
Read-only review and judgement skills default to
tools: chat.Active skills (like
test-hardening) requiretools: fullto execute mutations and run tests on disk rather than hallucinating outputs.