8000 docs: separate global and Agent OS guidance by dominicnunez · Pull Request #226 · dominicnunez/agentos · GitHub
[go: up one dir, main page]

Skip to content

docs: separate global and Agent OS guidance - #226

Merged
dominicnunez merged 5 commits into
mainfrom
docs/consolidate-agent-guidance
Oct 4, 2026
Merged

dominicnunez merged 5 commits into
mainfrom
docs/consolidate-agent-guidance

Conversation

@dominicnunez
@dominicnunez dominicnunez commented Oct 4, 2026 •
Copy link
Copy Markdown
Owner

Consolidate reusable working preferences in the user's global AGENTS.md and keep the project file focused on Agent OS architecture, ledger-selection and recovery rules, provider coverage, documentation layout, security backlog and certification decisions.

The maintainer explicitly confirmed that this workflow assumes the global guidance is available; supporting agents without it is outside this change's scope. Generic engineering evidence, final-head reviews, live-operation authorization, self-contained documentation, skills/model preferences, review automation and threat-model maintenance remain in the global guidance. The global file stays local and is not published in this PR. No duplicate public workflow documents are added. Existing repository change-control requirements remain unchanged.

Validation: independent review of the full migration; staged diff integrity; staged AIMS document verification; 78 document verification and assessment-bundle tests passed. The repository's staged-content materializer avoids Windows checkout line-ending conversion in the unchanged manifest. Existing documentation CI remains applicable; no runtime or race tests were added.

Update the CI documentation to remove its obsolete reference to project AGENTS.md review sequencing; the existing threat-model and workflow security-review requirements remain intact.

@chatgpt-codex-connector
chatgpt-codex-connector Bot commented Oct 4, 2026 •
Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-04T12:20:42.816398Z 9c266a4 New commits
🔒 Security Review ✅ Completed 2026-10-04T12:27:05.485066Z 9c266a4 Manual request

Security findings

Advisory findings (1)

ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@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: 6925929ab1

ℹ️ 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 AGENTS.md Resolved
Comment thread AGENTS.md Resolved
Comment thread AGENTS.md Resolved
@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: 4559ac4e74

ℹ️ 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 AGENTS.md Resolved
Comment thread AGENTS.md Resolved
@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: ad6550dd5b

ℹ️ 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 AGENTS.md Resolved
Comment thread AGENTS.md Resolved
Comment thread docs/development/ledger-verification.md Resolved
Comment thread AGENTS.md Resolved
@dominicnunez
Copy link
Copy Markdown
Owner Author

Scope clarification for the final head 43146d9: the maintainer explicitly confirmed that this workflow assumes the global AGENTS.md is available and does not need to support agents without it. The generic final-head review, live-operation authorization, shutdown/replay/performance evidence, self-contained documentation, authorized issue handling and threat-model maintenance rules are preserved in verified global guidance.

Accordingly, the extra public documents introduced during earlier review iterations are removed/restored to their original main content; those earlier thread replies describe superseded intermediate fixes. The final PR-base diff changes only AGENTS.md, retaining Agent OS-specific rules. The remaining missing-public-duplicate findings are intentional migration decisions, not missing instructions in the combined workflow. The new-document index finding is obsolete because no new document remains. Independent review verified the combined global/project coverage under this clarified scope.

@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: 43146d9552

ℹ️ 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 AGENTS.md Resolved
@dominicnunez
Copy link
Copy Markdown
Owner Author

@codex security review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment
Copy link
8000 Copy Markdown

Choose a reason for hiding this comment

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

🛡️ Codex Security Review

Here are some automated security review suggestions for this pull request.

Reviewed commit: 9c266a4c4c

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

Comment thread AGENTS.md Resolved
@dominicnunez
dominicnunez merged commit d940514 into main Oct 4, 2026
10 checks passed
@dominicnunez
dominicnunez deleted the docs/consolidate-agent-guidance branch October 4, 2026 12:32
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

0