Skip to content

refactor: reuse PatternKit behavior chains#108

Merged
JerrettDavis merged 1 commit into
mainfrom
simplify-patternkit-behavior-pipeline
Jun 2, 2026
Merged

refactor: reuse PatternKit behavior chains#108
JerrettDavis merged 1 commit into
mainfrom
simplify-patternkit-behavior-pipeline

Conversation

@JerrettDavis

Copy link
Copy Markdown
Owner

Summary

  • build PatternKit behavior chains once per BehaviorPipeline instance
  • keep per-execution state isolated while avoiding repeated chain construction
  • preserve behavior ordering and cancellation semantics

Validation

  • dotnet test tests\QuickApiMapper.UnitTests\QuickApiMapper.UnitTests.csproj --configuration Release --no-restore --filter "FullyQualifiedName~Behavior"
  • git diff --check

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Test Results

118 tests  ±0   118 ✅ ±0   6s ⏱️ -1s
  2 suites ±0     0 💤 ±0 
  2 files   ±0     0 ❌ ±0 

Results for commit 62a35fe. ± Comparison against base commit 5eee41f.

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

🔍 PR Validation Results

Version: 0.0.0-gef6af5ad1b

📦 Detected NuGet Packages (17)

  • Demo.JsonApi
  • Demo.SoapApi
  • QuickApiMapper.Application
  • QuickApiMapper.Behaviors
  • QuickApiMapper.Contracts
  • QuickApiMapper.CustomTransformers
  • QuickApiMapper.Extensions.RabbitMQ
  • QuickApiMapper.Extensions.ServiceBus
  • QuickApiMapper.Extensions.gRPC
  • QuickApiMapper.Management.Contracts
  • QuickApiMapper.MessageCapture.Abstractions
  • QuickApiMapper.MessageCapture.InMemory
  • QuickApiMapper.Persistence.Abstractions
  • QuickApiMapper.Persistence.PostgreSQL
  • QuickApiMapper.Persistence.SQLite
  • QuickApiMapper.StandardTransformers
  • QuickApiMapper.Tools.Migrator

🚀 Detected Executables (2)

  • QuickApiMapper.Host.AppHost
  • QuickApiMapper.Tools.Migrator

✅ Validation Steps

  • Build solution
  • Run unit tests
  • Run integration tests
  • Dry-run NuGet packaging
  • Dry-run executable publishing

📊 Artifacts

Dry-run artifacts have been uploaded and will be available for 7 days.


This comment was automatically generated by the PR validation workflow.

@JerrettDavis JerrettDavis merged commit affcf3b into main Jun 2, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant