Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Ports are not abstracting adaptors

オープン
#2 コメント 0 件 リアクション 9 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
35/100
issue の種類
リファクタリング
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
javascript, node.js

調査の方向性

issue でリンクされている 6 行目の ports/Repository.js から始め、その契約が DynamoDB スキーマとどのように結合しているかを追跡してください。issue の input/output 契約に関する議論を使って、ドメインロジックとアダプターの境界を評価してください。完了の条件は、アダプターのスキーマ変更に対応するドメインの変更が不要になることです。

索引モデルが issue の本文から書いたものです。

説明

I used internally this sample for training with my team, and they got confused by the responsibility of ports. They struggled to understand the abstraction and to translate it to typescript. This issue is an attempt to make things more clear for developers new to hexagonal architecture.

I believe ports should enforce a contract between Domain logic and Adaptors, and protect changes to happen on domain logic when the implementation details present in Adaptors change.

Here the port is not only enforcing the contract but tied to the adaptor on itself.

for example, the Repository port is tied to the implementation of a DynamoDB table with a defined schema.

https://github.com/aws-samples/aws-lambda-hexagonal-architecture/blob/1e9ec9a4e05479533ac01018adadf8e544b3a156/hexagonal-architecture/ports/Repository.js#L6

If we happen to change the database schema (changing key names, moving to a single table design, etc..), the port is not protecting the Domain logic from the implementation detail in the Adaptor. Moving to a single table design will imply changing Adaptor/ports and domain logic.

If we were in a typed language, the port would be represented by an Interface, that defines the contract (Input/Output), and the adaptor would implement this interface (the contract).

主要言語
JavaScript
スター
159
フォーク
27
PR マージ指標
30日以内にマージされた PR はありません

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。