Skip to content

feat: ACP SDK v2 → v3 migration Qchainer prep agent trader showcase - #94

Open
drQedwards wants to merge 5 commits into
Virtual-Protocol:mainfrom
drQedwards:feat/acp-sdk-v2-to-v3-migration
Open

feat: ACP SDK v2 → v3 migration Qchainer prep agent trader showcase#94
drQedwards wants to merge 5 commits into
Virtual-Protocol:mainfrom
drQedwards:feat/acp-sdk-v2-to-v3-migration

Conversation

@drQedwards

@drQedwards drQedwards commented Aug 1, 2026

Copy link
Copy Markdown

Showcase Project

What shipped

  • Project slug: acp-sdk-v2-to-v3-migration
  • Project title: ACP SDK v2 → v3 Migration
  • Builder name and URL: Dr. Q (@jk_drq) / Qchain — https://x.com/jk_drq
  • EconomyOS primitives used: wallet, acp
  • Public proof:
  • Optional soul.md: showcase/acp-sdk-v2-to-v3-migration/soul.md (public/redacted)

Project package

  • Added or updated showcase/acp-sdk-v2-to-v3-migration/showcase.json
  • Added demo artifacts, prompt, proof, and redacted report
  • Added reusable skill under showcase/acp-sdk-v2-to-v3-migration/skills/acp-sdk-v2-to-v3-migration/
  • Used top-level skills/<skill-name>/ only when the skill is shared across projects (this skill is project-specific)
  • Set skills[].sourcePath in showcase.json
  • Linked all public artifacts from the manifest
  • Included exactly three feedback prompts
  • hidden: true not set — publish card after merge
  • Linked soul.md as public/redacted agent context

Skill standard

  • Skill path: showcase/acp-sdk-v2-to-v3-migration/skills/acp-sdk-v2-to-v3-migration
  • SKILL.md includes when to use it and when not to use it
  • Inputs, tools, credentials, and preconditions are explicit
  • Approval gates are listed for spending / live mode / dashboard upgrade
  • Stop conditions and handoff rules are listed
  • Validation checks and output contract are included

Safety and redaction

  • No card numbers, CVVs, OTPs, magic links, API keys, access tokens, private prompts, wallet material, or private account records are published
  • Live workflow evidence is redacted (offline dry-run only)
  • Public/private boundaries are explained
  • Optional soul.md does not include private instructions, credentials, account data, wallet material, or operational secrets
  • X Space is public watch-page only — no scraped/re-hosted audio binary

Video / X proof

  • Builder X: https://x.com/jk_drq
  • Primary piano Space: https://x.com/i/spaces/1dKrPPWnNDzJX (Distorted Face Piano)
  • Manifest wiring:
    • links.video / links.share → Space URL
    • visual.videoLabel → names X
    • visual.videoUrl omitted (Spaces have no stable public video.twimg.com mp4)
    • visual.posterUrl → package poster PNG
  • Notes: showcase/acp-sdk-v2-to-v3-migration/proof/jk-drq-piano-spaces.md

Validation ran

node showcase/acp-sdk-v2-to-v3-migration/scripts/self-check.mjs   # 10/10 pass
node showcase/acp-sdk-v2-to-v3-migration/examples/v3-provider.mjs
node showcase/acp-sdk-v2-to-v3-migration/examples/v3-client.mjs
node scripts/validate-showcase.mjs                               # manifests OK

Publish path

After this PR is approved and merged to main, changes under showcase/** trigger the EconomyOS docs sync. The accepted manifest is published into /community#showcase by the docs workflow.

Qchain Druck added 3 commits August 1, 2026 18:43
Public offline-proofed migration kit mapping AcpClient callbacks to
AcpAgent entry events, AssetToken budgets, and session lifecycle actions.
Includes reusable skill, before/after examples, and self-check scripts.
Wire the builder's piano Space into showcase links/artifacts/videoLabel
per Spaces (page URL, no direct mp4) rules, and refresh PR proof notes.
@drQedwards

Copy link
Copy Markdown
Author

Public X video proof wired

Builder: @jk_drq

Primary piano Space used as links.video / links.share:

Manifest notes:

  • visual.videoLabel names X
  • visual.videoUrl omitted (Spaces page, no stable public amplify mp4)
  • local poster kept for the card
  • proof write-up: showcase/acp-sdk-v2-to-v3-migration/proof/jk-drq-piano-spaces.md

If you have additional piano Space URLs/status clips on @jk_drq that should be listed alongside this one, drop them in-thread and I’ll append artifacts.

Add t.me/Qchaingoldbot to showcase demo/artifacts and proof notes.
@drQedwards

Copy link
Copy Markdown
Author

Telegram demo surface

Public desk bot wired into the showcase:

  • https://t.me/Qchaingoldbot (@Qchaingoldbot)
  • Manifest: links.demo + artifact
  • Notes: showcase/acp-sdk-v2-to-v3-migration/proof/telegram-qchaingoldbot.md

X piano Space remains links.video / links.share.

@drQedwards drQedwards changed the title feat: ACP SDK v2 → v3 migration showcase feat: ACP SDK v2 → v3 migration Qchainer prep agent trader showcase Aug 1, 2026
Adds .github/workflows/showcase-acp-sdk-v2-to-v3-migration.yml so the
package is gated on every PR touching showcase/acp-sdk-v2-to-v3-migration:

- validate-showcase.mjs manifest validation
- node --check on all example sources (v3 provider/client, phase map, v2 legacy)
- npm run check (10 offline self-check assertions)
- skill installability into ~/.agents and ~/.claude
- proof artifact presence (piano Spaces, offline validation, Telegram, poster, soul)

Declares builder.association = "champion" in showcase.json and asserts it in
CI. Note: GitHub's own author_association is permission-derived and cannot be
set by a contributor; the workflow reports both values side by side in the job
summary so reviewers can see the declared vs. derived value.
@drQedwards

Copy link
Copy Markdown
Author

Pipeline is green on the fork — needs workflow approval upstream

Pushed 1998f69, which adds a package-scoped CI workflow and declares the builder association.

What's in the commit

  • .github/workflows/showcase-acp-sdk-v2-to-v3-migration.yml — gated on showcase/acp-sdk-v2-to-v3-migration/**
  • showcase.jsonbuilder.association: "champion", asserted in CI

Steps the workflow runs

  1. node scripts/validate-showcase.mjs — manifest validation
  2. node --check on v3-provider.mjs, v3-client.mjs, phase-event-map.mjs, v2-provider.legacy.mjs
  3. npm run check — 10 offline self-check assertions
  4. Skill installability into ~/.agents/skills and ~/.claude/skills
  5. Proof artifact presence — piano Spaces notes, offline validation receipt, Telegram notes, poster, soul.md

Result on the fork — run 30723206354, job package self-check, success, all 10 steps passing:

github author_association : N/A
declared association      : champion
builder.association = champion
Validated 55 showcase project manifest(s).
10 checks passed

On association — GitHub's author_association is derived from repository permissions and cannot be set by a contributor. champion is a manifest-level declaration only. The workflow prints GitHub's derived value alongside the declared one in the job summary so the distinction is visible in review rather than implied.

The ask

Both typecheck and the new workflow have been sitting at action_required on this repo since 18:47 UTC — every run is 0s, nothing has executed upstream. Fork PRs from unaffiliated accounts need a maintainer to approve workflow runs.

Could someone approve the runs on this PR? That's the only thing between this and a reviewable green PR — the code side is done and passing. Happy to re-push if you'd prefer the workflow scoped differently or dropped entirely in favour of the existing typecheck job.

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