Fix theme skill overlap and individually installed reference links - #25
Merged
Merged
Conversation
Route shared styling to theme skills and widget token defects to plugin QML. Bundle the two cross-skill review references locally for individually installed skills. Update workflow navigation and record the bounded diagnosis exercise.
tcballard
marked this pull request as ready for review
September 14, 2026 09:13
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.
Individually installed Plugin skills cannot rely on sibling directories retaining their skill names. Two review-reference links therefore broke after installation. Styling requests also need a clearer boundary between shared theme values and widget QML.
This change bundles the existing review guidance inside Plugin Test and Service IPC, using local links. Bar Widget, QML Patterns and Plugin Debug distinguish shared theme styling from widget token consumption and preserve diagnosis-only scope. Portable and OpenAI adapter copies match. Workflow navigation and the bounded colour-diagnosis evaluation are updated.
Validation: ./scripts/test passes all 55 tests and all portable, packaging, metadata and adapter checks. An installed-directory scan of both bundles found 24 unique skills and no broken or escaping local Markdown links. An independent diagnosis-only exercise identified a hard-coded widget colour, proposed a semantic token binding, preserved the fixture and reported live-host evidence limits. The five affected Plugin skills are already updated and verified in the personal installation.
Companion Theme fixes: tcballard/build-omarchy-themes#1
No new Omarchy schema or marketplace assumptions, version changes, tags or releases. Fresh ChatGPT automatic selection and live desktop behaviour remain untested.