Skip to content

AD-802: Add patch proposal service#46

Open
alph-cmky wants to merge 1 commit into
feat/ad-801-workflow-canvas-mvpfrom
feat/ad-802-patch-proposal-service
Open

AD-802: Add patch proposal service#46
alph-cmky wants to merge 1 commit into
feat/ad-801-workflow-canvas-mvpfrom
feat/ad-802-patch-proposal-service

Conversation

@alph-cmky

Copy link
Copy Markdown
Owner

Summary

  • Add PatchProposal schema with target metadata, base version, JSON patch operations, validation result, approval state, and diff preview.
  • Add an in-memory patch proposal service for workflow patch preview without mutating the target workflow.
  • Validate patched workflow previews through workflow-core and require approval before apply.
  • Stabilize the daemon SSE disconnect test by waiting for the close cleanup instead of relying on a fixed 10ms delay.
  • Update the AD-802 backlog checklist.

Testing

  • TDD red check: pnpm test apps/server/src/patches/patch-service.test.ts initially failed because patch-service.js did not exist.
  • pnpm test apps/server/src/patches/patch-service.test.ts
  • pnpm test apps/daemon/src/routes/events.test.ts
  • pnpm lint && pnpm format && pnpm typecheck && pnpm test && pnpm build

Stacked on #45.

Closes #21

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