Skip to content

feat(codex): add support for codex openai#153

Merged
scottschreckengaust merged 29 commits intoawslabs:mainfrom
dineshSajwan:main
Apr 23, 2026
Merged

feat(codex): add support for codex openai#153
scottschreckengaust merged 29 commits intoawslabs:mainfrom
dineshSajwan:main

Conversation

@dineshSajwan
Copy link
Copy Markdown
Contributor

Summary

Adds OpenAI Codex as a supported coding agent for AIDLC. Codex uses AGENTS.md at the project root to load instructions, which maps cleanly onto the existing .aidlc-rule-details/ path resolution already used by Claude Code, Cursor, Cline, and GitHub Copilot, so no new path or rule changes were required.

Changes

  • README.md: Added OpenAI Codex to the Platform-Specific Setup table of contents and added a full ### OpenAI Codex setup section (between GitHub Copilot and Other Agents) with

  • User experience

Before: Users who wanted to use AIDLC with OpenAI Codex had no documented setup path and would need to figure out the AGENTS.md convention themselves.

After: Users can follow the new ### OpenAI Codex section in the README to set up AI-DLC in two commands. Codex sessions in the project will automatically load the full AIDLC three phase workflow (Inception → Construction → Operations) and resolve rule details from .aidlc-rule-details/.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • [ Y ] I have reviewed the contributing guidelines
  • [Y ] I have performed a self-review of this change
  • [Y ] Changes have been tested
  • [ Y] Changes are documented

Test Plan

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation/support references for using AI-DLC with OpenAI Codex by leveraging Codex’s AGENTS.md convention while keeping the existing .aidlc-rule-details/ rule-details resolution approach.

Changes:

  • Document OpenAI Codex setup steps in the README (including verification and directory layout).
  • Update the core workflow docs to list OpenAI Codex among agents that use .aidlc-rule-details/.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
aidlc-rules/aws-aidlc-rules/core-workflow.md Adds OpenAI Codex to the documented .aidlc-rule-details/ consumers list.
README.md Adds an OpenAI Codex section and TOC entry with setup/verify instructions using AGENTS.md.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings March 30, 2026 17:28
Update the note about the AGENTS.md file size limit.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md Outdated
Copy link
Copy Markdown
Member

@scottschreckengaust scottschreckengaust left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@leandrodamascena leandrodamascena added the codebuild A label to signal a request for the "CodeBuild" workflow label Apr 18, 2026
Comment thread aidlc-rules/aws-aidlc-rules/core-workflow.md Dismissed
Comment thread README.md Dismissed
Comment thread README.md Dismissed
@leandrodamascena leandrodamascena removed the codebuild A label to signal a request for the "CodeBuild" workflow label Apr 18, 2026
Copy link
Copy Markdown
Contributor

@leandrodamascena leandrodamascena left a comment

Choose a reason for hiding this comment

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

Hey folks, this PR had been stalled for a few weeks due to merge conflicts and a feedback #153 (review) from @raj-jain-aws. I went ahead and resolved the conflicts and addressed the review feedback.

Since @scottschreckengaust and @MichaelWalker-git had already approved, I believe this is ready to merge.

I'm approving and waiting to someone else to merge.

Copilot AI review requested due to automatic review settings April 18, 2026 15:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

Copy link
Copy Markdown
Member

@scottschreckengaust scottschreckengaust left a comment

Choose a reason for hiding this comment

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

LGTM

Copilot AI review requested due to automatic review settings April 21, 2026 20:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

Copilot AI review requested due to automatic review settings April 22, 2026 18:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
auto-merge was automatically disabled April 22, 2026 19:12

Head branch was pushed to by a user without write access

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread README.md Dismissed
Comment thread README.md Dismissed
Comment thread README.md Dismissed
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@scottschreckengaust scottschreckengaust left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@harmjeff harmjeff left a comment

Choose a reason for hiding this comment

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

Another platform is good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants