2 Commits

Include renames

Author	SHA1	Message	Date
Accessibility ModandClaude Opus 5	5778e86e8e	orientation: the travel map, unexplored ways, the nearest exit ... Prompted by "do sighted players have a minimap we don't have access to?" No widget, but two map surfaces were only half exposed: the world map screen, and the level view itself, which IS the explored-floor map -- rememberedFoV keeps everything ever seen drawn, and a sighted player pans over it and reads the fog edge at a glance. The world map dropped the three things the picture carries: what kind of place each area is (the icon shape), which way it lies, and how long travelling there takes. The last is the number the decision turns on and was only ever in the side panel. states/worldmap.lua is now a read() plus details() rather than probe/render -- necessarily, because render() gets no state table and the travel time needs the screen's currentAreaId, an upvalue like options and selectedOption. Travel time is the game's own getDistanceMessage, the same call the screen makes. The blurb runs to several sentences, so it goes on = and the settle timer instead of the focus line. Two gotchas: options[i].icon is already an image object, so the icon id string has to come off the model area; and describe.direction only tests the sign of each delta, which on the map's floating-point coordinates called almost everything diagonal, hence a 25% deadzone carrying the same 4/3 x-correction the game's own distance maths uses. New frontier.lua answers "where haven't I been", which nothing did: surroundings is local, examine is one tile, and the tracker only finds things it already knows about. A frontier tile is seen, not blocking movement, with an in-bounds unseen CARDINAL neighbour -- cardinal because the diagonal test fires on the unseen tile behind every wall corner, so a lit corridor would report an unexplored way per corner. Clustered 8-connected and reduced to each cluster's nearest member, so a wide doorway is one way but two openings that both lie north stay two. Reads only FoV/rememberedFoV and reports only THAT an opening exists, never what is beyond it -- exactly what the fog edge shows. Shift+End names the nearest exit without disturbing the tracker selection. Note for later: level.exits does not exist at runtime, it is packed as nil and never assigned. The primitive is the game's own getListByTag("exit"), which already gates on FoV or rememberedFoV, skips hidden tiles and sorts by distance. Exits were already a tracker category, so this is a shortcut rather than a new capability. "remembered" moves from verbose to normal verbosity. tileFacts reads actors only when a tile is currently VISIBLE, so a remembered tile is silent about whatever may be standing on it now, and at normal verbosity that silence was indistinguishable from "nothing there". A safety flag, not a nicety. Also adds descriptor.probeNames: a read() descriptor has no probe table, so menus.selfTest could only report "custom read()" and the version-bump canary was lost on exactly the screens reaching for the most upvalues. worldmap and research now declare their game locals and get the same PROBE BROKEN check. 120 testbed checks, 0 failures. It caught two things review had not. The world map said "You are currently here." twice, because getDistanceMessage and getLastVisitedMessage return the same sentence for the area you are standing in -- fine as two panels on screen, a stutter in one utterance. And the first frontier check walked a step east and demanded the numbers change; it failed on a run where nothing was wrong, the step landing but both openings keeping the same Chebyshev distance. Replaced with the deterministic property: every reported distance must equal Chebyshev from the player's current position, nearest first. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-03 16:05:15 +02:00
Accessibility ModandClaude Opus 5	0477097e29	Phase 4: menu descriptors for 16 states ... Game-file footprint unchanged: still one line in main.lua. Adds descriptors for actions_menu, worldmap, alchemy_table, journal, character, inventory, and a shared one for the full-screen prose states (menu_text, ingame_message, menu_message). The prose states also push their text into the console history, so the existing review keys walk the manual paragraph by paragraph rather than delivering it as one unusable utterance. Two driver improvements, both prompted by real failures the self-test caught: - menus.getAnyUpvalue finds a file-local across all of a state's functions and one level into inner closures. Safe because a Lua file-local is a single shared upvalue. Needed because actions_menu and worldmap build their lists in open() and never reference them in keypressed, so the named probe function found nothing. The generic probe path now falls back to this. - menus.hasUpvalue checks that a local exists rather than that it holds a value. getUpvalue cannot distinguish 'no such local' from 'declared but currently nil', which made the self-test report PROBE BROKEN for worldmap, whose 'local options' is nil until the screen is first opened. patches/popups.lua now stands down for any state that has a descriptor. inventoryCursorPopup.set only fires when an item is under the cursor, so empty slots and empty tabs were silent and you could not tell the cursor had moved. The inventory descriptor reports item-or-empty plus slot and tab instead. 21/21 testbed checks, history unit tests passing. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-01 22:00:52 +02:00