Move investigation-log docs out of the public repo - #107
Merged
Conversation
… real-paper mentions Same class as the previous .claude/ privacy cleanup (PR #105), found while checking docs/kg-dead-letter-triage-2026-07-07.md at cservinl's flag: these are one-off engineering investigation/benchmark logs, not standing project documentation, and several reveal real vault paper filenames (the user's private research reading list) or personal machine specs incidentally, as test subjects/context for the investigation: - docs/kg-dead-letter-triage-2026-07-07.md -> .claude/ (dozens of real vault paper filenames, the user's thesis folder structure, username) - docs/kg-extraction-context-length.md -> .claude/ (real vault paper as the test subject) - docs/qwen3-family-evaluation.md -> .claude/ (personal GPU/hardware specs) - docs/llamacpp-vulkan-home-server-vs-desktop-client-benchmark.md -> .claude/ (home server + desktop client hardware specs) - docs/nuextract-2.0-kg-extraction-evaluation.md -> .claude/ (personal hardware specs; uses a public paper as its test content, but still an investigation log, not standing docs) - docs/ollama-concurrency.md -> .claude/ (personal GPU/hardware specs) Updated every cross-reference across TODO.md, config.example.toml, 3 ADRs, docs/wiki/configuration.md, the compute-pool-contention diagram source, the openrouter-free-models test harness, supervisor.py, knowledge_graph_ service.py, config.py, and their tests -- all comment/prose references, nothing load-bearing at runtime. Also genericized two more real-paper-filename mentions found in the same sweep, in docs that correctly stay public (ADR-013, TODO.md) -- the underlying architectural point (a paper too large for one extraction chunk) doesn't need the specific filename. .gitignore: added .claude/ on this branch too (it was cut before PR #105 merged that same line) -- caught a real near-miss while staging: `git add -A -- .claude/` briefly staged an unrelated stray full-repo worktree snapshot sitting under .claude/worktrees/ before this line existed on this branch; reset and re-staged explicit paths only. Full suite 1290 passed (this branch's baseline, off main).
…ntation Genericized (hardware specs, real-paper filenames -> academic citations) one-off engineering investigation/benchmark logs still belong in the public repo -- they're cited as the evidence base for real, shipped config decisions (token_budget, model_affinity, max_concurrent) -- but mixed in at docs/'s top level they read as product documentation, which they aren't. Same treatment for docs/tests/openrouter-free-models/: also a technical log (a benchmark run + its results), not a test suite (never in pytest's testpaths). - docs/kg-dead-letter-triage-2026-07-07.md -> docs/logs/ - docs/kg-extraction-context-length.md -> docs/logs/ - docs/qwen3-family-evaluation.md -> docs/logs/ - docs/llamacpp-vulkan-home-server-vs-desktop-client-benchmark.md -> docs/logs/ - docs/nuextract-2.0-kg-extraction-evaluation.md -> docs/logs/ - docs/ollama-concurrency.md -> docs/logs/ - docs/tests/openrouter-free-models/ -> docs/logs/openrouter-free-models/ (docs/tests/ is now gone entirely) Updated every cross-reference across TODO.md, config.example.toml, 3 ADRs, docs/wiki/configuration.md, the compute-pool-contention diagram source, supervisor.py, knowledge_graph_service.py, config.py, their tests, and the moved files' own internal cross-references to each other. Full suite 1290 passed.
Owner
Author
|
Update: revised per feedback -- instead of moving to |
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
1 participant
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.
Genericized, then reorganized — not moved to
.claude/. These are one-off engineering investigation/benchmark logs cited as the evidence base for real, shipped config decisions (token_budget,model_affinity,max_concurrent), so they belong in the public repo, just not mixed in with product documentation atdocs/'s top level, and not with real vault/hardware specifics in them.Genericized:
Meng_2023_MEMIT_Mass_Editing_Memory.md) → normal academic citations ("Meng et al. 2023's MEMIT paper") — keeps every finding fully verifiable without exposing what's specifically in the vault.thesis/Resources/Papers/) → "the vault's papers folder".Reorganized into
docs/logs/(separate from product documentation):kg-dead-letter-triage-2026-07-07.md,kg-extraction-context-length.md,qwen3-family-evaluation.md,llamacpp-vulkan-home-server-vs-desktop-client-benchmark.md,nuextract-2.0-kg-extraction-evaluation.md,ollama-concurrency.mddocs/tests/openrouter-free-models/→docs/logs/openrouter-free-models/(same class — a benchmark run + its results, never in pytest'stestpaths, not a test suite)Updated every cross-reference across
TODO.md,config.example.toml, 3 ADRs,docs/wiki/configuration.md, the compute-pool-contention diagram source,supervisor.py,knowledge_graph_service.py,config.py, their tests, and the moved files' own internal cross-references to each other.Also genericized two more real-paper-filename mentions in docs that stay at their original location (ADR-013,
TODO.md).Full suite 1290 passed.