Uh oh!
There was an error while loading. Please reload this page.
llp: OpenClaw full-capture spec (0157) + backfill decisions (0158, 0159) - #488
Merged
Conversation
Spec 0157 turns the Accepted decision set (0143-0149, 0152) into requirements: steering plugin, adapter rework, core json_path removal, warning ledger, backfill route, coverage statement, acceptance procedure. Decision 0158: one core reader for the OpenClaw session JSONL, shared by the settlement enricher and the backfill provider (the LLP 0150 shape). Decision 0159: backfill route agreement rides native-identity settlement plus the existing part_id dedupe, because live OpenClaw rows have no native identity at capture time (LLP 0144). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
From /llp-grill against the corpus and code: - 0158: reader moves from core to plugin-local, matching the Claude (transcripts.js) and Codex (backfill.js) precedent of plugin-owned format readers; promotion trigger to src/core/openclaw/ named explicitly (a cross-plugin consumer such as hyp session). - 0159: settlement upgrade scope settled as full (message id + session_id + cwd); enricher also applies the LLP 0085 flush-time policy drop, without which .hypignore fails open for all live OpenClaw capture (proxy rows carry no cwd). Restart seed-miss churn verified as self-healing and documented. Cross-partition twin restated on the real partition keys (session_id per LLP 0030, cwd), with a message-grain sweep upgrade as the duplication-preventing option. Match-key normalization across wire vs transcript shapes flagged as an implementation risk with a fallback shape. - 0157: new R14 (settlement cwd + policy drop); reader wording follows 0158's placement. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
Follow-up to #475 / #486: the Accepted OpenClaw full-capture decision set (LLP 0143-0149, 0152) gets its spec, plus the backfill design those decisions anticipated. All three documents were grilled against the corpus and code (
/llp-grill, 2026-07-30) and are Accepted.What's here
LLP 0157 (spec, Accepted) - requirements for implementing the set, citing the decisions via
constrained-byrather than restating them:before_model_resolvesteering,prepareRuntimeAuthcredential borrow + gateway baseUrl,wrapStreamFnparity mirror (concrete beta list verified againstextensions/anthropic/stream-wrappers.ts2026-07-30).@hypaware/openclawadapter rework: settings.js + undo record retired,attach_probedropped,openai-completionsprojection branch, match-key stamping, settlement enricher, backfill provider.json_pathprobe format (OpenClaw was the last consumer).openclaw_captureacceptance procedure mirroringcodex_desktop_capture, proving BOTH routes.@refs.LLP 0158 (decision, Accepted) - one shared reader for the OpenClaw session JSONL (
~/.openclaw/agents/<id>/sessions/*.jsonl), consumed by the settlement enricher and the backfill provider. Placement is plugin-local (inside@hypaware/openclaw), matching the Claude (transcripts.js) / Codex (backfill.js) precedent of plugin-owned format readers; the promotion trigger tosrc/core/openclaw/is named explicitly (a cross-plugin consumer, e.g.hyp sessionlearning OpenClaw sessions). Read rules mirror LLP 0150, including the absolute-cwd predicate for the.hypignoregate.LLP 0159 (decision, Accepted) - backfill route agreement. Codex's routes agree by construction (native ids on both sides); OpenClaw's live rows have no native identity at capture (LLP 0144: prompt-head hash + gateway fallback ids). Chosen mechanism: the LLP 0027 pattern - projector stamps a match key, a settlement enricher performs a full identity upgrade at flush (native message id +
session_id, the LLP 0030 partition key + cwd from the session header), and the existingpart_iddedupe layers (committed + spool scan) resolve overlap.rows_written: 0on overlap becomes the acceptance pass condition, matching codex.The enricher also applies the LLP 0085 flush-time usage-policy drop: live OpenClaw proxy rows carry no cwd, so without this seam
.hypignorewould fail open for the entire client (spec R14).Grill outcomes folded in (second commit)
part_iddedupe); the cross-partition twin restated on the real partition keys (session_idper LLP 0030 + cwd) with a message-grain sweep upgrade as the duplication-preventing option; match-key normalization across wire vs transcript shapes (tool_usevstoolCall) flagged as an implementation risk with timestamp-and-order matching as the fallback shape.Verified facts backing the docs
id+cwd+ timestamp; assistant messages carrymodel,provider,api,stopReason, fullusage; tool calls/results in-transcript.conversation_idunset, hardcodedprovider: 'anthropic', no cwd (plugins-workspace/openclaw/src/projector.js).part_id-only route separation) confirmed inplugins-workspace/codex/src/backfill.js+ai-gateway/src/dataset.js.extensions/anthropic/stream-wrappers.ts:8-18.Checks
ref-check: the 3 files contribute zero errors at every commit (74 pre-existing corpus errors unchanged).test/core/llp-ref-hygiene.test.js: 9/9 pass (number uniqueness holds; 0157-0159 were verified free on origin/master and all remote branches).[inferred]claims survive into Accepted.🤖 Generated with Claude Code