Repository navigation
feat: Track 1 End-to-End Execution Modes, Capability Registry & UI Integration - #38
Merged
Merged
Conversation
…lity registry, and offline test suite
…se 5 React diagnostics UI
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7567d5d909
ℹ️ 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".
Remove the "inheritMcp" option from the YAML frontmatter of all agents in .agents/agents, and change "model" to "flash"
…its, and solo runs - Constrain collaborative artifact reads to explicitly tracked files within safe subpaths (output_generator.track_generated_artifact) - Enforce max_steps and initial_budget caps across adjust_step_budget, main loop, and execution dispatcher - Guard zero-agent solo reasoning against UnboundLocalError and record solo attribution - Add comprehensive security and regression tests (+4 tests, 210 passed offline)
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.
Summary
This pull request implements Track 1: End-to-End Execution Modes, Capability Registry, and UI Integration (addressing Issue #35 Phases 3–5 and Issue #21).
It unifies the application's runtime reasoning surfaces into a strongly typed, observable execution model spanning backend schemas, GraphQL APIs, and the React frontend.
Key Changes
Phase 3: Shared Run Contracts & Capability Registry (
execution_modes.py)RunRequest,RunResult,RunEvent,RunDiagnostics,PlanStepDiagnostic,SubtaskDiagnostic,ReflectionDiagnostic, andAgentAttribution.CAPABILITY_REGISTRYcataloging supported execution modes (collaborative,advanced_pipeline) and gated future modes (tree_of_thoughtgated on Issue Implement true Tree-of-Thought search with bounded branching, evaluation, pruning, and end-to-end integration #36).RunResult(status="gated")with zero LLM provider or external network calls.RunRequestnormalizes tasks and conversations bidirectionally, accepting dicts, strings, and GraphQL input types.conversation_manager.run_conversation,advanced_prompting.AdvancedPromptEngineer) inside dispatcher functions, keeping module imports completely pure.Phase 4: GraphQL Schema Extensions (
schema.py)Query.execution_modes: Exposes registered capabilities, enabled status, descriptions, and gating issue references.Mutation.run_execution: Dispatches arbitrary registered modes, returning unifiedRunResultTypewith diagnostics, events, and telemetry.Mutation.run_advanced_pipeline: Dedicated resolver forAdvancedPromptEngineer.main(task)with structured telemetry logging.Query.hello,Mutation.run_conversation, andMutation.submit_feedback.Phase 5: React UI Observability & Diagnostics (
collaborative-assistant-frontend/)AssistantConfigPanel.tsxwith an Execution Mode selector (collaborative,advanced_pipeline, disabledtree_of_thoughtwith Issue Implement true Tree-of-Thought search with bounded branching, evaluation, pruning, and end-to-end integration #36 badge).ReasoningDiagnosticsPanel.tsxdisplaying complexity scores, step budgets, reflections, step rewards, plan subtasks, agent attributions, and a collapsible chronological event log.StructuredOutputRenderer.tsxwith dedicated formatters for code syntax highlighting, JSON tree/copy, CSV table/raw view, and Markdown.useRunExecution.tsproviding typed Apollo execution with loading states and error handling.ConversationPage.tsxpreserves legacy collaborative integration test mocks while enabling multi-mode execution.Offline Qualification & Verification Evidence
All tests were executed under the fail-closed network denial harness (
tools/run-offline.ps1andtools/run-frontend-offline.ps1):test_execution_modes.py, 13 tests intest_empirical_challenger_2.py, 6 intest_graphql_contract.py, and 48 intest_advanced_prompting.py.Related Issues