php-bug-catcher / bug-catcher / 33502827443

85%

main: 84%

Build Type

push

github

Committed by tito10047

Commit Message

fix(detail): keep shortening trace paths when a frame is not a path

A single frame whose `file` is not a path -- a "Caused by: ..." label, an
`[internal function]` marker, an empty string left by a half-parsed trace line --
dropped the common prefix to nothing, so every other frame was rendered with the
full deploy path of the reporting server. The prefix is now derived only from
frames that look like paths, and stripped only from those.

Two more failure modes fixed on the way:

  * the prefix was seeded from `trace[0]`, so any frame array that was not zero
    indexed produced a null prefix and, again, full paths;
  * `strrpos($prefix, '/')` found nothing in Windows paths, which disabled
    shortening for them -- a regression from the directory boundary cut in the
    previous commit. Both separators are now honoured.

Stripping uses substr() behind a str_starts_with() guard instead of str_replace(),
which would also have rewritten an occurrence of the prefix in the middle of a
path.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Coverage Stats

21 of 22 new or added lines in 1 file covered. (95.45%)

1 existing line in 1 file now uncovered.

859 of 1011 relevant lines covered (84.97%)

6.78 hits per line