🟠 Removal of focused_view() breaks external callers #22

Open xenolandscapes.com
Open opened by xenolandscapes.com

Summary #

The focused_view() method was removed and replaced with current_view(). All callers within the changed files have been updated, but any external code or unmodified internal modules that call editor:focused_view() will receive nil and likely crash.

The Problem #

The function focused_view() was collapsed into current_view(). The editor.lua diff removes the focused_view annotation and implementation. While the diff updates all visible call sites, there may be other code (user init.lua, external scripts, or unmodified modules) that still references focused_view(), leading to runtime errors.

Impact #

  • category: correctness
  • description: focused_view() removal breaks backward compatibility; unmodified callers in user code or other modules may crash

Add a backward-compatible alias: editor.focused_view = editor.current_view in Editor.new, then remove it in a later cleanup commit after confirming no external callers remain.


Commit range: c022da8..c022da8
Code Review: CR-2026-07-27-11

Sign up or Sign in to add to the discussion.