Repository navigation
docs: add 'For LLM Agents' setup guide section - #1
Merged
Merged
Conversation
- Add step-by-step guide for LLM agents to setup oh-my-opencode - Include OpenCode installation check with link to official docs - Include oh-my-opencode plugin configuration script - Update Table of Contents for both EN/KO README files
|
I have successfully cloned, built, and installed Since Here is what I did:
The project is now ready for development or use (provided you have the actual You can run the CLI using: bun run dist/cli/index.js --helpThe mock binary is located in |
sssgun
pushed a commit
to sssgun/oh-my-opencode
that referenced
this pull request
Jan 18, 2026
- Add step-by-step guide for LLM agents to setup oh-my-opencode - Include OpenCode installation check with link to official docs - Include oh-my-opencode plugin configuration script - Update Table of Contents for both EN/KO README files
5 of 6 tasks
Closed
qwertystars
referenced
this pull request
in qwertystars/oh-my-opencode
Jan 29, 2026
- Remove sisyphusJuniorModel fallback bypass (BUG #1) - userModel fallback was incorrectly bypassing category fallbackChain - Now correctly uses category requirements before system default - Fix variant preservation in actualModel fallback path (BUG #2) - Variant was lost when categoryModel undefined but actualModel existed - Now preserves variant when falling back to actualModel parsing - Add variant to session continuation (BUG code-yeongyu#3) - Session resume didn't pass variant from message history - Now extracts resumeVariant and passes as body.variant - Remove unused inheritedModel parameter (BUG code-yeongyu#4) - resolveCategoryConfig had misleading unused parameter - Cleaned up function signature and all call sites - Add variant-model compatibility validation (BUG code-yeongyu#5) - No warning when variant might not be supported by provider - Added isVariantLikelySupported() validation with console.warn Test coverage: - Updated 10+ existing tests to remove inheritedModel references - Added 9 new tests for variant validation functions - All 110 tests pass (77 delegate-task + 33 model-requirements)
This was referenced Feb 13, 2026
youngbinkim0
added a commit
to youngbinkim0/oh-my-opencode
that referenced
this pull request
Feb 20, 2026
…ts, abort before retry - Swap resolveAgentForSession priority: session agent before event agent (Bug code-yeongyu#2) - Add pinnedSessionAgentMap so /start-work agent can't be overwritten by SDK events (Bug code-yeongyu#3) - Abort in-flight request before fallback promptAsync to prevent silent drops (Bug code-yeongyu#1) - Add session.get fallback for agent resolution when chat.message never fires (Bug code-yeongyu#4) - Fix agentPattern regex: replace \b with custom boundary for underscore-delimited session IDs - Revert resolveAgentFromModel reverse-lookup (unreliable with overlapping models) - Add tests for pinned agents (8), agent-resolver (17), abort ordering, session.get resolution
luobosibing2
referenced
this pull request
in luobosibing2/oh-my-opencode
Feb 23, 2026
…-yeongyu#309) Critical fixes: - Bug #1: SubagentStop hook defaulted all agents to 'failed' because SDK doesn't provide `success` field. Now defaults to 'completed' when undefined. - Bug code-yeongyu#4: Token stats lost across TokenTracker instances. Constructor now restores session stats from global state for the same session ID. - Bug code-yeongyu#5: Ultrawork session isolation bypassed when both session IDs were undefined (undefined === undefined). Now rejects all falsy session IDs. High priority fixes: - Bug code-yeongyu#6: Cancel skill force-clear missed 12+ state files (boulder, hud-state, subagent-tracking, checkpoints, etc). Added comprehensive list. - Bug code-yeongyu#7: HUD semverCompare() returned NaN on pre-release versions like "3.9.5-beta". Fixed to use parseInt and handle pre-release ordering. - Bug code-yeongyu#8: Silent JSON parse failures in critical state readers. Added error logging to ralph and ultrawork state readers. - Bug code-yeongyu#9: Stale task detection had no default behavior when onStaleSession callback was not configured. Now auto-cleans after 2x threshold. - Bug code-yeongyu#10: Hardcoded 3-architect assumption in validation. Extracted to REQUIRED_ARCHITECTS constant. Medium priority fixes: - Bug code-yeongyu#11: Auto-invoke history used non-atomic writes. Now uses atomicWriteJson to prevent corruption from concurrent sessions. - Bug code-yeongyu#12: Ecomode docs said "all tasks" use Haiku, contradicting the escalation paths. Clarified to "most tasks" with upgrade criteria. - Bug code-yeongyu#13: Added safeUnlinkSync/safeRmSync utilities to prevent ENOENT crashes during cleanup operations. - Bug code-yeongyu#14: State files containing user prompts written with 0644 permissions. Now writes with 0600 (owner-only read/write). - Bug code-yeongyu#15: Model names recorded inconsistently (e.g., 'claude-3-5-haiku' vs 'claude-haiku-4'). Now normalizes at recording time via exported normalizeModelName(). Co-authored-by: Claude Opus 4.5 <[email protected]>
4 tasks done
3 tasks done
4 tasks done
Vacbo
referenced
this pull request
in Vacbo/oh-my-opencode
Mar 12, 2026
Eliminate dual manager instances: - Removed Manager #1 from create-managers.ts - Hook now uses coordinator.resolve() to find binding - Hook uses binding.manager and binding.rlmSessionId Fixed applyFeedback session ID path: - turn-feedback.ts now uses binding.rlmSessionId for createBlobVariable Updated tests to reflect correct RLM behavior: - RLM sessions use applyFeedback, not generic distillation - Tests verify coordinator-based session lookup All 3,608 tests pass.
Vacbo
referenced
this pull request
in Vacbo/oh-my-opencode
Mar 12, 2026
Eliminate dual manager instances: - Removed Manager #1 from create-managers.ts - Hook now uses coordinator.resolve() to find binding - Hook uses binding.manager and binding.rlmSessionId Fixed applyFeedback session ID path: - turn-feedback.ts now uses binding.rlmSessionId for createBlobVariable Updated tests to reflect correct RLM behavior: - RLM sessions use applyFeedback, not generic distillation - Tests verify coordinator-based session lookup All 3,608 tests pass.
Vacbo
referenced
this pull request
in Vacbo/oh-my-opencode
Mar 12, 2026
Eliminate dual manager instances: - Removed Manager #1 from create-managers.ts - Hook now uses coordinator.resolve() to find binding - Hook uses binding.manager and binding.rlmSessionId Fixed applyFeedback session ID path: - turn-feedback.ts now uses binding.rlmSessionId for createBlobVariable Updated tests to reflect correct RLM behavior: - RLM sessions use applyFeedback, not generic distillation - Tests verify coordinator-based session lookup All 3,608 tests pass.
DarkFunct
added a commit
to DarkFunct/oh-my-opencode
that referenced
this pull request
Mar 25, 2026
…tracking - Remove overly broad /not found/i regex that matched grep 'No matches found' output - Replace /expect.*to/i test pattern with /AssertionError/i for precise test failure detection - Narrow trackBuildResult regex from /error/i to /(?:compilation|type)\s+error/i - Add bash quiet-success handler to reset lastBuildResult from 'fail' to 'unknown' Fixes P-007 (false-positive error tracking chain) bugs code-yeongyu#1, code-yeongyu#2, code-yeongyu#4 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <[email protected]>
andrescera
referenced
this pull request
in andrescera/oh-my-cursor
Apr 17, 2026
…fy narrative, reconcile S5 count) - Fix #1: Filled `## Top-3 Selected Ports` with the 3 executed ports (Sisyphus 3bae6dd2, Hephaestus 0dca1c4e, Prometheus 9393ac6c), their sources, targets, and tier-2 classification. - Fix #2: Unified stale narrative cycle-2-dep labels post-Oracle reassignment — ralph-loop: N→Y in Top-ROI bullets and Top-5 candidates (uses subagentStop.followup_message); session-recovery: field corrected from sessionStart.user_message to sessionStart.env + additional_context in Top-5 candidates. unstable-agent-babysitter: no narrative drift found (data table already Y, reconciliation note already correct). - Fix code-yeongyu#3: S5 count reconciliation — option (b): S5 table has exactly 52 data rows (verified by manual count lines 146–197); section header and per-surface table both say 52. FV4 likely miscounted the reconciliation blockquote or header separator as a data row. No file changes needed. Post-FV cleanup commit. No tier reassignments, no port-content changes. Only doc housekeeping.
code-yeongyu
added a commit
that referenced
this pull request
Apr 17, 2026
Two order-breaking bugs remained in flushPending: 1. drained items after the failing write were dropped. pendingDeltaBuffer.drain removes the entire queue, writePendingEvent re-enqueued only the failed item, and the remaining items after it (already pulled out of the buffer) were never written or requeued. 2. The handler wrote the current event after a failed flush, so a buffered predecessor that just failed to write would land in the pane AFTER the new event on retry, breaking stream order. Replace flushPending with drainAndWrite which re-enqueues every item remaining after the failing write in their original order. Handlers now enqueue the current event before draining so any write failure defers the current event too, preserving order when the retry scheduler eventually drains everything. Extracted pending-retry-scheduler.ts to keep hook.ts under the 200 LOC architecture limit, and unified the delta/updated branches through a new handlePendingEvent helper. Two new regression tests: - 'does not lose buffered events after the second drained write fails and retries them in order' (4 pending + 1 new, write #2 fails, retry replays [b, c, d] so panes see A B B C D) - 'defers the current event when a buffered drain fails so stream order is preserved on retry' (1 pending + 1 new, write #1 fails, retry replays [a, b] so panes see A A B - the new event never precedes the failed one)
code-yeongyu
added a commit
that referenced
this pull request
Apr 18, 2026
Two order-breaking bugs remained in flushPending: 1. drained items after the failing write were dropped. pendingDeltaBuffer.drain removes the entire queue, writePendingEvent re-enqueued only the failed item, and the remaining items after it (already pulled out of the buffer) were never written or requeued. 2. The handler wrote the current event after a failed flush, so a buffered predecessor that just failed to write would land in the pane AFTER the new event on retry, breaking stream order. Replace flushPending with drainAndWrite which re-enqueues every item remaining after the failing write in their original order. Handlers now enqueue the current event before draining so any write failure defers the current event too, preserving order when the retry scheduler eventually drains everything. Extracted pending-retry-scheduler.ts to keep hook.ts under the 200 LOC architecture limit, and unified the delta/updated branches through a new handlePendingEvent helper. Two new regression tests: - 'does not lose buffered events after the second drained write fails and retries them in order' (4 pending + 1 new, write #2 fails, retry replays [b, c, d] so panes see A B B C D) - 'defers the current event when a buffered drain fails so stream order is preserved on retry' (1 pending + 1 new, write #1 fails, retry replays [a, b] so panes see A A B - the new event never precedes the failed one)
code-yeongyu
added a commit
that referenced
this pull request
Apr 18, 2026
Two order-breaking bugs remained in flushPending: 1. drained items after the failing write were dropped. pendingDeltaBuffer.drain removes the entire queue, writePendingEvent re-enqueued only the failed item, and the remaining items after it (already pulled out of the buffer) were never written or requeued. 2. The handler wrote the current event after a failed flush, so a buffered predecessor that just failed to write would land in the pane AFTER the new event on retry, breaking stream order. Replace flushPending with drainAndWrite which re-enqueues every item remaining after the failing write in their original order. Handlers now enqueue the current event before draining so any write failure defers the current event too, preserving order when the retry scheduler eventually drains everything. Extracted pending-retry-scheduler.ts to keep hook.ts under the 200 LOC architecture limit, and unified the delta/updated branches through a new handlePendingEvent helper. Two new regression tests: - 'does not lose buffered events after the second drained write fails and retries them in order' (4 pending + 1 new, write #2 fails, retry replays [b, c, d] so panes see A B B C D) - 'defers the current event when a buffered drain fails so stream order is preserved on retry' (1 pending + 1 new, write #1 fails, retry replays [a, b] so panes see A A B - the new event never precedes the failed one)
code-yeongyu
added a commit
that referenced
this pull request
Apr 19, 2026
Two order-breaking bugs remained in flushPending: 1. drained items after the failing write were dropped. pendingDeltaBuffer.drain removes the entire queue, writePendingEvent re-enqueued only the failed item, and the remaining items after it (already pulled out of the buffer) were never written or requeued. 2. The handler wrote the current event after a failed flush, so a buffered predecessor that just failed to write would land in the pane AFTER the new event on retry, breaking stream order. Replace flushPending with drainAndWrite which re-enqueues every item remaining after the failing write in their original order. Handlers now enqueue the current event before draining so any write failure defers the current event too, preserving order when the retry scheduler eventually drains everything. Extracted pending-retry-scheduler.ts to keep hook.ts under the 200 LOC architecture limit, and unified the delta/updated branches through a new handlePendingEvent helper. Two new regression tests: - 'does not lose buffered events after the second drained write fails and retries them in order' (4 pending + 1 new, write #2 fails, retry replays [b, c, d] so panes see A B B C D) - 'defers the current event when a buffered drain fails so stream order is preserved on retry' (1 pending + 1 new, write #1 fails, retry replays [a, b] so panes see A A B - the new event never precedes the failed one)
code-yeongyu
added a commit
that referenced
this pull request
Apr 20, 2026
Two order-breaking bugs remained in flushPending: 1. drained items after the failing write were dropped. pendingDeltaBuffer.drain removes the entire queue, writePendingEvent re-enqueued only the failed item, and the remaining items after it (already pulled out of the buffer) were never written or requeued. 2. The handler wrote the current event after a failed flush, so a buffered predecessor that just failed to write would land in the pane AFTER the new event on retry, breaking stream order. Replace flushPending with drainAndWrite which re-enqueues every item remaining after the failing write in their original order. Handlers now enqueue the current event before draining so any write failure defers the current event too, preserving order when the retry scheduler eventually drains everything. Extracted pending-retry-scheduler.ts to keep hook.ts under the 200 LOC architecture limit, and unified the delta/updated branches through a new handlePendingEvent helper. Two new regression tests: - 'does not lose buffered events after the second drained write fails and retries them in order' (4 pending + 1 new, write #2 fails, retry replays [b, c, d] so panes see A B B C D) - 'defers the current event when a buffered drain fails so stream order is preserved on retry' (1 pending + 1 new, write #1 fails, retry replays [a, b] so panes see A A B - the new event never precedes the failed one)
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
This was referenced Oct 22, 2026
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
# Conflicts: # CHANGELOG.md # packages/omo-senpi/plugin/extensions/gateway-store-worker.mjs # packages/omo-senpi/plugin/extensions/omo-onboarding-scan.js # packages/omo-senpi/plugin/extensions/omo.js # packages/omo-senpi/plugin/runtime/thread-sdk/sdk.js
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
…on current dev (bun 1.4.3)
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
…on the rebuilt batch #1 (bun 1.4.3)
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
chore(bundle-train): batch #1 integration and generated-bundle regen
code-yeongyu
added a commit
that referenced
this pull request
Oct 22, 2026
# Conflicts: # CHANGELOG.md # packages/omo-codex/scripts/install-dist/install-local.mjs # packages/omo-senpi/plugin/extensions/memory-run-supervisor.mjs # packages/omo-senpi/plugin/extensions/omo-feedback.js # packages/omo-senpi/plugin/extensions/omo-init-deep-advisor.js # packages/omo-senpi/plugin/extensions/omo-member.js # packages/omo-senpi/plugin/extensions/omo-memory-doctor.js # packages/omo-senpi/plugin/extensions/omo-memory-memfs.js # packages/omo-senpi/plugin/extensions/omo-memory-palace.js # packages/omo-senpi/plugin/extensions/omo-memory-sync.js # packages/omo-senpi/plugin/extensions/omo-onboarding-scan.js # packages/omo-senpi/plugin/extensions/omo-recall-store.js # packages/omo-senpi/plugin/extensions/omo-side-panel.js # packages/omo-senpi/plugin/extensions/omo-task.js # packages/omo-senpi/plugin/extensions/omo-thread-client.js # packages/omo-senpi/plugin/extensions/omo.js # packages/omo-senpi/plugin/runtime/rollback-migrate.js
This was referenced Oct 22, 2026
Merged
danielperezr88
added a commit
to danielperezr88/oh-my-openagent
that referenced
this pull request
Oct 29, 2026
… build - discover_tools plugin tool: Code Mode-callable semantic tool discovery over the live tool catalog via the fork's LocalToolEmbedder (all-MiniLM-L6-v2, local inference), with exact call expressions, calibrated 0.12 threshold, and a stopword-filtered keyword fallback - discover-catalog.ts holds the pure seams (catalog build, call-path formatting, fallback ranking, output format); 9 new unit tests - build.ts index node: --external @huggingface/transformers. Bundling inlined the onnxruntime-node wrapper whose native binding then resolved against dist/index.js and failed; external resolution from node_modules fixes it and shrinks dist 9.06 MB to 7.66 MB Upstream research recorded in evidence: codemode search() is lexical with merged incremental fixes; BM25 PR #57666 closed unmerged by its author; no semantic search attempts upstream. In-plugin tool chosen over an upstream PR. QA: live Code Mode catalog registration on the real service; sandbox A/B runs show builtin search() missing doc_store_query for an intent query while discover_tools surfaces it semantically; grep ranks code-yeongyu#1 for a duplication query. 31/31 plugin-v2 tests; 3 script/ failures verified pre-existing at HEAD via scratch worktree. Evidence: .omo/evidence/20261021-v2-discover-tools/ (local, untracked)
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
Changes
Target