Repository navigation
Add AI-generated model summaries, JSON index, and RuleHub explanation skills - #6
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5356cc0660
ℹ️ 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".
| ] | ||
| }, | ||
| { | ||
| "model_id": "lac_operon_dreisigmeyer2008", |
There was a problem hiding this comment.
Use metadata ids for summary entries
When consumers join this new index to the existing RuleHub manifest/gallery by model id, this entry will not match: Published/Dreisigmeyer2008/metadata.yaml defines id: "Dreisigmeyer_2008", and manifest.json/gallery.json are keyed by that metadata id, but the summary index uses the BNGL basename here. The same mismatch occurs for the Gardner, Goldstein, Harmon, Hlavacek 1999, and Hlavacek 2001 entries, so those summaries become orphaned in site integration or programmatic lookup by the repository's canonical ids.
Useful? React with 👍 / 👎.
Motivation
data/ai_summaries.json.Description
data/ai_summaries.jsoncontaining summarised entries (model id, paths, one-line summary, andkey_events) for models and tutorials.data/summaries/anddata/tutorial_summaries/to document individual published models and tutorial examples (new*_aigenerated.mdfiles).skills/explain-rulehub-model-coders/includingSKILL.mdand anagents/openai.yamldescriptor to support technical BNGL/YAML explanations for experienced modelers.skills/explain-rulehub-model/SKILL.mdto align guidance and output structure with the generated summaries and JSON index expectations.Testing
Codex Task