Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Comment |
purpshell
force-pushed
the
codex/oa-round2-cli-20261021
branch
from
October 28, 2026 04:17
c71cdcd to
8563080
Compare
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
Adds Number catalog list/get/products/sync and Marketing Messages status, Official multipart and Linked Devices group-picture commands, and Platform webhook batching flags/details. Existing Graph and contact block/unblock commands remain available. Server scopes, beta/enrollment checks and Hybrid linked-only routing remain authoritative.
Current head:
6ac56fea4fd9e0e7afe92bcc9d2fd0a339a4e34e, tree1612c6fe041dee17be5a42ffff0f92c44af668be. Current main044f8b75ae8b0689664fe489c22d4a3c64486717is an ancestor. Normal integration commit0ed399bfdf77793fb18e18bfff1ec539cc1b5ab1has ordered parents85630800ea40022046c7d5fc635e6ac238092ef5and044f8b75ae8b0689664fe489c22d4a3c64486717, with no conflicts. Authored history and prior rebase mappings are preserved. No main rewrite or new force rewrite occurred.The integration includes current main's browser shell, command-reference producer and accepted Zod/MCP repairs. The actual browser-registry producer added four missing loaders: cloud catalog get/sync, cloud group picture and group picture. HTTP fixtures now use canonical synthetic opaque credentials, without relaxing authentication. The campaign recipient-reason flag now uses Oclif's typed option parser for the accepted SDK's eleven existing values; unknown values fail during parsing, before command dispatch.
Remain draft: matching-dependency native/browser/HTTP/compiled-reference qualification is unavailable under the no-install restriction. The installed SDK and security dependencies differ from the accepted lock. The smallest missing action is admission or provision of one source-bound installed qualification environment using accepted SDK source
b3bd05843342275f25f304bfe1fc67ca872740baand current-main locked security dependencies. This permits the real builds, fixture tests and CLI-reference producer; it does not authorize registry publication or deployed acceptance.Accepted SDK source through sdks#362 contains all seven required methods/types. The committed registry SDK pin remains
0.1.0-dev.20261004024706, which lacks those exports. No invented version, Git/file dependency, donor lockfile or type bypass was committed. The round-2 manifest'sf9937116b17f9962de8a3eb632c44d0399b05002remains historical candidate evidence. Actual accepted package pinning/publication is a separate merge/release gate; publication alone is not the reason to withhold source Ready.Impact
Impact table
src/commands/cloud/,src/commands/group/picture.ts,src/commands/campaign/recipients/list.ts,src/developer/batching.ts, parser regressioncontracts/cli-reference.jsonis inherited current-main output and does not yet cover this PR. No hand-edit or stale-dist generation was used.scripts/gen-browser-registry.mjs,scripts/cli-reference.mjs,contracts/cli-reference.json,CLI-CURRENT-CHECKS.jsonpackage.json,package-lock.json,contracts/official-api-round2.json, sdks#362044f8b75; installed/locked table belowdocs/official-api-round2.md,contracts/cli-reference.json,scripts/cli-reference.mjspublished:false, source-bound SDK/environment acceptance, actual SDK/CLI package publication and independent exact-head CI/review remain required. Beta/server authority and uncertainty reconciliation remain enforced.scripts/prepare-release.mjs, release evidence guardsNo impact table
contracts/official-api-round2.jsonand CLI-only pathstest/helpers/opaque-credential.ts, two HTTP fixture files,scripts/verify-browser.mjs, existing command clientsgit diff 044f8b75 HEAD -- package.json package-lock.json .githubpublished:falseVerification
Fresh checks on exact head
6ac56fea4fd9e0e7afe92bcc9d2fd0a339a4e34e, under one serialized standardlocal-validation/run.pyclaim, Node 24.15.0 and one Vitest worker:tsc -p tsconfig.json --noEmit --incremental falsepassed against the existing installed candidate; matching lock/runtime qualification remains openEarlier in this continuation, five fixture assertions ran against the unmodified credential-shape implementation fetched from accepted SDK b3bd0584: valid opaque key accepted, old invalid fixture rejected, corrupted checksum rejected, canonical client credential accepted and old client fixture rejected. This is component validation, not an installed SDK/build/HTTP acceptance claim. Actual producer-file SHA-256:
efd2195c0fad2d4bb80fb97794d44acd6718a9d70c7765eb5971e0b315dd9838. The reason options were compared with the accepted SDK campaign-contract enum; all eleven values match.No install/download, stale-dist command execution, native/browser build, HTTP fixture run, compiled-reference regeneration, production audit, package dry-run or live provider test is claimed. No source assertion or backend/API semantics was weakened to silence a check.
Historical CI 40631452866, Verify Node 22 job 122628536690 failed prepare/build on old head
85630800because the registry SDK lacks four catalog operations, Official group picture, Marketing status and WebhookBatchingInput. Audit was not reached. Accepted SDK b3bd0584 source contains those exports; old CI remains a failure and does not qualify this head. Current-head hosted CI and independent review are unqualified here. There were no inline review threads in the owner read; no review request, CI rerun/dispatch or source watch was added.Session provenance and UI evidence
Registration #511,
purpshell-t3-codex, Codexgpt-6.1-sol, xhigh reasoning, STANDARD/default service tier, thread4af7f220-6221-461f-a59a-6bce7c722622. No child or review delegate. Existing authors retain their attribution.Current reviewed manual checkpoint, commit
ba153fa8ee25077d0d4be8f9f0f3085195f585ee, records four selected assistant checkpoints, with no claim of full user/tool/native capture or subsequent test results. Prior rebase and patch/author receipt retains the earlier exact mapping. Current commands/log hashes/push receipts remain inoa-readiness-20261030/. No hidden reasoning, customer content, real credentials, GUI screenshots or runtime acceptance is claimed.Rollout and rollback
Keep draft until the genuine matching environment permits native/browser builds, scoped HTTP fixtures and actual compiled CLI-reference generation/check. One owner/environment action is missing: admit or provide that installed source-bound environment for accepted SDK b3bd0584 plus CLI main 044f8b7 security dependencies. No installation permission is inferred from this PR. This is source qualification; registry publication and deployment need their own authority.
For merge/release, bind the actual accepted published containing SDK version in package/lock and source-bound acceptance, complete exact-head CI/independent review, and obtain Release's merge decision. Nightlies require matching staging acceptance; stable packages require production acceptance. Release alone merges; staging stays deferred. PG/MR/mount/POL-23/default-off and provider acceptance remain with the separate dark Groups surface and are not qualified by these CLI checks.
No package/tag, registry, provider, flag, secret, host, deployment or DB action occurred. Old remote
85630800ea40022046c7d5fc635e6ac238092ef5is preserved atbackup/oa-cli89-ready-20261030-headand in a pre-integration bundle. Rollback can revert this consumer support but cannot reverse an applied catalog sync, block or group-picture effect. Uncertain provider effects require reconciliation; Hybrid never retries group operations through Official.