Skip to content

Add Codex MCP multi-LLM workflow builder, guide, and provider agents#151

Merged
NestorMonroy merged 2 commits intodevelopfrom
feature/check-if-changes-are-implemented-06-08-45
Nov 13, 2025
Merged

Add Codex MCP multi-LLM workflow builder, guide, and provider agents#151
NestorMonroy merged 2 commits intodevelopfrom
feature/check-if-changes-are-implemented-06-08-45

Conversation

@2-Coatl
Copy link
Copy Markdown
Collaborator

@2-Coatl 2-Coatl commented Nov 13, 2025

Summary

  • add the CodexMCPWorkflow builder with provider-aware briefs and expose it from the orchestrators package
  • document Codex MCP single and multi agent workflows, update the ExecPlan, and publish the new agent catalog entry
  • extend documentation alignment tests to protect the new guide and cross-linking
  • add ClaudeAgent, ChatGPTAgent, and HuggingFaceAgent fichas while updating the SDLC guide, ExecPlan, and alignment tests to keep multi-LLM workflows consistent

Testing

  • pytest scripts/coding/tests/ai/orchestrators/test_codex_mcp_workflow.py
  • pytest docs/testing/test_documentation_alignment.py

Codex Task

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

https://github.com/2-Coatl/IACT---project/blob/f7f20aec6ceba22f507e4c508225e350324923b9/scripts/coding/ai/generators/llm_generator.py#L150-L152
P1 Badge Fix .env lookup path in LLMGenerator

_load_env_variables assumes the repository root is only three directories above llm_generator.py (project_root = current_dir.parent.parent.parent). Because this module actually lives under scripts/coding/ai/generators, walking up three levels lands in the scripts/ folder rather than the repo root, so the function always looks for scripts/.env instead of the real ./.env. As a result the promised automatic API‑key loading never happens unless contributors duplicate their .env file inside scripts/, and the subsequent validation logic will keep flagging missing keys even though they exist at the repository root. Please compute the root with one more .parent (or via Path(__file__).resolve().parents[...]) so the correct .env is read.

ℹ️ 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".

@NestorMonroy NestorMonroy merged commit 94618b8 into develop Nov 13, 2025
2 of 29 checks passed
@NestorMonroy NestorMonroy deleted the feature/check-if-changes-are-implemented-06-08-45 branch November 13, 2025 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants