Skip to content

fix: sync beacon/ add-on tree with root and fail CI on drift - #52

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/fix-beacon-sync-drift-562f
Oct 12, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
cursor/fix-beacon-sync-drift-562f

Conversation

@asachs01

Copy link
Copy Markdown
Owner

Summary

Fixes #47 — Syncs the beacon/ add-on directory with root src/ after PRs #48-#51 updated translations in root but missed the mirrored beacon/src/ tree that the HA add-on builds from.

Changes

Synced Files (24 files)

  • beacon/src/App.tsx
  • beacon/src/api/photos.ts
  • beacon/src/api/photos.test.ts (new)
  • beacon/src/components/CountdownWidget.tsx
  • beacon/src/components/PhotoFrame.tsx
  • beacon/src/components/Timer.tsx
  • beacon/src/components/WeatherView.tsx
  • beacon/src/components/cards/CardConfigModal.tsx
  • beacon/src/components/cards/CardPickerModal.tsx
  • beacon/src/components/cards/registry.test.ts
  • beacon/src/components/cards/registry.tsx
  • beacon/src/hooks/useNotifications.ts
  • beacon/src/hooks/useNotifications.test.ts (new)
  • beacon/src/i18n/translations/de.ts
  • beacon/src/i18n/translations/en.ts
  • beacon/src/i18n/translations/es.ts
  • beacon/src/i18n/translations/fr.ts
  • beacon/src/i18n/translations/it.ts
  • beacon/src/i18n/translations/ja.ts
  • beacon/src/i18n/translations/ko.ts
  • beacon/src/i18n/translations/nl.ts
  • beacon/src/i18n/translations/pt.ts
  • beacon/src/styles/beacon.css
  • beacon/src/types/dashboard-cards.ts

CI Protection

Testing

The CI check passes after this sync and will fail if directories drift in future PRs.

Open in Web Open in Cursor 

Fixes #47

The i18n PRs #48-#51 only updated root src/ but the HA add-on builds
from beacon/src/, so users were missing those translation changes.

Changes:
- Synced 24 files from src/ to beacon/src/ using rsync
- Added scripts/check-beacon-sync.sh to detect future drift
- Added CI check step to fail builds when directories are out of sync

This ensures the dual-directory structure documented in CLAUDE.md
stays in sync for all future changes.

Signed-off-by: Cursor Agent <cursoragent@cursor.com>

Co-authored-by: Aaron Sachs <asachs01@users.noreply.github.com>
@asachs01
asachs01 marked this pull request as ready for review October 12, 2026 16:14
@cursor
cursor Bot merged commit 9391c1f into main Oct 12, 2026
1 check passed
github-actions Bot pushed a commit that referenced this pull request Oct 12, 2026
## [1.39.4](v1.39.3...v1.39.4) (2026-10-12)

### Bug Fixes

* sync beacon/ add-on tree with root and fail CI on drift ([#52](#52)) ([9391c1f](9391c1f)), closes [#47](#47) [48-#51](https://github.com/48-/issues/51)

Co-Authored-By: claude-flow <ruv@ruv.net>
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.39.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

i18n: remaining untranslated screens after #45

2 participants