GitNexus/gitnexus/test/helpers/detect-reject-warning.ts
Gergő Magyar 9d95af9fc3
refactor(analyze): move detected-branch sanitization off CLI config (#3325)
* refactor(analyze): load detected-branch sanitization from core git-ref

Keep the never-throw helper next to validateBranchName so run-analyze no longer imports CLI config parsing.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(analyze): warn once when a checkout name cannot label the index

After the write lock settles, emit a single onLog warning and keep writing the workspace slot. Pin that run-analyze does not import CLI analyze-config.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(analyze): escape hidden checkout names in the detect-reject warning

Keep the rejected ref visible in onLog without replaying bidi or quote characters, and document that sanitizeDetectedBranch rethrows unexpected errors.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(analyze): keep detect-reject warnings on one line (#3325)

Git-legal U+2028/U+2029 checkout names were rejected as whitespace but left raw in the new onLog warning, so the message split across two lines. Escape those code points in the formatter without changing validateBranchName.

Co-authored-by: Cursor <cursoragent@cursor.com>

* chore(autofix): apply prettier + eslint fixes via /autofix command

* fix(analyze): keep C1 and Unicode spaces in detect-reject warnings

Escape NEL and remaining whitespace as \uXXXX so stripControlCharacters cannot drop or disguise the rejected checkout name.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Gergo Magyar <gergomagyar0@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
2026-09-18 11:03:24 +01:00

6 lines
282 B
TypeScript

/**
* Matches the detect-reject warning emitted by `run-analyze` after lock settle.
* Shared so unit and integration tests keep one regex.
*/
export const isDetectRejectWarning = (message: string): boolean =>
/^Warning:.*not a usable index label.*continuing\.$/.test(message);