Skip to content

docs(msdmd): align public contract with native readers - #114

Merged
erinepshovel-code merged 21 commits into
mainfrom
repair/msdmd-readme-reader-contract
Oct 2, 2026
Merged

erinepshovel-code merged 21 commits into
mainfrom
repair/msdmd-readme-reader-contract

Conversation

@erinepshovel-code

Copy link
Copy Markdown
Contributor

Summary

  • repair README's opening MSDMD contract so it no longer claims the shipped collector is block-only
  • refresh the README implementation-status/core-idea sections to match the schema-2 native-and-supplemental collector
  • update CLAUDE.md, ORG_DISTRIBUTION.md, and runner guidance so assistant/distribution docs no longer contradict skills.json, msdmd/SKILL.md, or merged fix(msdmd): integrate native standards readers and fail-closed coverage #111

Scope

This is a documentation/public-contract repair. It does not add reader behavior. The implemented reader surface remains exactly the existing schema-2 supported subset in msdmd/collect.py/msdmd/readers.py, plus the separate Python module projection runner.

Verification

GitHub checks are required for terminal verification. I could not run the repository's local stdlib suite from this connector-only path.

hmmm

Downstream propagation into Stack and other consumers remains a separate exact-source-pinned refresh after this PR merges.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 2, 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-02T01:43:29.600295Z 3ea19c0 New commits
ℹ️ 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: 5de6068b58

ℹ️ 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 ORG_DISTRIBUTION.md
Comment thread ORG_DISTRIBUTION.md
Comment thread CLAUDE.md
Comment thread CLAUDE.md
Comment thread ORG_DISTRIBUTION.md
@erinepshovel-code
erinepshovel-code merged commit ad895d2 into main Oct 2, 2026
6 checks passed

@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: 3ea19c02c1

ℹ️ 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 ORG_DISTRIBUTION.md
Comment thread README.md
Comment thread ORG_DISTRIBUTION.md

Copy link
Copy Markdown
Contributor Author

Follow-up #115 repaired the remaining post-merge findings and merged at 3024df2b91a2cf7f103dc60640081ec03505335a. Post-merge hygiene and full skill-lib-ci both pass on that exact main commit. Resolving the superseded #114 threads against the repaired source.

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