Skip to content

test(specs): wire-spec creator is a plain member — closes the original skill-creation story - #3420

Merged
benjaminshafii merged 1 commit into
devfrom
feat/spec-member-creator
Aug 2, 2026
Merged

test(specs): wire-spec creator is a plain member — closes the original skill-creation story#3420
benjaminshafii merged 1 commit into
devfrom
feat/spec-member-creator

Conversation

@benjaminshafii

Copy link
Copy Markdown
Member

One-scene change to skill-grant-access.test.ts: the skill creator is now a bootstrapped plain member (casey.spec@acme.test) instead of the seeded owner — possible since #3411 (member plugin creation) and meaningful since #3410 (grant-native index). The spec now proves the product story end-to-end exactly as originally asked: any member creates a skill over the chat wire, uses it immediately (marketplace: null), and another member can neither discover nor execute it.

Tests

pnpm --dir evals run spec specs/skill-grant-access.test.ts   # ran 3× against merged-dev den stack:
# bootstrap path + direct sign-in path (executor) + independent rerun (orchestrator) — all pass
pnpm --dir evals run typecheck                               # clean

No product code touched; spec + comment only.

@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview Aug 2, 2026 5:42am
openwork-den Ready Ready Preview Aug 2, 2026 5:42am
openwork-den-worker-proxy Ready Ready Preview Aug 2, 2026 5:42am
openwork-diagnostics Ready Ready Preview Aug 2, 2026 5:42am
openwork-landing Ready Ready Preview, v0 Aug 2, 2026 5:42am

@diff-warden diff-warden Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warden security clearance: clear. No new security issues found in this diff (6c2472be61a0070968b1bf8f24311c1f1a341943). Automated clearance satisfies the required-review gate only — a human still reviews and merges. Analysis run

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