Awesome Reviewers

When creating plugin outlets or extension points, avoid exposing entire component instances, controllers, or large objects. Instead, pass explicit, minimal arguments that represent only the data and actions that plugins actually need.

Exposing entire objects makes them part of the public API, making future refactoring difficult and creating tight coupling between internal implementation and external consumers.

Bad:

<PluginOutlet
  @name="sub-category-item"
  @outletArgs={{hash subCategoryItem=this}}
/>

<PluginOutlet
  @name="exception-wrapper"
  @outletArgs={{lazyHash controller=@controller}}
/>

Good:

<PluginOutlet
  @name="sub-category-item"
  @outletArgs={{lazyHash 
    category=this.category
    unreadCount=this.unreadTopicsCount
    newCount=this.newTopicsCount
    hideUnread=this.hideUnread
  }}
/>

<PluginOutlet
  @name="exception-wrapper"
  @outletArgs={{lazyHash
    errorHtml=@controller.errorHtml
    isForbidden=@controller.isForbidden
    reason=@controller.reason
    requestUrl=@controller.requestUrl
  }}
/>

This approach creates stable, documented interfaces that can evolve independently of internal implementation changes.

This site is open source. Improve this page.