onlyMember

[jordaniza]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces a new permission condition contract called OnlyMemberCondition that restricts permissions to members of a target contract implementing the IMembership interface.

• Adds a new permission condition contract that validates membership status
• Includes comprehensive test coverage with both unit tests and test specification
• Implements proper interface validation and error handling for invalid addresses

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
src/OnlyMemberCondition.sol	Main contract implementing membership-based permission condition
test/OnlyMemberCondition.t.sol	Unit tests covering deployment, membership validation, and interface support
test/OnlyMemberCondition.t.yaml	Test specification defining test cases and expected behaviors
test/mocks/MembershipMock.sol	Mock contract for testing membership functionality

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The indentation uses tabs mixed with spaces. This line should use consistent spacing (spaces) to match the rest of the codebase.

Suggested change

	target = _target;
	target = _target;

[Copilot]

The test file uses a different Solidity version (0.8.23) than the main contract and mock (^0.8.22). Consider using consistent versioning across the codebase.

Suggested change

	pragma solidity 0.8.23;
	pragma solidity ^0.8.22;