feat(cli): show author and source after run and install - #130
Open
harlan-zw wants to merge 3 commits into
Open
Conversation
Every surface that shows a remote Skill now names the GitHub owner and Repository, the commit, and the exact SKILL.md URL at that commit. - `skilld run`: headline `name · owner/repository @ commit`, then `Read it first: <SKILL.md URL>` after the source status caution. JSON origin gains owner, repository, skillPath, commit, sourceUrl. - `skilld install`: the same block plus the source status and its meaning for verified, local, and unverified. Replaces the bare "Review the unverified Skill before use." hint. - `skilld search`: `owner/repository` before the star count in human, plain, and JSON rows. - `skilld view`: the Source link points at the exact SKILL.md. `Host::install` and `Host::install_request` return `InstalledSkill` (name, locked source, source status) instead of names. Claude-Session: https://claude.ai/code/session_018T67Ndp8FAjnHWthXABbJW
Contributor
🤖 READY · 88/100
|
…output # Conflicts: # crates/skilld-command/src/lib.rs
The merge with main brought in the openclaw target test from #129, which compared the install result with plain names. The result now carries author and source per Skill. Claude-Session: https://claude.ai/code/session_018T67Ndp8FAjnHWthXABbJW
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.
What
From any CLI surface that shows a Skill, the user reaches the exact SKILL.md in the author's Repository in one click (VISION principle 1, rebrand plan Workstream 2).
skilld run: after the transient notice, a headlinename · owner/repository @ commit, andRead it first: <SKILL.md URL at that commit>after the source status caution. Also on--filereads.skilld install: the same block, plusSource statusand its plain meaning forverified,local, andunverified. Replaces the bare "Review the unverified Skill before use." hint.skilld search:owner/repositorybefore the star count (human,--plain,--json).skilld view: the Source link now points at the exact SKILL.md, not the Repository root.--jsonforrun: origin gainsowner,repository,skillPath,commit,sourceUrl.sourceStatuswas already present.Host::installandHost::install_requestreturnInstalledSkill(name, locked source, source status) instead of bare names.Real output
Built from this branch and run against
skilld-dev/skills. The hosted path (skilld:skilld-dev/skills/find-skill) returnedRESOLUTION_TIMEOUTfrom skilld.dev during capture, so the capture below uses--direct. The verified variant is covered by tests (cli_install_shows_the_author_the_source_status_and_the_exact_skill_file).skilld run github:skilld-dev/skills/skills/find-skill --direct --plainskilld install github:skilld-dev/skills/skills/find-skill --direct --agent codex --plainskilld search vue --plain(first rows)skilld run ... --direct --json(origin only){ "_tag": "remote", "source": "github:skilld-dev/skills/skills/find-skill", "direct": true, "owner": "skilld-dev", "repository": "skills", "skillPath": "skills/find-skill", "commit": "806d812729d24cbd3b504aeb7fdcb3e1f71ca031", "sourceUrl": "https://github.com/skilld-dev/skills/blob/806d812729d24cbd3b504aeb7fdcb3e1f71ca031/skills/find-skill/SKILL.md" }The URL returns HTTP 200.
Gaps
ResolvedSourcehas owner, repository, commitSha, skillPath only). The GitHub owner stands in for the author. TheUpdated ... · MITline from the plan is omitted rather than faked. Adding those fields to the contract is a follow-up.skilld installhas no--jsonmode today; that is unchanged.--plainprints the lines without styling.repository, notrepo, to matchSourceRequestand the glossary.Tests
New:
a_remote_run_names_the_author_and_links_the_exact_skill_file,a_remote_file_read_links_the_exact_skill_file,cli_install_shows_the_author_the_source_status_and_the_exact_skill_file. Updated the install, search, view link, and native probe expectations for the new lines.cargo test --workspace,cargo clippy --workspace --all-targets -- -D warnings, andcargo fmt --checkpass.Written with Claude Code.
https://claude.ai/code/session_018T67Ndp8FAjnHWthXABbJW