Skip to content

Add AI-generated model summaries, coder-facing explanations, and explanation skills - #7

Merged
Psuedonerd merged 2 commits into
masterfrom
codex/inspect-model-folders-in-rulehub-repo-r7dj0y
Jul 13, 2026
Merged

Psuedonerd merged 2 commits into
masterfrom
codex/inspect-model-folders-in-rulehub-repo-r7dj0y

Conversation

@Psuedonerd

Copy link
Copy Markdown
Owner

Motivation

  • Provide machine-generated, biologist- and coder-facing explanations for RuleHub models to support site integration and curator workflows.
  • Index the produced summaries in a JSON index to enable programmatic consumption by the website and tools.
  • Add skill descriptions and agent interface metadata so the repository can generate consistent summaries for different audiences.

Description

  • Added a machine-readable index data/ai_summaries.json containing summary metadata for many models.
  • Added biologist-facing Markdown summaries under data/summaries/*.md that follow the required human-readable structure for each model.
  • Added coder-facing, technical Markdown explanations under data/summaries_coders/*.md that enumerate molecule types, rules, observables, and actions.
  • Introduced new skill definitions and agent metadata under skills/ (explain-rulehub-model-coders, explain-rulehub-model-simple) including SKILL.md guidance and an openai.yaml agent interface, and removed the deprecated skills/explain-rulehub-model/SKILL.md file.

Testing

  • Validated the new JSON with jq to ensure it is syntactically valid (success).
  • Spot-checked several new Markdown files in a local Markdown viewer to confirm they render and the required sections are present (success).
  • No automated unit tests were added for this change; there are no failing test runs associated with these content additions.

Codex Task

@Psuedonerd
Psuedonerd merged commit 816a41f into master Jul 13, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

"model_id": "lac_operon_dreisigmeyer2008",

P2 Badge Use metadata ids in the summary index

For entries like this one, model_id is the BNGL basename, but yaml_path points to metadata whose id is Dreisigmeyer_2008, and the generated manifest uses metadata.id as the model id in scripts/generate-manifest.js:87-90. In consumers that join data/ai_summaries.json to manifest/gallery records by id, these entries will not match and the summaries for Dreisigmeyer, Gardner, Goldstein, Harmon, Hlavacek 1999, and Hlavacek 2001 will be hidden. Please populate model_id from the metadata id consistently.



P2 Badge Add frontmatter to the simple skill

In environments that load Codex skills from skills/*/SKILL.md, this file will not be discoverable because it starts directly with a Markdown heading instead of the required YAML frontmatter. The adjacent coder skill has name and description frontmatter, but this replacement simple skill does not, so the biologist-facing explanation skill cannot be selected by its trigger description. Please add name: explain-rulehub-model-simple and a description before the heading.

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

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.

1 participant