Repository navigation
knowledge: recount a relayed figure from its raw output, re-verify an interrupted run's draft before an automated reset, init scripts skipped by page.setContent - #271
Merged
Conversation
…om its raw output, re-verify an interrupted run's draft before an automated reset, init scripts skipped by page.setContent
This was referenced Oct 11, 2026
Merged
Merged
This was referenced Oct 18, 2026
Merged
log.md: union of both appends. INGEST_REPORT.md: this PR's report. Other conflicts (index rows, related: lists) resolved as unions. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MWyLAeAi5szx683XVs4cbM
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.
Knowledge flush — 4 claimed rows: 2 merged into existing pages, 1 new page, 1 anhinga plan-gap kept local-layer
Run
20261011-213417-11470(the headless auto-flush child; the lock owner id arrived inDEV_LOOP_FLUSH_RUN_ID, so step 0's acquire with a fresh id reportedheld … 24sand the re-run with the inherited id reportedalready-owned). Claimed 4 rows — every row in the queue:5e9bdfb102f22421(qa),eec9adde8f6cf9c3(infrastructure),cb99014ed0e30451(qa),73c466a4b014e5c2(anhinga plan-gap).~/.dev-loop/repowas clean before the reset (git status --porcelainempty; the previous branch's commit is PR #270), so nothing needed salvaging.Verified best-practice
~/.claude/projects/…/2af9c97c-….jsonl):gh pr view 270body line 21 reads "24 of 24 distinct source URLs returned HTTP 200" and "the 8 remaining URLs in the diff are … fixture URLs";jqover every Bashtool_usefound one command containinghttp_code(the loop overurls-recovered-20261011.txt) and zeroAgentcalls; itstool_resulthas 32 lines starting with a 3-digit code: 21200, 11000, 0 other; all 11000hosts areexample.com/evil.testfixtures, andhttps://example.com/+https://evilexample.com/(both200) are fixture rows on #270'shost-disguise-fixtures-for-url-allowlistspage (lines 40, 61, 69) — so the output supports 19 source URLs all200, and neither 24 nor 8 is derivable from it; the first "24 of 24" in the transcript (line 424) sits inside the run's own INGEST_REPORT heredoccheckout main+reset --hard origin/mainon a reused checkout, inspect the tree and preserve leftovers; new material: inputs leased with a TTL come back on their own (the processing does not), and a leftover draft with cited sources is re-verified by re-fetching its URLs, re-running lint and a sample of its reproductions before adoptiongit diff HEADwording was not imported because the page's Instead-of row already corrects it (untracked files are left out). Field figures recounted here:~/.dev-loop/salvage/knowledge-20261011-132856.patchholds 23diff --githeaders;git show --stat e43e647→ 23 files changed, 421 insertions, 95 deletions; "five reproductions re-run" is the 19:29 run's own report (PR #270 body line 28) and is attributed as such on the pagepage.setContentisdocument.write()on the existing document, not a navigation, so an init script registered on the page is never evaluated for it; serve the fixture throughpage.route+page.goto. Agoto('about:blank')beforesetContentkeeps the script's globals butdocument.open()erases its listenersaddInitScript"Whenever the page is navigated … after the document was created but before any of its scripts were run";setContent"This method internally calls document.write(), inheriting all its specific characteristics and behaviors"), microsoft/playwright#13274 (viagh api; maintainer pavelfeldman, 2022-04-03: "page.setContentdoes not run the init scripts, because JS world is not re-created, only navigation such aspage.gotore-creates it"), https://html.spec.whatwg.org/multipage/dynamic-markup-insertion.html (HTTP 200; document open steps: "For each shadow-including inclusive descendant node of document, erase all event listeners and handlers given node")~/.dev-loop/scratch/flush-20261011-2134/probe-setcontent.js(playwright-core 1.63.0 from the originating worktree'snode_modules, Chromium 153.0.8010.12 headless shell): Apage.addInitScript+setContent→__probeundefined, inline script sawundefined,__eventsundefined; B +route/goto→1,number,["animationstart"]; Cgoto('about:blank')thensetContent→1,number,[]; Dcontext.addInitScriptbeforenewPage()thensetContent→1,number,[]; E no init script +route/goto(control) → undefined throughout; FsetContenttwice → undefined. Known-good (B) and known-bad (A, E) were both observed, so the probe itself is provendocs/backends.mdsection-8 capability table (Korean text, lnpl outbox/relay rows, lnpl-postgres#8)Nothing was upgraded on the originating session's say-so: the two statements the queue rows carried that could not be recounted here ("five reproductions"; the harvester's note that its blog draft omitted the figure) are attributed to their report or left out.
Existing-layer check
Route:
INDEX.md→ qa ("evidence for completion claims", "verifying the quantitative claims in a document before publishing it") and infrastructure ("multi-agent orchestration … shared run state") →wiki/qa/index.md,wiki/infrastructure/index.md;wiki/testing/index.mdread for the Playwright candidate (e2e / mocking categories).Pages read: qa-process-completion-claims, qa-deliverables-quantitative-claims-in-a-published-document, infrastructure-agent-orchestration-resetting-a-reused-checkout-after-an-interrupted-run, qa-environments-browser-console-capture-gaps, testing-quality-markup-contract-assertions, testing-e2e-e2e-stability, testing-quality-harness-reverse-controls
wiki_search(k=5) per candidate, plus a whole-wikigrepfor each candidate's key terms:wiki_searchhitsIndex rows:
wiki/qa/index.mdcompletion-claims row extended and one environments row added at the end of that table;wiki/infrastructure/index.mdresetting row extended (maintenance invariant 1). Checks on this branch:node scripts/wiki-lint-prohibitions.js wiki→compliant: 84, violations: 0, info: 1;node scripts/wiki-structure-checks.js wiki→pages: 427, indexes: 13, findings: 0(baseline onorigin/mainf7c865c: 84 / 0 / 1 and 426 / 13 / 0); body lines 66 / 63 / 68 (limit 120); no banned qualifier in any added directive (the one hit, "probably fixed", is the page's pre-existing quoted hedge-word example); everyrelated:id resolves on main.Open-PR check
Open
knowledge/*heads fromgh pr list --repo choiyounggi/dev-loop --state open --json number,headRefNamefiltered on the head prefix: #265knowledge/choiyounggi-20261009-222812, #266knowledge/choiyounggi-20261010-021455, #268knowledge/choiyounggi-20261011-002014, #269knowledge/choiyounggi-20261011-022522, #270knowledge/choiyounggi-20261011-193253(all OPEN, none merged; 6 open PRs in total). Each head was fetched and the added lines ofgit diff origin/main...origin/<head> -- wiki/were scanned for every candidate's terms (recount|restat|raw output|denominator|pass count|…,reset --hard|porcelain|salvage|abandoned|uncommitted|…,setContent|addInitScript|init script|route.fulfill|page.route|…), and each matching line was read.wiki/infrastructure/index.mdat lines 35/49 and 30, not the resetting row (line 28)proving-nothing-moves-under-reduced-motionpage serves its fixture "throughpage.route+page.goto" and says an init script "re-runs on each navigation", but never states thatsetContentskips the init script, nor the listener erasure — it is the field origin of this candidate, not a carrier of its lesson. No other head mentionssetContent/addInitScriptrelated:link to #270's page is deferred: its id does not exist on main yet)Nothing folded, nothing dropped as a pending duplicate.
git merge-tree --write-treeof a temporary commit of this tree against each of the five heads: every wiki page and both indexes merge cleanly; the only conflict with each head islog.md, the append-only log every knowledge PR conflicts on with every other.Routing decision
wiki/qa/process/completion-claims.mdwiki/infrastructure/agent-orchestration/resetting-a-reused-checkout-after-an-interrupted-run.mdwiki/qa/environments/init-scripts-skipped-by-setcontent.mdNo new category.
Local-layer candidates
docs/backends.mdcontract table/Users/choeyeong-gi/Desktop/workspace/linkly/anhinga)wiki-local/backend/common/api-design/t1-driver-capability-contract-table.md— run wiki-ingest inside that project. Its research notes (JDBCDatabaseMetaData.supportsXxx, PEP 544runtime_checkable, the transactional-outbox relay, the IETF Idempotency-Key draft) carry a general kernel — a driver contract self-reports optional capabilities through predicates, and a conformance kit reports declared-unsupported rather than skip — that no bundled page owns (the row's own semantic search found none); it was not harvested as a candidate and is noted here as a gap, not ingestedNotes for the reviewer
DEV_LOOP_FLUSH_RUN_ID, so the fresh-id acquire reportsheld(the bundled pageinherited-lock-ownership-in-a-spawned-sessioncovers this). This run re-acquired with the inherited id and releases with it.🤖 Generated with Claude Code