Skip to content

feat(cli): show author and source after run and install - #130

Open
harlan-zw wants to merge 3 commits into
mainfrom
brand/cli-provenance-output
Open

feat(cli): show author and source after run and install#130
harlan-zw wants to merge 3 commits into
mainfrom
brand/cli-provenance-output

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

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 headline name · owner/repository @ commit, and Read it first: <SKILL.md URL at that commit> after the source status caution. Also on --file reads.
  • skilld install: the same block, plus Source status and its plain meaning for verified, local, and unverified. Replaces the bare "Review the unverified Skill before use." hint.
  • skilld search: owner/repository before the star count (human, --plain, --json).
  • skilld view: the Source link now points at the exact SKILL.md, not the Repository root.
  • --json for run: origin gains owner, repository, skillPath, commit, sourceUrl. sourceStatus was already present.

Host::install and Host::install_request return InstalledSkill (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) returned RESOLUTION_TIMEOUT from 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 --plain

skilld loaded the transient Skill find-skill for this session.
skilld retained no Skill files.
It created no lockfile entry, Agent target, or project file.
find-skill · skilld-dev/skills @ 806d812
Source: github:skilld-dev/skills/skills/find-skill
Revision: 806d812729d24cbd3b504aeb7fdcb3e1f71ca031
Source status: unverified
skilld did not check this source. Read this Skill before you follow it.
Read it first: https://github.com/skilld-dev/skills/blob/806d812729d24cbd3b504aeb7fdcb3e1f71ca031/skills/find-skill/SKILL.md

--- SKILL.md ---
...

skilld install github:skilld-dev/skills/skills/find-skill --direct --agent codex --plain

Installed Skill find-skill.
find-skill · skilld-dev/skills @ 806d812
Source: github:skilld-dev/skills/skills/find-skill
Source status: unverified
skilld did not check this source. Read this Skill before you follow it.
Read it first: https://github.com/skilld-dev/skills/blob/806d812729d24cbd3b504aeb7fdcb3e1f71ca031/skills/find-skill/SKILL.md

skilld search vue --plain (first rows)

vue	skilld:antfu/skills/vue	antfu/skills	5822	Vue 3 Composition API, ...
vue-testing-best-practices	skilld:antfu/skills/vue-testing-best-practices	antfu/skills	5822	Use for Vue.js testing. ...
vueuse-functions	skilld:vueuse/skills/vueuse-functions	vueuse/skills	385	Apply VueUse composables ...

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

  • The Artifact wire contract carries no author, licence, or source updated time (ResolvedSource has owner, repository, commitSha, skillPath only). The GitHub owner stands in for the author. The Updated ... · MIT line from the plan is omitted rather than faked. Adding those fields to the contract is a follow-up.
  • skilld install has no --json mode today; that is unchanged. --plain prints the lines without styling.
  • JSON uses repository, not repo, to match SourceRequest and 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, and cargo fmt --check pass.

Written with Claude Code.

https://claude.ai/code/session_018T67Ndp8FAjnHWthXABbJW

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
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Sep 1, 2026
@harlan-github-agent

harlan-github-agent Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

🤖 READY · 88/100

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. A person still decides the merge.

  • Merge gate: Passed.
  • Review gate: Passed. No material issues.
  • CI gate: Passed.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Sep 1, 2026
…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
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-ready The automated Review passed every gate on this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. labels Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

harlan-agent-ready The automated Review passed every gate on this head commit.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant