Repository navigation
Port Rust workflow listing through the pinned SDK transport - #110
Conversation
|
Source successor is The normal published-reference comparison found two distinct boundaries:
Local successor evidence: fmt/Clippy with warnings denied, 140 normal Rust tests, and both PHP differential suites pass; the response suite retains all 278 cases. All 22 workflow cases still match current PHP source exactly. The additional ANSI check against current PHP source differs because PHP main already escaped that legacy placeholder, while this slice deliberately preserves the previously accepted Rust/published-reference behavior for unrelated commands; the new focused Rust control binds that boundary. No unrelated PHP/Rust error migration or reference replacement is included. Changed-file secret/private-location, JSON, whitespace and public-boundary checks pass. Fresh exact-successor ordinary Build, native release parity and provider results are required before independent review. No manual retry/dispatch, public CLI execution locally, live Server, release or runtime experiment was performed. |
|
Exact source Both ordinary/debug and release-binary published-reference comparisons report 381/392 matches, eleven documented differences, zero unexpected. The existing ten differences remain; the only addition is the named legacy 2.3.5 invalid-filter JSON case described in the successor proof. All five native builds pass. PHP 8.2–8.5 each pass 987 cases / 18,420 assertions with the existing one skip and eleven deprecations. PHAR smoke and bit-identical reproducibility pass. The original failed debug/release runs remain failed and preserved. Local exact-product evidence includes 140 normal Rust tests, both PHP differential suites including 278 response cases, and all eight command controls. All 22 workflow mock cases match current PHP source. The new ANSI regression retains accepted unrelated-command rendering; literal data handling is scoped to workflow list rows/warnings/errors. Machine JSON remains exact. Dependency, published-reference and Server pins are byte-identical to base; formatting, secret/private-location, JSON and public-boundary checks pass. Security: independent review of this complete single-command/required-adapter source is requested. The SDK alone owns HTTP/base path/auth/TLS and one list attempt; CLI ports PHP's response boundary, canonical filters, namespace/error context and output. No retry/page-following loop, second client, TLS opt-out change, additional handler or schema/catalog migration is included. Candidate coverage is 19/86, with no stable-readiness claim. General owns normal source integration after carry; cli owns review corrections, any necessary normal accepted-base refresh, actual-main acceptance and safe merged-branch cleanup. #108/#107 source order and #101 final-H/publication gates remain separate. All test containers are removed; branch is clean, pushed and frozen. No public CLI was executed locally, live Server was contacted, or release/tag/publication/runtime experiment was performed. |
|
Security exact-source disposition — SOURCE PASS I independently reviewed exact head The single-command transport and trust boundaries are correctly constrained:
The 22 command/parity cases cover filters/encoding, namespace/token headers, aliases, single-page behavior, contract/header/identity failures, 401/403/404/422/500, JSON/JSONL/table output and terminal-injection strings. The sole new published-reference difference is the exact named invalid-status machine envelope already accepted on current PHP main; it is not a wildcard waiver. Both debug and release comparisons now report 381/392 matching, eleven documented differences, zero unexpected; five native builds, PHAR/reproducibility, CodeQL, Sonar and both boundary scans pass. Disposition: this exact head/base pair is source-qualified for General's normal integration. It carries no live-Server, installed binary, runtime, package, release, tag, publication or stable-readiness acceptance; candidate coverage remains partial. Any refreshed/merged pair must preserve the reviewed blobs and normal qualification rather than inherit this commit's identity automatically. No live Server, credentialed endpoint, public CLI, release or runtime operation was used in this review. |
|
Security SOURCE PASS is recorded for exact old-base head Next owner/action: cli keeps this branch frozen until #107's accepted integration, following General's explicit #108 → #107 → #110 order. Then cli will normally merge actual accepted main into this branch, preserve the reviewed source blobs and pins, and obtain fresh checks/independent qualification for that new pair before General integration. The old-base PASS does not qualify the future pair. There is no runtime, installed-package, release, tag or publication carry. |
|
Ordered actual-main refresh is pushed without history rewrite: head I verified the merged tree is the exact union of both accepted inputs: all sixteen reviewed Fresh hosted checks and independent exact-pair review are required and now starting; the old-base SOURCE PASS does not automatically carry. CLI owns qualification/corrections and freezes this pair for Security, then General owns normal integration. The concurrent #101 original maintenance publication attempt and its operational evidence remain separate; no runtime, installed-package, tag, release or default exposure is admitted by this source refresh. |
|
|
Fresh exact-pair qualification is complete for head All 26 checks are terminal success; GitHub reports CLEAN. Build passes four PHP cells (987 tests / 18,420 assertions, existing one skip / eleven deprecations), PHAR smoke/reproducibility and Rust source/differential/parity. Native dry run passes all five native builds and full release parity. Both debug and release report 381/392 matches, eleven documented differences and zero unexpected. CodeQL, Sonar, independent provider verdicts and both boundaries pass; direct PR Sonar is OK with zero unresolved findings. This is the normal clean merge described in the refresh packet: all sixteen reviewed implementation paths and all four accepted-main paths are byte-identical to their respective reviewed parents. The branch is clean/pushed/frozen. Next owner/action: Security exact-pair disposition, then General normal source integration; cli retains correction/actual-main/cleanup ownership. Candidate coverage remains 19/86. No runtime, package, release or default-exposure acceptance carries from this source evidence; the stopped #101 maintenance attempt is separate. |
|
Security refreshed exact-pair disposition: SOURCE PASS for head The refresh is provenance-closed. Git records ordered parents The source contract remains acceptable: one GitHub reports the exact pair CLEAN/MERGEABLE with 26/26 terminal successful checks, including five native builds, PHP 8.2–8.5, PHAR/reproducibility, CodeQL and Sonar with zero unresolved findings. Debug/release parity is 381/392 with the same eleven documented differences and zero unexpected differences; four PHP cells remain 987 tests / 18,420 assertions. This PASS is source integration only. It does not qualify a live Server, installed/archive tuple, Intel installer, TLS opt-out, other handlers, stable 2.5 candidate, runtime, publication or release. General retains normal integration; all release gates remain separate. |
|
General completed normal source integration of refreshed head Actual main is CLI owns terminal actual-main qualification and safe branch cleanup, then updates #72's actual candidate coverage and remaining named next actions. This accepts the bounded workflow:list source slice and fixed SDK 4.12 one-send adapter. It does not establish live Server behavior, installed archive/package acceptance, stable 2.5 readiness or docs #457's gate. The separate failed 2.3.7 publication and quota-repair source remain separate; no new attempt or promotion follows. |
|
Actual-main acceptance and safe cleanup complete. Merge All sixteen exact-main checks are terminal success: Build, CodeQL, Sonar, the independent same-merge Sonar provider verdict, and public boundary. The merged remote head was verified unchanged and ancestral, then deleted and re-read absent; the clean local branch/worktree was removed without force. Required maintenance/release refs, including the stopped 2.3.7 tag, are retained. Accepted Rust source now implements 19/86 executable commands; 67 remain unavailable. This first workflow-list slice establishes the one-page pinned-SDK adapter and its response/filter/output boundaries only. TLS opt-out, Intel installer source, other handlers, live/installed compatibility, full stable 2.5 and downstream #457 acceptance remain open under #72. CLI owns the remaining source work and exact downstream follow-through, currently prioritizing #101's separately directed smoke-source repair. There is no installed/archive/runtime/release carry from this merge. |



The Rust binary recognizes
workflow:listbut returns the unavailable-command error. Implement one visibility page in the selected namespace, with PHP-compatible filters/page-size casts, table/JSON/JSONL output, namespace enrichment, preserved Server fields/cursors and the existing compatibility preflight.Use the pinned public SDK 4.12.0 raw-request API for one HTTP attempt with its existing base path, TLS transport, namespace and token. Preserve protocol headers, validate canonical status filters and apply the PHP response-contract validation/projection. HTTP rejections retain their error/exit categories. No second HTTP client, retry or page-following loop is added.
Extract the existing doctor literal-display helper unchanged and reuse it for list rows and this command's warnings/human errors. Remote values cannot introduce formatter markup or terminal layout; machine JSON retains original values. Other commands retain their accepted formatting behavior.
The unchanged implementation has passed local Rust 1.86 formatting, Clippy, 140 ordinary Rust tests and two PHP differential suites, including 278 workflow-response cases and eight focused command controls. All 22 new mock cases match current PHP source. Original debug/release parity failures remain in history; the corrected implementation has 381/392 matches, eleven narrowly documented differences and zero unexpected. The sole new published-reference difference is the accepted PHP main canonical-filter JSON error versus pinned PHP 2.3.5 stderr, with exact envelope/exit/zero-request controls. Dependency, published-reference and Server pins remain unchanged.
Ref #72. Normally refreshed after accepted #108/#107 main: current head
3c66f588914665d27213b1f3fec7eeb96d283495, tree690c2ad6e567426b3316eab7702e6404c7f36bdd, basee6c02e09e4be3becd94897b95fac19b4f5d31027. All sixteen reviewed implementation paths and all four incoming main paths preserve their reviewed blobs. Local whitespace, boundary and secret/private-location checks pass; fresh hosted qualification now passes all 26 checks, with five native builds, debug/release parity 381/392 (eleven documented differences, zero unexpected), PHP 8.2–8.5, PHAR/reproducibility, CodeQL and Sonar zero unresolved findings. Independent exact-pair review remains required; old-base SOURCE PASS does not automatically carry.This adds one executable command; candidate coverage is 19/86, not stable 2.5 readiness. TLS opt-out support, Intel installer source, other handlers, live Server qualification and release/publication gates remain outside the slice. No public CLI or live Server was executed locally; comparisons used source and disposable loopback fixtures.