Skip to content

Publish workflows from the skill form#61

Closed
mega123-art wants to merge 1 commit into
mainfrom
fix/workflow-selling-publish
Closed

Publish workflows from the skill form#61
mega123-art wants to merge 1 commit into
mainfrom
fix/workflow-selling-publish

Conversation

@mega123-art

Copy link
Copy Markdown
Contributor

Summary

  • parse publish form frontmatter for type: workflow
  • route workflow publishes through the existing publishWorkflow path
  • keep normal skill publishes on publishSkill
  • add focused routing tests for workflow, skill, and no-frontmatter publishes

Verification

  • pnpm --filter @iqlabs-official/agent-sdk exec tsc --noEmit
  • pnpm --filter @iqlabs-official/agent-sdk exec vitest run src/skill-market/ingest/env.spec.ts

Note: an earlier broad vitest invocation also ran unrelated session tests and hit an existing /init ENOENT failure outside this slice.

Split out of #56 per review feedback.

zo-sol added a commit that referenced this pull request Jun 20, 2026
Parse the SKILL.md frontmatter on publish: type: workflow routes to
publishWorkflow (carrying requiredSkills), everything else stays a skill mint.
Lets the existing publish form sell workflows, not just skills.

Cherry-picked the core change from #61.

Co-authored-by: mega123-art <agrawalparth326@gmail.com>
@zo-sol

zo-sol commented Jun 20, 2026

Copy link
Copy Markdown
Collaborator

Thanks @mega123-art — nice work. Routing publish by SKILL.md frontmatter (type: workflowpublishWorkflow carrying requiredSkills, else a skill mint) is clean and lets the existing form sell workflows too.

I cherry-picked the core of this onto main in 5603611 (with you as co-author), keeping your env.ts + the new env.spec.ts as-is (113-line spec passes). Same reason as the sibling PRs: this branch predates the market RPC refactor (3b6b601) and #60, so a full merge would have silently reverted attachMarketHandlers (confirmed by a trial merge). The webview parts are superseded by main.

Closing as cherry-picked into main. Thanks again — credit's on the commit. 🚀

@zo-sol zo-sol closed this Jun 20, 2026
@zo-sol zo-sol deleted the fix/workflow-selling-publish branch June 20, 2026 11:56
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.

2 participants