docs: stop enumerating private repositories in ui-usage - #201
Merged
Conversation
Both files were merged into this public repository while describing seven private repositories in detail: enumerating them by name, and documenting their internal directory layouts, file names and per-repo differences. Removing them from the tree. This does not remove them from this repository's history, which is a separate decision. Reverts the content added in #199. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The header listed seven consuming apps by name in this public repository. Six of the seven are private repos; naming the products is fine, but publishing the roster is not. Replaced with "every consuming application" — the sentence made the same point without the list. Predates the frontend reference docs removed in the previous commit; same class of disclosure. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
|
🚀 Preview deployment is ready! You can view the preview at: https://pr-ui-preview-201.surge.sh |
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.
Follow-up to #200, which removed the two frontend reference docs from this public repository.
docs/ui-usage.mdstill opens by listing seven consuming apps by name. Six of those repositories are private. Naming the products is fine — they're public-facing sites — but publishing the roster of repositories is a different thing, and this is a public repo.The sentence made its point without the list, so the list is simply gone:
This predates the docs removed in #200; same class of disclosure, found while auditing for the rest.
Also checked
Scanned every tracked file for the seven names. What remains is deliberate and harmless:
Secured by Honey.idandhttps://honey.id/in the auth components (public product branding), andteam@pathscale.complaceholders in the playground.Two comments in
src/components/language-switcher/createI18n.tsxreference a private repo's internali18nStoreAPI. Low signal, and changingsrc/in a docs PR mixes concerns — flagging rather than folding it in. Say the word and it's a two-line follow-up.🤖 Generated with Claude Code