Repository navigation
knowledge: freshness check for an uncommitted generated file; test module written during a baseline run - #319
Open
choiyounggi wants to merge 1 commit into
Open
choiyounggi wants to merge 1 commit into
choiyounggi wants to merge 1 commit into
Conversation
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 — 2 insight(s) (headless auto-flush run 20261030-153306-64550)
35 queue rows were claimed this run: 2 ingested, 31 dropped as project-specific
plan-gap rows, 2 released because they plausibly overlap open PRs. Nothing was folded
into another branch.
The 31 plan-gap rows are lines 278–308 of
~/.dev-loop/queue/plan-gaps.jsonl. Thisheadless session cannot list the queue directory, so the rows were located by
filename: claim order follows sorted filenames and file order, the first and last six
claimed ids match lines 278 and 303–308, and the 31 ids fill that range exactly.
Verified best-practice
9950a83a55ea53e7).Claim: before a generated file's first commit, compare a saved copy with the
regenerated file (
cmp);git diff --exit-codeaftergit add -Nis always red.-N: "An entry for the path is placed inthe index with no content."
index";
--exit-codeexits 1 on differences.tests expect an intent-to-add path to diff as a new file with all lines added.
new file mode 100644and 444 addedlines on a byte-identical regeneration; the
cmpcheck exited 0.was re-run in this headless session, which has no shell for git.
bc0e0b2a5d0c2dca).Claim: when a full-suite baseline runs in the background, wait for it or run it in
a separate copy before creating test modules in the same tree; a test that
re-discovers the tree at runtime imports the half-written file.
"If importing a module fails … this will be recorded as a single error and
discovery will continue."
test_readme_currencyfailures citingFailed to import test module: tests.test_mcp_exec_tools(the file beingwritten); with the module complete, the full run reached 5959 OK.
added as one edge row on a
verifiedpage, and its source line marks it as afield incident.
Existing-layer check
Pages read: qa-document-verification-generated-reference-drift-gates, platforms-tools-staging-tracked-files-under-an-ignored-directory, testing-data-test-data-and-isolation
Also read the domain indexes
wiki/testing/index.md,wiki/qa/index.mdandwiki/platforms/index.md, and ranwiki_searchfor each candidate's trigger.generated-reference-drift-gatesowns--checkgates for closed-vocabularyreferences (a narrower trigger), and
staging-tracked-files-under-an-ignored-directoryowns
git addexit codes. → new pageplatforms/tools/freshness-check-of-an-uncommitted-generated-file.md,related-linked to both, to
platforms-toolchains-regeneration-silently-drops-hand-edited-state,and to
testing-quality-checks-that-cannot-pass.testing-data-test-data-and-isolationowns "suite behaves differently because ofits environment". → merged as one Edge-cases row plus two source lines, a
related link to
qa-process-scope-purity-checks(same root cause: a suite readingthe live tree), and an extended load-when line in the testing index.
Open-PR check
Open
knowledge/*heads: #318, #317, #315, #310, #309, #308, #307, #306, #305, #304,#301, #300, #299, #298, #293. This headless run read their changed-file lists
(
gh pr list --json files), not their diffs.9950a83a55ea53e7git add -N freshness check--require-hashesknown-bad test) touch other mechanisms and pagesbc0e0b2a5d0c2dcatest file written mid-baselinetesting/data/artifact-leakage-from-a-suite.md(leftover files), a different triggerfc76dc4dfa08ecdedocker schema validator: skip vsREQUIREfail, known-bad documenttesting/quality/tests-that-cannot-fail.mdandchecks-that-cannot-pass.md)122dd81477d4a447absence-in-source assertion fails on its own explanatory commentchecks-that-cannot-pass.md)The released rows are left
pending, not retired, so the manual flush can diff themagainst those PRs.
Routing decision
Row 1 →
platforms/tools/freshness-check-of-an-uncommitted-generated-file.md(newpage). It is a git-behaviour edge case that applies to any repository, which is
the scope of the
toolscategory; no new category needed.Row 2 →
testing/data/test-data-and-isolation.md, Edge cases (merge).31 plan-gap rows → not ingested (see below).
Local-layer candidates
31 rows from
plan-gaps.jsonl(reposhag= linkly, task plans of orchestrationrun oi1029; the 7 rows read in full come from t1, t5 and t9), ids
1d01cb971e07f6cb…9bf49772c47a2672. Each recordsone plan decision (
Planning tN: deciding … (no owning wiki page)) that nameslinkly's own files, flags and plan ids (
cmd_token,docs/backends.md,benchmarks/token/,plans/t9/design.md D23), so it would be wrong in anothercodebase. Target, if anyone wants them kept:
wiki-local/<domain>/<category>/<slug>.mdinside linkly — run wiki-ingest inside that project. Each decision is already
recorded in that run's
plans/tN/design.md, so they were retired asdropped: project-specific plan-gap (local layer of its repo), the same outcomeearlier flushes gave such rows.