Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 28, 2026

Description

Phone functionality was deprecated - endpoints return errors, DB columns dropped in V82/V84. This removes the module and associated dead code while preserving backward compatibility.

Removed:

  • Wire.API.User.Phone module
  • PhoneUpdate, SendLoginCode types and error variants
  • Endpoints: change-phone, remove-phone, send-login-code (all stub handlers)
  • Golden tests and test data (43 files)

Retained:

  • Minimal Phone type moved inline to Wire.API.User.Identity
  • Used only for deserializing old data in UserEvents, EJPD, and password reset
  • All usages are read-only (phone fields always Nothing)

Additional dead code identified:

  • tools/db/phone-users/ - queries phone-only users, but phone columns no longer exist

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines
Original prompt

there is a module Wire.API.User.Phone in this repo, and there shouldn't. please remove it and see if you can find any other dead code around phone numbers.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 28, 2026
Copilot AI and others added 3 commits January 28, 2026 16:14
…one module

Co-authored-by: fisx <10210727+fisx@users.noreply.github.com>
Co-authored-by: fisx <10210727+fisx@users.noreply.github.com>
Co-authored-by: fisx <10210727+fisx@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove Wire.API.User.Phone module and clean dead code Remove Wire.API.User.Phone module and phone-related dead code Jan 28, 2026
Copilot AI requested a review from fisx January 28, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants