githubnext/ace2Public

Ace: a collaborative coding environment on the team's own machines

Do not reject editable text areas when AXEnabled is absent

IA

A standard editable AppKit NSTextView can omit AXEnabled. Peekaboo reports that as is_enabled=false together with axEnabledKnown=false, but Ace's action wrapper currently treats the raw Boolean as a known disabled state and refuses the control before the native action validator can run.

Found while validating #65 against a real signed AppKit form. The text area was focused, editable, and reported isValueSettable=true, but desktop_click returned “The observed element is disabled.” The existing guard also applies to whole-value edits and the new text selection tool.

Peekaboo exposes DetectedElement.knownIsEnabled for this distinction: nil means unknown; false means explicitly disabled. Use that authoritative property to reject only a known-disabled target, while retaining exact target validation and the native action-specific checks. Do not override the test control's accessibility flags or add an input fallback.

Implemented in draft #67. The signed rebuild passes, and real native validation now accepts the editable multiline text area while refusing an explicitly disabled NSButton without activating its Save callback. This fix is pending merge with #65; broader keyboard validation remains blocked by #64. Tracked by #8 / #5.

  1. iamnbutlerreferenced this in [Meta] Complete native computer use in Ace #8last month on GitHub

  2. iamnbutlerreferenced this in [Meta] Dogfood Ace end to end #5last month on GitHub

  3. iamnbutlerreferenced this in Validate native text insertion on hosted channels and under interruption #65last month on GitHub

  4. iamnbutlerreferenced this in Add literal desktop insertion with reserved clipboard ownership #67last month on GitHub

  5. iamnbutlerreferenced this in Add native text selection and keyboard shortcuts #81last month on GitHub

  6. IA
    Author

    Fixed by merged PR #81 on feat/github-cache. Text controls with absent AXEnabled metadata remain usable; explicitly disabled controls remain refused. No new canary has been published for this merge.

  7. iamnbutlerclosedlast month on GitHub