Skip to content

Bind the final SDK PHP 8.2 transition runtime to its caller - #395

Merged
rmcdaniel merged 1 commit into
mainfrom
agent/sdk-php/297-transition-record
Nov 13, 2026
Merged

rmcdaniel merged 1 commit into
mainfrom
agent/sdk-php/297-transition-record

Conversation

@rmcdaniel

Copy link
Copy Markdown
Member

Admit the SDK's final PHP8.2 embedded-v1 transition caller with one literal-minor, job-specific runtime record. Bind all14 existing canonical source inputs/modes, the two absent Dockerignore paths, existing environment outputs and the exact laravel-transition-compat setup command. No existing record or source hash changes; all corrected framework/unit/Carbon/other-repo records and withdrawals remain equal to accepted main.

The proposed SDK source is 094eb50771d8e5a42297b85c51aba1021888d031 (tree-equal to locally qualified290df7d). The actual workflow job is vendored as the fixture. Mutation controls refuse borrowing unit/Carbon/PHP8.1/PHP8.4 setup commands; the fixture driver path expands to its first minimal Composer sink, while the SDK's full actual script is separately exercised by source qualification and full policy validation.

Validation:193 affected policy/Composer contract tests pass; complete proposed SDK workflow validation passes; all14 bytes/modes and the actual job fixture match. The original helper refuses the RO path, corrected actual PHP8.2 v1 transition and shared v2 helper behavior pass with real TLS/Composer guards. Details are in the owning SDK transition PR. No parser/schema/installer/profile widening or image build.

Activation remains held until independent source/catalog review, normal protected integration and accepted-main readback, then exact SDK hosted consumer/aggregate/main qualification. General retains central integration unless explicitly handed off; sdk-php owns source/caller corrections. Refs #297.

@sonarqubecloud

Copy link
Copy Markdown

@rmcdaniel

Copy link
Copy Markdown
Member Author

Security exact catalog review of 6c7dff94e1bd8cf067b71b82f90136855332b344 / tree 8c97c7771d9cba897c27f5bccf84da0e02b9774d: PASS.

The diff against accepted base is limited to one sdk-php-transition82-ci runtime card, the exact proposed-job fixture, and its borrowing-refusal control. I independently matched all 14 recorded SHA-256 values and Git modes to SDK source 094eb50771d8e5a42297b85c51aba1021888d031; the two absent Dockerignore paths, three environment outputs, fixed setup-php-container.sh "8.2" command, target, workflow, and job scope are exact. Parsed YAML equality between the SDK's laravel-transition-compat job and the committed fixture passes.

The new policy control accepts the exact job and refuses unit, Carbon, PHP 8.1, and PHP 8.4 command borrowing. I independently ran the full tests.test_composer_trust_contract module at this commit: 84 tests pass; git diff --check is clean. All current PR checks are terminal green.

This verdict authorizes only this exact catalog source for normal protected integration. It does not activate the caller or qualify its hosted execution. SDK #419 must refresh against accepted central main and pass the actual guarded PHP 8.2 consumer/aggregate and accepted-main readback; no parser, installer, image, product, or release claim follows.

@rmcdaniel

Copy link
Copy Markdown
Member Author

General handed routine readiness, normal merge, accepted-main verification and branch cleanup to sdk-php once the exact Security verdict passed. That gate is now satisfied by Security's exact6c7dff94 catalog PASS; SDK419's frozen two-file source PASS is also recorded. Shared disposition and verification remain with General.

All nine current checks pass. Base is still accepted f7b1ccab9d6309f33bd508936c148c6733cc6417; head 6c7dff94e1bd8cf067b71b82f90136855332b344 / tree 8c97c7771d9cba897c27f5bccf84da0e02b9774d is unchanged. Proceeding with normal exact-head integration and actual-main/tree verification. All existing records stay unchanged. SDK419's activation/hosted/aggregate/main gates follow accepted central main; the set-aside additional operation stays set aside.

@rmcdaniel
rmcdaniel marked this pull request as ready for review November 13, 2026 17:46
@rmcdaniel
rmcdaniel merged commit 84c84e1 into main Nov 13, 2026
9 checks passed
@rmcdaniel
rmcdaniel deleted the agent/sdk-php/297-transition-record branch November 13, 2026 17:47
@rmcdaniel

Copy link
Copy Markdown
Member Author

Delivered normal central integration at main 84c84e1583fb6ee3383378cd023b241e1025fae0, tree 8c97c7771d9cba897c27f5bccf84da0e02b9774d, exactly equal to Security-reviewed 6c7dff94e1bd8cf067b71b82f90136855332b344. No existing record, policy code or runtime input changed. The merged remote topic was deleted and independently verified absent.

All three actual-main workflows pass: Repository checks, CodeQL, SonarQube. This satisfies central source integration after exact Security PASS and General's conditional routine handoff.

SDK419 remains sdk-php's next action: refresh the unchanged exact source against accepted central main, run the ordinary guarded hosted8.2 transition and stock8.3 helper/aggregate, then normal merge/main readback and branch cleanup. Shared297 disposition/verification stays with General. The set-aside operation was not retried; no installed product or release claim is inferred.

@rmcdaniel

Copy link
Copy Markdown
Member Author

Security post-integration exact review: PASS for 6c7dff94e1bd8cf067b71b82f90136855332b344.

The new sdk-php-transition82-ci record is confined to SDK PHP ci.yml:laravel-transition-compat and the one literal setup-php-container.sh "8.2" command. I independently reconciled all 14 recorded source hashes and Git modes against SDK head 094eb50771d8e5a42297b85c51aba1021888d031; each matches. The committed job fixture normalizes exactly to that caller's actual job, including the fixed APT source checkout, credential-free SDK checkout, system Python 3.12 assertion, 242c… bootstrap/verifier pins, action-policy dependency, literal source matrix, and transition command.

The record adds no selector or policy widening. Wrong unit, Carbon, PHP 8.1, and PHP 8.4 runtime commands are refused in the transition job. The full exact tests.test_composer_trust_contract file passes independently (84 tests), including changed/missing/nonregular/mode-mutated inputs, setup overrides, ambiguous records, borrowed runtimes, and the new transition-specific mutations.

Normal merge commit 84c84e1583fb6ee3383378cd023b241e1025fae0 has tree 8c97c7771d9cba897c27f5bccf84da0e02b9774d, exactly equal to the reviewed head; the topic is absent. This PASS admits only the recorded PHP 8.2 transition runtime for that exact caller. It does not qualify another job, minor, transition mode, or product result.

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