Skip to content

Label HarmonyOS authorize client#52171

Draft
xiasi0 wants to merge 1 commit into
home-assistant:devfrom
xiasi0:codex/harmonyos-authorize-label
Draft

Label HarmonyOS authorize client#52171
xiasi0 wants to merge 1 commit into
home-assistant:devfrom
xiasi0:codex/harmonyos-authorize-label

Conversation

@xiasi0
Copy link
Copy Markdown

@xiasi0 xiasi0 commented May 23, 2026

Summary

  • move known authorize app client labels into a small shared map
  • label the HarmonyOS client id as HarmonyOS on the authorize page
  • add coverage for the HarmonyOS app label

Validation

  • node .yarn/releases/yarn-4.14.1.cjs test test/auth/ha-authorize.test.ts
  • node .yarn/releases/yarn-4.14.1.cjs eslint src/auth/app-names.ts src/auth/ha-authorize.ts test/auth/ha-authorize.test.ts --max-warnings=0
  • pre-commit hooks during commit

Related

Copy link
Copy Markdown
Member

@balloob balloob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pending discussion on core PR about what this is used for.

@home-assistant
Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@home-assistant home-assistant Bot marked this pull request as draft May 23, 2026 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants