Repository navigation
Update: product-verified catalog skill content - #13
Conversation
Cross-check every catalog skill against live Fledge plugins and product repos, then fix wrong flags, lifecycle guidance, install paths, and host prompt stubs so agents get accurate public commands.
69869e3 to
29470d3
Compare
0xLeif
changed the title
Clarify that catalog skills install via fledge-plugin-skills, while product CLIs use separate Fledge plugins (let, rune, augur, attest, atlas) or standalone installs (specsync, threemd, agent3md) when they are not plugins.
💡 Codex ReviewLine 450 in 69869e3  Detect metadata changes before replacing managed copies
The Line 589 in 69869e3  Propagate manifest parsing failures from status
Because ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Skills assume they are already loaded. Document only product CLI/plugin install where relevant. README clarifies per-skill install (no install --all).
Document agent-coordination + fledge-workflows as the default install set; product skills stay opt-in per tool.
Support installing, status, update, and uninstall under \$HOME (or \$CORVID_SKILLS_HOME) for agent host skill dirs, tracked in ~/.corvid-skills.json. Document and test the new scope.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cd36546d75
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| local is_global="$1" | ||
| local repo_arg="$2" | ||
| if [[ "$is_global" == "true" ]]; then | ||
| [[ "$repo_arg" == "." ]] || fail "--global cannot be combined with --repo" |
There was a problem hiding this comment.
When a caller explicitly supplies --repo . --global, repo_arg is indistinguishable from its default value, so this check passes and the command silently targets $HOME instead of rejecting the conflicting scope flags. For install, update, and uninstall, this can mutate machine-global skills when the caller intended repository-local operation; track whether --repo was provided rather than comparing its value with ..
Useful? React with 👍 / 👎.
| fledge skills uninstall augur --dry-run | ||
| fledge skills uninstall augur | ||
| fledge skills uninstall augur --global |
There was a problem hiding this comment.
Following this documented preview-and-remove sequence checks the repository-local manifest during the dry run, but then removes the machine-global installation in the next command. When augur exists in both scopes, the preview provides no evidence about the copy that will actually be deleted; add --global to the dry-run command (or keep both commands repository-local).
Useful? React with 👍 / 👎.
1 participant
Summary
main(provider hosts + managed lifecycle from Add OpenAI provider integration #11/Fix Skills provider review findings #12).--gaps, Attestexport, public Rune surface, Spec Sync accept→merge→archive, agent-3md crates.io lag, Augur/Attest macOS plugin notes, Let dual-path discovery).agent-coordinationas a hub and keeps host OpenAI prompts on$skill-nameform.Test plan
fledge run plugin-validatefledge run lintfledge run test(installer tests passed)fledge let|rune|augur|attest|atlas --helpandspecsync --help