Skip to content

fix(R-SLOP-08): remove Meta.Cursor from hallucinated API pattern#157

Closed
ZviBaratz wants to merge 1 commit intomainfrom
fix/meta-cursor-slop-08
Closed

fix(R-SLOP-08): remove Meta.Cursor from hallucinated API pattern#157
ZviBaratz wants to merge 1 commit intomainfrom
fix/meta-cursor-slop-08

Conversation

@ZviBaratz
Copy link
Copy Markdown
Owner

Problem

R-SLOP-08 incorrectly flags Meta.Cursor as a hallucinated API. In reality, Meta.Cursor is a valid GJS enum for cursor shapes used with global.display.set_cursor(). This caused a false positive on PaperWM (field test 2026-03-27).

Only Meta.Screen is nonexistent in GJS.

Changes

  • Remove Meta.Cursor from the R-SLOP-08 pattern regex
  • Update the message to clarify only Meta.Screen is invalid
  • Update the fix hint to note that Meta.Cursor IS valid

Tests

  • All existing hallucinated-apis fixture tests pass (Meta.Screen still detected)
  • 583 passed, 127 failed — identical to baseline (0 regressions)

Meta.Cursor is a valid GJS enum for cursor shapes used with
global.display.set_cursor(). Only Meta.Screen is nonexistent.

Update message and fix hint to clarify that Meta.Cursor IS valid,
to prevent future confusion about the API.

Discovered during PaperWM field test (2026-03-27).
@ZviBaratz
Copy link
Copy Markdown
Owner Author

Duplicate of #156 — both fix the same R-SLOP-08 Meta.Cursor false positive. Closing this one; #156 has a cleaner message.

@ZviBaratz ZviBaratz closed this Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant