Make the documented upstream behaviour verifiable instead of asserted.
voyager_runner.py
skill_manager_retrieval_top_k now follows the upstream FAQ, which says it is 'essential' to set it <= 2 for a non-GPT-4 model to reduce prompt length. This deployment runs granite-4.0-h-tiny with an 8192-token window and the action prompt was measured at 6909 tokens. Overridable with VOYAGER_SKILL_TOP_K.
voyager_verification.py
task_spec() no longer swallows the purpose clause into the item name. It was feeding the official Critic a false world statement, e.g. 'Live Inventory has 0/1 diamond_pickaxe_to_upgrade_your_mining_capabilities...'.
tests/test_voyager_constructor_defaults.py (rewritten)
Was three substring checks on a hardcoded Windows path. Now parses the upstream Voyager.__init__ signature and the local Voyager(...) call with ast and compares every passed parameter against the upstream default, with an explicit reviewed departure list. Portable via VOYAGER_UPSTREAM_REF.
tests/tes…
Show the Action agent its full previous program and restore upstream defaults
naldo
008c4fa
The previous-code bound truncated the model's last program to 3200 characters. A
~1000-token program is ~4000 characters, so the model read its own function cut
off mid-body and redeclared it on the next round. That is exactly the
'Identifier has already been declared' SyntaxError upstream's Babel rejects, and
it was happening on a third of all attempts. Upstream sends the whole program;
restore that.
Also restore the three constructor values that had been narrowed without cause:
action_agent_task_max_retries 3 -> 4, action_agent_show_chat_log False -> True,
skill_manager_retrieval_top_k 1 -> 5. With one retrieved skill the Action agent
could not reuse what it had learned, which pushed it to redeclare skills inline.
max_iterations, learn(reset_env=False), the Creative evidence layer, the
exploreUntil bounds, the creative toolkit and the neutralised /pause are left
exactly as they were: they define how this Space operates, not how it matches
upstream, and changing them would disturb b…
