Singleton-variable warnings on every library(rlm) load (`a3628d3`): rlm_skill.pl:881, rlm_prompt_compiler.pl:1941 #301

New issue

Open

opened 2026-09-02 17:25:36 +00:00 by nsaspy · 0 comments

nsaspy commented

2026-09-02 17:25:36 +00:00

Owner

Summary

a3628d3 ("fix(runtime): propagate wall-time limits through skill compilation") added exception-propagation clauses with an unused first argument, which makes every use_module(library(rlm)) print singleton-variable warnings:

Warning: .../prolog/rlm_skill.pl:881:
Warning:    Singleton variables: [Phase]
Warning: .../prolog/rlm_prompt_compiler.pl:1941:
Warning:    Singleton variables: [Phase]

Affected clauses

% prolog/rlm_skill.pl:881
skill_exception(Phase, Exception, _) :-
    time_limit_exception(Exception),
    !,
    throw(Exception).

% prolog/rlm_prompt_compiler.pl:1941
compiler_exception(Phase, Exception, _) :-
    time_limit_exception(Exception),
    !,
    throw(Exception).

Impact

Hosts that load the pack (CI logs, embedders, the Agent Zero runtime worker) get warning noise on every load; it also hides genuine future warnings in the same output. Not present at b891c27, so it regressed with a3628d3.

Fix

Rename the unused head argument to _Phase (or _) in both clauses.

## Summary a3628d3 ("fix(runtime): propagate wall-time limits through skill compilation") added exception-propagation clauses with an unused first argument, which makes every `use_module(library(rlm))` print singleton-variable warnings: ``` Warning: .../prolog/rlm_skill.pl:881: Warning: Singleton variables: [Phase] Warning: .../prolog/rlm_prompt_compiler.pl:1941: Warning: Singleton variables: [Phase] ``` ## Affected clauses ```prolog % prolog/rlm_skill.pl:881 skill_exception(Phase, Exception, _) :- time_limit_exception(Exception), !, throw(Exception). % prolog/rlm_prompt_compiler.pl:1941 compiler_exception(Phase, Exception, _) :- time_limit_exception(Exception), !, throw(Exception). ``` ## Impact Hosts that load the pack (CI logs, embedders, the Agent Zero runtime worker) get warning noise on every load; it also hides genuine future warnings in the same output. Not present at b891c27, so it regressed with a3628d3. ## Fix Rename the unused head argument to `_Phase` (or `_`) in both clauses.

nsaspy added the

documentation

label

2026-09-02 17:25:36 +00:00