docs: Claude skill 'prefer-sedx-over-sed' + README refresh#6
Merged
Conversation
Mirrors sel's claude-skill layout: contrib/claude-skill/prefer-sedx-over-sed/SKILL.md contrib/claude-skill/install.sh contrib/claude-skill/README.md The skill nudges Claude to prefer sedx over GNU/BSD sed for file-editing work whenever sedx is installed, highlighting --dry-run / automatic backup / rollback as the safety wins. Deliberately sits alongside (not replaces) docs/CLAUDE_SKILL.md — the former is a focused 'when to switch' trigger, the latter is the long-form reference.
- README: new 'Claude Code integration' section with one-liner install for the prefer-sedx-over-sed skill, cross-links to the contrib/ files and the long-form docs/CLAUDE_SKILL.md reference. - docs/CLAUDE_SKILL.md: add a pointer at the top clarifying that this is the long-form reference and the installable skill lives in contrib/claude-skill/ (single source of truth). - TEST_SUITE_SUMMARY.txt: removed — it describes the old shell test suite (tests/fixtures/, tests/scripts/) that was replaced by the Rust integration tests in Spec A. Obsolete.
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
1 participant
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.
Summary
Adds a focused "when to use sedx instead of sed" Claude skill (with one-liner installer) mirroring sel's
contrib/claude-skill/pattern, plus README and docs refresh to surface it.contrib/claude-skill/prefer-sedx-over-sed/SKILL.md— short, trigger-focused skill with a translation table and "when not to rewrite" guardrails.contrib/claude-skill/install.sh— POSIX installer with--user/--project/--dest/--uninstall/--force, supports local-repo copy ANDcurl | shremote fetch (viaSEDX_SKILL_REPO/SEDX_SKILL_REF).contrib/claude-skill/README.md— one-liner install instructions + pointers.docs/CLAUDE_SKILL.md— pointer at the top clarifying this is the long-form reference; the installable skill lives incontrib/.TEST_SUITE_SUMMARY.txt— removed, described the old shell test suite replaced in Spec A.Test plan
./contrib/claude-skill/install.sh --dest /tmp/testround-trip (install + uninstall) works locallycargo test,cargo clippy --all-targets -- -D warnings,cargo fmt --checkgreencurl | shone-liner works againstmain