Inconsistency between addMember() implementation and documentation

[sourcery-ai]

The addMember() method in the codebase has an inconsistency between its implementation and its documentation. The method is currently implemented to always return true, regardless of whether the member already exists. However, the documentation specifies that the method should return false if the member already exists. This discrepancy could potentially lead to bugs, especially in security-related code where the existence of a member needs to be accurately checked.

Suggested Actions:

1. Review the implementation of addMember() to ensure it aligns with the documented behavior.
2. Decide whether the documentation should be updated to reflect the current implementation or if the code should be modified to match the documentation.
3. Consider the implications of this inconsistency on security and other critical areas of the codebase.
4. Track this issue as a pre-existing condition if it is not addressed immediately, to ensure it is revisited in future updates.

---

I created this issue for @yingbull from #45 (comment).

Tips and commands

Interacting with Sourcery

• Generate a plan of action: Comment @sourcery-ai plan on this issue.
• Generate a pull request for this issue: Comment @sourcery-ai develop to
  generate a PR that addresses this issue.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.