Skip to content

refactor(tinyagents): complete provider cutover and lifecycle consolidation#5143

Merged
senamakel merged 120 commits into
tinyhumansai:mainfrom
senamakel:feat/tinyagents-provider-cleanup
Jul 24, 2026
Merged

refactor(tinyagents): complete provider cutover and lifecycle consolidation#5143
senamakel merged 120 commits into
tinyhumansai:mainfrom
senamakel:feat/tinyagents-provider-cleanup

Conversation

@senamakel

Copy link
Copy Markdown
Member

Summary

  • cut every production and test construction path from the legacy host Provider abstraction to TinyAgents-native ChatModel, then delete the compatibility clients, routing wrappers, and provider adapter
  • preserve OpenHuman-owned auth, temperature, thread context, tool timeout, tool-status, and durable message policy at explicit host seams
  • adopt TinyAgents invalid-argument recovery and detached-task runtime ownership, retaining only OpenHuman product metadata, durable task projection, RPC, and the RunQueue compatibility fallback
  • reconcile the migration/drift/deletion ledgers and record the WP-4 tool-model decision as proposed; no WP-4 implementation is included without explicit approval

Upstream dependencies

Validation

  • cargo test --test inference_provider_e2e — 14 passed
  • cargo test --test agent_harness_e2e — 18 passed, including after merging current upstream main
  • focused model-council tests — 23 passed
  • focused tool-timeout tests — 14 passed
  • focused tool-status tests — 20 passed
  • focused running_subagents tests — 17 passed, including after merging current upstream main
  • TinyAgents Enhance CI workflows: multi-platform build, Rust quality gates, Windows release #75: focused registry tests pass; cargo clippy --all-targets --all-features -- -D warnings passes; Rust SDK CI and CodeRabbit green

Deliberate follow-ups / gates

  • WP-4 implementation remains design-gated pending explicit approval of docs/tinyagents-tool-model-decision-2026-07-23.md
  • progress-tracing deletion remains blocked by the recorded one-release C4 shadow-parity gate
  • final upstream gitlink normalization waits for the TinyAgents PRs to merge

App-wide and full-suite validation is intentionally delegated to GitHub Actions.

senamakel added 30 commits July 22, 2026 16:38
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jul 23, 2026

@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: 9e2e4f171d

ℹ️ 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 src/openhuman/inference/provider/factory.rs Outdated
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jul 23, 2026

@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: c089d96e48

ℹ️ 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 src/openhuman/inference/provider/factory.rs Outdated
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jul 23, 2026

@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: 63af5fdc9c

ℹ️ 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 src/openhuman/inference/provider/factory.rs Outdated
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jul 23, 2026

@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: fbba4882af

ℹ️ 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 src/openhuman/inference/provider/factory.rs
Comment thread src/openhuman/inference/provider/factory.rs Outdated

@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: 07d91b8876

ℹ️ 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 src/openhuman/tinyagents/mod.rs Outdated
@senamakel senamakel self-assigned this Jul 24, 2026
# Conflicts:
#	docs/tinyagents-drift-ledger.md
#	docs/tinyagents-full-migration-plan/99-deletion-ledger.md
#	docs/tinyagents-migration-plan-2026-07-22.md
#	src/openhuman/agent_orchestration/tools/spawn_async_subagent.rs
#	src/openhuman/agent_orchestration/tools/spawn_parallel_agents_tests.rs
#	src/openhuman/model_council/council.rs
#	src/openhuman/proc_metrics/mod.rs
@greptile-apps

greptile-apps Bot commented Jul 24, 2026

Copy link
Copy Markdown

Too many files changed for review. (211 files found, 100 file limit)

Bypass the limit by tagging @greptile-apps to review.

@senamakel
senamakel merged commit 7db104b into tinyhumansai:main Jul 24, 2026
20 checks passed
senamakel added a commit to senamakel/openhuman that referenced this pull request Jul 24, 2026
… merge

The upstream merge bumped vendor/tinyagents to the tinyhumansai#5143 provider refactor
which removed the with_requires_streaming builder method. Drop the call since
the new OpenAiModel transport already handles streaming correctly.
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