Skip to content

Feat/683 684 680 685 concurrency lifecycle - #763

Merged
0xNgoo merged 15 commits into
Anchor-kit:mainfrom
AbilityTechy:feat/683-684-680-685-concurrency-lifecycle
Oct 23, 2026
Merged

0xNgoo merged 15 commits into
Anchor-kit:mainfrom
AbilityTechy:feat/683-684-680-685-concurrency-lifecycle

Conversation

@AbilityTechy

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Makes idempotent deposit creation atomic, using a dedicated PostgreSQL connection per transaction.
  • Makes first webhook persistence race-safe and ensures one delivery claims processing.
  • Adds reverse-order plugin cleanup on shutdown and initialization rollback, preserving cleanup errors.
  • Reports PostgreSQL tests as skipped locally, and fails when required database setup is missing or broken.

How to test?

Run bun test. The last full run passed 409 tests; 5 PostgreSQL tests were skipped because no local PostgreSQL URL was configured. With PostgreSQL available, run npm exec -- vitest run tests/runtime/atomic-idempotency-concurrency.test.ts. Run npm exec -- vitest run tests/core/plugin-lifecycle.test.ts for the plugin lifecycle checks.

Changed-file ESLint and Prettier checks passed. The full bun run lint gate remains unverified in this Windows environment due to repository-wide line-ending formatting errors.

Checklist

  • Changed files pass lint and formatting checks.
  • Tests added for concurrency and plugin cleanup.
  • Documentation updated.
  • Ran both bun run test and bun run lint successfully locally.

Issue Reference

Closes #683,
Closes #684,
Closes #680,
Closes #685.

@drips-wave

drips-wave Bot commented Oct 20, 2026

Copy link
Copy Markdown

@AbilityTechy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@0xNgoo

0xNgoo commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Correction: I verified maintainer edits are enabled for this PR. I’ll resolve the conflicts and Type Check failures on the existing branch as 0xNgoo, preserving your PR and issue attribution. Please disregard my request for you to push the fix.

@0xNgoo
0xNgoo merged commit 609ac66 into Anchor-kit:main Oct 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants