feat: add oh-my-pi (omp) platform to install scripts - #425
Closed
0xnayuta wants to merge 1 commit into
Closed
Conversation
5 tasks
Add dedicated 'omp' platform entry targeting omp's native skills directory (~/.omp/agent/skills) for better isolation and discoverability. Changes: - install.sh: add omp to platforms_table() and document omp/pi relationship in usage() - install.ps1: add omp to table - README.md: add oh-my-pi / omp to platform table, supported list, and section heading - READMEs/*.md: add oh-my-pi / omp row to all 7 translated platform tables Closes Egonex-AI#424
0xnayuta
force-pushed
the
feat/oh-my-pi-platform
branch
from
August 10, 2026 09:15
341dbdf to
30bbf66
Compare
There was a problem hiding this comment.
Pull request overview
Adds explicit support for the oh-my-pi (omp) platform across the repo’s install surface area, so omp users can install into oh-my-pi’s native skills directory (~/.omp/agent/skills) instead of needing to know to use the pi target.
Changes:
- Adds an
ompplatform entry to bothinstall.shandinstall.ps1, targeting~/.omp/agent/skills(per-skill linking). - Updates the main README to list
ompas a supported<platform>value and to documentinstall.sh omp. - Updates several translated READMEs to include
oh-my-pi / ompin the platform compatibility table.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Documents omp as a supported platform and adds it to the compatibility table and one-line install section. |
| install.sh | Adds omp to the platform table (targeting ~/.omp/agent/skills) and expands help text notes. |
| install.ps1 | Adds omp to the Windows platform table (targeting $HOME\.omp\agent\skills). |
| READMEs/README.zh-TW.md | Adds oh-my-pi / omp to the compatibility table. |
| READMEs/README.zh-CN.md | Adds oh-my-pi / omp to the compatibility table. |
| READMEs/README.tr-TR.md | Adds oh-my-pi / omp to the compatibility table. |
| READMEs/README.ru-RU.md | Adds oh-my-pi / omp to the compatibility table. |
| READMEs/README.ko-KR.md | Adds oh-my-pi / omp to the compatibility table. |
| READMEs/README.ja-JP.md | Adds oh-my-pi / omp to the compatibility table. |
| READMEs/README.es-ES.md | Adds oh-my-pi / omp to the compatibility table. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| ### One-line install (Codex / OpenCode / OpenClaw / Antigravity / Gemini CLI / Pi Agent / Vibe CLI / VS Code Copilot / Hermes / Cline / KIMI CLI / Trae / Nanobot / Kiro) | ||
|
|
||
| ### One-line install (Codex / OpenCode / OpenClaw / Antigravity / Gemini CLI / Pi Agent / omp / Vibe CLI / VS Code Copilot / Hermes / Cline / KIMI CLI / Trae / Nanobot / Kiro) |
Comment on lines
31
to
36
| gemini|$HOME/.agents/skills|per-skill | ||
| codex|$HOME/.agents/skills|per-skill | ||
| opencode|$HOME/.agents/skills|per-skill | ||
| pi|$HOME/.agents/skills|per-skill | ||
| omp|$HOME/.omp/agent/skills|per-skill | ||
| openclaw|$HOME/.openclaw/skills|folder |
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
Add a dedicated
ompplatform entry to both installers, targeting oh-my-pi's native user-level skills directory (~/.omp/agent/skills). oh-my-pi (omp) is a fork of Pi (11.6k+ stars) at https://github.com/can1357/oh-my-pi.omp already discovers skills from
~/.agents/skills(the existingpitarget), soinstall.sh pitechnically works, but is non-obvious for omp users. This PR adds a dedicatedompplatform using omp's native path for better isolation and discoverability.Linked issue(s)
Closes #424
How I tested this
pnpm lintpnpm --filter @understand-anything/core testpnpm testplatforms_table(),platform_ids(), and usage outputVersioning