Repository navigation
Conversation
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds an opt-in
NEXENT_AGENT_RUNTIME=toolcallingruntime based on smolagents ToolCallingAgent alongside the default CodeAgent. Existing authorized tools work as native calls and as prebound functions inside optional restrictedcode_execution; persisted Agent Definitions require no migration.The native loop owns structured provider history, streamed call assembly and correlated results. It reuses Nexent model configuration/retries/concurrency, tool factories, sandbox lifecycle/bridge, context selection, observer events and tracing. NL2Agent/NL2Skill retain CodeAgent for their platform authoring protocols.
Validation:
docs/experimental-native-runtime/evidence/.This remains an architecture spike, not production parity. Native current-run compaction and multimodal projection are incomplete; dynamic MCP container journeys, live planning/verifier/clarification journeys and concurrent child lifetimes remain unverified. Provider reasoning controls/fields are recorded independently from tool support, including inconsistent gateway observations. Both runtimes retain the existing unauthenticated-run HTTP 500 defect while denying execution.
Based on remote tag v2.8.4 (
c43d2d995b15c9005c0d736cd60200a22305de74). Final experimental deployment stays on the host by explicit user preference; Nexent-Test remains stopped. Investigation, design, compatibility, validation and develop migration findings are included in the five runtime documents. No production migration or dependency upgrade is proposed.Current test-only head is
1ba6aa0a490945dd92a510e877b16ea7b4293222. Tested product source matches commit128f22ee1d45e89ae77e8029544a1add37a5e562; final main imagesha256:cd0e5604e99984eb0f21db8c01703100861387183299538f923247f0b37a51e5. Existing CI filters omit main-targeted PRs; equivalent branch workflows were dispatched manually without configuration changes.Final CI on
1ba6aa0a4: unit tests and coverage upload, amd64/arm64 image builds, changed-source copyright checks and CodeQL passed. Codecov PR report reports 96.96312% patch coverage, above the workspace 90% gate; 4 missing and 10 partially covered lines remain in the native loop. No full production parity is claimed.