Skip to content

fix(ios): remove super invalidate (NativeFlic2SpecBase) - #57

Merged
XGManuelJager merged 2 commits into
developfrom
feature/auto-20260716-171051
Jul 16, 2026
Merged

fix(ios): remove super invalidate (NativeFlic2SpecBase)#57
XGManuelJager merged 2 commits into
developfrom
feature/auto-20260716-171051

Conversation

@jdegger

@jdegger jdegger commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • iOS builds fail because NativeFlic2SpecBase (NSObject) does not declare invalidate, so [super invalidate] does not compile.
  • Keep -invalidate cleanup (reject pending init); RN still invokes it via respondsToSelector.

Test plan

  • npm run lint / npm run typecheck
  • iOS build of a consumer app (e.g. xgac-rn-alarm) with this pod compiles past Flic2.mm
  • Module remount still rejects in-flight initialize with MODULE_INVALIDATED

Made with Cursor

NativeFlic2SpecBase is NSObject and does not declare invalidate, which
breaks Xcode builds. Keep our invalidate cleanup; RN still finds it via
respondsToSelector.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings July 16, 2026 15:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes an iOS compilation failure in the native module by removing an invalid [super invalidate] call from Flic2’s -invalidate implementation while keeping the module’s cleanup behavior (rejecting a pending initialize promise) intact.

Changes:

  • Remove [super invalidate] from -invalidate to avoid calling a selector not declared on the generated base class.
  • Preserve cleanup behavior by continuing to reject any in-flight initialize with MODULE_INVALIDATED.
  • Add an explanatory comment clarifying why -invalidate exists without a super call.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings July 16, 2026 15:14
@XGManuelJager
XGManuelJager merged commit f454f8a into develop Jul 16, 2026
4 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.

@jdegger
jdegger deleted the feature/auto-20260716-171051 branch July 17, 2026 08:41
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.

3 participants