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 |
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
Refresh eight profile/consent/import commands against main
301bd6aand API dev9bb1e2e. Gate--tagand--phone-prefix: help says "requires API support not yet available" and either flag fails before HTTP. The adapter sends only cursor/limit, so API #567 is required before enabling filters. No filtering claim.SDK pin remains
0.1.0-dev.20261004024706; local types/authenticated raw transport bridge unpublished methods. Profile mutations disable retries. Includes the same MCP audit patch as #80.Impact
Impact table
0.1.0-dev.20261004024706; local types/authenticated raw transport bridge missing methods. TS/Go/Python/PHP/C#/Rust package parity and acceptance remain separate SDK work (#348/#350/#351/#352).No impact table
9bb1e2e; no routes, scopes, protocol, workflow or storage changes.Verification
Node 24:
npm run verifypasses (1,074 tests / 111 files, format, lint, typecheck, build, pack dry-run, zero production audit vulnerabilities). Three release-evidence tests and Actionlint pass. The 23 focused command tests cover all eight paths, pagination, CSV mapping/replay identity, revisions, API failures, unsupported-filter refusal with zero HTTP and four transient mutation failures sent once. Built help verifies both unavailable flag descriptions. API paths/scopes/fields/errors checked against dev9bb1e2e; filters are absent there. No deployed API, import-worker execution, real send, hosted enrollment, registry install or publication exercised.Session provenance and UI evidence
Parent thread
646a0c0a-e900-4d0f-9a25-0b8834d7d4e6; implementation agent gpt-6.1-sol through T3 Code. Sequential work, no requested automated reviews. Coordinator owns registration/checkpoint publication; that evidence is pending. Synthetic terminal/HTTP fixtures only; no GUI changes or screenshots.Rollout and rollback
Leave draft; orchestrator merges. All PRs target main per
RELEASING.md. Main push runs CI, not publication.publish-dev.ymlruns at 02:30 UTC or manually and selects an exact registry SDK in disposable build inputs, requiring matching staging acceptance for nightly or production for stable. The existingcontracts/abprops-capabilities.jsonhaspublished: false, which independently blocks the publication gate. Merging source does not publish or prove availability. No workflow changes. Revert client changes before publication; published fixes need a new package. No migrations or flag changes.Independent of #80/#81; README placement avoids their append conflict. Suggested order: #80, #81, #79. Keep filters gated until API #567 is merged, deployed and accepted; enabling them requires a separate verified change. SDK #350/#348 publication/acceptance remains external.