Skip to content

Fix: Bisect 95ebc2f7#786

Closed
xsscx wants to merge 1 commit intomasterfrom
fix-bisect-95ebc2f7

Hidden character warning

The head ref may contain hidden characters: "fix-bisect-\ufeff95ebc2f7"
Closed

Fix: Bisect 95ebc2f7#786
xsscx wants to merge 1 commit intomasterfrom
fix-bisect-95ebc2f7

Conversation

@xsscx
Copy link
Copy Markdown
Member

@xsscx xsscx commented Apr 6, 2026

Pull Request Checklist

#785

  • Have you followed the guidelines in Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you built your Pull Request locally with the Build Instructions?
  • Have you added or updated relevant tests?
  • Have you added or updated relevant docs?

@xsscx xsscx requested a review from ChrisCoxArt April 6, 2026 16:36
@xsscx xsscx self-assigned this Apr 6, 2026
@xsscx xsscx added PR Pull Request Merge Ready Maintainer indicates Merge Ready labels Apr 6, 2026
@xsscx xsscx linked an issue Apr 6, 2026 that may be closed by this pull request
@xsscx xsscx closed this Apr 6, 2026
@xsscx xsscx deleted the fix-bisect-95ebc2f7 branch April 6, 2026 20:31
@xsscx xsscx removed the request for review from ChrisCoxArt April 6, 2026 20:32
@xsscx xsscx mentioned this pull request Apr 6, 2026
5 tasks
xsscx pushed a commit that referenced this pull request Apr 6, 2026
Section 10: add PR #786 BOM canary check that verifies
detect_hidden_chars() is functional before trusting ref
validation results. Injects exact U+FEFF pattern from the
known-bad branch name.

Section 10: add emit_ref_finding() helper with raw hex dump
and diagnostic category output. Makes invisible bytes
forensically visible in the step summary.

Section 6: upgrade unsanitized write listing from bullet
list to table with File, Line, and Content columns.

Fix: add elements-sanitized marker to Windows Section 10
Out-File write (closes 105/105 gap).

Tested: 84/84 pass against xsscx/fuzz full-unicode.txt corpus
(5.5MB, all 8 detection categories verified).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
xsscx pushed a commit that referenced this pull request Apr 6, 2026
…tics

- sanitize-sed.sh v4: detect_hidden_chars() (8 categories), validate_ref()
- sanitize.ps1 v4: Detect-HiddenChars, Validate-Ref (PowerShell parity)
- ci-pr-risk-security-analysis.yml: Section 6 table output, Section 10
  canary check (PR #786 BOM pattern), emit_ref_finding() with hex dump
- ci-pr-action.yml: ref validation using detect_hidden_chars
- test_sanitization.sh: version v3->v4 fix, grep pipefail guard

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@xsscx xsscx added ML Maintainer indicates topic related to Machine Learning SCAP Maintainer indicates topic related to Security Content Information Protocol (SCAP) and removed Merge Ready Maintainer indicates Merge Ready pending labels Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ML Maintainer indicates topic related to Machine Learning PR Pull Request SCAP Maintainer indicates topic related to Security Content Information Protocol (SCAP)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant