Repository navigation
Conversation
Everything here is a private trace, not a behavior change: the maintainer's name, local absolute paths, the out-of-repo private corpus location, and a planning document that only ever existed on one machine. - Person: "Lman" -> "the maintainer" / 「維護者」 in comments, .gitignore, tools/eval docs, and the review reports. Copyright lines keep the real name; so do the lmanchu GitHub URLs and the bundle id. - Paths: /Users/lman/... -> ~/... or a description; a scratch-directory script reference loses its path. - Design notes: ~/.claude/plans/zhuyin-ime-personal.md (and its bare filename) never existed in this repo, so every reference now reads "the design notes". The README sentence citing it is gone. - Eval corpus: the two XCTest eval measurements read BOPOMIX_EVAL_CORPUS and XCTSkip when it is unset or missing, instead of hardcoding ~/Dev/mixime-private/eval200.tsv. check_plist_unchanged.sh forwards the variable as TEST_RUNNER_BOPOMIX_EVAL_CORPUS, which is the only spelling xcodebuild passes to the test host. build_corpus.py defaults to the gitignored eval-corpus/ directory; .gitignore's comment no longer names a private repo path. docs/REVIEW-* and docs/REVERIFY-* keep their content and conclusions verbatim -- only the private details inside them changed. Verified: xcodebuild build succeeds; the suite is 255 tests green under check_plist_unchanged.sh both with and without BOPOMIX_EVAL_CORPUS (2 skipped without it), PLIST_UNCHANGED and DATA_FOLDER_UNCHANGED each time, and BASELINE.md's numbers regenerate identically apart from latency. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…odenames Follow-up to the previous scrub. The review reports quote samples of the maintainer's private eval corpus verbatim, which leaked colleague names and the customers/partners those notes are about; the shipped tech word list carried the maintainer's own product codenames. - Corpus token dumps and frequency counts in docs/REVIEW-P3-2026-09-11.md and docs/REVERIFY-P3-2026-09-12.md now use stable placeholders: people become <name-N>, the maintainer's company/customers/partners become <company-N>, their products become <product-N>. The same word always maps to the same placeholder, so the counts and the "how many of these are real proper nouns" conclusions still read correctly. Generic global brands already in the shipped lexicon (google, computex, Stripe, Apple, Electron) stay as they are. - Source/Data/latin-tech-seed.txt: dropped irisgo, tachikoma, hermes, gloria, apollo, argus, wells and magi. The first two are private names that exist nowhere else; the other six are internal system codenames that were outranking the whole dictionary for words a public user has no reason to want boosted. All six remain available from latin-words.txt. Ranks are explicit per row, so the resulting gaps change nothing. - tools/eval/build_corpus.py: the synthetic-padding token list no longer names the maintainer's company, and says to swap in your own. Verified: xcodebuild build succeeds; 255 tests green under check_plist_unchanged.sh without BOPOMIX_EVAL_CORPUS (2 skipped), with PLIST_UNCHANGED and DATA_FOLDER_UNCHANGED; the built app's bundled latin-tech-seed.txt no longer contains any of the removed terms. Co-Authored-By: Claude Opus 5 (1M context) <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.
What
Removes the maintainer's local traces so the repository can go public. Stacked on #4 (
rename-bopomix).BOPOMIX_EVAL_CORPUSand skip with a pointer totools/eval/build_corpus.pywhen it is unset.xcodebuilddoes not pass its environment to the test host, sotools/eval/check_plist_unchanged.shforwards the variable asTEST_RUNNER_BOPOMIX_EVAL_CORPUS; documented intools/eval/README.md.docs/are published as they are (decision: keep the honest record), with one change: the corpus token dumps no longer name real people or the maintainer's employer, customers and partners — placeholders such as<name-2>/<company-1>keep the counts and conclusions intact.Source/Data/latin-tech-seed.txtdrops eight internal codenames that would have shipped inside the app bundle and outranked the dictionary (irisgo,tachikoma,hermes,gloria,apollo,argus,wells,magi); the six that are ordinary English words remain in the main dictionary at their normal rank..gitignoregains a genericeval-corpus/entry;build_corpus.py's proper-noun example no longer names the maintainer's company.Untouched on purpose: copyright lines,
lmanchuin URLs and the bundle id, test method names, and the dictionary data.Verification
BOPOMIX_EVAL_CORPUS: 255 XCTest (2 skipped, the corpus tests) + 87 Swift Testing,PLIST_UNCHANGED/DATA_FOLDER_UNCHANGED.git grepaudit for every scrubbed pattern: remaining hits are dictionary rows and-isubstring false positives only.🤖 Generated with Claude Code