Say exactly what can leave the device, because "only" was wrong - #46
Merged
Conversation
The README claimed the paraphrase check was the only feature that sends text anywhere. There are four: paraphrase, the perplexity measurement, the live rewrite with your own key — the key stays put, the text does not — and the optional web spot-check for a distinctive phrase. Every one is off until switched on and disclosed at the moment of choosing, so the spirit of the claim held. The letter did not, and this is the one sentence in the project a reader is entitled to take literally. A privacy promise wider than the code is the sort of thing this project exists to complain about elsewhere. The MCP README listed nine tools and said "the first seven"; write_report shipped in 0.4.0 and eight of the ten now run offline. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
This commit was written on 5 Aug, approved, and never reached
main— it was part of PR #45,which merged while it was still in flight.
maintoday still carries the sentence this fixes.That is the second time a merge has silently dropped work on this repository, so it is worth naming
the check that caught it:
git merge-base --is-ancestor <sha> origin/main. The PR badge said mergedand was green. The commit was not there.
What it fixes
README.mdclaimed:There are four features that can send text off the device, not one:
EmbeddingClientPerplexityClientHumanizerService— the key stays, the text goesWebCheckClientEvery one of them is off until switched on, and each discloses itself in the interface, so the
spirit of the claim held. The letter did not. A privacy promise broader than the code is exactly
what this project criticises in other tools, and it is the one sentence a reader is entitled to
take literally.
Also corrects
src/SignsOfAI.Mcp/README.md, which still said "the first seven" tools while listingnine.
No behaviour change; documentation only.