yylego / runpath / 23178122135

59%

push

github

revert to original Skip-based implementation and add shallow-stack usage note

The previous commit changed all functions to invoke runtime.Caller
instead of going through the package-level Skip function to avoid
potential inlining issues. However, the actual problem was that
PARENT.Skip method calls consume one extra stack frame, which fails
in shallow-stack scenarios (e.g., package-level variable init) —
this has nothing to do with inlining. The original Skip-based
implementation is correct. Added a note on Path() recommending
filepath.Dir(runpath.Skip(n)) in shallow-stack contexts.

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

27 of 28 new or added lines in 3 files covered. (96.43%)

92 of 176 relevant lines covered (52.27%)

13.27 hits per line