Skip to content

fix: Missing CHANGELOG.md#46

Merged
simongonzalezdc merged 2 commits into
mainfrom
fix/issue-45-missing-changelog-md
May 21, 2026
Merged

fix: Missing CHANGELOG.md#46
simongonzalezdc merged 2 commits into
mainfrom
fix/issue-45-missing-changelog-md

Conversation

@simongonzalezdc
Copy link
Copy Markdown
Member

@simongonzalezdc simongonzalezdc commented May 16, 2026

Summary

  • Fixes Missing CHANGELOG.md #45 by adding the missing root CHANGELOG.md.
  • Removes the accidental .omc runtime/session artifacts that did not satisfy the repository-health issue.

Files Changed

  • CHANGELOG.md

Evidence

  • test -f CHANGELOG.md passed locally.
  • git diff --cached --check passed before commit.
  • Codex review thread is now outdated on GitHub after commit df4cb5e.
  • GitHub check agent-law passed after push.

Closes #45

(Issue Closer / Matt Pocock / Deep modules, domain language, disciplined diagnosis)


View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

Closes #45

_(🧰 Issue Closer · 🎨 Matt Pocock)_
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

Here are some automated review suggestions for this pull request.

Reviewed commit: aa024b77dc

ℹ️ 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 .omc/project-memory.json Outdated
The branch was carrying OMX runtime/session state while the issue and review asked for the root changelog artifact. This swaps the PR payload to the repository-health file the automation actually expects.

Constraint: PR 46 exists to satisfy the missing CHANGELOG.md repository standard.

Rejected: Keep .omc runtime artifacts | they do not satisfy the requested repository artifact.

Confidence: high

Scope-risk: narrow

Directive: Keep runtime session metadata out of repository health PRs.

Tested: test -f CHANGELOG.md; git diff --cached --check

Not-tested: GitHub Actions, doc-only metadata change

Co-authored-by: OmX <omx@oh-my-codex.dev>
@simongonzalezdc simongonzalezdc merged commit 85ac06a into main May 21, 2026
1 check passed
@simongonzalezdc simongonzalezdc deleted the fix/issue-45-missing-changelog-md branch May 21, 2026 04:41
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.

Missing CHANGELOG.md

1 participant