Skip to content

Review and classify AZM scratch refinements - #921

Merged
AndreaCuneo merged 10 commits into
masterfrom
copilot/review-azm-scratch-items
Aug 29, 2026
Merged

Review and classify AZM scratch refinements#921
AndreaCuneo merged 10 commits into
masterfrom
copilot/review-azm-scratch-items

Conversation

Copilot AI commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Reviews all AZM scratch items before the Mediator Framework’s first release, separating necessary contract decisions from incremental future work.

  • Release classification

    • Marks each item as pre-release, future, or split by scope.
    • Identifies changes that would create avoidable API, wire-format, or topology breaks.
  • Technical recommendations

    • Covers attributes, composition, naming, transport sizing, capabilities, deferral, topology, serialization, and OpenTelemetry.
    • Corrects inaccurate MessagePack, Protobuf, and payload-limit assumptions.
  • Open decisions

    • Adds focused clarification questions where requirements or delivery guarantees remain ambiguous.
    • Records criteria needed before converting pre-release findings into implementation tasks.

Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Copilot AI and others added 5 commits August 29, 2026 16:04
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Comment thread docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Outdated
Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Copilot AI requested a review from AndreaCuneo August 29, 2026 17:57
@AndreaCuneo
AndreaCuneo marked this pull request as ready for review August 29, 2026 18:16
@AndreaCuneo
AndreaCuneo requested a review from a team as a code owner August 29, 2026 18:16
Copilot AI lite review requested due to automatic review settings August 29, 2026 18:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR formalizes the “AZM scratch” pre-release review by classifying refinements into concrete pre-release tasks, future work, and open decisions, and by adding new task writeups (AZM-17 through AZM-23) to the Mediator Framework task board.

Changes:

  • Updates the AZM scratch document to a reviewed/refined structure with pre-release vs future classification and explicit open questions.
  • Adds new pre-release task specs (AZM-17..AZM-23) covering naming, payload sizing, non-JSON validation, OTel metrics, generic declarations, fluent composition, and multiline API-surface format.
  • Updates the tasks index to include AZM-17..AZM-23 entries.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
docs/mediator-framework/progress/tasks/README.md Adds task-board entries for AZM-17..AZM-23.
docs/mediator-framework/progress/tasks/azure-functions/AZM-scratch.md Rewrites scratch notes into reviewed, release-classified refinements with assignments and open questions.
docs/mediator-framework/progress/tasks/azure-functions/AZM-17-logical-names-and-native-entity-mapping.md New task spec for separating logical names from transport-native naming constraints.
docs/mediator-framework/progress/tasks/azure-functions/AZM-18-transport-contract-and-payload-sizing.md New task spec for transport-owned payload limits and runtime sizing/offload rules.
docs/mediator-framework/progress/tasks/azure-functions/AZM-19-non-json-contract-validation.md New task spec for compile-time validation of non-JSON wire contract shapes.
docs/mediator-framework/progress/tasks/azure-functions/AZM-20-opentelemetry-messaging-metrics.md New task spec for defining a stable, low-cardinality messaging metrics contract.
docs/mediator-framework/progress/tasks/azure-functions/AZM-21-generic-messaging-declarations.md New task spec for constrained generic declaration attributes and static interfaces.
docs/mediator-framework/progress/tasks/azure-functions/AZM-22-fluent-messaging-composition.md New task spec for a single fluent IServiceCollection composition entry point.
docs/mediator-framework/progress/tasks/azure-functions/AZM-23-multiline-messaging-api-surface.md New task spec for deterministic multiline messaging snapshot grammar + parser updates.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@github-code-quality

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: C#

C# / Ark.Tools

The overall line coverage in commit 4136159 in the copilot/review-azm-s... branch remains at 47%, unchanged from commit d70743c in the master branch.

Show a line coverage summary of the most impacted files.
File master d70743c copilot/review-azm-s... 4136159 +/-
/home/runner/wo...MemTransport.cs 93% 86% -7%
/home/runner/wo...ultRetryStep.cs 69% 72% +3%
/home/runner/wo...ctionManager.cs 72% 86% +14%
/home/runner/wo...ntextFactory.cs 57% 73% +16%

Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 29, 2026 19:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Initialize containers custom setup step from your .github/workflows/copilot-setup-steps.yml file failed.

…ot-setup-steps

Co-authored-by: AndreaCuneo <5227688+AndreaCuneo@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 29, 2026 19:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.

@AndreaCuneo
AndreaCuneo merged commit 5d32e27 into master Aug 29, 2026
4 checks passed
@AndreaCuneo
AndreaCuneo deleted the copilot/review-azm-scratch-items branch August 29, 2026 20:16
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.

3 participants