Skip to content

api: drop User.phone and the delivered order status - #28

Open
zsobpeter-code wants to merge 6 commits into
mainfrom
api/drop-user-phone-delivered
Open

zsobpeter-code wants to merge 6 commits into
mainfrom
api/drop-user-phone-delivered

Conversation

@zsobpeter-code

Copy link
Copy Markdown
Member

Removes the optional phone property from the User schema (and from the GET /users/{id} response example) and removes delivered from the Order.status enum. Note: these are breaking contract changes. CreateUserRequest.phone is left unchanged.

@coderifts

coderifts Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

STOP · merge · sha256:1297fd9…6ea049 · expires 2026-10-11T16:02:18Z · FIELD_REMOVAL, TOOL_RESULT_SHAPE_DRIFT, AGENT_PROTOCOL_DRIFT
signature_present_not_locally_verified
n/a · issued in this run — not in the commit
npx @coderifts/receipt-verifier
the approver is not in the signature · https://coderifts.com/docs/proof-boundaries/
remedy: this receipt does not allow the merge; re-preflight authorize for this diff and use that receipt

Next step for an agent: revert, then call preflight_change_set.
Suggested fix: Keep field phone on /users/{id}; mark deprecated; add replacement field alongside; remove later.
Suggested fix: Keep field phone on /users; mark deprecated; add replacement field alongside; remove later.

The long report

Mumblemanatee
Mumblemanatee previously approved these changes Oct 10, 2026
@zsobpeter-code

Copy link
Copy Markdown
Member Author

/rerun

@zsobpeter-code
zsobpeter-code dismissed Mumblemanatee’s stale review October 10, 2026 20:31

dismissal test 2026-10-10 (R5)

@zsobpeter-code

Copy link
Copy Markdown
Member Author

/rerun

Mumblemanatee
Mumblemanatee previously approved these changes Oct 10, 2026
@zsobpeter-code
zsobpeter-code dismissed Mumblemanatee’s stale review October 10, 2026 22:19

R5 live test 2026-10-11

@zsobpeter-code

Copy link
Copy Markdown
Member Author

/rerun

Mumblemanatee
Mumblemanatee previously approved these changes Oct 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants