docs(prd): 0018 take what omp got right - #538
Merged
Merged
Conversation
Cross-engine session handoff, stream rules at the PTY layer, and provider limits folded into `moshcode cost`, drawn from omp (oh-my-pi) after #536 added it as an engine. Grounded in three things already in the tree: cost.mjs already parses every engine's transcript format, pty.mjs already sits below every engine's output stream, and cost.mjs already reports spend but not headroom. Records two non-starters so they are not re-proposed: completion.mjs already generates no-drift completions from cli-schema.mjs, and advisor.mjs is advis0r.com equity research rather than an advisor model, so that name is taken. Conforms to profullstack.com/stack: MCP bridge on every verb, hqtui rather than a localhost web dashboard, node --test, and no em dashes. Regenerating the index with prd.mjs also corrected a stale row: 0017 has said Accepted on disk while README said Draft. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ThreatCrush Security Scan0 finding(s) in the 2 file(s) this pull request changes. Nothing in the files this pull request changes. 101 pre-existing finding(s) elsewhere in the repository — **HIGH/CRITICAL**: 8 | **MEDIUM**: 82 | **LOW**: 11Not introduced by this pull request. The full set is in the Security tab.
…and 81 more. Full results in the Security tab. Snippets are redacted; ThreatCrush never prints matched credential material. |
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
1 participant
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.
PRD 0018, drawn from reading omp (oh-my-pi) after #536 added it as an engine.
What it proposes
omp is the most interesting harness in the engine set, but most of it is not ours to take. The Rust core, in-process ripgrep, LSP and DAP are engine internals, and moshcode inherits those by wrapping omp. What is worth taking is the handful of omp ideas that are only half-built, because omp can only ever apply them to omp:
moshcode cost limits/cost route(P0/P1). Spend is already reported. Remaining headroom is not, and that is the number that matters on one Anthropic key shared across 13 production vaults.pty.mjsis engine-agnostic by construction.auth serve(P1), so no engine process reads a credential from a .env.grievances,cleanse(P2).Why it is cheap rather than speculative
Three things are already in the tree:
cost.mjsparses every engine's transcript format,pty.mjssits below every engine's output stream, andcost.mjsalready reports spend.Two are already done and the PRD records them so they are not re-proposed:
completion.mjsgenerates no-drift completions fromcli-schema.mjs, andadvisor.mjsis advis0r.com equity research rather than an advisor model, so that name is taken.Conformance
Follows profullstack.com/stack: MCP bridge on every verb, hqtui rather than a localhost web dashboard (omp's
stats -pis a browser page),node --test, and no em dashes.Also in this diff
Regenerating the index with
prd.mjscorrected a stale row. 0017 has saidAcceptedon disk while README saidDraft.Testing
node --test test/prd.test.mjs test/prd-index-cell.test.mjs test/tui-prd-errors.test.mjspasses, 39/39.🤖 Generated with Claude Code