Skip to content

Support oh-my-pi agent sessions - #519

Merged
powerfooI merged 3 commits into
powerfooI:mainfrom
piqusy:feat/omp-session-inspection
Oct 30, 2026
Merged

powerfooI merged 3 commits into
powerfooI:mainfrom
piqusy:feat/omp-session-inspection

Conversation

@piqusy

@piqusy piqusy commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Recognize oh-my-pi sessions and project their Pi-compatible transcripts.
  • Find local sessions in default, profile, XDG, PI_CONFIG_DIR, and configured agent directories. Remote sessions use the transcript path reported by Herdr; remote session ID-only discovery is not supported.
  • Preserve Bash and Python execution commands, output, exit status, and cancellation in session inspection and history.
  • Preserve user-invoked OMP /skill: requests in History, ATIF, and turn counts. Use the original draft when available and reconstruct older prompts from their skill name and arguments.

Verification

  • bun test server/src/agent: 101 passed, including regression cases for original skill drafts, legacy arguments, no arguments, and history/summary endpoints.
  • Confirmed the regression tests fail before the skill-request fix.
  • Formatting checks passed for the three files changed by the fix.
  • Pre-commit lint and bun run typecheck:bun passed. Lint reports one existing warning in TerminalView.tsx.

Real OMP/Herdr browser integration was not re-tested for the skill-request fix.

@github-actions github-actions Bot added the enhancement New feature or request label Oct 30, 2026
@powerfooI
powerfooI merged commit 8a2a4c7 into powerfooI:main Oct 30, 2026
5 checks passed
@piqusy

piqusy commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

Thank you @powerfooI for allowing this feature contribution

@piqusy
piqusy deleted the feat/omp-session-inspection branch October 31, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants