Skip to content

chore(manifest): set source_of_truth type to git-repository - #21

Open
lukisch wants to merge 1 commit into
masterfrom
chore/manifest-git-repository-source-of-truth
Open

lukisch wants to merge 1 commit into
masterfrom
chore/manifest-git-repository-source-of-truth

Conversation

@lukisch

@lukisch lukisch commented Nov 9, 2026

Copy link
Copy Markdown
Member

Update source_of_truth.type from local-directory to git-repository for catalog alignment.

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

Welcome! Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what and why

Thanks for contributing.

lukisch commented Nov 10, 2026

Copy link
Copy Markdown
Member Author

Zweitmodell-Review (Cloud-Bot): ✅ inhaltlich unbedenklich – formales Gate bleibt offen.

  • Diff: eine Zeile in ellmos-module.v2.json, source_of_truth.type local-directory → git-repository. path und repository (https://github.com/ellmos-ai/clutch) bleiben unverändert und passen zum neuen Typ.
  • Konsistenz: Im Repo selbst wertet kein Code diesen Wert aus. In den Nachbar-Manifesten ist git-repository bereits der überwiegende Wert (35 von 54 lokal geprüften ellmos-module.v2.json), system-explorer (manifests.py) und ellmos-unified-gui (config.py) kennen ihn.
  • CI: alle Test-Jobs (Ubuntu/macOS/Windows, Python 3.10–3.13) und CodeQL grün.
  • Offen: der Status ellmos/review-gate steht auf pending („Awaiting a signed exact-SHA reviewer or owner receipt“). Diese signierte Quittung kann der Cloud-Bot nicht ausstellen; sie muss vom Owner bzw. dem signierenden Reviewer für Head 3ce49b7 kommen.

Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zweitprüfung (Head 3ce49b7): keine Einwände, freigabefähig.

  • Einzeilige Manifest-Änderung in ellmos-module.v2.json: source_of_truth.type von local-directory auf git-repository. Das passt zum bereits gesetzten repository-Feld.
  • Im Repo liest kein Code und kein Test den alten Wert local-directory; es bricht also nichts.
  • Lokal am Head ausgeführt: pytest -q → 388 passed, 4 skipped (fehlende optionale Abhängigkeiten fastapi und companion-for-agy).
  • CI am Head: alle Testjobs (3 OS × Python 3.10–3.13) und CodeQL grün.

Nicht geprüft: ob externe Konsumenten des Manifests (z. B. Registry oder Validator in anderen Repos) den Wert git-repository im Schema zulassen. Formales Approve ist nicht möglich, weil der PR vom selben Account stammt.


Generated by Claude Code

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