Repository navigation
openehr-specs: class tables from the repo's BMM, scaffold revision 2, no-ticket amendments - #8
Merged
Conversation
class-generation and regen-classes mounted no schema files and passed bundled schema ids, so bmm-publisher rendered its own, older copy of the BMM without any error; the suggested mount over /app/resources hid the bundled dependencies (exit 1). Both now mount the component repo's BMM and its sibling dependencies under /in/ and pass paths; regen-classes stops when a file is missing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…epo tooling AGENTS.md build region: no longer claims the repo has no build tooling, and the class-table command loads the sibling BASE BMM with -d (new variable base_bmm_schema_id, inferred from the specifications-BASE clone), so links to BASE types resolve. New repos get a TODO(scaffold) marker for their own tooling; .claude/CLAUDE.md and the skill's rules say repo guidance goes into AGENTS.md outside the regions; first_release is confirmed only when manifest.json is created. Upgrade tests now follow the latest revision instead of assuming revision 1. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
No invented or placeholder key; offer to raise a ticket; otherwise an entry without a Jira reference (266 of 974 existing entries have none); no entry when the author says none is needed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…mple The example used the retired MagicDraw uml expressions; it now shows RM's ITS-BMM-RM url expression and ITS-BMM-BASE dependency. Co-Authored-By: Claude Opus 5.5 <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
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.
Summary
Fixes found while running the
openehr-specsworkflow end to end onspecifications-TERM.class-generationandregen-classesnow generate from the repo's own BMM (fix). They passed bundled schema ids, sobmm-publisherrendered its own copy of the BMM, which lags the repos. A run with the documented command exited 0 but did not show a fix just made tospecifications-TERM's BMM. The suggested mount over/app/resourceshid the bundled dependencies and exited 1. Both skills now mount the component'scomputable/BMM/<id>.bmm.jsonand its sibling dependencies under/in/and pass paths.regen-classesstops when a file is missing instead of falling back to an id. Ids remain documented for reproducing published output.scaffoldtemplate set revision 2 (feat):AGENTS.mdbuild region no longer says "No build tooling lives in this repo", which is wrong for TERM's PHP generator. New repos get aTODO(scaffold)marker asking them to describe their own tooling.-d, through a new variablebase_bmm_schema_id. It is inferred from thespecifications-BASEclone, highest version first, and is empty for BASE itself. Without it, links to BASE types come out as/classes/String..claude/CLAUDE.mdand the skill's rules say that repo guidance, such as/init's, goes intoAGENTS.mdoutside the regions.first_releaseis confirmed only whenmanifest.jsonis created.migrations/0002.jsoncarries a note for existing repos, which do not get the new marker. A dry-run plan upgrades BASE's build region (without-d) and leaves TERM's pinned one alone.amendment-recordcovers changes without a ticket (docs). No invented or placeholder key, an offer to raise a ticket, otherwise an entry without a reference (266 of the 974 existing entries have none), and no entry when the author says none is needed.manifest-spec-entry.mdexample shows the currentITS-BMM-RMurl expression andITS-BMM-BASEdependency, copied fromspecifications-RM/manifest.json, instead of MagicDrawumlexpressions (docs).Five upgrade tests hard-coded revision numbers that assumed the shipped set is revision 1. They now follow the latest revision, so the next revision will not break them.
Checklist
python3 scripts/validate.pyandpython3 -m unittest discover -s scripts -p 'test_*.py'pass, on each of the four commits (117 tests;scaffold.py checkclean)Unreleased; no version bumpbmm-publisher0.11.0, the manifest example is RM's actual manifest, and the no-ticket count comes from a survey of the localspecifications-*clones🤖 Generated with Claude Code