Repository navigation
Reconcile rewrite acceptance and publish migration guidance - #62
Merged
Merged
Conversation
Map every WIS criterion, verify integrated operation coverage, and document current-source consumer installation and live verification boundaries. Refs #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
Reconciles all WIS criteria against the integrated rewrite and adds the missing 1.x migration guide and public coverage/limitations reference. The migration read is executable and checked, and every included inventory entry now has a repeatable check against real typed methods, documentation and contract-test classes.
Upload implementation resolutions are recorded separately from the original upstream documentation gaps, preserving source provenance without keeping resolved implementation contracts blocked. The acceptance map, project context and Unreleased changelog are updated.
Verification
All 3,357 offline tests passed in default and strict runs on PHP 8.4 and 8.5 with networking denied. Documentation checks passed. Separate independent Standards and Spec reviews found no issues, including renewed reviews of the fresh-install evidence.
Separately approved fresh temporary consumers installed the current local-source package, resolved public dependencies and passed synthetic reads on both runtimes. This is not published-2.0 artifact verification. Broad live API behavior, S3 execution and plan entitlement remain unverified as documented.
Merging publishes the updated docs through the existing workflow; it does not publish a library release.
Closes #29
Refs #8
Validation details
6a4a2980f1a43eea93e06911b4769aea72082043; delivered commit:c2010a44818d5c1bc28f0cbc9535f9a7589e4c42. All 1,143 reviewed source/support content, mode and absence fingerprints were compared after commit and preserved. The working tree is clean.sandbox-exec -p '(version 1)(allow default)(deny network*)'. PHP 8.4.26 and 8.5.10 each passed bothcomposer testandphp -d allow_url_fopen=0 vendor/bin/phpunit --fail-on-warning --fail-on-risky --fail-on-deprecation --fail-on-notice: 3,357 tests / 31,076 assertions per run. The test bootstrap enables captured arguments for redaction checks. No legacy/live/destructive tests ran.php -d allow_url_fopen=0 tools/check-inventory.php --self-test --require-complete: accounting/mapping and 23 malformed-inventory checks passed. A separate negative CLI fixture missing resolution evidence exited 1 and retained its visible blocker.php -d allow_url_fopen=0 tools/check-delivery.php: 160 non-Upload catalog methods plus six Upload workflows map to actual method signatures, request/result classes, documentation files and executable PHPUnit classes. These are mapping checks, not a substitute for behavioral contract execution.composer validate --no-check-publish --no-interactionpassed with the existingGPLv2SPDX warning; license identity was not changed. Installed-platform checks passed on both runtimes.dev-main, with no lock/vendor copied. Onlysrc/, manifest, LICENSE and README were exported; all 769 files were byte-compared against the checkout. Plugins/scripts were disabled; Composer home/cache were isolated. Public dependencies resolved Guzzle 8.3.0. Installation, non-dev platform checks, default client construction without HTTP and one injected typed media read passed on PHP 8.4.26/8.5.10. Smoke reads had OS network denial; dependency downloads required network. Repository dependencies were not modified.dev-mainselection, published-release verification and a full dependency-version matrix. Root PHPUnit runs used the provisioned Guzzle 8.2.0; Guzzle 8.3 evidence is consumer installation/configuration/smoke coverage only.php tools/docs/build.php && php tools/docs/check.php: passed, 1,606 HTML pages.check-examples.php: 35 PHP snippets, with the tutorial and migration reads each executed through a synthetic transport.check-repository-links.php: 495 local links passed.git diff --check: passed. Runtime/test/config/public-site inputs were unchanged by the final two internal evidence updates; relevant checks were retained and repository links renewed.