Add OMP provider support#298
Open
jenkei wants to merge 1 commit into
Open
Conversation
Add OMP provider metadata, generated skill output, CLI install/update path handling, and test coverage for build, install, and pin behavior.
|
Too many files changed for review. ( |
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.
Scope
Add OMP provider support.
Fixes #297.
.omp/skills/impeccableoutput..omp/skillswithdist/omp/.omp/skills.Validation
bun run buildbun run test(repo-supported default harness perDEVELOP.md)diff -qr .omp/skills dist/omp/.omp/skills; targeted OMP tests pass).omp/skills/.Notes
bun run test:live-svelte-adapter-deepseekfailed on a stale assertion: the UI now uses an icon button witharia-label="Generate variants"instead of visible "Go" text, soassertSelectedElementChrome'shasGocheck is outdated. This is unrelated to the OMP provider changes.Note
Low Risk
Mostly generated docs and install/pin tooling; no runtime auth or data paths. Risk is limited to wrong invoke strings in pinned shortcuts if the prefix map drifts from a harness.
Overview
Adds OMP as a first-class install target with generated
.omp/skills/impeccablecontent, wired through the existing cross-provider build (alongside CLI aliases, install/update paths, and download lookup described in the PR).Pin shortcuts (
pin.mjsacross harness copies) now recognize.github,.qoder, and.omp, and embed the correct invoke syntax per harness instead of unresolved{{command_prefix}}placeholders. OMP pinned skills point at/skill:impeccable(and subcommands as/skill:…); Codex/Agents keep$, most others keep/.The OMP skill bundle mirrors the full impeccable surface—main
SKILL.md, scripts, and reference flows—with paths and command examples rewritten for.omp/skills/...and/skill:routing.Reviewed by Cursor Bugbot for commit a9b6b2f. Bugbot is set up for automated code reviews on this repo. Configure here.