Repository navigation
docs: make consumer skill self-contained and highlight breaking changes - #53
Merged
Merged
Conversation
Separate published 3.0.0 guidance from unreleased fixes, correct standalone and central package installation, expose analyzer diagnostic properties, and record upgrade effects. Keep the consumer skill focused on supported usage. Co-authored-by: Codex <codex@openai.com>
Describe current release behavior directly, keep older versions in migration guidance, and bundle all skill documentation. Add prominent conditional compatibility warnings to the changelog and record validation boundaries. Co-authored-by: Codex <codex@openai.com>
AGiorgetti
marked this pull request as ready for review
October 12, 2026 14:29
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CompilerVisibleProperty.Seven documentation/control files only. No library source, public API, specification semantics, package version, workflow, or protection-setting change.
Review corrections at
c00821a6bb6504fcc1d621caf335f2aa9818c137dotnet add package Mammoth.LiteMappercommand succeeded against the existing configured feed, resolved3.0.0, and built/ran the extracted quickstart with zero warnings/errors andId=42. This is a factual feed-resolution check, not a claim that another version is published.3.0.0behavioral-break reproduction was not established.0329bf637032b04c39503f382e5f587db90b26c2. Library source, tests, build settings, workflows and benchmarks are unchanged frombd7b366.Exact correction-head CI passed at
c00821a6bb6504fcc1d621caf335f2aa9818c137after one unchanged-job retry:Documentation validation
The initial documentation revision at
bd7b366passed the following checks; changed-skill revalidation is recorded below.Id=42against published3.0.0: standalone, central management, hidden/visible analyzer properties, and central management with visible properties.Milestone16UsageDocumentationTests: 1 passed, zero failed/skipped, including Basic, Collections, and ASP.NET Core smoke samples.fa6e5ab62475eee670ec567cd84b8eb4e6a8b8a8matched the first uploaded commit.Initial-head CI passed for
bd7b366509072617d2785686e021b9e9a34f0ec1:Prior-head local artifact rehearsal
A local provisional
3.1.0artifact rehearsal was attempted frombd7b366509072617d2785686e021b9e9a34f0ec1. This is a packaging assumption, not an approved release version or a publication.Passed:
3.0.0.netstandard2.0, .NET 8/9/10; C# 9/14/latest), independent abstractions-only use, and all three unchanged source samples.Local trimmed and Native AOT publication both failed at the known MSBuild task-host creation limitation (
MSB4216/MSB4027,ComputeManagedAssemblies). Neither local executable ran. One supported tool-level escalation retry passed explicit restore but failed with the same trimming task-host errors. Ordinary exact-head CI passed its own trimming/AOT coverage, but does not prove those provisional-version release artifacts. One recovered checker error and one interrupted/resumed restore are retained in the evidence; this is not represented as a single uninterrupted all-green invocation.Performance evidence remains separate
All measurements below target
bd7b366versus actual published3.0.0; the subsequent correction changes documentation only.No measured allocation increase occurred. Overall performance acceptance is not established; no further benchmarks or threshold changes are part of this documentation correction. The repository script has now run on Windows. Cloud PowerShell remains uninstalled.
Remaining release gates
publish=false,dry_run=true; dispatch is awaiting secure GitHub sign-in. The workflow has no final-version input.protection_rules: []and no repository/inherited rulesets. No protection or credential setting was changed.The requested documentation corrections are ready for human review. Overall release readiness is not established. No merge, release, tag or NuGet publication was performed.