Regenerate HTML for all 61 communities including new EcoFAB#5
Merged
Conversation
- Add missing HTML for EcoFAB_Ring_Trial_SynCom17 - Update all existing community HTML pages with template improvements - Regenerate index.html with complete community listing - Includes security fixes: rel="noopener noreferrer" on external links - Includes case-insensitive DOI detection - Includes extracted CSS classes for better maintainability This completes the PR #4 requirements for the AssociatedDataset schema extension and EcoFAB Ring Trial SynCom17 community. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merged
6 tasks
realmarcin
added a commit
that referenced
this pull request
May 26, 2026
Two small follow-ups to the recent schema-gap audit (#84): - F7: scripts/audit_writers.py's writer-detection heuristic only caught yaml.dump / yaml.safe_dump / path.write_text(yaml.dump(...)) patterns. scripts/clean_metals_inplace.py mutates 149 community YAMLs via regex/line-edit substitutions + path.write_text(string), bypassing the detector entirely. Broaden the heuristic: a file that references the kb/communities path literal AND round-trips the same path variable through .read_text() and .write_text(...) is also a community-YAML writer. clean_metals_inplace.py is the only script that matches this pattern today; it now surfaces in the audit TSV (correctly flagged validates_before_write=no, appends_curation_history=no — slated for conversion as separate work). - Fix #5: GrowthMedia.culturemech_id and GrowthMediaComponent.media_ingredient_mech_id had no pattern declared, while the sibling RelatedMedia.culturemech_id / RelatedIngredient.mediaingredientmech_id slots do. Live data already conforms to the standard ^CultureMech:\d{6}$ / ^MediaIngredientMech:\d{6}$ format — just lock it in to catch future drift at write time. Schema regenerated via `just gen-python`; `validate_strict.py` still reports 0 ERROR rows across 265 files. This PR does NOT convert clean_metals_inplace.py itself — that's a separate, more invasive change (regex surgery → load/dump roundtrip across 149 files). The audit-blind-spot fix above ensures the next audit cycle surfaces it. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This completes the PR #4 requirements for the AssociatedDataset schema extension and EcoFAB Ring Trial SynCom17 community.