Skip to content

Summary comment title issues#25

Open
Amazing-Stardom wants to merge 6 commits intomasterfrom
summary-comment-title-issues
Open

Summary comment title issues#25
Amazing-Stardom wants to merge 6 commits intomasterfrom
summary-comment-title-issues

Conversation

@Amazing-Stardom
Copy link
Copy Markdown
Contributor

@Amazing-Stardom Amazing-Stardom commented Apr 3, 2026

Random Name Added in Summary Title

What Is the Issue?

This issue was created based on a PR review trigger.

In the GitHub PR, random names were added to the title.

image

What Is the RCA?

In the RCA, random names were added in the comment due to the library aliengiraffe/deidentify.

image image

Here is the exact position at which the comment content was changed due to regex issues.

Due to the name regex,

Refactor LED Blinking to State Machines

was changed to

Refactor LED Blinking to Non-Arden Thomas

This function was called like this:

redacted, err := deidentifier.Text(out)

Where the out variable is the original content and redacted is the output.

What Is the Fix?

As this library didn't have any configuration, it was forked from HexmosTech/deidentify/.

A small configurable function was added.

For example, in the current problem, since the name function is causing issues, I will skip it by explicitly defining SkipNames: true in deidentify.TextOptions:

opts := deidentify.TextOptions{SkipNames: true}
redacted, err := d.Text(input, opts)

configuration added to forked library here is the
Commit Link

LiveReview Pre-Commit Check: ran (iter:2, coverage:94%)
LiveReview Pre-Commit Check: skipped
@Amazing-Stardom Amazing-Stardom requested a review from shrsv April 3, 2026 15:52
@Amazing-Stardom Amazing-Stardom marked this pull request as ready for review April 3, 2026 15:55
LiveReview Pre-Commit Check: ran (iter:1, coverage:0%)
LiveReview Pre-Commit Check: ran (iter:1, coverage:0%)
LiveReview Pre-Commit Check: ran (iter:1, coverage:0%)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant