Turn the MCP, A2A and inference parsers into libraries, with saved real-traffic test cases
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start by reading the three parser plugins under core/plugins/inferenceparser, core/plugins/mcpparser and core/plugins/a2aparser, plus the data package from #1502 and the Settle work in #1501. Extract request and response parsing into libraries with explicit streaming state, then add saved request, JSON response and SSE fixtures under testdata for each library. Done means the plugins delegate parsing, cpex's ParseResponse still works, and the libraries do not import pricing or settle.
由索引模型根据 Issue 内容生成。
描述
Part of #1437 (assessment: laptop data plane on Praxis) · prep work, useful to Cortex on its own
Status: there is no shipped Cortex–Praxis integration today (the paused
cmd/cortex-praxisconverter is not one), and the interim path is not decided (LLM traffic to praxis-ai as an inference server, or all traffic through Praxis via #1491). This is a refactor with no behaviour change: it is useful to Cortex on its own, and would make a possible move to Praxis easier whichever path is taken.
Problem
Parsing an LLM, MCP or A2A request is mostly pure work (bytes in, structured result out), but most of it is reachable only as a plugin. One exported entry point exists: inferenceparser.ParseResponse(path, body), which cpex already calls. Everything else is unexported or inline:
- inference (mostly pure):
- requests:
parseAnthropicRequestandparseOpenAIRequesttake only the body;parseResponsesRequestalso needs headers (zstd decode); - responses:
parseAnthropicJSON,parseAnthropicSSE,parseResponsesJSON,parseResponsesSSE,parseInferenceJSON,parseInferenceSSE. The SSE parsers write into the request's existing result; - per-frame folds:
foldAnthropicFrame,foldOpenAIFrame,foldResponsesFrame(responses.go:499). They need stream state that the plugin keeps onpctx, andfoldResponseFrametakespctxitself; - plugin-only logic: the dialect is chosen from the path, the response arm from
Content-Typeand whether a stream ran, andAgentRoleis set inOnRequest.
- requests:
- MCP (not pure today): request parsing is inline in
OnRequest(mcpparser/plugin.go:166-266), and needs the method, path, the configured path matcher and theMcp-Session-Idheader.parseMCPResponsereturns an unexported type thatapplyMCPResponseRPCwrites intopctx. - A2A (not pure today): request parsing is inline in
OnRequest(a2aparser/plugin.go:35-127). Response parsing isextractSendResponse,extractStreamEventandsessionIDFromJSON(:259-430).parseA2APartsis only a helper over already-decoded data.
The inference parser also prices the request (cost.go:64-113), so parsing and pricing are one plugin. It imports core/cost/pricing and core/cost/settle from cost.go, cacheblind.go and plugin.go (pricing.Resolver, settle.IsEventStream).
Apart from ParseResponse, nothing outside the plugin can parse a body: not a replay tool, not a test, not another proxy. And there are no saved real-traffic test cases: inference-parser's 18 test files (about 4,900 lines) build inputs as inline JSON, with a few verbatim captured blocks pasted in (e.g. anthropic_test.go:166, :624). The only testdata/ in the repo is core/cost/pricing/testdata.
Proposal
- Parser libraries (e.g.
core/parse/inference,core/parse/mcp,core/parse/a2a): exportedParseRequest(method, path, headers, body, config),ParseResponse(...)and a streaming fold with explicit state, returning the parsed-request types. The three plugins become thin wrappers.- MCP and A2A request parsing must be extracted from
OnRequestfirst. - The libraries depend on the data package from #1502, so #1502 should land first.
- MCP and A2A request parsing must be extracted from
- Pricing out of the parser library. The library computes no cost. Pricing stays in the plugin wrapper (or moves to a separate cost step), calling #1501's plain-value
Settle. Moving it changes when the settle-once latch runs, so it needs its own test. - Saved test cases as the safety net: real captured request and response pairs (JSON and SSE, per agent) stored under
testdata/, each with the expected parsed result, run by one table test per library. They prove the move changed nothing, and they also show exactly what Cortex needs from a parser if another one (e.g. praxis-ai's) were ever used instead. A request to praxis-ai to expose parsed content to other filters is drafted as a Discussion, not yet posted; see #1437's tracker.
Design questions
- Library API: whole-body functions only, or also the incremental frame fold the streaming path uses?
- Should captured fixtures be scrubbed (no real prompts), and where do they come from — a capture tool reading the session archive?
- Fixture format: one file per case (request, response, expected), or one golden file per parser?
Done when
- The three parser plugins contain no parsing logic, only calls into the libraries;
ParseResponsekeeps working for cpex. - Each library has saved real-traffic test cases covering request, JSON response and SSE response.
- The parser libraries import neither
core/cost/settlenorcore/cost/pricing. The plugin wrapper may.
Evidence read at main @ e455a04 (2026-10-25); not built.
- 主要语言
- Go
- 星标
- 21
- 派生
- 43
- 平均合并
- 14 小时 26 分钟
- 30 天内合并 PR
- 239
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
rossoctl/cortex 的其他 Issue
-
UI shows costs for all sessions even when drilled into a session可能已有人在做 @esnible 于 14 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 70/100
rossoctl/cortex#1116 · 2 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
agentop configure codex/opencode can wait on itself when an agent runs it可能已有人在做 @Alan-Cha 今天认领。 未关闭
rossoctl/cortex#1530 · 已指派 1 人 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 25/100
维护者通常 1 天内回复
-
WebSocket turns carry only new input, so conversation-level plugins see less of a Codex session可能已有人在做 @Alan-Cha 于 1 天前认领。 未关闭
难度 5/5 一周以上 新手友好度 35/100
rossoctl/cortex#1515 · 已指派 1 人 ·
维护者通常 1 天内回复
-
enhancement laptop
难度 5/5 一周以上 新手友好度 35/100
维护者通常 1 天内回复
相似的 Issue
-
VLAgent `spec.k8sCollector.includePodLabels: false` has no effect可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
VictoriaMetrics/operator#2772 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
Gaurav-Gosain/tuios#1316 ·
维护者通常 1 天内回复
-
needs: investigation
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
area:auth documentation priority:low
难度 1/5 1-3 小时 新手友好度 86/100
维护者通常 1 天内回复
-
Mend: dependency security vulnerability
难度 2/5 1-3 小时 新手友好度 65/100
kube-tarian/sigrun#203 ·