Replace the context roadmap with a problem inventory - #83
Merged
Conversation
Owner
Author
|
Reopening to re-fire the CI event — no workflow run was dispatched for this PR. |
The context-handling roadmap and its test-harness companion had accumulated three rounds of re-baselining, two withdrawn designs, and a ranking built on a premise that has since drifted (heartbeat was 83% of input at the 07-16 measurement; it is 63% now, because user-scope spend nearly doubled). Two of ten planned items shipped in five weeks. The documents had become hard to act on, and parts of them were actively misleading. Both are deleted rather than archived: a stale plan sitting in docs/plans/ costs reader attention every time someone opens the directory, and git history is a perfectly good archive. What replaces them, in docs/plans/context/: - PROBLEMS.md — 33 problems in seven clusters, each stated without its proposed solution, and tagged MEASURED / STALE / ASSERTED / RESOLVED so a claim carries its own evidence status. Includes a fresh telemetry reading and an issue map that is the single place issue numbers appear. - RESEARCH.md — what outlives the plans: the OpenClaw / hermes-agent comparison, how each measurement was actually taken, and the code-level findings discovered while planning (notably that a deterministic probe cannot call fetch_upcoming_arbox_classes, because it deletes the dropped rows and returns them for a one-shot notice). Two corrections found while compiling this. Most issue cross-references in the deleted plans did not resolve: they cite #33, #54 and #20 for the umbrella, compaction and event-driven heartbeat, but in this repository those numbers belong to unrelated issues, two of them closed. Only #18 was correct. And turns.jsonl contains an unparseable line, so any reader must parse tolerantly. Issues were brought into line with the same discipline — a title should state what is wrong, such that someone who rejects every proposed fix would still agree it is true. #18 rewritten, #50 reframed around cross-scope visibility (the real problem behind "channel awareness"), #81/#61/#60 retitled, #59 folded into #36 as it described the same failure split on shippability grounds, and #1/#5 kept as standing concepts with pointers rather than test or gate lists. The comment in tools/web/fetch.py referencing the deleted plan is rewritten to describe the behaviour instead. Claude-Session: https://claude.ai/code/session_01RdXUG9jwvgS2yu4VPjiNer
roiguri
force-pushed
the
docs/context-problems
branch
from
August 6, 2026 18:27
d8555bc to
a5341d4
Compare
Owner
Author
|
Merging with an admin bypass — GitHub Actions is in a major outage (qcvjkzcs7j74, started 2026-08-06 15:22:49 UTC), so the required checks cannot dispatch. This repo's last successful run was 15:23:23Z, ~30s after the incident opened. All three CI guards were run locally against
Bypass authorized by the owner. |
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.
Deletes
CONTEXT_HANDLING_PLAN.mdandTEST_HARNESS_PLAN.md; addsdocs/plans/context/in their place.The roadmap had accumulated three rounds of re-baselining, two withdrawn designs, and a ranking resting on a premise that has since drifted — the heartbeat was 83% of input at the 07-16 measurement and is 63% now, because user-scope spend nearly doubled. Two of ten planned items shipped in five weeks.
Deleted rather than archived, on purpose: a stale plan in
docs/plans/costs reader attention every time the directory is opened, and git history is a perfectly good archive.What replaces them
PROBLEMS.md— 33 problems in seven clusters, each stated without its proposed solution, and taggedMEASURED/STALE/ASSERTED/RESOLVEDso every claim carries its evidence status. Includes a fresh telemetry reading, and an issue map that is the single place issue numbers appear in the file.RESEARCH.md— what outlives the plans: the OpenClaw / hermes-agent comparison, how each measurement was actually taken, and the code-level findings discovered while planning.Two corrections found while compiling this
turns.jsonlcontains an unparseable line — a strictjson.loadsover the file raises. Any reader must parse tolerantly.Issues brought into line
Same test applied throughout: a title should state what is wrong, such that someone who rejects every proposed fix would still agree it is true.
turn_id, split on shippability groundsNote
PR #70 edits
CONTEXT_HANDLING_PLAN.md, which this deletes, and will conflict.https://claude.ai/code/session_01RdXUG9jwvgS2yu4VPjiNer