Skip to content

Add async runtime and concurrent vault support - #2

Merged
AronAxe merged 1 commit into
mainfrom
feat/async-concurrency
Aug 25, 2026
Merged

Add async runtime and concurrent vault support#2
AronAxe merged 1 commit into
mainfrom
feat/async-concurrency

Conversation

@AronAxe

@AronAxe AronAxe commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • add an executor-backed AsyncRuntime façade while preserving the synchronous API
  • propagate task and thread-safe token cancellation and kill/reap RTK subprocesses
  • enable SQLite WAL concurrency and stress-test multi-thread/process deduplication, integrity, and lease claims
  • document async usage, streaming boundaries, cancellation semantics, and local-vault concurrency

Verification

  • 141 passed, 2 skipped
  • ruff check and format check pass
  • structural benchmark unchanged at 82.8% aggregate token reduction
  • wheel/sdist build, twine check, release verification, and wheel module inspection pass
  • independent Gemini 3.1 Pro review completed; actionable findings addressed

@AronAxe
AronAxe merged commit e58e05a into main Aug 25, 2026
6 checks passed
@AronAxe
AronAxe deleted the feat/async-concurrency branch August 25, 2026 06:37

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 40160b68df

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
Wrap the same synchronous runtime when the host owns an asyncio event loop:

```python
from rtk_hermes_plus import AsyncRuntime, CancellationToken, Runtime

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update the install pin before advertising async imports

When a user follows the immediately preceding adapter installation command, pip installs commit e02a035d52cc2b0e6e95748b35deb1f61656a4a3, whose package tree contains neither async_runtime.py nor these exports from rtk_hermes_plus.__init__. The documented import therefore raises ImportError for exactly the users this new section targets; update the installation reference to a revision containing the async API, or clearly require an unreleased/current checkout.

Useful? React with 👍 / 👎.

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