Skip to content

Remove Kayak adapter pattern#120

Merged
hibri merged 3 commits intomasterfrom
copilot/remove-adapters-kayak-dependency
Mar 28, 2026
Merged

Remove Kayak adapter pattern#120
hibri merged 3 commits intomasterfrom
copilot/remove-adapters-kayak-dependency

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 28, 2026

  • Create IHttpRequestHead.cs interface with Method, Uri, Headers, HasEntityBody properties
  • Have HttpRequestHead implement IHttpRequestHead
  • Update interface/abstract signatures to use IHttpRequestHead: IRequestProcessor, IRequestHandler, IRequestVerify, IMatchingRule, IStubResponse, IRequestMatcher
  • Update ReceivedRequest to use IHttpRequestHead
  • Update concrete implementations: RequestProcessor, RequestHandler, RequestMatcher, EndpointMatchingRule, StubNotFoundResponse
  • Build and run all tests (35 unit + 62 integration = 97 total, all passing)

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Copilot AI and others added 2 commits March 28, 2026 15:17
…questHeadAdapter; rename SimpleHttpRequestHead to HttpRequestHead

Agent-Logs-Url: https://github.com/hibri/HttpMock/sessions/f9f4bdf9-f737-47fb-89bd-4f315612b6cd

Co-authored-by: hibri <122442+hibri@users.noreply.github.com>
Copilot AI requested a review from hibri March 28, 2026 15:29
@hibri hibri marked this pull request as ready for review March 28, 2026 15:40
@hibri hibri merged commit 890260b into master Mar 28, 2026
1 check passed
@hibri hibri deleted the copilot/remove-adapters-kayak-dependency branch March 28, 2026 15:51
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