Repository navigation
Add Copilot repository guidance and tuned Scout knowledge - #32
Conversation
Add repository-specific Copilot instructions, a source-backed Scout memory bank, and a reusable 20-query search evaluation set. Persist search parameters selected by the 48-round autotune sweep. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
sean-parent
left a comment
There was a problem hiding this comment.
Talos local review
Verdict: No actionable findings accepted within the reviewed scope.
Reviewed SHA: 847b2912af42de996589b9f157d78b53d8579b47
Scope: Full branch diff against the PR's actual base, main (merge base 2068f2e6522ead4577f147f886e264b66f7db07e). Local HEAD matched the pushed PR head.
Reviewed by: Security, correctness, standards, architecture, operations, and documentation personas, followed by the challenge reviewer.
Counts: 3 provisional observations; 2 excluded because they target the generated synonyms.json artifact; 1 sent to challenge and rejected for lack of a documented rule or concrete consequence. Accepted: 0. Posted inline: 0. Overflow: 0. No previous review comments existed for duplicate/recurrence matching.
Coverage: 14 changed files in the complete snapshot. The generated .scout/memory/synonyms.json artifact is marked skipped for finding eligibility; the other 13 files were eligible. Base-revision CLAUDE.md context was loaded. GitHub custom-instructions matching returned zero instructions at the base revision, so newly added guidance was reviewed as content rather than used to govern its own review. No active OpenSpec context was found.
Verification: The user-approved repository-native replacement for the PR skill's inapplicable Cargo suite ran all seven documented CMake regression scripts successfully under x64 MSVC, without detected compiler warnings. Scout query-set evaluation passed; the persisted compact autotune configuration reproduced NDCG@5 0.587805 twice. Staged whitespace checks passed. Scout memory lint returned zero errors and five known orphan warnings; no automatic maintenance was applied.
Limitations: Architecture and operations personas did not consume the Scout packet/leads. Documentation review checked selected public contracts but did not exhaustively trace install internals; correctness review verified the deferred install ordering against source. Search relevance judgments and Scout's external implementation were not independently certified. Scout reported conservative whole-file roots for 13 changed files. Filtered DOCS_OPTIONS retrieval remains a known zero-scoring evaluation case. Hosted CI results are not claimed by this local review.
Persona routing: static roster (DCAP routing off; rerun with --dcap to enable).
Scout
Used: risk LOW, evidence HIGH, status COMPLETE (scout 0.9.130, 1s).
Cross-file leads given to reviewers: 0 changed symbol(s) with uses outside the diff, 0 new value(s) not handled everywhere, 0 possibly unused symbol(s).
Summary
Validation
The user approved repository-native CMake and Scout checks in place of the invoked PR skill's inapplicable Cargo workspace commands. All seven CMake scripts passed under an x64 MSVC environment, with no detected compiler warnings:
cmake -P tests/install/CMakeLists.txtcmake -P tests/install/test_provider_merge.cmakecmake -P tests/install/test_nested_install.cmakecmake -P tests/setup/test_target_type.cmakecmake -P tests/setup/test_runtime_dlls.cmakecmake -P tests/setup/test_setup_version_resolution.cmakecmake -P tests/docs/test_docs.cmakegit diff --cached --checkKnown limitations
DOCS_OPTIONSretrieval still scores zero.Generated with GitHub Copilot CLI