feat(runtime): scheduler-via-runSkill + Pi banter escalation + base-site (follow-ups) (#20)
Three closures of remaining PRD follow-ups, one merge:
1. Reflex scheduler now drives behaviour from the curriculum.
- reflex.js: replaced ad-hoc techTreeReflex + autonomousReflex with
curriculumReflex that dispatches the skill suggested by
snapshot.curriculum.plan via runSkill. Per-skill backoff for
missing_tool / missing_material / no_target / no_food_source /
unsupported_version. recover() hint with `{hint:"wander"}` swaps
the next tick to wander for 60 s.
- Chain is now: defend > eat > sleep > curriculum > idle.
- reflex.test.js: 11 new tests covering busy/disconnected,
defend/eat preemption, dispatch by id, unknown-skill fallback,
per-skill + wander-hint backoffs, onComplete updating backoff.
2. Pi escalation for ADDRESSED_BANTER with hard rate limit.
- bot.js: when generateReply returns {escalate:true}, spawn askPi
with bot state + last 5 lines from that speaker (redacted via
chatMemory). Reply capped at 200 chars, sent as one chat line.
- Rate cap: 6 calls/hour, 90 s min gap. Suppressed escalations
log once and silently drop.
3. Phase 4 substrate.
- runtime/locations.js: atomic JSON store
(state/<host>/locations.json) with setLocation / getLocation /
nearestLocation / removeLocation; 6 tests.
- runtime/base-site.js: scoreCurrentPosition(bot) + pure scoreSite
bundle (wood / stone / water / flatness / no-players /
no-foreign-builds, owned-blocks excluded from claim penalty);
6 tests.
- runtime/skills/choose-base.js: village.choose-base skill — scores
the current spot, writes locations.base if score ≥ 8, otherwise
returns code:"too_weak" with a wander recover hint.
- curriculum.js: new final milestone village.base-site fires
village.choose-base until a base location exists.
- bot.js: stamps snapshot.locations from listLocations() each tick
so the curriculum can read it without coupling to disk.
docs/runtime.md updated with three new sections.
npm test now 116/116.
Co-authored-by: Yuriy Mayatnikov <mayatnikov@me.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit was merged in pull request #20.
This commit is contained in:
Yuriy Mayatnikov
@@ -0,0 +1,227 @@
|
||||
// Tests for the new curriculum-driven scheduler in runtime/reflex.js.
|
||||
// We exercise runTick with synthetic snapshots + a recording ctx so we can
|
||||
// assert which reflex fires and what gets dispatched.
|
||||
|
||||
import { test } from "node:test";
|
||||
import assert from "node:assert/strict";
|
||||
|
||||
import { runTick } from "./reflex.js";
|
||||
|
||||
function makeCtx({
|
||||
snapshot,
|
||||
busy = false,
|
||||
bot = { entities: {}, entity: { position: { x: 0, y: 64, z: 0 } } },
|
||||
skillBackoff,
|
||||
lastEatAt = 0,
|
||||
lastSleepAttemptAt = 0,
|
||||
lastCurriculumAt = 0,
|
||||
} = {}) {
|
||||
const dispatches = [];
|
||||
const ctx = {
|
||||
bot,
|
||||
snapshot,
|
||||
busy,
|
||||
currentActionLabel: null,
|
||||
lastEatAt,
|
||||
lastSleepAttemptAt,
|
||||
lastCurriculumAt,
|
||||
skillBackoff,
|
||||
dispatch(fn, label, opts = {}) {
|
||||
dispatches.push({ label, opts });
|
||||
},
|
||||
};
|
||||
return { ctx, dispatches };
|
||||
}
|
||||
|
||||
test("busy ctx returns skipped without dispatching", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: { connected: true, curriculum: { plan: { skillId: "gather.logs" } } },
|
||||
busy: true,
|
||||
currentActionLabel: "chopping",
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "busy");
|
||||
assert.equal(out.action, "skipped");
|
||||
assert.equal(dispatches.length, 0);
|
||||
});
|
||||
|
||||
test("disconnected snapshot → no dispatch", () => {
|
||||
const { ctx, dispatches } = makeCtx({ snapshot: { connected: false } });
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out, null);
|
||||
assert.equal(dispatches.length, 0);
|
||||
});
|
||||
|
||||
test("defend wins over curriculum when hostile in melee", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
closestHostile: { name: "zombie", distance: 3 },
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "defend");
|
||||
assert.match(dispatches[0].label, /attack zombie/);
|
||||
});
|
||||
|
||||
test("eat wins over curriculum when food low and bot has food", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 10,
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "eat");
|
||||
assert.equal(dispatches[0].label, "eat");
|
||||
});
|
||||
|
||||
test("curriculum dispatches suggested skill by id", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "curriculum");
|
||||
assert.equal(dispatches[0].label, "gather.logs");
|
||||
assert.ok(typeof dispatches[0].opts.onComplete === "function");
|
||||
});
|
||||
|
||||
test("curriculum falls back to wander when no plan", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: null,
|
||||
},
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "curriculum");
|
||||
assert.equal(dispatches[0].label, "wander");
|
||||
});
|
||||
|
||||
test("curriculum falls back to wander when skill id is unknown", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "shelter.assemble" } }, // not registered yet
|
||||
},
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "curriculum");
|
||||
assert.match(dispatches[0].label, /^wander/);
|
||||
});
|
||||
|
||||
test("curriculum honours per-skill backoff", () => {
|
||||
const future = Date.now() + 30_000;
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.stone" } },
|
||||
},
|
||||
skillBackoff: { "gather.stone": future },
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
// Backoff blocks the curriculum reflex; idle still won't fire on this
|
||||
// tick (idleCounter not at the 20-tick mark) so runTick yields.
|
||||
assert.equal(out, null);
|
||||
assert.equal(dispatches.length, 0);
|
||||
});
|
||||
|
||||
test("wander-hint backoff swaps skill for wander on the next tick", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
skillBackoff: { __wander_hint__: Date.now() + 30_000 },
|
||||
});
|
||||
const out = runTick(ctx);
|
||||
assert.equal(out.reflex, "curriculum");
|
||||
assert.equal(dispatches[0].label, "wander");
|
||||
});
|
||||
|
||||
test("onComplete sets wander hint when skill recovery says so", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
});
|
||||
runTick(ctx);
|
||||
const cb = dispatches[0].opts.onComplete;
|
||||
cb({ ok: false, code: "no_target", recovery: { hint: "wander" } });
|
||||
assert.ok((ctx.skillBackoff?.__wander_hint__ ?? 0) > Date.now());
|
||||
});
|
||||
|
||||
test("onComplete clears wander hint on success", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
skillBackoff: { __wander_hint__: Date.now() + 30_000 },
|
||||
});
|
||||
// With wander hint active, first tick will choose wander, not gather.logs.
|
||||
runTick(ctx);
|
||||
// Simulate the wander completing and clearing the hint manually since
|
||||
// wander is dispatched via actions.js directly (no recover hint).
|
||||
ctx.skillBackoff.__wander_hint__ = 0;
|
||||
|
||||
// Next tick — past the curriculum cooldown.
|
||||
ctx.lastCurriculumAt = 0;
|
||||
const { ctx: ctx2, dispatches: dispatches2 } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.logs" } },
|
||||
},
|
||||
});
|
||||
runTick(ctx2);
|
||||
dispatches2[0].opts.onComplete({ ok: true, code: "done" });
|
||||
assert.equal(ctx2.skillBackoff.__wander_hint__, 0);
|
||||
});
|
||||
|
||||
test("onComplete sets per-skill backoff on cooldown-class failures", () => {
|
||||
const { ctx, dispatches } = makeCtx({
|
||||
snapshot: {
|
||||
connected: true,
|
||||
health: 20,
|
||||
food: 20,
|
||||
isDay: true,
|
||||
curriculum: { plan: { skillId: "gather.stone" } },
|
||||
},
|
||||
});
|
||||
runTick(ctx);
|
||||
const cb = dispatches[0].opts.onComplete;
|
||||
cb({ ok: false, code: "missing_tool", detail: "no pickaxe" });
|
||||
assert.ok((ctx.skillBackoff?.["gather.stone"] ?? 0) > Date.now());
|
||||
});
|
||||
Reference in New Issue
Block a user