Toilal / rebulk / 28641828000

98%

master: 97%

Build Type

Pull #78

github

Committed by Toilal

Commit Message

perf: cache getfullargspec on the hot matching path

getfullargspec is called for every validator/formatter/conflict-solver
invocation via loose.call. Its result depends only on the callable, and
those callables are stable singletons for a Rebulk instance's lifetime,
so the introspection is fully redundant across matching calls.

Wrap it in an unbounded lru_cache: ~11% faster repeated parses for a
one-line change. The set of distinct callables is small and bounded,
callables are hashable, and getfullargspec is a pure function of its
argument, so caching by identity cannot change behaviour.

closes #77

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ah1tBeqoVa8sM2CP5ibSJQ

Pull Request Pull Request #78: perf: cache getfullargspec on the hot matching path

Coverage Stats

4 of 4 new or added lines in 1 file covered. (100.0%)

4 existing lines in 2 files now uncovered.

1913 of 1948 relevant lines covered (98.2%)

6.86 hits per line