Skip to content

fix(manifest): surfaces 'gui' -> 'ui' (schema enum) - #8

Merged
lukisch merged 1 commit into
mainfrom
fix/manifest-surfaces-ui
Oct 12, 2026
Merged

lukisch merged 1 commit into
mainfrom
fix/manifest-surfaces-ui

Conversation

@lukisch

@lukisch lukisch commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

ellmos-module.v2.json declared surfaces: [..., "gui"]. The ellmos.module.v2 schema enum is library, cli, service, ui, workflow, skill, dataset, template, mcp-adapter, so gui is invalid and the module catalog build (build_catalog.py, error surfaces/2) fails. llm-note's GUI is a local web interface (llm_note/gui.py, stdlib http.server), which is the ui surface. Only this one value changes; 34 tests pass locally. Needed for BACH task #2068 (catalog reconciliation). Draft, not merged.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XW6jSy5hqFumsSYisAd9fC

The module schema allows library, cli, service, ui, workflow, skill, dataset,
template, mcp-adapter; 'gui' is invalid and made build_catalog.py fail
(surfaces/2). llm-note's GUI is a local web interface (llm_note/gui.py,
http.server), i.e. the 'ui' surface. Only this value changes.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XW6jSy5hqFumsSYisAd9fC
@lukisch
lukisch marked this pull request as ready for review October 11, 2026 17:56
@lukisch

lukisch commented Oct 11, 2026

Copy link
Copy Markdown
Contributor Author

Integrator-Abnahme (claude-code-opus@ASUS-GEI, BACH-TaskDB #2068): Diff nur surfaces: "gui" → "ui" in ellmos-module.v2.json. Schema-Enum (library, cli, service, ui, workflow, skill, dataset, template, mcp-adapter) kennt kein "gui"; llm-note hat eine lokale Weboberfläche (llm_note/gui.py, stdlib ThreadingHTTPServer) → "ui" ist der passende Wert. CI 20/20 grün. Wirkung: build_catalog.py --check im OneDrive-Modulkatalog wird wieder grün (bricht heute nur an diesem Manifest ab).

@lukisch

lukisch commented Oct 11, 2026

Copy link
Copy Markdown
Contributor Author

@claude Bitte reviewen und mergen (BACH-TaskDB #2068). Einzeilige Manifest-Korrektur surfaces "gui"→"ui" (Schema-Enum, llm-note hat eine lokale Weboberfläche); CI 20/20 grün; Integrator-Abnahme oben. Danach spiegelt ein Worker das Manifest nach OneDrive und erzeugt den Modulkatalog neu.

lukisch commented Oct 11, 2026

Copy link
Copy Markdown
Contributor Author

Zweitreview (Cloud-Bot): Einzeiler gui → ui im Manifest (Schema-Enum). Alle 20 Checks sind grün. Aus Sicht des Zweitreviews merge-fähig. Eine formale Approval ist nicht möglich, weil der Bot dasselbe Konto wie der Autor nutzt.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Zweitmodell-Review: Einzeiler gui -> ui laut Schema-Enum, alle 20 Checks grün. Freigabe empfohlen.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Zweitmodell-Review (LGTM, mergebereit): Einzeiliger Manifest-Fix (gui → ui, Schema-Enum), CI auf allen OS/Python-Versionen grün.


Generated by Claude Code

lukisch commented Oct 11, 2026

Copy link
Copy Markdown
Contributor Author

Zweitmodell-Review (Cloud-Bot) – Einzeilige Manifest-Korrektur (surfaces: gui → ui, Schema-Enum), CI vollständig grün. Hinweis: falls README/Docs die Surface "gui" erwähnen, sind die Bezeichnungen nicht betroffen (Schema-Enum). Freigabefähig.


Generated by Claude Code

@lukisch
lukisch merged commit ce5d1a1 into main Oct 12, 2026
20 checks passed
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