Make Step 2's KB duplicate search visible with a printed log line - #25
Merged
Conversation
Co-authored-by: bguidolim <987360+bguidolim@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix capture skill's duplicate search enforcement
Make Step 2's KB duplicate search visible with a printed log line
Aug 27, 2026
bguidolim
approved these changes
Aug 27, 2026
There was a problem hiding this comment.
Pull request overview
This PR updates the continuous-learning capture skill to make Step 2’s knowledge-base duplicate search auditable by requiring a single printed log line that records the search and the chosen branch (skip/edit/create), aligning Step 2 with Step 4’s “visible checks” rationale from Issue #22.
Changes:
- Add a Step 2 requirement to print one log line summarizing the KB search and the decision branch taken.
- Provide a concrete example format for that log line in the skill instructions.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| Print one line recording this search before continuing, matching the Step 4 checks — hidden reasoning is easy to skip, printed output is reviewable: | ||
|
|
||
| ``` | ||
| KB search: "<query>" -> <n> hits, <what they covered> -> <branch taken, and the file edited or created> |
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.
Step 2 of the continuous-learning capture skill decides whether a session skips, edits, or creates a memory based on a knowledge-base search — but unlike Step 4's pre-
Writechecks, it never required printed output. A search that never ran was indistinguishable from one that ran and found nothing, weakening the audit trail behind the duplication (C.2) and narrow-learning (H) DROP categories.Changes
skills/continuous-learning/SKILL.md— Step 2 now requires one printed line recording the search and the branch taken, before continuing to Step 3:This mirrors the existing rationale for Step 4's visible checks ("Hidden reasoning is easy to skip; printed output is reviewable") and adds no new mechanism — just makes an already-mandatory step auditable.