Skip to content

fix/issue 5840 - #5893

Closed
nghetienhiep wants to merge 1 commit into
JSONbored:mainfrom
nghetienhiep:fix/issue-5840
Closed

fix/issue 5840#5893
nghetienhiep wants to merge 1 commit into
JSONbored:mainfrom
nghetienhiep:fix/issue-5840

Conversation

@nghetienhiep

Copy link
Copy Markdown
Contributor

FORBIDDEN_EXTENSION_TERMS in extension-contributor-context.ts was a
separately hand-maintained regex that had drifted from the canonical
PUBLIC_UNSAFE_TERMS vocabulary in redaction.ts, missing bare cohort,
bare ranking, miner-originated/human-originated, and standalone
reviewability. Compose it from PUBLIC_UNSAFE_TERMS instead (importing
just the string constant keeps the module cycle-free, since
redaction.ts has no imports of its own), plus the couple of
extension-only terms not in that shared vocabulary.

Adds regression tests for each newly-covered term and a drift-guard
test that checks a representative sample of every PUBLIC_UNSAFE_TERMS
entry is caught by redactExtensionText, so this can't silently drift
again.

Closes JSONbored#5840
@nghetienhiep
nghetienhiep requested a review from JSONbored as a code owner July 14, 2026 21:42
@JSONbored

Copy link
Copy Markdown
Owner

Your PR descriptions are wrong, and your titles do not follow our conventional commit structure. Closing.

@JSONbored JSONbored closed this Jul 14, 2026
@loopover-orb loopover-orb Bot added the gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. label Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants