Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: cortexkit/magic-context
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: mittalsuraj18/magic-context
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 8 commits
  • 98 files changed
  • 1 contributor

Commits on May 10, 2026

  1. feat(oh-my-pi): add full oh-my-pi harness support

    - Port entire pi-plugin source (~50 files) to oh-my-pi-plugin with:
      - Package imports: @mariozechner/pi-coding-agent → @oh-my-pi/pi-coding-agent
      - Config paths: .pi/ → .omp/, ~/.pi/ → ~/.omp/
      - Harness identity: pi → oh-my-pi
      - API adaptation: before_agent_start returns string[] (not string)
    
    - Bootstrap oh-my-pi-plugin package:
      - package.json with peer deps on @oh-my-pi/pi-coding-agent
      - tsconfig.json path-mapped to @magic-context/core
      - Added to root workspace build/test/lint scripts
    
    - Integrate CLI for oh-my-pi:
      - New adapter: OhMyPiAdapter (adapters/oh-my-pi.ts)
      - New helpers: oh-my-pi-helpers.ts (binary detection, model listing)
      - New commands: setup-oh-my-pi.ts, doctor-oh-my-pi.ts
      - New diagnostics: diagnostics-oh-my-pi.ts, logs-oh-my-pi.ts
      - Wire into harness-select.ts, setup.ts, doctor.ts dispatchers
    
    - Update shared harness.ts to include 'oh-my-pi' in HarnessId union
    
    - Fix: Remove pi-tui runtime import (causes Bun segfault with pi-natives)
    - Fix: Update ctx-status to skip TUI dialog for oh-my-pi compatibility
    
    All 1728 tests pass, build and lint clean.
    Suraj Mittal committed May 10, 2026
    Configuration menu
    Copy the full SHA
    0de0e6d View commit details
    Browse the repository at this point in the history
  2. docs: add oh-my-pi installation instructions to READMEs

    - Create packages/oh-my-pi-plugin/README.md with detailed local install steps
    - Add oh-my-pi section to root README.md with quick local install guide
    - Document the isolated-copy requirement (avoids Bun segfault from pi-natives)
    - Include rebuild alias for rapid development iteration
    Suraj Mittal committed May 10, 2026
    Configuration menu
    Copy the full SHA
    246170b View commit details
    Browse the repository at this point in the history
  3. docs(oh-my-pi): add comprehensive testing and troubleshooting guide

    - 10-step verification checklist for testing oh-my-pi plugin installation
    - Commands: /ctx-status, /ctx-dream
    - Tools: ctx_memory, ctx_search, ctx_note
    - Cross-session persistence test
    - Cross-harness sharing test (OpenCode <-> oh-my-pi)
    - Troubleshooting section for common issues:
      - Bun segfault (pi-natives)
      - Plugin not showing in list
      - Commands/tools not found
      - Database errors
      - Build errors
    Suraj Mittal committed May 10, 2026
    Configuration menu
    Copy the full SHA
    b4cdb00 View commit details
    Browse the repository at this point in the history
  4. test(e2e): add oh-my-pi end-to-end test suite

    Add comprehensive e2e tests for oh-my-pi harness:
    
    - src/oh-my-pi-runner/spawn.ts: Spawn isolated omp turns with plugin
      - Creates isolated plugin copy (NOT symlink) to avoid pi-natives segfault
      - Uses ~/.omp/ config paths and oh-my-pi binary detection
      - Adapts before_agent_start return to string[] for oh-my-pi API
    
    - src/oh-my-pi-harness.ts: Test harness facade
      - countTags, countPendingOps, countDroppedTags with harness='oh-my-pi'
      - waitFor, mock provider integration, DB assertions
    
    Test coverage:
    - oh-my-pi-smoke: Plugin loads, tools registered, no crash
    - oh-my-pi-tagging: §N§ tags persisted with harness='oh-my-pi'
    - oh-my-pi-drops: pending_ops drain at execute threshold
    - oh-my-pi-cross-harness: Memory sharing OpenCode <-> oh-my-pi
    - oh-my-pi-conflict: Built-in compaction conflict detection
    - oh-my-pi-system-prompt: before_agent_start string[] injection
    
    Typecheck passes clean.
    Suraj Mittal committed May 10, 2026
    Configuration menu
    Copy the full SHA
    8d183a0 View commit details
    Browse the repository at this point in the history

Commits on May 11, 2026

  1. docs(oh-my-pi): add comprehensive configuration guide and fix e2e tests

    - Add CONFIGURATION.md with detailed explanations of every magic-context.jsonc setting
    - Update README.md to reference local CONFIGURATION.md and add quick example
    - Fix e2e test: remove --no-extensions flag (breaks oh-my-pi's --extension loading)
    - Fix e2e test: use full path to extension entry file for oh-my-pi
    - Adjust system-prompt test expectations for oh-my-pi print mode behavior
    Suraj Mittal committed May 11, 2026
    Configuration menu
    Copy the full SHA
    a6dd819 View commit details
    Browse the repository at this point in the history
  2. fix(oh-my-pi): make dreamer registration lazy for directory changes

    The dreamer was only registered once at plugin load time with process.cwd().
    When users changed directories or worked in multiple projects, /ctx-dream
    failed with 'Pi dreamer not registered for project ...' because the
    registration lookup used the initial directory's projectIdentity.
    
    Fix:
    - Extract doRegisterPiDreamerProject helper for on-demand registration
    - Make runPiDreamForProject accept fallback PiDreamerOptions for lazy reg
    - Update /ctx-dream handler to resolve projectIdentity from ctx.cwd at
      runtime (not the static plugin-load value)
    - Pass dreamer config through command deps so lazy registration has the
      model, tasks, and schedule it needs
    - Move dreamerConfig resolution before /ctx-dream command registration
    
    Now /ctx-dream works in any directory, registering the dreamer on-the-fly
    if needed. E2E tests pass (7/7).
    Suraj Mittal committed May 11, 2026
    Configuration menu
    Copy the full SHA
    477279f View commit details
    Browse the repository at this point in the history
  3. test(oh-my-pi): add tests for dreamer lazy registration and multi-dir…

    …ectory support
    
    Unit tests:
    - dreamer/index.test.ts: test runPiDreamForProject lazy registration
      - with fallbackOpts: registers on-the-fly and returns null (empty queue)
      - without fallbackOpts: throws 'not registered' as expected
      - reuses existing registration when already registered
    
    - commands/ctx-commands.test.ts: test /ctx-dream dynamic cwd resolution
      - verifies projectIdentity is resolved from ctx.cwd at runtime
      - proves static projectIdentity is NOT used for queueing
    
    E2E tests:
    - oh-my-pi-multi-dir.test.ts: verifies plugin works across directories
      - plugin loads without crash from different directory
      - /ctx-dream does not crash from different project directory
    
    All oh-my-pi-plugin tests pass: 240/240.
    Suraj Mittal committed May 11, 2026
    Configuration menu
    Copy the full SHA
    bf9bdf0 View commit details
    Browse the repository at this point in the history
  4. Merge remote-tracking branch 'origin/master'

    Suraj Mittal committed May 11, 2026
    Configuration menu
    Copy the full SHA
    127e447 View commit details
    Browse the repository at this point in the history
Loading