Add auth-integration-engineer and webhook-engineer subagents - #310
Open
Sabeekhann wants to merge 1 commit into
Open
Add auth-integration-engineer and webhook-engineer subagents#310Sabeekhann wants to merge 1 commit into
Sabeekhann wants to merge 1 commit into
Conversation
Both fill workflows that no existing agent owns: - auth-integration-engineer: builds auth (OAuth 2.1/OIDC/PKCE, SAML SSO, SCIM, sessions, multi-tenant authorization, MFA/passkeys). The existing security-engineer, security-auditor and penetration-tester agents audit security posture; none implement identity flows. SAML currently appears in zero agent files, OIDC in one. - webhook-engineer: covers both consuming and producing webhooks — raw-body signature verification, idempotency under at-least-once delivery, out-of-order handling, retry/backoff, dead-letter replay, and SSRF hardening on outbound delivery. Webhooks appear as a stray bullet in ten agent files but are owned by none. Both are protocol/integration implementation agents, placed in 01-core-development alongside api-designer and websocket-engineer. Updates per CONTRIBUTING.md: main README (list + count badge 158 -> 160), category README (descriptions, Quick Selection Guide, Common Combinations), plugin.json (agents array + version 1.0.2 -> 1.0.3), and marketplace.json version sync. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds two subagents to 01-core-development, both targeting workflows that no existing agent currently owns.
Why these two
I audited all 158 existing agents (every agent's
name/descriptionfrontmatter, plus keyword sweeps acrosscategories/) before picking, specifically to avoid overlap:security-engineer,security-auditorandpenetration-testeraudit security posture; none build identity flowsSAML: 0 files,OIDC: 1 filewebhook: 10 files, 0 own itBoth are protocol/integration implementation agents, so they sit naturally alongside
api-designerandwebsocket-engineerrather than in a security or vertical category.auth-integration-engineerScoped deliberately to building auth, not auditing it — that boundary is what keeps it from colliding with
security-engineer/security-auditor.Covers OAuth 2.1/OIDC with PKCE, token strategy (rotation, reuse detection, signature/issuer/audience validation), session and cookie lifecycle, SAML 2.0 assertion validation and SCIM provisioning for enterprise SSO, multi-tenant authorization models, and MFA/WebAuthn. Includes an explicit "common failure modes" section for the mistakes that actually ship — unvalidated
state, missingnonce,alg: none, open redirect on callback, tenant ID read from the request body instead of the verified token.webhook-engineerCovers both sides of the boundary, since most real bugs live at that seam.
Consuming: raw-body HMAC verification (verify before parsing — the classic bug), constant-time comparison, timestamp tolerance, fast-ack with async processing. Plus idempotency under at-least-once delivery, out-of-order arrival handling, retry/backoff with jitter, dead-letter queues with replay tooling, and reconciliation against the provider API as source of truth. Producing: event versioning, per-subscriber backoff, secret rotation, and SSRF hardening on delivery to subscriber-controlled URLs.
Files updated per CONTRIBUTING.md
categories/01-core-development/auth-integration-engineer.md(new)categories/01-core-development/webhook-engineer.md(new)README.md— added to the Core Development list in alphabetical order; count badge and intro line158→160(verified against the actual file count, which was exactly 158 before this PR)categories/01-core-development/README.md— descriptions with Use when: lines, two Quick Selection Guide rows, two new Common Combinationscategories/01-core-development/.claude-plugin/plugin.json— both agents added toagents, version1.0.2→1.0.3.claude-plugin/marketplace.json—voltagent-core-devversion synced to1.0.3, keywords extendedVerification
api-designer.mdandwebsocket-engineer.mdagentsarray