Skip to content

AD-602: Add workflow validator#42

Open
alph-cmky wants to merge 1 commit into
feat/ad-601-workflow-dag-schemafrom
feat/ad-602-workflow-validator
Open

AD-602: Add workflow validator#42
alph-cmky wants to merge 1 commit into
feat/ad-601-workflow-dag-schemafrom
feat/ad-602-workflow-validator

Conversation

@alph-cmky

Copy link
Copy Markdown
Owner

Summary

  • Adds workflow safety validation with actionable error codes/messages.
  • Returns schema errors before deeper safety checks.
  • Detects DAG cycles.
  • Validates node input/output compatibility.
  • Validates referenced agents and tools through injected resolvers.
  • Validates requested tool permissions do not exceed workflow permissions.
  • Marks AD-602 complete in the development backlog.

Stack

Verification

  • TDD red check: pnpm test packages/workflow-core/src/validator.test.ts initially failed because validator.js did not exist yet.
  • pnpm test packages/workflow-core/src/validator.test.ts
  • pnpm lint && pnpm format && pnpm typecheck && pnpm test && pnpm build

Closes #17

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