Skip to content

fix(skills): support connector creation from an ordinary Vana request - #105

Closed
callumflack wants to merge 2 commits into
mainfrom
callum/connector-agent-workflow
Closed

callumflack wants to merge 2 commits into
mainfrom
callum/connector-agent-workflow

Conversation

@callumflack

@callumflack callumflack commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Superseded by the combined CLI PR #106. Its connector skills, README and obsolete-reference removals are included there, with the subsequent self-contained Builder verification fix. Keep review and further changes on #106.

Summary

The public connector skills still teach the removed CJS format and leave users to coordinate authoring, authentication and collection with repeated prompts. Replace that guidance with public Collection Profile tools and one request that an agent carries through to CLI collection and server storage.

- Legacy scripts and user-managed implementation prompts
+ Discover, create if missing, collect once with Vana, verify storage

The accompanying README shows the CLI commands and both skill installers. Factory access is optional. Human authentication uses the existing hidden terminal prompt or browser handoff, with visible progress.

Evidence

  • Native and skills.sh installers produced byte-identical skill files. Installation from this public branch into a temporary project passed.
  • A fresh agent given an ordinary request created a missing export connector without private factory access or follow-up implementation prompts. Its single Vana CLI run uploaded all three synthetic records unchanged to an isolated fixture server.
  • The connector's 72 focused tests and package verification passed. This controlled trial does not establish a second real-provider integration.
  • Skill frontmatter, Markdown formatting, local links and the repository's P0 review passed.

Merge Danger

Door: two-way.

Blast Radius: agent instructions and skill discovery metadata. No CLI runtime code changes.

Created with Codex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant