-
Notifications
You must be signed in to change notification settings - Fork 0
feat(skills): add just-commands skill with dynamic context injection #133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
3077f9c to
33d8b2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 2 files
Add a new skill that uses Claude Code's dynamic context injection feature to embed the output of `just --list` at runtime. This replaces the static command list in CLAUDE.md with a reference to the skill. Benefits: - Commands stay in sync with justfile automatically - Reduces manual maintenance of CLAUDE.md - Demonstrates Claude Code's !`command` syntax for dynamic context
33d8b2c to
aa80fd4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds a Claude Code “skill” that dynamically injects the current just --list output at runtime, and removes the now-stale hardcoded command list from CLAUDE.md.
Changes:
- Add
.claude/skills/just-commands/SKILL.mdusing dynamic context injection to embedjust --list. - Update
CLAUDE.mdto point developers (and Claude) tojust --list/ the new skill instead of a static list.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
CLAUDE.md |
Replaces the hardcoded “Essential Development Commands” block with guidance to use just --list / the new skill. |
.claude/skills/just-commands/SKILL.md |
Introduces a skill that injects just --list output dynamically. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | Skill | Description | | ||
| | ----------------- | -------------------------------------------- | | ||
| | **just-commands** | Available just commands (dynamically loaded) | |
Copilot
AI
Jan 31, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sentence claims Claude will “automatically reference” the just-commands skill, but earlier in this same file skills are described as “Manually invoked”. Please align the documentation (either clarify that this skill is auto-loaded, or update this line to describe the manual invocation mechanism).
| | Skill | Description | | |
| | ----------------- | -------------------------------------------- | | |
| | **just-commands** | Available just commands (dynamically loaded) | | |
| | Skill | Description | | |
| | ----------------- | -------------------------------------------------------- | | |
| | **just-commands** | Manually invoked skill that lists available just commands | |
| just lint # Run ruff linting | ||
| just lint-fix # Auto-fix linting issues | ||
| just ty # Run type checking | ||
| ## Project Overview |
Copilot
AI
Jan 31, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR removes the static command list from CLAUDE.md to avoid drift, but there is still a static “Essential Commands” list in .claude/rules/development-workflow.md that appears out of sync with the current justfile (e.g., it references just lint-fix but the justfile defines format). Consider updating that rule to reference this skill or to use just --list as well, so there is a single source of truth.
| --- | ||
| name: just-commands | ||
| description: Available just commands for this project. Use when you need to know what development commands are available. | ||
| user-invocable: false |
Copilot
AI
Jan 31, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
user-invocable: false conflicts with CLAUDE.md referencing this skill as a way to discover commands. If the intent is for developers to manually invoke it, set it to user-invocable; if the intent is automatic loading, update the surrounding docs in CLAUDE.md to reflect that skills are not purely manual in this repo.
| user-invocable: false | |
| user-invocable: true |
Summary
Add a new skill that uses Claude Code's dynamic context injection feature (
!command`` syntax) to embedjust --listoutput at runtime, replacing the static command list in CLAUDE.md.What Changed
.claude/skills/just-commands/SKILL.mdwith dynamicjust --listinjectionCLAUDE.mdto reference the skill instead of hardcoding commandsWhy
The static command list in CLAUDE.md required manual updates whenever the justfile changed. Using dynamic context injection keeps commands in sync automatically and demonstrates a useful Claude Code feature.
Summary by cubic
Adds a just-commands skill that injects the output of
just --listat runtime, replacing the static command list in CLAUDE.md. This keeps commands in sync and removes manual updates.just --list.Written for commit aa80fd4. Summary will update on new commits.