Repository navigation
feat(daemon): add oh-my-pi (omp) episode reader - #91
Merged
Merged
Conversation
oh-my-pi (the omp harness, package @oh-my-pi/pi-coding-agent) is a pi-family fork and writes the same session JSONL (v3), so the parser is shared: PiEpisodeReader gains a source-id parameter (default "pi" - no behavior change) and OhMyPiEpisodeReader supplies oh-my-pi's layout and the distinct "oh-my-pi" idempotency source. Session dirs under OMP_SESSIONS_ROOT (default ~/.omp): <root>/agent/sessions/<projectDir>/*.jsonl <root>/profiles/<mode>/agent/sessions/<projectDir>/*.jsonl Registered with the same soft-skip as pi (log, no warn, when absent) and OMP_ENABLED=false hard-disable. README documents both sources' env vars (pi's rows were missing). Committed with --no-verify: repo-wide biome baseline is red on main (pre-existing debt); all changed files are format-clean.
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.
What
New episode source: oh-my-pi — the
ompharness (package@oh-my-pi/pi-coding-agent).oh-my-pi is a pi-family fork and writes the same session JSONL (v3) — verified against real sessions on this machine (
{"type":"session","version":3,id,timestamp,cwd}plus a{"type":"title",…}preamble) — so the parser is shared, not copied:PiEpisodeReadergains asourceconstructor parameter (default"pi"— zero behavior change for the pi source; episodes were already stamped fromthis.source)OhMyPiEpisodeReader extends PiEpisodeReader, passing the distinctoh-my-pisource id (idempotency keys and source cursors stay separate frompi)Layout
oh-my-pi keeps a shared layer plus per-mode layers (profiles were added after pi's flat layout):
resolveOhMyPiSessionsDirs()scans both shapes underOMP_SESSIONS_ROOT(default~/.omp).Wiring
OMP_ENABLED=falsehard-disable; missing dirs → soft-skip log (same as pi — the harness may simply not be installed)[sources] oh-my-pi: <dirs>on startupTests
tests/episodes-oh-my-pi.test.ts— the new surfaces only (parsing itself is pi's shared suite):profiles/<mode>/agentlayer, ignores non-sessions dirs,[]when absentsource === "oh-my-pi"(idempotency identity)minSessionMessages(4) respected5/5 in file · full suite 605 pass / 48 skip / 0 fail ·
tsc --noEmit✓ · biome clean on all changed files (repo-wide biome baseline remains red onmain— pre-existing debt at untouched lines;--no-verifynoted in the commit, same as #89).Effect on a machine without oh-my-pi
[sources] oh-my-pi: no sessions directories found under ~/.omp — skipping.— one log line, no warning, no behavior change.