Skip to content

Add Objective-C info conversion options - #29

Merged
p-x9 merged 5 commits into
mainfrom
codex/objc-info-options
Jun 29, 2026
Merged

Add Objective-C info conversion options#29
p-x9 merged 5 commits into
mainfrom
codex/objc-info-options

Conversation

@p-x9

@p-x9 p-x9 commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add non-breaking options for Objective-C class, category, and protocol info conversion
  • Keep existing info(in:) behavior as recursive full protocol expansion
  • Add options to limit referenced protocol traversal depth
  • Add a direct protocol name mode to avoid recursively materializing adopted protocols

Validation

  • swift build
  • swift test

p-x9 added 4 commits June 28, 2026 03:56
- Updated ObjCInfoOptions to conform to Sendable
- Updated ObjCProtocolInfoOptions to conform to Sendable
- Updated Traversal and ReferencedProtocolInfo enums to conform to Sendable

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 introduces configurable conversion options for Objective-C class/category/protocol metadata so callers can limit protocol traversal depth and optionally avoid recursively materializing adopted protocol details, while preserving the existing default recursive behavior.

Changes:

  • Added ObjCInfoOptions and ObjCProtocolInfoOptions to control protocol traversal depth and referenced protocol detail level.
  • Extended info(in:) conversion APIs for protocols/classes/categories to accept the new options (defaulting to the prior recursive behavior).
  • Implemented option-aware protocol reference expansion, including a “name-only” referenced protocol mode.

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

Comment thread Sources/MachOObjCSection/Support/ObjCDump.swift
Comment thread Sources/MachOObjCSection/Support/ObjCDump.swift
@p-x9
p-x9 merged commit d3e6f43 into main Jun 29, 2026
2 checks passed
@p-x9
p-x9 deleted the codex/objc-info-options branch June 29, 2026 14:04
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.

2 participants