-
Notifications
You must be signed in to change notification settings - Fork 55
Comparing changes
Open a pull request
base repository: cortexkit/magic-context
base: master
head repository: mittalsuraj18/magic-context
compare: master
- 8 commits
- 98 files changed
- 1 contributor
Commits on May 10, 2026
-
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 committedMay 10, 2026 Configuration menu - View commit details
-
Copy full SHA for 0de0e6d - Browse repository at this point
Copy the full SHA 0de0e6dView commit details -
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 committedMay 10, 2026 Configuration menu - View commit details
-
Copy full SHA for 246170b - Browse repository at this point
Copy the full SHA 246170bView commit details -
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 committedMay 10, 2026 Configuration menu - View commit details
-
Copy full SHA for b4cdb00 - Browse repository at this point
Copy the full SHA b4cdb00View commit details -
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 committedMay 10, 2026 Configuration menu - View commit details
-
Copy full SHA for 8d183a0 - Browse repository at this point
Copy the full SHA 8d183a0View commit details
Commits on May 11, 2026
-
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 committedMay 11, 2026 Configuration menu - View commit details
-
Copy full SHA for a6dd819 - Browse repository at this point
Copy the full SHA a6dd819View commit details -
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 committedMay 11, 2026 Configuration menu - View commit details
-
Copy full SHA for 477279f - Browse repository at this point
Copy the full SHA 477279fView commit details -
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 committedMay 11, 2026 Configuration menu - View commit details
-
Copy full SHA for bf9bdf0 - Browse repository at this point
Copy the full SHA bf9bdf0View commit details -
Merge remote-tracking branch 'origin/master'
Suraj Mittal committedMay 11, 2026 Configuration menu - View commit details
-
Copy full SHA for 127e447 - Browse repository at this point
Copy the full SHA 127e447View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...master