feat(conjure): add the conjure plugin - #39
Conversation
c69ef6d to
db04322
Compare
Conjure's skill has only ever reached people through `conjure init`, which writes it into one folder and refuses to run where an index.html already exists. That leaves every user carrying whatever copy their binary held on the day they ran it, with nothing to refresh it. Publishing it here makes it installable on its own and, once marketplaces sync, keeps it current. The skill is a copy; the conjure repo remains the source of truth and a workflow there opens a PR here whenever it changes. The README says so, because a plausible-looking edit here would be reverted without warning. Also drops skills/wiki, a symlink whose target left in #14. It is unrelated to this plugin, but check-skills walks every link in skills/, so master is already failing and this PR could not go green beside it. Part of AUT-1829 Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
db04322 to
d301b15
Compare
|
Closing: the conjure plugin is going to Factory-AI/factory-internal-plugins instead.
Two marketplaces shipping a skill with the same One thing here was not about conjure and is worth keeping: this PR also deleted the dangling Branch |
What
Publishes the conjure skill as a marketplace plugin, so people get it without running
conjure init.Today the skill only reaches someone who already has the binary and runs
conjure initin a folder with noindex.html(it aborts otherwise). It lands in that one folder, frozen at whatever version the binary held. One copy in the wild was already 15 days and two login fixes behind the source, with nothing to signal it.Contents
plugins/conjure/withplugin.json,README.md, andskills/conjure/SKILL.mdskills/conjuresymlink, per the conventioncheck-skillsenforces.factory-plugin/marketplace.jsonand a section in the rootREADME.mdTwo things worth a reviewer's attention
1. This also deletes
skills/wiki. That symlink points at../plugins/droid-evolved/skills/wiki, whose target was removed in #14.check-skillswalks every link inskills/, so master is already failing and this PR could not go green next to it. Unrelated to the plugin; called out rather than buried.2. The SKILL.md here is a copy. The source of truth is
internal/cli/scaffold_skill.mdinFactory-AI/conjure, where it is also embedded in the CLI. A companion PR there adds a workflow that opens a sync PR here whenever it changes.plugins/conjure/README.mdsays so, because an edit made here looks reasonable and would be silently overwritten.Public-repo check
This marketplace is public, so the content was reviewed for exposure: no credentials, no internal hostnames beyond
conjure.factory.aianddownloads.factory.ai(both already public; the CLI binaries are served unauthenticated), and the only warehouse example queries a table calledwh_test. Conjure is Okta-gated, so the skill is useful only to Factory employees, but nothing in it is sensitive.Verified
check-skillsreplicated locally against this branch: PASS (and reproduced the pre-existing failure on master).Part of AUT-1829