refactor: Use RequiredField util for username enforcement in API definition

[Copilot]

Replaces the verbose inline type Omit<IUser, 'username'> & Required<Pick<IUser, 'username'>> with the existing RequiredField utility from @rocket.chat/core-typings, as flagged in review.

Changes

• apps/meteor/app/api/server/definition.ts
  • Import RequiredField from @rocket.chat/core-typings
  • Replace all three occurrences of the inline pattern with RequiredField<IUser, 'username'>

// Before
user: TOptions extends { userWithoutUsername: true } ? IUser : Omit<IUser, 'username'> & Required<Pick<IUser, 'username'>>;

// After
user: TOptions extends { userWithoutUsername: true } ? IUser : RequiredField<IUser, 'username'>;

No behaviour change — RequiredField<T, K> is defined as T & Required<Pick<T, K>>.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[dionisio-bot]

Looks like this PR is not ready to merge, because of the following issues:

• This PR is missing the 'stat: QA assured' label
• This PR is missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

⚠️ No Changeset found

Latest commit: efb3287

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[cubic-dev-ai]

No issues found across 1 file