Skip to content

Promote ha-ui and npm prerelease automation to master - #13

Merged
efraespada merged 60 commits into
masterfrom
develop
Nov 5, 2026
Merged

efraespada merged 60 commits into
masterfrom
develop

Conversation

@efraespada

@efraespada efraespada commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Promote the shared UI package and its reviewed npm publication workflow to master. This includes the Lit and DOM core and React, Vue, Angular and Svelte adapters reviewed in #10, plus the prerelease publishing workflow from #12.

Production CI will retain the five reproducible tarballs for the merge SHA. A reviewed prerelease tag will trigger npm Trusted Publishing after the package names and publisher permissions are available. Installed Domus adoption remains a separate step.

Refs #7 and #11.

efraespada and others added 30 commits November 4, 2026 03:12
Propagate the unchanged web-installed tree reviewed in #5. Native run 41758842863 completed all 3/3 diff partitions with zero candidate/open/reopened/unknown findings. Its independent metadata publication failure is the reported vypdev/copilot#882 issue covered by the owner-authorized Copilot operational-check exception. No generated Copilot asset was edited.
…d-ui-specification-catalog-and-product-standard

docs: bootstrap the shared UI specification catalog

@vypbot vypbot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Current status: All findings originating in this review are resolved.
Last reconciled on 202fe15. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 4 potential problems when commit 7d56566 was analyzed. This snapshot is historical; use the status block above for current state. 4 findings are linked to changed code.

Findings

  • medium: Canceled story asserts a nonexistent button value — stories/select.stories.ts:33
  • medium: Next.js SSR fallback assertion cannot match the rendered text — tests/consumers/ssr/fixtures/next/app/boundary.jsx:5
  • medium: Configure the SvelteKit adapter through Svelte config — tests/consumers/ssr/fixtures/sveltekit/fixture.vite.config.js:1
  • low: Navigation visual gate never compares captured pixels to the reference — tests/storybook/navigation-visual.ts:36

Comment thread stories/select.stories.ts Resolved
Comment thread tests/storybook/navigation-visual.ts Resolved
@vypbot

vypbot commented Nov 5, 2026 •

Copy link
Copy Markdown

Copilot approval · Human review needed

Status: This pull request is outside the configured approval scope.
Completed: 1/1 observed checks succeeded; Bugbot complete.
Action required: Ask a human reviewer to decide.
Impact: No new approval is confirmed for this revision.
See checks

Technical evidence

Revision: 202fe1598e6f; base: ba91a389df44; reason: scope.

@vypbot

vypbot commented Nov 5, 2026 •

Copy link
Copy Markdown

Bugbot: 1 finding needs attention

Current status: 1 finding requires attention on 202fe15.

Action: Review the linked threads or comment /copilot fix all.

Findings

Pull request · Verified commit · Workflow run

@vypbot vypbot added state:changes-requested Review identified changes that are required. state:awaiting-issue-author The next action requires more information or changes from the issue author. and removed state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 5, 2026
@efraespada

Copy link
Copy Markdown
Member Author

/copilot dismiss stories/select.stories.ts:canceled-play-button-value tests/consumers/ssr/fixtures/next/app/boundary.jsx:5:loading-fallback-ellipsis-mismatch fixture.vite.config.js:1:adapter-not-configured

@vypbot vypbot added state:reviewing A pull request is being reviewed. and removed state:changes-requested Review identified changes that are required. state:awaiting-issue-author The next action requires more information or changes from the issue author. labels Nov 5, 2026
@vypbot

vypbot commented Nov 5, 2026

Copy link
Copy Markdown

3 findings were dismissed.

…nshots-with-original-home-assistant-pixels

test: compare Home Assistant navigation pixels
@vypbot vypbot added the state:ai-processing A Copilot agent is analyzing or working on the issue or change. label Nov 5, 2026

@vypbot vypbot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Current status: 1 finding originating in this review requires attention.
Last reconciled on 202fe15. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 202fe15 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • low: A drag can suppress the next tab click — packages/core/src/components/navigation-overflow.ts:107

@vypbot vypbot added state:changes-requested Review identified changes that are required. state:awaiting-issue-author The next action requires more information or changes from the issue author. and removed state:reviewing A pull request is being reviewed. state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 5, 2026
@efraespada

Copy link
Copy Markdown
Member Author

/copilot dismiss packages/core/src/components/navigation-overflow.ts:107:stale-drag-click-suppression

@vypbot vypbot added state:reviewing A pull request is being reviewed. and removed state:changes-requested Review identified changes that are required. state:awaiting-issue-author The next action requires more information or changes from the issue author. labels Nov 5, 2026
@vypbot

vypbot commented Nov 5, 2026

Copy link
Copy Markdown

1 finding was dismissed.

@efraespada
efraespada merged commit 35b4197 into master Nov 5, 2026
13 of 14 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in vypdev Nov 5, 2026
@vypbot vypbot added state:verified The change has passed Copilot verification. and removed state:reviewing A pull request is being reviewed. labels Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:verified The change has passed Copilot verification.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants